Skip to content

Backport tsgo PR 3301 for testing#63328

Closed
jakebailey wants to merge 1 commit intomicrosoft:mainfrom
jakebailey:port-tsgo3301
Closed

Backport tsgo PR 3301 for testing#63328
jakebailey wants to merge 1 commit intomicrosoft:mainfrom
jakebailey:port-tsgo3301

Conversation

@jakebailey
Copy link
Copy Markdown
Member

Want to check perf on microsoft/typescript-go#3301

@jakebailey
Copy link
Copy Markdown
Member Author

@typescript-bot test it

@typescript-bot
Copy link
Copy Markdown
Collaborator

typescript-bot commented Mar 31, 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
Copy Markdown
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
Copy Markdown
Collaborator

@jakebailey Here are the results of running the user tests with tsc comparing main and refs/pull/63328/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
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 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 275,071k (± 0.49%) 276,200k (± 0.69%) ~ 274,404k 277,994k p=0.378 n=6
Parse Time 1.30s (± 0.79%) 1.31s (± 0.31%) ~ 1.30s 1.31s p=0.461 n=6
Bind Time 0.75s (± 1.00%) 0.75s ~ ~ ~ p=0.598 n=6
Check Time 14.16s (± 0.47%) 14.12s (± 0.25%) ~ 14.06s 14.16s p=0.377 n=6
Emit Time 2.66s (± 0.60%) 2.66s (± 0.41%) ~ 2.65s 2.68s p=1.000 n=6
Total Time 18.87s (± 0.32%) 18.84s (± 0.18%) ~ 18.78s 18.87s p=0.293 n=6
angular-1 - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 959,140 959,140 ~ ~ ~ p=1.000 n=6
Types 416,033 416,033 ~ ~ ~ p=1.000 n=6
Memory used 1,261,175k (± 0.00%) 1,261,108k (± 0.00%) -67k (- 0.01%) 1,261,044k 1,261,180k p=0.045 n=6
Parse Time 6.53s (± 0.77%) 6.52s (± 0.59%) ~ 6.50s 6.60s p=0.936 n=6
Bind Time 1.97s (± 0.56%) 1.97s (± 0.21%) ~ 1.96s 1.97s p=0.445 n=6
Check Time 32.45s (± 0.51%) 32.36s (± 0.52%) ~ 32.14s 32.55s p=0.423 n=6
Emit Time 15.08s (± 0.82%) 15.09s (± 0.41%) ~ 15.01s 15.17s p=0.687 n=6
Total Time 56.02s (± 0.47%) 55.94s (± 0.39%) ~ 55.68s 56.29s p=0.689 n=6
mui-docs - node (v18.15.0, x64)
Errors 11,136 11,136 ~ ~ ~ p=1.000 n=6
Symbols 2,425,801 2,425,801 ~ ~ ~ p=1.000 n=6
Types 768,504 768,504 ~ ~ ~ p=1.000 n=6
Memory used 2,753,432k (± 0.00%) 2,753,429k (± 0.00%) ~ 2,753,335k 2,753,494k p=0.810 n=6
Parse Time 7.87s (± 0.13%) 7.86s (± 0.20%) ~ 7.84s 7.88s p=0.458 n=6
Bind Time 2.05s (± 0.50%) 2.04s (± 0.74%) ~ 2.02s 2.06s p=0.346 n=6
Check Time 73.27s (± 0.32%) 73.99s (± 1.48%) ~ 72.72s 75.14s p=0.521 n=6
Emit Time 0.30s (± 1.70%) 0.30s (± 2.52%) ~ 0.29s 0.31s p=0.241 n=6
Total Time 83.49s (± 0.28%) 84.19s (± 1.29%) ~ 82.92s 85.32s p=0.574 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,227,363 1,227,363 ~ ~ ~ p=1.000 n=6
Types 267,198 267,198 ~ ~ ~ p=1.000 n=6
Memory used 2,352,459k (± 0.02%) 2,473,654k (± 7.60%) ~ 2,351,381k 2,717,012k p=0.689 n=6
Parse Time 5.22s (± 0.72%) 5.22s (± 1.10%) ~ 5.13s 5.30s p=1.000 n=6
Bind Time 1.87s (± 1.92%) 1.84s (± 1.01%) ~ 1.82s 1.87s p=0.223 n=6
Check Time 35.82s (± 0.24%) 35.79s (± 0.83%) ~ 35.23s 36.13s p=0.575 n=6
Emit Time 3.08s (± 3.77%) 3.11s (± 3.85%) ~ 2.99s 3.27s p=0.575 n=6
Total Time 46.01s (± 0.47%) 45.97s (± 0.73%) ~ 45.33s 46.26s p=0.936 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,227,363 1,227,363 ~ ~ ~ p=1.000 n=6
Types 267,198 267,198 ~ ~ ~ p=1.000 n=6
Memory used 2,812,056k (± 8.47%) 2,811,949k (± 8.45%) ~ 2,423,268k 3,147,953k p=1.000 n=6
Parse Time 6.78s (± 0.69%) 6.77s (± 0.88%) ~ 6.73s 6.89s p=0.810 n=6
Bind Time 2.27s (± 1.59%) 2.27s (± 1.20%) ~ 2.24s 2.31s p=0.747 n=6
Check Time 42.90s (± 1.26%) 42.96s (± 1.24%) ~ 42.39s 43.79s p=0.936 n=6
Emit Time 3.47s (± 1.72%) 3.57s (± 2.51%) ~ 3.45s 3.69s p=0.066 n=6
Total Time 55.43s (± 1.09%) 55.57s (± 1.08%) ~ 54.91s 56.49s p=0.689 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 261,660 261,660 ~ ~ ~ p=1.000 n=6
Types 104,323 104,323 ~ ~ ~ p=1.000 n=6
Memory used 439,388k (± 0.01%) 439,362k (± 0.01%) ~ 439,325k 439,381k p=0.377 n=6
Parse Time 3.50s (± 0.82%) 3.54s (± 0.77%) ~ 3.50s 3.58s p=0.091 n=6
Bind Time 1.37s (± 1.78%) 1.37s (± 1.03%) ~ 1.35s 1.39s p=1.000 n=6
Check Time 19.28s (± 0.42%) 19.33s (± 0.48%) ~ 19.23s 19.49s p=0.419 n=6
Emit Time 1.54s (± 0.67%) 1.55s (± 1.26%) ~ 1.52s 1.58s p=0.220 n=6
Total Time 25.69s (± 0.38%) 25.79s (± 0.35%) ~ 25.69s 25.93s p=0.107 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 271 271 ~ ~ ~ p=1.000 n=6
Symbols 225,897 225,897 ~ ~ ~ p=1.000 n=6
Types 93,564 93,564 ~ ~ ~ p=1.000 n=6
Memory used 371,631k (± 0.01%) 371,663k (± 0.02%) ~ 371,582k 371,780k p=0.688 n=6
Parse Time 2.86s (± 0.75%) 2.87s (± 0.85%) ~ 2.84s 2.89s p=0.935 n=6
Bind Time 1.63s (± 1.52%) 1.64s (± 1.02%) ~ 1.62s 1.67s p=0.464 n=6
Check Time 17.48s (± 0.56%) 17.49s (± 0.36%) ~ 17.40s 17.55s p=0.936 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 21.97s (± 0.46%) 22.00s (± 0.33%) ~ 21.89s 22.08s p=0.689 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,840 408,840 ~ ~ ~ p=1.000 n=6
Types 186,909 186,909 ~ ~ ~ p=1.000 n=6
Memory used 571,744k (± 0.02%) 571,818k (± 0.02%) ~ 571,714k 571,970k p=0.471 n=6
Parse Time 4.71s (± 0.40%) 4.73s (± 1.04%) ~ 4.67s 4.79s p=0.419 n=6
Bind Time 2.05s (± 0.91%) 2.05s (± 1.18%) ~ 2.03s 2.10s p=0.744 n=6
Check Time 24.43s (± 0.36%) 24.54s (± 0.46%) ~ 24.42s 24.70s p=0.125 n=6
Emit Time 0.01s (±77.38%) 0.00s (±154.76%) ~ 0.00s 0.01s p=0.311 n=6
Total Time 31.19s (± 0.28%) 31.33s (± 0.39%) ~ 31.19s 31.50s p=0.127 n=6
xstate-main - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 782,787 782,787 ~ ~ ~ p=1.000 n=6
Types 228,290 228,290 ~ ~ ~ p=1.000 n=6
Memory used 698,151k (± 0.00%) 698,189k (± 0.01%) ~ 698,097k 698,370k p=0.936 n=6
Parse Time 4.94s (± 0.58%) 4.95s (± 0.58%) ~ 4.91s 4.99s p=0.293 n=6
Bind Time 1.60s (± 0.83%) 1.59s (± 0.76%) ~ 1.58s 1.61s p=0.315 n=6
Check Time 23.04s (± 0.35%) 23.20s (± 1.55%) ~ 22.99s 23.93s p=0.296 n=6
Emit Time 0.00s (±244.70%) 0.00s ~ ~ ~ p=0.405 n=6
Total Time 29.58s (± 0.31%) 29.74s (± 1.27%) ~ 29.54s 30.51s p=0.334 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
Collaborator

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

Something interesting changed - please have a look.

Details

SBoudrias/Inquirer.js

1 of 23 projects failed to build with the old tsc and were ignored

tsconfig.json

packages/testing/tsconfig.json

packages/rawlist/tsconfig.json

packages/prompts/tsconfig.json

packages/inquirer/tsconfig.json

packages/i18n/tsconfig.json

packages/demo/tsconfig.json

@RyanCavanaugh
Copy link
Copy Markdown
Member

Reduced version from that repo

 type MakePartial<T> = T extends object ? { [P in keyof T]?: T[P] } : T;
 
 declare function merge<T extends object>(x: MakePartial<T>, y: MakePartial<T>): T;
 
 declare const a: { fn: () => void };
 declare const b: MakePartial<{ fn: () => void; extra: number }>;
 
 const m = merge(a, b);
 m.fn(); // TS2722: Cannot invoke an object which is possibly 'undefined'.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants