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

Backport #85277 #85422

Merged
merged 1 commit into from
Mar 19, 2024
Merged

Backport #85277 #85422

merged 1 commit into from
Mar 19, 2024

Conversation

Patryk27
Copy link
Contributor

Backporting #85277, at the request of rust-lang#171 (comment).

@Patryk27
Copy link
Contributor Author

cc @benshi001 - not sure if you can approve backports as well, but maybe you'll know who to ping if so 😅

@llvmbot
Copy link
Collaborator

llvmbot commented Mar 16, 2024

/pull-request #85512

@brad0 brad0 reopened this Mar 16, 2024
@brad0
Copy link
Contributor

brad0 commented Mar 16, 2024

Sorry, I didn't mean to mess that up.

@tstellar tstellar requested a review from benshi001 March 17, 2024 01:30
@benshi001 benshi001 requested a review from brad0 March 18, 2024 03:47
LDDRdPtrQ was marked as `earlyclobber`, which doesn't play well with
GreedyRA (which can generate this instruction through `loadRegFromStackSlot()`).

This seems to be the same case as:

https://github.com/llvm/llvm-project/blob/a99b912c9b74f6ef91786b4dfbc25160c27d3b41/llvm/lib/Target/AVR/AVRInstrInfo.td#L1421

Closes llvm#81911.
@tstellar tstellar merged commit fd9f1fa into llvm:release/18.x Mar 19, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

6 participants