diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2014-08-25 18:06:11 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2014-08-25 18:06:11 +0000 |
commit | 70f81a98ca014d0425efb6b3d2cc2a347ca871f0 (patch) | |
tree | c2139c93a429421cbdd2b429cb7c4f906d0873a3 /llvm | |
parent | e2a1fa35dfba8efbec2d4468d1dc44c7ecf7f2d7 (diff) | |
download | bcm5719-llvm-70f81a98ca014d0425efb6b3d2cc2a347ca871f0.tar.gz bcm5719-llvm-70f81a98ca014d0425efb6b3d2cc2a347ca871f0.zip |
[x86] Fix a bug in r216319 where I was missing a 'break'.
This actually was caught by existing tests but those tests were disabled
with an XFAIL because of PR20736. While working on fixing that,
I noticed the test failure, and tracked it down to this.
We even have a really nice Clang warning that would have caught this but
it isn't enabled in LLVM! =[ I may look at enabling it.
llvm-svn: 216391
Diffstat (limited to 'llvm')
-rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 66c00fb3507..5dd542842e4 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -19467,11 +19467,13 @@ combineRedundantDWordShuffle(SDValue N, MutableArrayRef<int> Mask, case X86ISD::UNPCKL: case X86ISD::UNPCKH: V = DAG.getNode(W.getOpcode(), DL, W.getValueType(), V, V); + break; case X86ISD::PSHUFD: case X86ISD::PSHUFLW: case X86ISD::PSHUFHW: V = DAG.getNode(W.getOpcode(), DL, W.getValueType(), V, W.getOperand(1)); + break; } } if (V.getValueType() != N.getValueType()) |