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

[monodroid] Use MONO_AOT_MODE_INTERP_ONLY when available #5635

Merged
merged 4 commits into from
Mar 3, 2021

Conversation

lambdageek
Copy link
Member

@lambdageek lambdageek commented Feb 18, 2021

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

@lambdageek
Copy link
Member Author

lambdageek commented Feb 18, 2021

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";
        }
  }

@lambdageek lambdageek force-pushed the interp-fix branch 2 times, most recently from 8e50608 to b5be8ed Compare March 2, 2021 20:56
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
@lambdageek lambdageek changed the title .NET6: Use MONO_AOT_MODE_INTERP_ONLY [monodroid] Use MONO_AOT_MODE_INTERP_ONLY when available Mar 3, 2021
@lambdageek lambdageek marked this pull request as ready for review March 3, 2021 04:35
@lambdageek
Copy link
Member Author

The "MSBuild Emulator Tests MSBuild With Emulator - macOS - One .NET" lane is failing InstallWithoutSharedRuntime and MonoAndroidExportReferencedAppStartsFalseAssembliesDexes which I don't think are related to this PR.

Not sure I understand what the "Smoke Tests APK Instrumentation - macOS" failure is saying.

@grendello
Copy link
Contributor

InstallWithoutSharedRuntime was known to be failing in #5665 and I don't think either of the failures here are due to this here PR. We can merge, @jonpryor?

@jonpryor
Copy link
Member

jonpryor commented Mar 3, 2021

The crash in Smoke Tests > APK Instrumentation for Mono.Android_Tests-Bundle is concerning. I've restarted those tests.

@jonpryor jonpryor merged commit 194a1e2 into dotnet:master Mar 3, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AndroidUseInterpreter should work in net6
3 participants