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-bottypescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Dec 2, 2023
@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.

CommandStatusResults
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!

@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
MetricbaselineprDeltaBestWorstp-value
Compiler-Unions - node (v18.15.0, x64)
Errors33~~~p=1.000 n=6
Symbols82,52182,521~~~p=1.000 n=6
Types125,280125,280~~~p=1.000 n=6
Memory used277,173k (± 0.51%)276,681k (± 0.64%)~274,360k277,972kp=0.936 n=6
Parse Time1.61s (± 1.02%)1.61s (± 0.93%)~1.59s1.63sp=0.508 n=6
Bind Time0.92s (± 1.27%)0.92s (± 1.15%)~0.90s0.93sp=0.619 n=6
Check Time17.87s (± 1.88%)17.69s (± 2.44%)~17.05s18.40sp=0.688 n=6
Emit Time3.35s (± 4.94%)3.26s (± 0.90%)~3.22s3.30sp=0.106 n=6
Total Time23.75s (± 1.25%)23.49s (± 1.96%)~22.78s24.22sp=0.378 n=6
angular-1 - node (v18.15.0, x64)
Errors33~~~p=1.000 n=6
Symbols959,140959,136-4 (- 0.00%)~~p=0.001 n=6
Types416,033416,033~~~p=1.000 n=6
Memory used1,261,071k (± 0.00%)1,261,018k (± 0.00%)-53k (- 0.00%)1,260,948k1,261,054kp=0.045 n=6
Parse Time8.02s (± 0.79%)8.07s (± 0.35%)~8.02s8.09sp=0.223 n=6
Bind Time2.37s (± 0.64%)2.37s (± 0.43%)~2.36s2.39sp=0.440 n=6
Check Time38.78s (± 0.18%)38.63s (± 0.32%)-0.15s (- 0.39%)38.51s38.82sp=0.045 n=6
Emit Time18.00s (± 0.46%)18.00s (± 0.48%)~17.88s18.08sp=1.000 n=6
Total Time67.17s (± 0.19%)67.07s (± 0.28%)~66.83s67.36sp=0.298 n=6
mui-docs - node (v18.15.0, x64)
Errors11,12911,129~~~p=1.000 n=6
Symbols2,209,0502,209,046-4 (- 0.00%)~~p=0.001 n=6
Types767,921767,921~~~p=1.000 n=6
Memory used2,656,208k (± 0.00%)2,656,288k (± 0.00%)~2,656,149k2,656,401kp=0.378 n=6
Parse Time9.48s (± 0.50%)9.45s (± 0.56%)~9.37s9.52sp=0.377 n=6
Bind Time2.51s (± 0.92%)2.54s (± 0.54%)~2.52s2.55sp=0.089 n=6
Check Time89.58s (± 2.47%)87.90s (± 1.38%)~86.96s89.48sp=0.230 n=6
Emit Time0.37s (± 2.02%)0.37s (± 3.96%)~0.35s0.39sp=0.934 n=6
Total Time101.94s (± 2.18%)100.24s (± 1.22%)~99.24s101.84sp=0.230 n=6
self-build-src - node (v18.15.0, x64)
Errors00~~~p=1.000 n=6
Symbols1,223,0061,222,954-52 (- 0.00%)~~p=0.001 n=6
Types267,163267,163~~~p=1.000 n=6
Memory used2,590,471k (± 7.23%)2,711,946k (± 0.03%)~2,710,548k2,712,905kp=0.128 n=6
Parse Time6.56s (± 1.40%)6.57s (± 0.57%)~6.53s6.63sp=0.689 n=6
Bind Time2.27s (± 0.99%)2.25s (± 1.14%)~2.21s2.27sp=0.221 n=6
Check Time43.08s (± 1.12%)42.83s (± 0.76%)~42.51s43.32sp=0.423 n=6
Emit Time3.60s (± 1.96%)3.52s (± 3.27%)~3.34s3.64sp=0.378 n=6
Total Time55.50s (± 0.72%)55.19s (± 0.84%)~54.63s55.86sp=0.173 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors00~~~p=1.000 n=6
Symbols1,223,0061,222,954-52 (- 0.00%)~~p=0.001 n=6
Types267,163267,163~~~p=1.000 n=6
Memory used2,804,534k (± 2.30%)2,717,338k (± 5.44%)~2,415,048k2,778,569kp=0.128 n=6
Parse Time6.71s (± 0.67%)6.77s (± 0.63%)+0.06s (+ 0.87%)6.72s6.82sp=0.020 n=6
Bind Time2.26s (± 1.00%)2.26s (± 1.42%)~2.22s2.31sp=0.688 n=6
Check Time42.64s (± 0.24%)42.82s (± 1.00%)~42.53s43.65sp=0.575 n=6
Emit Time3.58s (± 4.77%)3.54s (± 2.52%)~3.43s3.70sp=0.630 n=6
Total Time55.19s (± 0.42%)55.40s (± 0.73%)~55.01s56.18sp=0.230 n=6
self-compiler - node (v18.15.0, x64)
Errors00~~~p=1.000 n=6
Symbols261,151261,147-4 (- 0.00%)~~p=0.001 n=6
Types104,322104,322~~~p=1.000 n=6
Memory used438,747k (± 0.01%)438,702k (± 0.01%)~438,670k438,772kp=0.173 n=6
Parse Time3.53s (± 1.22%)3.49s (± 1.16%)~3.44s3.55sp=0.092 n=6
Bind Time1.34s (± 0.78%)1.36s (± 1.04%)~1.34s1.38sp=0.086 n=6
Check Time19.27s (± 0.44%)19.32s (± 0.59%)~19.16s19.49sp=0.471 n=6
Emit Time1.54s (± 0.89%)1.55s (± 0.71%)~1.53s1.56sp=0.491 n=6
Total Time25.69s (± 0.29%)25.72s (± 0.45%)~25.58s25.90sp=0.809 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors271271~~~p=1.000 n=6
Symbols225,897225,893-4 (- 0.00%)~~p=0.001 n=6
Types93,56493,564~~~p=1.000 n=6
Memory used371,623k (± 0.01%)371,623k (± 0.03%)~371,513k371,790kp=0.689 n=6
Parse Time2.86s (± 1.14%)2.86s (± 0.96%)~2.82s2.90sp=0.936 n=6
Bind Time1.62s (± 1.20%)1.64s (± 1.94%)~1.60s1.69sp=0.411 n=6
Check Time17.45s (± 0.50%)17.49s (± 0.38%)~17.39s17.56sp=0.377 n=6
Emit Time0.00s0.00s~~~p=1.000 n=6
Total Time21.93s (± 0.43%)21.99s (± 0.36%)~21.89s22.09sp=0.378 n=6
vscode - node (v18.15.0, x64)
Errors00~~~p=NaN n=0
Symbols00~~~p=NaN n=0
Types00~~~p=NaN n=0
Memory used0k0k~~~p=NaN n=0
Parse Time0s0s~~~p=NaN n=0
Bind Time0s0s~~~p=NaN n=0
Check Time0s0s~~~p=NaN n=0
Emit Time0s0s~~~p=NaN n=0
Total Time0s0s~~~p=NaN n=0
webpack - node (v18.15.0, x64)
Errors88~~~p=1.000 n=6
Symbols408,831408,827-4 (- 0.00%)~~p=0.001 n=6
Types186,899186,899~~~p=1.000 n=6
Memory used570,936k (± 0.02%)570,895k (± 0.01%)~570,770k570,957kp=0.688 n=6
Parse Time4.74s (± 0.29%)4.73s (± 0.59%)~4.67s4.75sp=0.932 n=6
Bind Time2.03s (± 1.41%)2.04s (± 1.18%)~2.01s2.07sp=0.629 n=6
Check Time24.44s (± 0.52%)24.51s (± 0.29%)~24.44s24.61sp=0.295 n=6
Emit Time0.00s (±244.70%)0.00s (±154.76%)~0.00s0.01sp=0.595 n=6
Total Time31.21s (± 0.41%)31.29s (± 0.27%)~31.19s31.38sp=0.470 n=6
xstate-main - node (v18.15.0, x64)
Errors3030~~~p=1.000 n=6
Symbols781,035781,031-4 (- 0.00%)~~p=0.001 n=6
Types227,795227,795~~~p=1.000 n=6
Memory used697,228k (± 0.01%)697,301k (± 0.02%)~697,212k697,549kp=0.378 n=6
Parse Time4.93s (± 1.02%)4.93s (± 0.49%)~4.89s4.96sp=0.572 n=6
Bind Time1.58s (± 0.62%)1.59s (± 0.47%)~1.58s1.60sp=0.102 n=6
Check Time23.12s (± 0.43%)23.09s (± 0.23%)~23.01s23.16sp=0.936 n=6
Emit Time0.00s0.00s~~~p=1.000 n=6
Total Time29.63s (± 0.41%)29.61s (± 0.13%)~29.57s29.68sp=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)
BenchmarkNameIterations
Currentpr6
Baselinebaseline6

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 rbucktonMarch 25, 2026 15:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For Uncommitted BugPR 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

@LongTengDao@RyanCavanaugh@typescript-bot@sandersn@rbuckton