-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Revert "Disable GetGCMemoryInfo on arm (#73477)" #73595
Conversation
This reverts commit 9865cc7. We believe the problematic change has been reverted so the test can be re-enabled.
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
@karelz - as new test-monitor can I hand this PR over to you to decide whether or not to merge it? I think the PR that reverted the pinvoke inlining did its ARM validation before this test got disabled and passed so I am reasonably confident that issue is solved. This PR just gets the test turned on again and confirms everything is still in good shape. |
Some of the pipelines unfortunately failed on the dotnet-install.sh script where HTML was received from the server instead of expected binary data... |
I retried the failing jobs, let's see. |
The Android JIT.Directed failures are tracked in #73539 |
@noahfalk sorry for late response, I didn't notice it until now. |
Fixes #73247
This reverts commit 9865cc7.
We believe the problematic change has been reverted so the test can be re-enabled.