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

[SystemZ] Require D12 for i128 accesses in isLegalAddressingMode() #79221

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

JonPsson1
Copy link
Contributor

For targets with vector support handle i128 in vector registers and therefore only have the small displacement available for memory accesses. Update isLegalAddressingMode() to reflect this.

@JonPsson1 JonPsson1 changed the title [SystemZ] Require D12 for i128 accesses in isLegalAddressingMode() [SystemZ] Require D20 for i128 accesses in isLegalAddressingMode() Jan 23, 2024
@JonPsson1 JonPsson1 changed the title [SystemZ] Require D20 for i128 accesses in isLegalAddressingMode() [SystemZ] Require D12 for i128 accesses in isLegalAddressingMode() Jan 23, 2024
Copy link
Member

@uweigand uweigand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@JonPsson1 JonPsson1 merged commit 84dcf3d into llvm:main Jan 24, 2024
3 of 4 checks passed
@JonPsson1 JonPsson1 deleted the LSRi128 branch January 24, 2024 19:16
@uweigand uweigand added this to the LLVM 18.X Release milestone Feb 26, 2024
@uweigand
Copy link
Member

/cherry-pick 84dcf3d

llvmbot pushed a commit to llvmbot/llvm-project that referenced this pull request Feb 26, 2024
…lvm#79221)

Machines with vector support handle i128 in vector registers and
therefore only have the small displacement available for memory
accesses. Update isLegalAddressingMode() to reflect this.

(cherry picked from commit 84dcf3d)
@llvmbot
Copy link
Collaborator

llvmbot commented Feb 26, 2024

/pull-request #83022

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

3 participants