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

[LLVM][AArch64] Backport fix for stack corruption issue when rewriting SVE stack offsets. #70457

Closed
paulwalker-arm opened this issue Oct 27, 2023 · 4 comments · Fixed by llvm/llvm-project-release-prs#754
Closed

[LLVM][AArch64] Backport fix for stack corruption issue when rewriting SVE stack offsets. #70457

paulwalker-arm opened this issue Oct 27, 2023 · 4 comments · Fixed by llvm/llvm-project-release-prs#754
Assignees
@paulwalker-arm
Labels
backend:AArch64 release:backport release:merged SVE ARM Scalable Vector Extensions
Milestone
LLVM 17.0.X Release

Comments

@paulwalker-arm
Copy link
Collaborator

paulwalker-arm commented Oct 27, 2023 •
edited

When computing scalable offsets that are larger than the immediate field of an SVE load/store instruction and the offset is not an integer multiple of the target vector length we are incorrectly calculating the offset remainder after the instructions immediate operand has been taken into account.
@paulwalker-arm paulwalker-arm added this to the LLVM 17.0.X Release milestone Oct 27, 2023
@paulwalker-arm paulwalker-arm self-assigned this Oct 27, 2023
@paulwalker-arm
Copy link
Collaborator Author

I plan on cherry-picking #70315 once the precommit testing has finished and I can land the patch.

@paulwalker-arm
Copy link
Collaborator Author

/cherry-pick 7c90be2

@llvmbot
Copy link
Collaborator

llvmbot commented Oct 27, 2023

/branch llvm/llvm-project-release-prs/issue70457

@llvmbot llvmbot mentioned this issue Oct 27, 2023
Merged
@llvmbot
Copy link
Collaborator

llvmbot commented Oct 27, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@paulwalker-arm paulwalker-arm

Labels
backend:AArch64 release:backport release:merged SVE ARM Scalable Vector Extensions
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#70457 llvm/llvm-project-release-prs
3 participants
@EugeneZelenko @paulwalker-arm @llvmbot