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

Bring up Android device aot on ci #50510

Closed
wants to merge 75 commits into from

Conversation

steveisok
Copy link
Member

@steveisok steveisok commented Mar 31, 2021

Addresses #33381

Some library tests were skipped:

  • due to "Attempting to JIT compile method ... while running in aot-only mode" (using conditional facts based on PlatformDetection.IsNotDeviceAOT; it's considered not to work on both Android and iOS devices)
  • due to a runtime crash occurred on Android AOT (using conditional active issues based on PlatformDetection.IsNotAndroidAOT)
  • due to the same failures as on Browser AOT (using conditional active issues based on PlatformDetection.IsNotAndroidAOT )

Some library test suites were skipped in tests.proj.

@ghost
Copy link

ghost commented Mar 31, 2021

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

Issue Details

Addresses #33381

Author: steveisok
Assignees: -
Labels:

area-Infrastructure-libraries

Milestone: -

@SamMonoRT
Copy link
Member

cc @fanyang-mono - something we may also need for enabling CI lanes for aot android validation

@MaximLipnin
Copy link
Contributor

After addressing the latest feedback the number of test failures increased up to ~114. Not sure if it's related to using EnableAggressiveTrimming property rather than just EnableTrimming one.

@steveisok
Copy link
Member Author

This is no longer needed. Closing.

@steveisok steveisok closed this Sep 7, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Oct 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants