-
Notifications
You must be signed in to change notification settings - Fork 12.9k
[WIP] Improve optional chaining checker performance #33794
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
Conversation
@typescript-bot perf test this |
Heya @rbuckton, I've started to run the perf test suite on this PR at f39b884. You can monitor the build here. It should now contribute to this PR's status checks. Update: The results are in! |
@rbuckton Here they are:Comparison Report - master..33794
System
Hosts
Scenarios
|
f39b884
to
27d142d
Compare
27d142d
to
b86485b
Compare
@typescript-bot perf test this |
Heya @rbuckton, I've started to run the perf test suite on this PR at b86485b. You can monitor the build here. It should now contribute to this PR's status checks. Update: The results are in! |
@rbuckton Here they are:Comparison Report - master..33794
System
Hosts
Scenarios
|
@typescript-bot perf test this |
Heya @rbuckton, I've started to run the perf test suite on this PR at f584fcb. You can monitor the build here. It should now contribute to this PR's status checks. Update: The results are in! |
@rbuckton Here they are:Comparison Report - master..33794
System
Hosts
Scenarios
|
@ahejlsberg: This seems to improve check time for the following scenarios:
However it seemed to have no effect on these scenarios:
This change also reduced overall memory consumption for all scenarios, versions, and cpu architectures. I'm not entirely sure why this change had little effect on Angular and TFS for those specific scenarios, but will continue to investigate. |
@typescript-bot perf test |
Heya @rbuckton, I've started to run the perf test suite on this PR at dbf67ac. You can monitor the build here. It should now contribute to this PR's status checks. Update: The results are in! |
@rbuckton Here they are:Comparison Report - master..33794
System
Hosts
Scenarios
|
I'm still looking for ways to improve the performance here, but these changes at least provide some minor improvements in some cases. |
Investigating possible performance improvements in the checker related to optional chaining.