Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Fix issue #13161. #13299

Merged
merged 1 commit into from
Aug 10, 2017
Merged

Fix issue #13161. #13299

merged 1 commit into from
Aug 10, 2017

Conversation

pgavlin
Copy link

@pgavlin pgavlin commented Aug 9, 2017

This issue was a crash when calling R2R-compiled code from JITted code
on Windows/ARM32. The caller had a live value in r4 that was scribbled
over by the callee. This change adds the necessary call to
regSet.rsTrackRegTrash which ensures that r4 is saved/restored in
the prolog/epilog when it is used for R2R indirect calls.

This issue was a crash when calling R2R-compiled code from JITted code
on Windows/ARM32. The caller had a live value in `r4` that was scribbled
over by the callee. This change adds the necessary call to
`regSet.rsTrackRegTrash` which ensures that `r4` is saved/restored in
the prolog/epilog when it is used for R2R indirect calls.
@pgavlin
Copy link
Author

pgavlin commented Aug 9, 2017

@BruceForstall @CarolEidt @dotnet/jit-contrib PTAL

Copy link

@CarolEidt CarolEidt left a comment

Choose a reason for hiding this comment

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

LGTM

@pgavlin
Copy link
Author

pgavlin commented Aug 10, 2017

@dotnet-bot test Ubuntu x64 Checked Build and Test

@pgavlin pgavlin merged commit 51ed16f into dotnet:master Aug 10, 2017
@pgavlin pgavlin deleted the GH13161 branch August 10, 2017 23:02
@karelz karelz modified the milestone: 2.1.0 Aug 28, 2017
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.

5 participants