Skip to content

feat: add StaticPropertyInitializationGeneratorBenchmarks - #4639

Merged
thomhurst merged 1 commit into
thomhurst:mainfrom
TimothyMakkison:static_prop_init_generator_bench
Feb 1, 2026
Merged

feat: add StaticPropertyInitializationGeneratorBenchmarks#4639
thomhurst merged 1 commit into
thomhurst:mainfrom
TimothyMakkison:static_prop_init_generator_bench

Conversation

@TimothyMakkison

Copy link
Copy Markdown
Contributor

Add StaticPropertyInitializationGeneratorBenchmarks

MethodMeanErrorStdDevMedianGen0Gen1Allocated
RunGenerator183.7 ms6.12 ms16.66 ms176.5 ms10000.00004000.000099.35 MB

@TimothyMakkison

TimothyMakkison commented Feb 1, 2026

Copy link
Copy Markdown
ContributorAuthor

After rebasing, these are the new benchmarks. Were more tests that used static property initializers added recently?

MethodMeanErrorStdDevGen0Gen1Allocated
RunGenerator290.6 ms5.78 ms10.42 ms11000.00003000.0000102.34 MB

@thomhurst

Copy link
Copy Markdown
Owner

@TimothyMakkison nope! Might be environment rated?

@TimothyMakkison

TimothyMakkison commented Feb 1, 2026

Copy link
Copy Markdown
ContributorAuthor

@TimothyMakkison nope! Might be environment rated?

Porbably! I reran the benchmarks under similar conditions on older commits and got the same results of around 290ms. Seems like I had a weird fluke earlier

@thomhurst
thomhurst merged commit 5f682ff into thomhurst:mainFeb 1, 2026
8 of 10 checks passed
This was referenced Feb 3, 2026
Sign up for freeto 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.

2 participants

@TimothyMakkison@thomhurst