Skip to content

Fix crash in CFA when for loop initializer throws#63108

Open
Andarist wants to merge 6 commits intomicrosoft:mainfrom
Andarist:fix/crash-cfa-throw-for-head
Open

Fix crash in CFA when for loop initializer throws#63108
Andarist wants to merge 6 commits intomicrosoft:mainfrom
Andarist:fix/crash-cfa-throw-for-head

Conversation

@Andarist
Copy link
Contributor

@Andarist Andarist commented Feb 5, 2026

fixes #63092

@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Feb 5, 2026
@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Feb 5, 2026
@DanielRosenwasser
Copy link
Member

@tyepscript-bot test it

@DanielRosenwasser
Copy link
Member

@DanielRosenwasser
Copy link
Member

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 6, 2026

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top400 ✅ Started
user test this ✅ Started ✅ Results
run dt ✅ Started ✅ Results
perf test this faster ✅ Started 👀 Results

@typescript-bot
Copy link
Collaborator

Hey @DanielRosenwasser, the results of running the DT tests are ready.

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@DanielRosenwasser Here are the results of running the user tests with tsc comparing main and refs/pull/63108/merge:

There were infrastructure failures potentially unrelated to your change:

  • 1 instance of "Git clone failed"

Otherwise...

Everything looks good!

@typescript-bot
Copy link
Collaborator

@DanielRosenwasser
The results of the perf run you requested are in!

Here they are:

tsc

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Compiler-Unions - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 82,521 82,521 ~ ~ ~ p=1.000 n=6
Types 125,280 125,280 ~ ~ ~ p=1.000 n=6
Memory used 276,733k (± 0.67%) 274,511k (± 0.06%) ~ 274,319k 274,699k p=0.230 n=6
Parse Time 1.31s (± 0.79%) 1.31s (± 0.31%) ~ 1.30s 1.31s p=0.924 n=6
Bind Time 0.75s 0.75s (± 0.54%) ~ 0.75s 0.76s p=0.405 n=6
Check Time 14.09s (± 0.35%) 14.08s (± 0.21%) ~ 14.04s 14.11s p=0.746 n=6
Emit Time 2.64s (± 0.50%) 2.64s (± 0.50%) ~ 2.62s 2.66s p=0.607 n=6
Total Time 18.79s (± 0.30%) 18.78s (± 0.09%) ~ 18.76s 18.80s p=0.936 n=6
angular-1 - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 955,823 955,823 ~ ~ ~ p=1.000 n=6
Types 415,853 415,853 ~ ~ ~ p=1.000 n=6
Memory used 1,254,966k (± 0.01%) 1,254,922k (± 0.01%) ~ 1,254,839k 1,255,034k p=0.810 n=6
Parse Time 7.94s (± 0.87%) 8.03s (± 0.33%) ~ 7.98s 8.05s p=0.052 n=6
Bind Time 2.38s (± 0.93%) 2.41s (± 0.97%) ~ 2.37s 2.44s p=0.104 n=6
Check Time 38.78s (± 0.30%) 38.91s (± 0.44%) ~ 38.62s 39.12s p=0.128 n=6
Emit Time 17.92s (± 0.50%) 17.88s (± 0.65%) ~ 17.78s 18.10s p=0.378 n=6
Total Time 67.02s (± 0.30%) 67.23s (± 0.21%) ~ 67.07s 67.48s p=0.128 n=6
mui-docs - node (v18.15.0, x64)
Errors 11,439 11,439 ~ ~ ~ p=1.000 n=6
Symbols 2,703,113 2,703,113 ~ ~ ~ p=1.000 n=6
Types 931,060 931,060 ~ ~ ~ p=1.000 n=6
Memory used 3,039,102k (± 0.00%) 3,039,096k (± 0.00%) ~ 3,038,963k 3,039,315k p=0.936 n=6
Parse Time 8.52s (± 0.30%) 8.52s (± 0.13%) ~ 8.51s 8.53s p=1.000 n=6
Bind Time 2.31s (± 0.27%) 2.31s (± 0.39%) ~ 2.30s 2.32s p=1.000 n=6
Check Time 92.86s (± 0.52%) 92.83s (± 0.15%) ~ 92.65s 92.97s p=0.810 n=6
Emit Time 0.32s (± 1.74%) 0.32s (± 1.74%) ~ 0.31s 0.32s p=1.000 n=6
Total Time 104.00s (± 0.48%) 103.97s (± 0.13%) ~ 103.78s 104.13s p=0.810 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,252,789 1,252,789 ~ ~ ~ p=1.000 n=6
Types 259,991 259,991 ~ ~ ~ p=1.000 n=6
Memory used 2,394,074k (± 0.02%) 2,454,922k (± 6.06%) ~ 2,393,635k 2,758,901k p=0.471 n=6
Parse Time 5.17s (± 1.17%) 5.12s (± 0.97%) ~ 5.07s 5.21s p=0.149 n=6
Bind Time 1.84s (± 1.22%) 1.83s (± 1.32%) ~ 1.81s 1.87s p=0.743 n=6
Check Time 35.54s (± 0.26%) 35.54s (± 0.64%) ~ 35.15s 35.80s p=0.810 n=6
Emit Time 3.01s (± 0.58%) 3.02s (± 1.08%) ~ 2.97s 3.06s p=0.520 n=6
Total Time 45.60s (± 0.28%) 45.55s (± 0.40%) ~ 45.26s 45.75s p=0.748 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,252,789 1,252,789 ~ ~ ~ p=1.000 n=6
Types 259,991 259,991 ~ ~ ~ p=1.000 n=6
Memory used 3,190,283k (± 0.02%) 2,948,170k (±12.69%) ~ 2,464,339k 3,190,639k p=0.173 n=6
Parse Time 6.79s (± 1.29%) 6.80s (± 1.54%) ~ 6.65s 6.92s p=0.810 n=6
Bind Time 2.24s (± 1.25%) 2.26s (± 1.44%) ~ 2.22s 2.31s p=0.296 n=6
Check Time 43.05s (± 0.32%) 43.07s (± 0.48%) ~ 42.75s 43.32s p=0.810 n=6
Emit Time 3.44s (± 1.73%) 3.47s (± 2.16%) ~ 3.39s 3.57s p=0.336 n=6
Total Time 55.53s (± 0.30%) 55.62s (± 0.54%) ~ 55.20s 55.87s p=0.378 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 264,703 264,703 ~ ~ ~ p=1.000 n=6
Types 104,091 104,091 ~ ~ ~ p=1.000 n=6
Memory used 443,675k (± 0.01%) 443,663k (± 0.01%) ~ 443,609k 443,725k p=0.810 n=6
Parse Time 4.33s (± 0.52%) 4.34s (± 0.62%) ~ 4.30s 4.37s p=0.370 n=6
Bind Time 1.71s (± 0.81%) 1.68s (± 1.27%) -0.03s (- 1.56%) 1.64s 1.70s p=0.029 n=6
Check Time 23.77s (± 0.31%) 23.76s (± 0.29%) ~ 23.67s 23.86s p=0.810 n=6
Emit Time 1.93s (± 1.38%) 1.92s (± 0.85%) ~ 1.90s 1.95s p=0.935 n=6
Total Time 31.73s (± 0.26%) 31.71s (± 0.30%) ~ 31.56s 31.81s p=0.936 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 271 271 ~ ~ ~ p=1.000 n=6
Symbols 225,118 225,118 ~ ~ ~ p=1.000 n=6
Types 93,554 93,554 ~ ~ ~ p=1.000 n=6
Memory used 370,533k (± 0.02%) 370,559k (± 0.01%) ~ 370,503k 370,638k p=0.575 n=6
Parse Time 2.83s (± 1.16%) 2.82s (± 0.81%) ~ 2.79s 2.84s p=0.569 n=6
Bind Time 1.63s (± 0.78%) 1.65s (± 2.09%) ~ 1.60s 1.70s p=0.280 n=6
Check Time 17.38s (± 0.35%) 17.40s (± 0.25%) ~ 17.34s 17.46s p=0.574 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 21.84s (± 0.39%) 21.87s (± 0.18%) ~ 21.83s 21.93s p=0.688 n=6
vscode - node (v18.15.0, x64)
Errors 11 11 ~ ~ ~ p=1.000 n=6
Symbols 4,221,104 4,221,104 ~ ~ ~ p=1.000 n=6
Types 1,340,209 1,340,209 ~ ~ ~ p=1.000 n=6
Memory used 4,015,297k (± 0.00%) 4,015,300k (± 0.00%) ~ 4,015,142k 4,015,488k p=0.810 n=6
Parse Time 16.21s (± 0.32%) 16.52s (± 3.25%) +0.32s (+ 1.94%) 16.20s 17.61s p=0.045 n=6
Bind Time 5.49s (± 0.71%) 5.50s (± 0.46%) ~ 5.48s 5.55s p=0.520 n=6
Check Time 121.88s (± 4.41%) 119.26s (± 2.22%) ~ 115.93s 123.50s p=0.378 n=6
Emit Time 53.25s (±15.86%) 55.24s (±13.02%) ~ 45.57s 62.96s p=0.810 n=6
Total Time 196.83s (± 4.53%) 196.53s (± 2.98%) ~ 190.45s 204.25s p=0.810 n=6
webpack - node (v18.15.0, x64)
Errors 41 41 ~ ~ ~ p=1.000 n=6
Symbols 402,249 402,249 ~ ~ ~ p=1.000 n=6
Types 178,120 178,120 ~ ~ ~ p=1.000 n=6
Memory used 559,133k (± 0.02%) 559,017k (± 0.01%) -115k (- 0.02%) 558,905k 559,107k p=0.031 n=6
Parse Time 4.76s (± 0.55%) 4.76s (± 0.76%) ~ 4.71s 4.82s p=0.570 n=6
Bind Time 2.04s (± 1.34%) 2.03s (± 2.17%) ~ 1.99s 2.11s p=0.746 n=6
Check Time 24.31s (± 1.41%) 24.38s (± 0.39%) ~ 24.20s 24.46s p=0.689 n=6
Emit Time 0.00s (±154.76%) 0.01s (±109.43%) ~ 0.00s 0.01s p=0.640 n=6
Total Time 31.11s (± 1.10%) 31.18s (± 0.40%) ~ 30.95s 31.30s p=1.000 n=6
xstate-main - node (v18.15.0, x64)
Errors 30 30 ~ ~ ~ p=1.000 n=6
Symbols 731,941 731,941 ~ ~ ~ p=1.000 n=6
Types 219,279 219,279 ~ ~ ~ p=1.000 n=6
Memory used 635,987k (± 0.01%) 635,897k (± 0.01%) -90k (- 0.01%) 635,829k 635,937k p=0.020 n=6
Parse Time 4.59s (± 0.51%) 4.59s (± 0.80%) ~ 4.55s 4.65s p=0.869 n=6
Bind Time 1.50s (± 0.42%) 1.50s (± 1.17%) ~ 1.47s 1.51s p=0.866 n=6
Check Time 22.17s (± 0.31%) 22.18s (± 0.52%) ~ 22.04s 22.33s p=0.748 n=6
Emit Time 0.00s (±244.70%) 0.00s ~ ~ ~ p=0.405 n=6
Total Time 28.26s (± 0.23%) 28.27s (± 0.36%) ~ 28.11s 28.36s p=0.688 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • Compiler-Unions - node (v18.15.0, x64)
  • angular-1 - node (v18.15.0, x64)
  • mui-docs - node (v18.15.0, x64)
  • self-build-src - node (v18.15.0, x64)
  • self-build-src-public-api - node (v18.15.0, x64)
  • self-compiler - node (v18.15.0, x64)
  • ts-pre-modules - node (v18.15.0, x64)
  • vscode - node (v18.15.0, x64)
  • webpack - node (v18.15.0, x64)
  • xstate-main - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Developer Information:

Download Benchmarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

Status: Not started

Development

Successfully merging this pull request may close these issues.

Crash: RangeError: Maximum call stack size exceeded in isReachableFlowNodeWorker with complex for loop headers

3 participants