Conversation
|
@typescript-bot perf test. |
|
Heya @Kingwl, I've started to run the perf test suite on this PR at 84bad5e. You can monitor the build here. Update: The results are in! |
|
@Kingwl Here they are:Comparison Report - master..39403
System
Hosts
Scenarios
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
sandersn
left a comment
There was a problem hiding this comment.
This looks like the right thing, but can you run the perf test again after merging from master? It looks a tiny bit slower, but it could just be noise.
|
@typescript-bot perf test. |
|
Heya @Kingwl, I've started to run the perf test suite on this PR at 660b399. You can monitor the build here. Update: The results are in! |
|
@Kingwl Here they are:Comparison Report - master..39403
System
Hosts
Scenarios
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
sandersn
left a comment
There was a problem hiding this comment.
The second round of perf results are not significantly slower except in a couple of cases. I think there is probably a very slight slowdown, but also that it's worth it to have the feature work right.
|
This is ready to merge whenever github's merge-checking code allows me to push the merge button. |
Fixes a part of #39374