Skip to content

GeneratorFunction should not be newable#56646

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

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

Conversation

@LongTengDao

Copy link
Copy Markdown
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
Copy Markdown
Member

@typescript-bot test it

@typescript-bot

typescript-bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

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
Copy Markdown
Contributor

@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
Copy Markdown
Contributor

@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
Copy Markdown
Contributor

@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

5 participants