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

[Mono] Enable runtime tests to run on Android x64 with interpreter #54084

Merged
merged 18 commits into from
Jun 21, 2021

Conversation

fanyang-mono
Copy link
Member

@fanyang-mono fanyang-mono commented Jun 11, 2021

Contributes to #33381

When working on this PR, I noticed that the mono interpreter lanes were not running with Mono interpreter. This PR also fixes that issue.

This PR also add an option to enable interpreter for Android sample.

2,427 runtime tests were enabled on Android x64 with interpreter. Comparing to Android x64 with JIT, 93 more tests were disabed.

@ghost
Copy link

ghost commented Jun 11, 2021

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

Issue Details
Author: fanyang-mono
Assignees: -
Labels:

area-Infrastructure-mono

Milestone: -

@imhameed
Copy link
Contributor

I just merged #53752; if you rebase on top of main you should have the changes necessary to run-test-job.yml to make RuntimeVariant accessible to helixpublishwitharcade.proj (and testenvironment.proj).

@naricc
Copy link
Contributor

naricc commented Jun 14, 2021

I'm not sure how the interpreter got disabled on the interpreter lanes. I will be adding a test to make sure that doesn't happen again though.

@fanyang-mono
Copy link
Member Author

/azp run runtime,runtime-staging

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@fanyang-mono
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@fanyang-mono fanyang-mono marked this pull request as ready for review June 18, 2021 22:03
Copy link
Member

@trylek trylek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you!

@fanyang-mono
Copy link
Member Author

The failures on windows arm64 were irrelevant to this PR. There are two tracking issues for it https://github.com/dotnet/core-eng/issues/13324 and https://github.com/dotnet/core-eng/issues/13292

@fanyang-mono fanyang-mono merged commit 213600c into dotnet:main Jun 21, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jul 21, 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.

4 participants