summaryrefslogtreecommitdiffstats
path: root/llvm
diff options
context:
space:
mode:
authorChandler Carruth <chandlerc@gmail.com>2014-08-25 18:06:11 +0000
committerChandler Carruth <chandlerc@gmail.com>2014-08-25 18:06:11 +0000
commit70f81a98ca014d0425efb6b3d2cc2a347ca871f0 (patch)
treec2139c93a429421cbdd2b429cb7c4f906d0873a3 /llvm
parente2a1fa35dfba8efbec2d4468d1dc44c7ecf7f2d7 (diff)
downloadbcm5719-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.cpp2
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())
OpenPOWER on IntegriCloud