Migrate GC benchmarks to .NET10 runtime#5235
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the GC benchmark projects and a subset of GC infrastructure configuration to target the .NET 10 runtime, aligning the GC benchmark suite with newer runtime tooling.
Changes:
- Retarget GCPerfSim and GC.Infrastructure projects (including test projects) from net7.0/net8.0/net9.0 to net10.0.
- Update selected GC infrastructure YAML configurations to reference the net10.0 GCPerfSim build output.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/benchmarks/gc/GCPerfSim/GCPerfSim.csproj | Retarget GCPerfSim to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Infrastructure/GC.Infrastructure.csproj | Retarget GC.Infrastructure CLI/app to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Infrastructure.NotebookTests/GC.Infrastructure.NotebookTests.csproj | Retarget notebook test project to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Infrastructure.MCPServer/GC.Infrastructure.MCPServer.csproj | Retarget MCP server project to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Infrastructure.Core/GC.Infrastructure.Core.csproj | Retarget core library to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Infrastructure.Core.UnitTests/GC.Infrastructure.Core.UnitTests.csproj | Retarget core unit tests to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Analysis.API/GC.Analysis.API.csproj | Retarget analysis API library to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/GC.Analysis.API.UnitTests/GC.Analysis.API.UnitTests.csproj | Retarget analysis API unit tests to net10.0. |
| src/benchmarks/gc/GC.Infrastructure/Configurations/Run.yaml | Update config to point at net10.0 GCPerfSim artifact path. |
| src/benchmarks/gc/GC.Infrastructure/Configurations/GCPerfSim/LowVolatilityRuns.yaml | Update config to point at net10.0 GCPerfSim artifact path. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The benchmarks / infra was using .NET 7 and .NET 8, this change moves them all to .NET 10
68c005b to
32870ea
Compare
mrsharm
approved these changes
Jun 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The benchmarks / infra was using .NET 7 and .NET 8, this change moves them all to .NET 10