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.

Sign up for GitHub

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

Jump to bottom

Backport e3c57fdd8439 (minor RISC-V ABI fix) #64486

Closed
asb opened this issue Aug 7, 2023 · 4 comments · Fixed by llvm/llvm-project-release-prs#536
Closed

Backport e3c57fdd8439 (minor RISC-V ABI fix) #64486

asb opened this issue Aug 7, 2023 · 4 comments · Fixed by llvm/llvm-project-release-prs#536
Labels
backend:RISC-V release:backport release:merged
Milestone
LLVM 17.0.X Release

Comments

@asb
Copy link
Contributor

asb commented Aug 7, 2023

/cherry-pick e3c57fd

This commit landed before the branch, but I had to revert it as an issue was found. That implementation issue has now been addressed, and we'd like to get this (minor) ABI fix in the LLVM 17 release.
@asb asb added this to the LLVM 17.0.X Release milestone Aug 7, 2023
@llvmbot
Copy link
Collaborator

llvmbot commented Aug 7, 2023

@llvm/issue-subscribers-backend-risc-v

@llvmbot
Copy link
Collaborator

llvmbot commented Aug 7, 2023

Failed to cherry-pick: e3c57fd

https://github.com/llvm/llvm-project/actions/runs/5783606412

Please manually backport the fix and push it to your github fork. Once this is done, please add a comment like this:

/branch <user>/<repo>/<branch>

@asb
Copy link
Contributor Author

asb commented Aug 7, 2023

/branch asb/llvm-project/2023q3-riscv-llvm17-abifix-backport

@llvmbot llvmbot mentioned this issue Aug 7, 2023
Merged
@llvmbot
Copy link
Collaborator

llvmbot commented Aug 7, 2023

/pull-request llvm/llvm-project-release-prs#536

asb added a commit that referenced this issue Aug 10, 2023
@asb
[clang][doc][RISCV][LoongArch] Remove release notes for changes backp…
ac2af81 
…orted to 17.0.x

The RISC-V and LoongArch ABI fixes were backported to 17.0.x
(<#64486>,
<#64514>) and so are now
covered in the release notes for 17.0.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend:RISC-V release:backport release:merged
Projects
LLVM Release Status
Status: Done
Milestone
LLVM 17.0.X Release
Development

Successfully merging a pull request may close this issue.

PR for llvm/llvm-project#64486 llvm/llvm-project-release-prs
3 participants
@asb @EugeneZelenko @llvmbot