Skip to content

Reduce allocations under CodeFixService.GetShouldIncludeDiagnosticPredicate#82492

Merged
ToddGrun merged 2 commits into
dotnet:mainfrom
ToddGrun:dev/toddgrun/CodeFixServiceAllocations
Feb 24, 2026
Merged

Reduce allocations under CodeFixService.GetShouldIncludeDiagnosticPredicate#82492
ToddGrun merged 2 commits into
dotnet:mainfrom
ToddGrun:dev/toddgrun/CodeFixServiceAllocations

Conversation

@ToddGrun

@ToddGrun ToddGrun commented Feb 22, 2026

Copy link
Copy Markdown
Contributor

Populating both an ImmutableDictionary and an ImmutableHashSet both show up in the LightBulbInvocationTestForCSharp speedometer profile I'm looking at, accounting for about 5 MB (0.5%) of allocations.

*** Before change ***
image

*** With change ***
image

…dicate

Populating both an ImmutableDictionary and an ImmutableHashSet both show up in the LightBulbInvocationTestForCSharp speedometer profile I'm looking at, accounting for about 5 MB (0.5%) of allocations.

In draft mode until I get back speedometer results.
@ToddGrun

Copy link
Copy Markdown
Contributor Author

/pr-val

@github-actions

Copy link
Copy Markdown
Contributor

View PR Validation Run triggered by @ToddGrun

Parameters
  • Validation Type: pr-val
  • Pipeline ID: 8972
  • Pipeline Version: main
  • PR Number: 82492
  • Commit SHA: 4f65395f082521c2b6fa6d614580e77a88d05b4b
  • Source Branch: dev/toddgrun/CodeFixServiceAllocations
  • Target Branch: main
  • Build ID: 13367472

@ToddGrun ToddGrun marked this pull request as ready for review February 22, 2026 21:39
@ToddGrun ToddGrun requested a review from a team as a code owner February 22, 2026 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants