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)