Skip to content

GeneratorFunction should not be newable#56646

Open
LongTengDao wants to merge 2 commits intomicrosoft:mainfrom
LongTengDao-fork-D:main
Open

GeneratorFunction should not be newable#56646
LongTengDao wants to merge 2 commits intomicrosoft:mainfrom
LongTengDao-fork-D:main

Conversation

@LongTengDao
Copy link
Contributor

Fixes #56645

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Dec 2, 2023
@sandersn sandersn requested a review from rbuckton December 12, 2023 17:07
@RyanCavanaugh
Copy link
Member

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 24, 2026

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

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

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

There were infrastructure failures potentially unrelated to your change:

  • 1 instance of "Git clone failed"

Otherwise...

Everything looks good!

@RyanCavanaugh RyanCavanaugh added this to the Post-7.0 lib candidates milestone Mar 24, 2026
@typescript-bot
Copy link
Collaborator

@RyanCavanaugh
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 277,173k (± 0.51%) 276,681k (± 0.64%) ~ 274,360k 277,972k p=0.936 n=6
Parse Time 1.61s (± 1.02%) 1.61s (± 0.93%) ~ 1.59s 1.63s p=0.508 n=6
Bind Time 0.92s (± 1.27%) 0.92s (± 1.15%) ~ 0.90s 0.93s p=0.619 n=6
Check Time 17.87s (± 1.88%) 17.69s (± 2.44%) ~ 17.05s 18.40s p=0.688 n=6
Emit Time 3.35s (± 4.94%) 3.26s (± 0.90%) ~ 3.22s 3.30s p=0.106 n=6
Total Time 23.75s (± 1.25%) 23.49s (± 1.96%) ~ 22.78s 24.22s p=0.378 n=6
angular-1 - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 959,140 959,136 -4 (- 0.00%) ~ ~ p=0.001 n=6
Types 416,033 416,033 ~ ~ ~ p=1.000 n=6
Memory used 1,261,071k (± 0.00%) 1,261,018k (± 0.00%) -53k (- 0.00%) 1,260,948k 1,261,054k p=0.045 n=6
Parse Time 8.02s (± 0.79%) 8.07s (± 0.35%) ~ 8.02s 8.09s p=0.223 n=6
Bind Time 2.37s (± 0.64%) 2.37s (± 0.43%) ~ 2.36s 2.39s p=0.440 n=6
Check Time 38.78s (± 0.18%) 38.63s (± 0.32%) -0.15s (- 0.39%) 38.51s 38.82s p=0.045 n=6
Emit Time 18.00s (± 0.46%) 18.00s (± 0.48%) ~ 17.88s 18.08s p=1.000 n=6
Total Time 67.17s (± 0.19%) 67.07s (± 0.28%) ~ 66.83s 67.36s p=0.298 n=6
mui-docs - node (v18.15.0, x64)
Errors 11,129 11,129 ~ ~ ~ p=1.000 n=6
Symbols 2,209,050 2,209,046 -4 (- 0.00%) ~ ~ p=0.001 n=6
Types 767,921 767,921 ~ ~ ~ p=1.000 n=6
Memory used 2,656,208k (± 0.00%) 2,656,288k (± 0.00%) ~ 2,656,149k 2,656,401k p=0.378 n=6
Parse Time 9.48s (± 0.50%) 9.45s (± 0.56%) ~ 9.37s 9.52s p=0.377 n=6
Bind Time 2.51s (± 0.92%) 2.54s (± 0.54%) ~ 2.52s 2.55s p=0.089 n=6
Check Time 89.58s (± 2.47%) 87.90s (± 1.38%) ~ 86.96s 89.48s p=0.230 n=6
Emit Time 0.37s (± 2.02%) 0.37s (± 3.96%) ~ 0.35s 0.39s p=0.934 n=6
Total Time 101.94s (± 2.18%) 100.24s (± 1.22%) ~ 99.24s 101.84s p=0.230 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,223,006 1,222,954 -52 (- 0.00%) ~ ~ p=0.001 n=6
Types 267,163 267,163 ~ ~ ~ p=1.000 n=6
Memory used 2,590,471k (± 7.23%) 2,711,946k (± 0.03%) ~ 2,710,548k 2,712,905k p=0.128 n=6
Parse Time 6.56s (± 1.40%) 6.57s (± 0.57%) ~ 6.53s 6.63s p=0.689 n=6
Bind Time 2.27s (± 0.99%) 2.25s (± 1.14%) ~ 2.21s 2.27s p=0.221 n=6
Check Time 43.08s (± 1.12%) 42.83s (± 0.76%) ~ 42.51s 43.32s p=0.423 n=6
Emit Time 3.60s (± 1.96%) 3.52s (± 3.27%) ~ 3.34s 3.64s p=0.378 n=6
Total Time 55.50s (± 0.72%) 55.19s (± 0.84%) ~ 54.63s 55.86s p=0.173 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,223,006 1,222,954 -52 (- 0.00%) ~ ~ p=0.001 n=6
Types 267,163 267,163 ~ ~ ~ p=1.000 n=6
Memory used 2,804,534k (± 2.30%) 2,717,338k (± 5.44%) ~ 2,415,048k 2,778,569k p=0.128 n=6
Parse Time 6.71s (± 0.67%) 6.77s (± 0.63%) +0.06s (+ 0.87%) 6.72s 6.82s p=0.020 n=6
Bind Time 2.26s (± 1.00%) 2.26s (± 1.42%) ~ 2.22s 2.31s p=0.688 n=6
Check Time 42.64s (± 0.24%) 42.82s (± 1.00%) ~ 42.53s 43.65s p=0.575 n=6
Emit Time 3.58s (± 4.77%) 3.54s (± 2.52%) ~ 3.43s 3.70s p=0.630 n=6
Total Time 55.19s (± 0.42%) 55.40s (± 0.73%) ~ 55.01s 56.18s p=0.230 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 261,151 261,147 -4 (- 0.00%) ~ ~ p=0.001 n=6
Types 104,322 104,322 ~ ~ ~ p=1.000 n=6
Memory used 438,747k (± 0.01%) 438,702k (± 0.01%) ~ 438,670k 438,772k p=0.173 n=6
Parse Time 3.53s (± 1.22%) 3.49s (± 1.16%) ~ 3.44s 3.55s p=0.092 n=6
Bind Time 1.34s (± 0.78%) 1.36s (± 1.04%) ~ 1.34s 1.38s p=0.086 n=6
Check Time 19.27s (± 0.44%) 19.32s (± 0.59%) ~ 19.16s 19.49s p=0.471 n=6
Emit Time 1.54s (± 0.89%) 1.55s (± 0.71%) ~ 1.53s 1.56s p=0.491 n=6
Total Time 25.69s (± 0.29%) 25.72s (± 0.45%) ~ 25.58s 25.90s p=0.809 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 271 271 ~ ~ ~ p=1.000 n=6
Symbols 225,897 225,893 -4 (- 0.00%) ~ ~ p=0.001 n=6
Types 93,564 93,564 ~ ~ ~ p=1.000 n=6
Memory used 371,623k (± 0.01%) 371,623k (± 0.03%) ~ 371,513k 371,790k p=0.689 n=6
Parse Time 2.86s (± 1.14%) 2.86s (± 0.96%) ~ 2.82s 2.90s p=0.936 n=6
Bind Time 1.62s (± 1.20%) 1.64s (± 1.94%) ~ 1.60s 1.69s p=0.411 n=6
Check Time 17.45s (± 0.50%) 17.49s (± 0.38%) ~ 17.39s 17.56s p=0.377 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 21.93s (± 0.43%) 21.99s (± 0.36%) ~ 21.89s 22.09s p=0.378 n=6
vscode - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=NaN n=0
Symbols 0 0 ~ ~ ~ p=NaN n=0
Types 0 0 ~ ~ ~ p=NaN n=0
Memory used 0k 0k ~ ~ ~ p=NaN n=0
Parse Time 0s 0s ~ ~ ~ p=NaN n=0
Bind Time 0s 0s ~ ~ ~ p=NaN n=0
Check Time 0s 0s ~ ~ ~ p=NaN n=0
Emit Time 0s 0s ~ ~ ~ p=NaN n=0
Total Time 0s 0s ~ ~ ~ p=NaN n=0
webpack - node (v18.15.0, x64)
Errors 8 8 ~ ~ ~ p=1.000 n=6
Symbols 408,831 408,827 -4 (- 0.00%) ~ ~ p=0.001 n=6
Types 186,899 186,899 ~ ~ ~ p=1.000 n=6
Memory used 570,936k (± 0.02%) 570,895k (± 0.01%) ~ 570,770k 570,957k p=0.688 n=6
Parse Time 4.74s (± 0.29%) 4.73s (± 0.59%) ~ 4.67s 4.75s p=0.932 n=6
Bind Time 2.03s (± 1.41%) 2.04s (± 1.18%) ~ 2.01s 2.07s p=0.629 n=6
Check Time 24.44s (± 0.52%) 24.51s (± 0.29%) ~ 24.44s 24.61s p=0.295 n=6
Emit Time 0.00s (±244.70%) 0.00s (±154.76%) ~ 0.00s 0.01s p=0.595 n=6
Total Time 31.21s (± 0.41%) 31.29s (± 0.27%) ~ 31.19s 31.38s p=0.470 n=6
xstate-main - node (v18.15.0, x64)
Errors 30 30 ~ ~ ~ p=1.000 n=6
Symbols 781,035 781,031 -4 (- 0.00%) ~ ~ p=0.001 n=6
Types 227,795 227,795 ~ ~ ~ p=1.000 n=6
Memory used 697,228k (± 0.01%) 697,301k (± 0.02%) ~ 697,212k 697,549k p=0.378 n=6
Parse Time 4.93s (± 1.02%) 4.93s (± 0.49%) ~ 4.89s 4.96s p=0.572 n=6
Bind Time 1.58s (± 0.62%) 1.59s (± 0.47%) ~ 1.58s 1.60s p=0.102 n=6
Check Time 23.12s (± 0.43%) 23.09s (± 0.23%) ~ 23.01s 23.16s p=0.936 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 29.63s (± 0.41%) 29.61s (± 0.13%) ~ 29.57s 29.68s p=1.000 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

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

Everything looks good!

@RyanCavanaugh RyanCavanaugh removed the request for review from rbuckton March 25, 2026 15:59
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: Waiting on author

Development

Successfully merging this pull request may close these issues.

GeneratorFunction should not be newable

5 participants