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

Jump to bottom

Instruction selection fails for %llvm.ppc.altivec.vct* with a non-zero argument #43584

Closed
llvmbot opened this issue Dec 6, 2019 · 3 comments
Closed

Instruction selection fails for %llvm.ppc.altivec.vct* with a non-zero argument #43584

llvmbot opened this issue Dec 6, 2019 · 3 comments
Labels
backend:PowerPC bugzilla Issues migrated from bugzilla

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Dec 6, 2019

Bugzilla Link 44239
Resolution FIXED
Resolved on Dec 15, 2019 18:25
Version trunk
OS Linux
Attachments C reproducer, IR reproducer, Patch
Reporter LLVM Bugzilla Contributor
CC @nemanjai
Fixed by commit(s) rG7e0fd77645cf

Extended Description

Use of __builtin_altivec_vctsxs, __builtin_altivec_vctsxs, __builtin_altivec_vctuxs, __builtin_altivec_vcfux with a non-zero immediate argument result in: LLVM ERROR: Cannot select: intrinsic %llvm.ppc.altivec.vctsxs.

Attached are C and IR reproducers.

Broken by r372285 (git d8399d1). This commit make similar changes to other architectures, so I wonder if there are any other cases.

The attached patch fixes the issue. Let me know if you want me to work up something more formal to submit for review.
@nemanjai
Copy link
Member

nemanjai commented Dec 6, 2019

The test case and the fix seem perfectly fine. Would you be interested/able to post the patch with the test case on Phabricator and we can get it reviewed and committed?

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 6, 2019

The test case and the fix seem perfectly fine. Would you be interested/able
to post the patch with the test case on Phabricator and we can get it
reviewed and committed?

Will do

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 6, 2019

Differential: https://reviews.llvm.org/D71138

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend:PowerPC bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nemanjai @llvmbot