-
Notifications
You must be signed in to change notification settings - Fork 12.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Only catalog types when tracing #43446
Conversation
(cherry picked from commit 9fd2b48)
@typescript-bot perf test this |
Heya @amcasey, I've started to run the perf test suite on this PR at 0ac4f88. You can monitor the build here. Update: The results are in! |
This is a memory usage win that seems safe enough for a patch release. |
@amcasey Here they are:Comparison Report - master..43446
System
Hosts
Scenarios
|
I really don't see how this could regress check time by 40%, especially since the same PR in master did not. However, I'll investigate. master perf results: #43304 (comment) Edit: These numbers don't make any sense. Even if some weird de-opt effect hurt check time, I definitely didn't improve emit time. This should affect memory usage and nothing else. Maybe the baselines are off? Edit 2: That's because the baseline is master. 🤦♂️ |
@typescript-bot perf test this |
Heya @amcasey, I've started to run the perf test suite on this PR at 0ac4f88. You can monitor the build here. Update: The results are in! |
So I guess the perf results actually show that this PR closes the memory perf gap that existed between release-4.2 and master. |
@amcasey Here they are:Comparison Report - master..43446
System
Hosts
Scenarios
|
(cherry picked from commit 9fd2b48)