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

JIT: Fix reordering with 'backfilled' arguments on x86 #71940

Merged
merged 3 commits into from
Jul 11, 2022

Conversation

jakobbotsch
Copy link
Member

@jakobbotsch jakobbotsch commented Jul 11, 2022

In #70931 I made the assumption that all register args come
before stack args on x86. This assumption is not true.

Fix #71939

Small handful of x86 diffs

In a previous change I made the assumption that all register args come
before stack args on x86. This assumption is not true.

Fix dotnet#71939
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Jul 11, 2022
@ghost ghost assigned jakobbotsch Jul 11, 2022
@ghost
Copy link

ghost commented Jul 11, 2022

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Issue Details

In #70931 I made the assumption that all register args come
before stack args on x86. This assumption is not true.

Fix #71939

Author: jakobbotsch
Assignees: -
Labels:

area-CodeGen-coreclr

Milestone: -

@jakobbotsch
Copy link
Member Author

cc @dotnet/jit-contrib PTAL @AndyAyersMS

@jakobbotsch jakobbotsch requested a review from AndyAyersMS July 11, 2022 13:12
@jakobbotsch jakobbotsch merged commit 52d9d0c into dotnet:main Jul 11, 2022
@jakobbotsch jakobbotsch deleted the fix-71939 branch July 11, 2022 17:30
@ghost ghost locked as resolved and limited conversation to collaborators Aug 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JIT: More unsafe call args reordering in x86
2 participants