Skip to content

Conversation

@hez2010
Copy link
Contributor

@hez2010 hez2010 commented Feb 12, 2025

Now that we have late devirt inlining, try #100041 again.
Depends on #112396 for the correctness fix.

cc: @MichalPetryka

@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Feb 12, 2025
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Feb 12, 2025
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

@hez2010
Copy link
Contributor Author

hez2010 commented Feb 12, 2025

@MihuBot

@hez2010
Copy link
Contributor Author

hez2010 commented Feb 12, 2025

We replaced the context which caused huge missed context, so spmi diff would not be meaningful.
While from MihuBot it doesn't seem to have meaningful diffs, which shares the same observation in #100041, closing for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants