Revert #35877 - fix receiver on calls of imported and exported functions - #43993
Conversation
|
TypeScript Bot (@typescript-bot) perf test |
|
Heya Ron Buckton (@rbuckton), I've started to run the perf test suite on this PR at 76b2930. You can monitor the build here. Update: The results are in! |
|
Re-running perf tests to verify this addresses the issue. |
76b2930 to
7b7a0ea
Compare
|
Ron Buckton (@rbuckton) Here they are:Comparison Report - master..43993
System
Hosts
Scenarios
Developer Information: |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Wesley Wigham (weswigham)
left a comment
There was a problem hiding this comment.
We should have some RWC baselines to accept once this gets merged~
|
TypeScript Bot (@typescript-bot) test this |
|
Heya Ron Buckton (@rbuckton), I've started to run the extended test suite on this PR at 7b7a0ea. You can monitor the build here. |
|
Heya Ron Buckton (@rbuckton), I've started to run the parallelized community code test suite on this PR at 7b7a0ea. You can monitor the build here. |
|
The user suite test run you requested has finished and failed. I've opened a PR with the baseline diff from master. |
|
Baselines seem fine, I can merge the RWC changes afterwards. |
|
Thank you! |
This reverts #35877 due to the severe negative performance impact caused by the change. We may attempt to reland this in TS 4.4 either without a flag (if we can mitigate the performance cost), or with a flag (so that you can opt in to the more expensive behavior).
Fixes #43486
/cc Daniel Rosenwasser (@DanielRosenwasser)