-
Notifications
You must be signed in to change notification settings - Fork 538
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
[monodroid] Use MONO_AOT_MODE_INTERP_ONLY when available #5635
Conversation
55d9329
to
848e22c
Compare
Haven't tried it yet. still need to read the documentation on how to run the changes locally. But it feels right. To test it, I would run this code: private static string OnInterp () {
if (RuntimeFeature.IsDynamicCodeSupported) {
if (RuntimeFeature.IsDynamicCodeCompiled) {
return "JIT";
} else {
return "Interp";
}
} else {
return "FullAOT";
}
} |
848e22c
to
7372da5
Compare
8e50608
to
b5be8ed
Compare
This depends on a fix to mono/mono#18893 (make interp only mode distinct from MONO_AOT_MODE_LAST) which was done in mono/mono#20159 but only for .NET 6, not previous Mono releases. Fixes dotnet#5633
The "MSBuild Emulator Tests MSBuild With Emulator - macOS - One .NET" lane is failing Not sure I understand what the "Smoke Tests APK Instrumentation - macOS" failure is saying. |
The crash in Smoke Tests > APK Instrumentation for |
This depends on a fix to mono/mono#18893 (make interp only mode distinct from
MONO_AOT_MODE_LAST
) which was done in mono/mono#20159 but only for .NET 6, not previous Mono releases.Fixes #5633