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

PR for llvm/llvm-project#65056 #661

Closed
wants to merge 1 commit into from
Closed

Conversation

llvmbot
Copy link
Member

@llvmbot llvmbot commented Aug 29, 2023

This adds the v9 SETX pseudoinstruction for convenient loading of 64-bit values.

Reviewed By: barannikov88

Differential Revision: https://reviews.llvm.org/D157230

(cherry picked from commit c8c5f317e4617062dab61be650cecbda9adb407a)
@llvmbot
Copy link
Member Author

llvmbot commented Aug 29, 2023

@s-barannikov What do you think about merging this PR to the release branch?

@tru
Copy link
Contributor

tru commented Aug 31, 2023

ping @s-barannikov

@s-barannikov
Copy link
Contributor

I don't mind, but I'd like to hear @brad0's justification why this change is needed in the release branch.

@tru
Copy link
Contributor

tru commented Sep 4, 2023

ping @brad0 ?

@tru
Copy link
Contributor

tru commented Sep 5, 2023

This issue missed RC4. The next release is 17.0.0-final. We only take on patches that fix crashes, severe miscompiles, or regressions for the next two weeks. Keep that in mind when reviewing this PR.

@tru
Copy link
Contributor

tru commented Dec 11, 2023

No more 17.x releases are planned, closing the open Pull Requests.

@tru tru closed this Dec 11, 2023
@tru tru deleted the llvm-issue65056 branch December 11, 2023 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge c8c5f317e4617062dab61be650cecbda9adb407a into 17.0.X
4 participants