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

[RISC-V] Missing defines in GC #88661

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

alpencolt
Copy link

Some defines were not ported to RISC-V in GC. This PR fix it.

Part of #84834

cc @jakobbotsch @wscho77 @HJLeee @JongHeonChoi @t-mustafin @clamp03 @gbalykov

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Jul 11, 2023
@ghost
Copy link

ghost commented Jul 11, 2023

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

Issue Details

Some defines were not ported to RISC-V in GC. This PR fix it.

Part of #84834

cc @jakobbotsch @wscho77 @HJLeee @JongHeonChoi @t-mustafin @clamp03 @gbalykov

Author: alpencolt
Assignees: -
Labels:

area-GC-coreclr, community-contribution

Milestone: -

src/coreclr/gc/gc.cpp Outdated Show resolved Hide resolved
@mangod9
Copy link
Member

mangod9 commented Jul 12, 2023

CI failures appear to be known issues.

@mangod9 mangod9 closed this Jul 12, 2023
@mangod9 mangod9 reopened this Jul 12, 2023
@mangod9 mangod9 merged commit a23c7a1 into dotnet:main Jul 12, 2023
97 of 107 checks passed
@ghost ghost locked as resolved and limited conversation to collaborators Aug 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-riscv Related to the RISC-V architecture area-GC-coreclr community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants