Revert "Remove unnecessary null check. NFC."
authorPete Cooper <peter_cooper@apple.com>
Mon, 27 Jul 2015 18:37:58 +0000 (18:37 +0000)
committerPete Cooper <peter_cooper@apple.com>
Mon, 27 Jul 2015 18:37:58 +0000 (18:37 +0000)
This reverts commit r243167.

Duncan pointed out that dyn_cast can return null in these cases, so this
was an unsafe commit to make.  Sorry for the noise.

Worryingly there were no tests which fail...

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

lib/Transforms/Scalar/LoopInterchange.cpp

index 75fe0c77e7b21e3c46bc8bb29ae94c2d1b66a5f2..9d7e57ffebac143a35ed4145450de8e3b2c5fd70 100644 (file)
@@ -817,6 +817,9 @@ bool LoopInterchangeLegality::currentLimitations() {
     InnerIndexVarInc =
         dyn_cast<Instruction>(InnerInductionVar->getIncomingValue(0));
 
+  if (!InnerIndexVarInc)
+    return true;
+
   // Since we split the inner loop latch on this induction variable. Make sure
   // we do not have any instruction between the induction variable and branch
   // instruction.