-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Limit native AOT to CORINFO_HELP_ALLOC_CONTINUATION
#121444
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
Limit native AOT to CORINFO_HELP_ALLOC_CONTINUATION
#121444
Conversation
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR restricts the use of specialized continuation allocation helpers to non-Native AOT scenarios. Native AOT doesn't require the loader keep-alive mechanism since it lacks a runtime MethodDesc and doesn't have a loader to maintain. The change adds IsTargetAbi(CORINFO_NATIVEAOT_ABI) checks to ensure Native AOT always uses CORINFO_HELP_ALLOC_CONTINUATION instead of the keep-alive variants.
- Added Native AOT ABI checks to skip loader keep-alive logic
- Forces Native AOT to use standard continuation allocation helper
Native AOT doesn't have a `MethodDesc` at runtime. But that's fine, we don't have a loader to keep alive either.
98c1b23 to
5aecbe1
Compare
|
/ba-g android timeout is dotnet/dnceng#6408 |
Native AOT doesn't have a
MethodDescat runtime. But that's fine, we don't have a loader to keep alive either.Cc @dotnet/ilc-contrib