Skip to content

Conversation

@rbuckton
Copy link
Contributor

@rbuckton rbuckton commented Oct 22, 2024

This reports an error when [Symbol.iterator]() or [Symbol.asyncIterator]() cannot safely be called by for..of, yield*, or ... due to a parameter count mismatch.

Fixes #57130

@typescript-bot typescript-bot added Author: Team For Milestone Bug PRs that fix a bug with a specific milestone labels Oct 22, 2024
@jakebailey
Copy link
Member

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 22, 2024

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

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

@typescript-bot
Copy link
Collaborator

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

Everything looks good!

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@jakebailey
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 31 31 ~ ~ ~ p=1.000 n=6
Symbols 62,340 62,340 ~ ~ ~ p=1.000 n=6
Types 50,379 50,379 ~ ~ ~ p=1.000 n=6
Memory used 193,484k (± 0.79%) 192,778k (± 0.01%) -706k (- 0.36%) 192,750k 192,805k p=0.045 n=6
Parse Time 1.31s (± 0.62%) 1.31s (± 0.96%) ~ 1.29s 1.32s p=0.796 n=6
Bind Time 0.72s 0.72s (± 0.57%) ~ 0.72s 0.73s p=0.405 n=6
Check Time 9.76s (± 0.28%) 9.73s (± 0.38%) ~ 9.67s 9.78s p=0.091 n=6
Emit Time 2.68s (± 1.35%) 2.73s (± 0.65%) ~ 2.70s 2.74s p=0.050 n=6
Total Time 14.48s (± 0.38%) 14.48s (± 0.26%) ~ 14.43s 14.53s p=0.873 n=6
angular-1 - node (v18.15.0, x64)
Errors 33 33 ~ ~ ~ p=1.000 n=6
Symbols 947,886 947,886 ~ ~ ~ p=1.000 n=6
Types 410,840 410,840 ~ ~ ~ p=1.000 n=6
Memory used 1,224,567k (± 0.00%) 1,224,624k (± 0.00%) +57k (+ 0.00%) 1,224,562k 1,224,653k p=0.031 n=6
Parse Time 6.67s (± 0.63%) 6.62s (± 0.62%) -0.06s (- 0.87%) 6.59s 6.69s p=0.043 n=6
Bind Time 1.88s (± 0.40%) 1.88s (± 0.29%) ~ 1.88s 1.89s p=0.476 n=6
Check Time 31.82s (± 0.53%) 31.88s (± 0.29%) ~ 31.73s 31.98s p=0.809 n=6
Emit Time 15.17s (± 0.26%) 15.19s (± 0.38%) ~ 15.10s 15.25s p=0.629 n=6
Total Time 55.54s (± 0.31%) 55.57s (± 0.28%) ~ 55.30s 55.78s p=0.748 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,494,498 2,494,498 ~ ~ ~ p=1.000 n=6
Types 908,298 908,298 ~ ~ ~ p=1.000 n=6
Memory used 2,307,104k (± 0.00%) 2,307,074k (± 0.00%) ~ 2,307,036k 2,307,113k p=0.298 n=6
Parse Time 9.31s (± 0.16%) 9.32s (± 0.21%) ~ 9.30s 9.35s p=0.325 n=6
Bind Time 2.14s (± 0.35%) 2.15s (± 0.39%) ~ 2.13s 2.15s p=0.432 n=6
Check Time 74.76s (± 0.33%) 75.09s (± 0.12%) +0.33s (+ 0.44%) 74.97s 75.21s p=0.013 n=6
Emit Time 0.28s (± 4.71%) 0.28s (± 4.27%) ~ 0.27s 0.30s p=0.865 n=6
Total Time 86.49s (± 0.28%) 86.84s (± 0.10%) +0.35s (+ 0.41%) 86.71s 86.92s p=0.013 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,258,033 1,258,041 +8 (+ 0.00%) ~ ~ p=0.001 n=6
Types 266,263 266,229 -34 (- 0.01%) ~ ~ p=0.001 n=6
Memory used 3,146,384k (± 0.01%) 3,146,885k (± 0.02%) ~ 3,145,940k 3,147,625k p=0.230 n=6
Parse Time 6.57s (± 0.59%) 6.62s (± 0.36%) +0.05s (+ 0.76%) 6.59s 6.64s p=0.030 n=6
Bind Time 2.34s (± 1.14%) 2.33s (± 0.97%) ~ 2.30s 2.36s p=0.748 n=6
Check Time 43.12s (± 0.16%) 42.98s (± 0.42%) ~ 42.81s 43.26s p=0.173 n=6
Emit Time 3.54s (± 1.57%) 3.50s (± 2.70%) ~ 3.37s 3.63s p=0.575 n=6
Total Time 55.58s (± 0.09%) 55.44s (± 0.30%) ~ 55.24s 55.68s p=0.128 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,258,033 1,258,041 +8 (+ 0.00%) ~ ~ p=0.001 n=6
Types 266,263 266,229 -34 (- 0.01%) ~ ~ p=0.001 n=6
Memory used 2,496,230k (± 0.01%) 2,496,475k (± 0.03%) ~ 2,495,703k 2,497,440k p=1.000 n=6
Parse Time 5.38s (± 0.92%) 5.40s (± 0.68%) ~ 5.34s 5.45s p=0.335 n=6
Bind Time 1.74s (± 0.47%) 1.74s (± 0.32%) ~ 1.73s 1.74s p=0.859 n=6
Check Time 35.85s (± 0.33%) 35.83s (± 0.15%) ~ 35.73s 35.88s p=1.000 n=6
Emit Time 3.02s (± 1.49%) 3.01s (± 1.79%) ~ 2.94s 3.10s p=0.470 n=6
Total Time 45.99s (± 0.39%) 45.98s (± 0.20%) ~ 45.88s 46.08s p=1.000 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 261,789 261,753 -36 (- 0.01%) ~ ~ p=0.001 n=6
Types 106,511 106,477 -34 (- 0.03%) ~ ~ p=0.001 n=6
Memory used 438,818k (± 0.01%) 438,702k (± 0.02%) -116k (- 0.03%) 438,565k 438,743k p=0.008 n=6
Parse Time 3.52s (± 0.91%) 3.50s (± 1.12%) ~ 3.46s 3.57s p=0.419 n=6
Bind Time 1.30s (± 0.64%) 1.32s (± 1.66%) ~ 1.30s 1.36s p=0.128 n=6
Check Time 18.85s (± 0.50%) 18.86s (± 0.24%) ~ 18.80s 18.90s p=0.686 n=6
Emit Time 1.54s (± 1.82%) 1.53s (± 1.04%) ~ 1.50s 1.54s p=0.114 n=6
Total Time 25.22s (± 0.31%) 25.21s (± 0.24%) ~ 25.16s 25.33s p=0.810 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 68 68 ~ ~ ~ p=1.000 n=6
Symbols 225,919 225,919 ~ ~ ~ p=1.000 n=6
Types 94,415 94,415 ~ ~ ~ p=1.000 n=6
Memory used 371,094k (± 0.01%) 371,080k (± 0.01%) ~ 371,056k 371,125k p=0.173 n=6
Parse Time 2.90s (± 1.37%) 2.90s (± 1.35%) ~ 2.84s 2.95s p=0.936 n=6
Bind Time 1.58s (± 1.68%) 1.60s (± 0.68%) ~ 1.58s 1.61s p=0.145 n=6
Check Time 16.38s (± 0.36%) 16.39s (± 0.28%) ~ 16.30s 16.43s p=0.335 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.86s (± 0.34%) 20.89s (± 0.21%) ~ 20.84s 20.94s p=0.333 n=6
vscode - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 3,126,375 3,126,375 ~ ~ ~ p=1.000 n=6
Types 1,077,686 1,077,686 ~ ~ ~ p=1.000 n=6
Memory used 3,218,522k (± 0.01%) 3,218,284k (± 0.01%) ~ 3,217,889k 3,218,838k p=0.471 n=6
Parse Time 14.13s (± 0.31%) 14.10s (± 0.16%) ~ 14.08s 14.13s p=0.146 n=6
Bind Time 4.47s (± 0.24%) 4.47s (± 0.31%) ~ 4.45s 4.49s p=0.504 n=6
Check Time 86.70s (± 2.31%) 85.38s (± 1.62%) ~ 83.82s 87.94s p=0.298 n=6
Emit Time 26.91s (± 3.42%) 26.73s (± 4.48%) ~ 25.03s 28.46s p=0.574 n=6
Total Time 132.21s (± 1.80%) 130.67s (± 1.29%) ~ 128.87s 133.44s p=0.298 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 286,375 286,375 ~ ~ ~ p=1.000 n=6
Types 116,078 116,078 ~ ~ ~ p=1.000 n=6
Memory used 437,033k (± 0.01%) 437,189k (± 0.03%) +156k (+ 0.04%) 437,003k 437,335k p=0.031 n=6
Parse Time 5.05s (± 1.22%) 5.05s (± 1.05%) ~ 4.98s 5.10s p=0.872 n=6
Bind Time 2.16s (± 1.29%) 2.18s (± 1.31%) ~ 2.15s 2.23s p=0.625 n=6
Check Time 22.88s (± 0.30%) 22.98s (± 0.58%) ~ 22.77s 23.10s p=0.199 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 30.10s (± 0.39%) 30.21s (± 0.53%) ~ 30.01s 30.35s p=0.172 n=6
xstate-main - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 543,130 543,130 ~ ~ ~ p=1.000 n=6
Types 181,889 181,889 ~ ~ ~ p=1.000 n=6
Memory used 485,509k (± 0.01%) 485,527k (± 0.01%) ~ 485,479k 485,564k p=0.378 n=6
Parse Time 4.17s (± 1.00%) 4.17s (± 0.25%) ~ 4.15s 4.18s p=0.807 n=6
Bind Time 1.46s (± 1.14%) 1.45s (± 1.07%) ~ 1.43s 1.47s p=0.282 n=6
Check Time 23.70s (± 0.29%) 23.69s (± 0.32%) ~ 23.60s 23.81s p=0.687 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 29.34s (± 0.28%) 29.30s (± 0.28%) ~ 29.23s 29.45s p=0.378 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

@typescript-bot
Copy link
Collaborator

@jakebailey Here are the results of running the top 400 repos with tsc comparing main and refs/pull/60321/merge:

Everything looks good!

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

Labels

Author: Team For Milestone Bug PRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implicit Symbol.iterator call does not check additional arguments

5 participants