Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make virtual delegate targets less costly #87308

Merged
merged 3 commits into from
Jun 16, 2023

Commits on Jun 9, 2023

  1. Make virtual delegate targets less costly

    Saves 1.5% on the Stage2 app.
    
    We make sure delegate targets are reflection visible to support Delegate.GetMethodInfo. When I originally did this work in dotnet#70198 I made a shortcut to avoid yet another node kind in the system (with a giant comment explaining what the problem is). But there's a lot of benefit in having this new node for apps with many reflection visible virtual methods.
    MichalStrehovsky committed Jun 9, 2023
    Configuration menu
    Copy the full SHA
    6d8fc12 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a5d58a6 View commit details
    Browse the repository at this point in the history

Commits on Jun 12, 2023

  1. Fix failing test

    Add a variation of the test which actually keeps the method
    vitek-karas committed Jun 12, 2023
    Configuration menu
    Copy the full SHA
    8d7822a View commit details
    Browse the repository at this point in the history