ARM: Don't assume an SDNode is a constant.
authorJim Grosbach <grosbach@apple.com>
Wed, 25 Jul 2012 17:02:47 +0000 (17:02 +0000)
committerJim Grosbach <grosbach@apple.com>
Wed, 25 Jul 2012 17:02:47 +0000 (17:02 +0000)
Before accessing a node as a ConstandSDNode, make sure it actually is one.
No testcase of non-trivial size.

rdar://11948669

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@160735 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/ARM/ARMISelLowering.cpp

index 04370c095cd76a1682aafcf547d3cc338e8b11c3..9d8b7dff108945c0cc3d4ac204137a6d3c49e2b4 100644 (file)
@@ -4271,6 +4271,10 @@ SDValue ARMTargetLowering::ReconstructShuffle(SDValue Op,
 
     // Record this extraction against the appropriate vector if possible...
     SDValue SourceVec = V.getOperand(0);
+    // If the element number isn't a constant, we can't effectively
+    // analyze what's going on.
+    if (!isa<ConstantSDNode>(V.getOperand(1)))
+      return SDValue();
     unsigned EltNo = cast<ConstantSDNode>(V.getOperand(1))->getZExtValue();
     bool FoundSource = false;
     for (unsigned j = 0; j < SourceVecs.size(); ++j) {