-
Notifications
You must be signed in to change notification settings - Fork 12.8k
Swap checkExpressionWorker
to use an array of functions instead of a switch
statement.
#50228
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
…a `switch` statement.
@typescript-bot perf test this |
Heya @DanielRosenwasser, I've started to run the perf test suite on this PR at 2b0503b. You can monitor the build here. Update: The results are in! |
@DanielRosenwasser Here they are:
CompilerComparison Report - main..50228
System
Hosts
Scenarios
TSServerComparison Report - main..50228
System
Hosts
Scenarios
Developer Information: |
@typescript-bot perf test this faster |
Heya @DanielRosenwasser, I've started to run the abridged perf test suite on this PR at ed827b2. You can monitor the build here. Update: The results are in! |
@DanielRosenwasser Here they are:Comparison Report - main..50228
System
Hosts
Scenarios
Developer Information: |
@typescript-bot perf test this faster |
Heya @DanielRosenwasser, I've started to run the abridged perf test suite on this PR at f0ab973. You can monitor the build here. Update: The results are in! |
@DanielRosenwasser Here they are:Comparison Report - main..50228
System
Hosts
Scenarios
Developer Information: |
No description provided.