Skip to content

Turn off slow analyzers to measure roslyn perf regression impact#1973

Closed
bkoelman wants to merge 1 commit into
masterfrom
roslyn-no-slow-analyzers
Closed

Turn off slow analyzers to measure roslyn perf regression impact#1973
bkoelman wants to merge 1 commit into
masterfrom
roslyn-no-slow-analyzers

Conversation

@bkoelman
Copy link
Copy Markdown
Member

@bkoelman bkoelman commented Apr 11, 2026

Turn off slow analyzers in Release builds to measure impact (using .NET SDK v10.0.201).

Removed the condition for the DisableSlowAnalyzers target to always execute before CoreCompile.
@codecov

This comment was marked as off-topic.

@bkoelman bkoelman changed the title Modify DisableSlowAnalyzers target execution condition Turn off slow analyzers to measure roslyn perf regression Apr 11, 2026
@bkoelman
Copy link
Copy Markdown
Member Author

bkoelman commented Apr 11, 2026

Baseline results at https://github.com/json-api-dotnet/JsonApiDotNetCore/actions/runs/24285202718/job/70913314037:

Ubuntu: 17m 37s
Windows: 20m 40s
macOS: 13m 21s

Results from this PR (slow analyzers off) at https://github.com/json-api-dotnet/JsonApiDotNetCore/actions/runs/24289878942/job/70925386527:

Ubuntu: 2m 20s
Windows: 3m 31s
macOS: 2m 18s

@bkoelman bkoelman changed the title Turn off slow analyzers to measure roslyn perf regression Turn off slow analyzers to measure roslyn perf regression impact Apr 11, 2026
@bkoelman bkoelman closed this Apr 16, 2026
@bkoelman bkoelman deleted the roslyn-no-slow-analyzers branch April 16, 2026 00:59
@bkoelman bkoelman restored the roslyn-no-slow-analyzers branch May 17, 2026 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant