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

Fix Win arm GC barrier stuff #56585

Merged
merged 1 commit into from
Jul 30, 2021
Merged

Conversation

janvorli
Copy link
Member

My recent change to enable using LLD on Unix inadvertedly broken
Windows ARM GC barriers. This change makes part of that change
Unix specific, reverting the Windows state to the previous way.

Close #56570

@janvorli janvorli added this to the 6.0.0 milestone Jul 29, 2021
@janvorli janvorli requested a review from AndyAyersMS July 29, 2021 22:04
@janvorli janvorli self-assigned this Jul 29, 2021
@AndyAyersMS
Copy link
Member

cc @dotnet/jit-contrib

Copy link
Member

@AndyAyersMS AndyAyersMS left a comment

Choose a reason for hiding this comment

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

LGTM.

Should we run the outerloop tests that were failing?

@janvorli
Copy link
Member Author

/azp run R2R-CG2 windows arm Checked

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@janvorli
Copy link
Member Author

We should, but I don't know how to launch it.

@AndyAyersMS
Copy link
Member

I think it's just

/azp run runtime-coreclr outerloop

@janvorli janvorli force-pushed the fix-win-arm64-gcbarrier branch from f009495 to 6e32bc3 Compare July 30, 2021 15:39
My recent change to enable using LLD on Unix inadvertedly broken
Windows ARM GC barriers. This change makes part of that change
Unix specific.
@janvorli janvorli force-pushed the fix-win-arm64-gcbarrier branch from 6e32bc3 to e24e9f7 Compare July 30, 2021 15:57
@janvorli
Copy link
Member Author

/azp run runtime-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@janvorli
Copy link
Member Author

Some of the outerloop test legs have failed due to some infra issues, but the Windows arm test leg that we needed to run completed green. Merging...

@janvorli janvorli merged commit 3bd0acf into dotnet:main Jul 30, 2021
@janvorli janvorli deleted the fix-win-arm64-gcbarrier branch July 30, 2021 21:21
@ghost ghost locked as resolved and limited conversation to collaborators Aug 29, 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.

windows arm/arm64: runtime-coreclr outerloop tests failing with stack overflow
3 participants