Skip to content

Fix arm64/arm32 cross-DAC #91425

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

Merged
merged 1 commit into from
Aug 31, 2023
Merged

Fix arm64/arm32 cross-DAC #91425

merged 1 commit into from
Aug 31, 2023

Conversation

mikem8361
Copy link
Contributor

The precode code was incorrectly using the HOST_ARM64, HOST_ARM, etc. defines instead of the TARGET_ ones. This means the cross-DAC was using the wrong architectures *Precode::Type codes.

Our vendors have been reporting this issue during their regular SOS testing for too long.

The precode code was incorrectly using the HOST_ARM64, HOST_ARM, etc. defines instead
of the TARGET_ ones. This means the cross-DAC was using the wrong architectures
*Precode::Type codes.
@mikem8361 mikem8361 requested a review from hoyosjs August 31, 2023 20:56
@ghost ghost added the area-VM-coreclr label Aug 31, 2023
@ghost ghost assigned mikem8361 Aug 31, 2023
@mikem8361 mikem8361 merged commit 0241bc1 into dotnet:main Aug 31, 2023
@mikem8361
Copy link
Contributor Author

mikem8361 commented Aug 31, 2023

/backport to release/8.0

@mikem8361 mikem8361 deleted the crossdacfix branch August 31, 2023 23:54
@mikem8361
Copy link
Contributor Author

/backport to release/8.0

@github-actions
Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/6043630327

@ghost ghost locked as resolved and limited conversation to collaborators Oct 1, 2023
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.

3 participants