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

MMX shuffle is not properly selected. #1799

Closed
asl opened this issue May 16, 2007 · 5 comments
Closed

MMX shuffle is not properly selected. #1799

asl opened this issue May 16, 2007 · 5 comments
Assignees
@lattner
Labels
backend:X86 bugzilla Issues migrated from bugzilla compile-fail Use [accepts-invalid] and [rejects-valid] instead

Comments

@asl
Copy link
Collaborator

asl commented May 16, 2007

Bugzilla Link 1427
Resolution FIXED
Resolved on Feb 22, 2010 12:41
Version trunk
OS Linux
Attachments Testcase

Extended Description

Consider attached testcase. We have:

Cannot yet select: 0x878fe20: v4i16 = vector_shuffle 0x878fb90, 0x878fd28, 0x878fda0
./llc((anonymous namespace)::PrintStackTrace()+0x1f)[0x866467f]
/lib/libc.so.6(abort+0xec)[0xb7c9bc0c]
./llc((anonymous
namespace)::X86DAGToDAGISel::Select_ISD_VECTOR_SHUFFLE_v4i16(llvm::SDOperand
const&)+0x436)[0x8385016]

This breaks Qt 4.3.0rc1
@asl
Copy link
Collaborator Author

asl commented May 16, 2007

assigned to @lattner

@lattner
Copy link
Collaborator

lattner commented May 17, 2007

renaming this - the maskmov is selected correctly now, but another shuffle feeding it isn't.

@lattner
Copy link
Collaborator

lattner commented May 17, 2007

Fixed. Patch here:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070514/049726.html

Testcase here: test/CodeGen/X86/mmx-shuffle.ll

-Chris

@asl
Copy link
Collaborator Author

asl commented May 17, 2007

This patch breaks almost all vector tests. Some of them just assert, others -
cycle, eat all memory and die. I've reverted it in order not to break nightly
testers.

@lattner
Copy link
Collaborator

lattner commented May 17, 2007

Fixed again, patch here:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070514/049750.html

-Chris

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lattner lattner

Labels
backend:X86 bugzilla Issues migrated from bugzilla compile-fail Use [accepts-invalid] and [rejects-valid] instead
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@asl @lattner