From 11bd958cb673b1af9892ed820087b8a4cc62a480 Mon Sep 17 00:00:00 2001 From: Pete Cooper Date: Mon, 27 Jul 2015 18:37:58 +0000 Subject: [PATCH] Revert "Remove unnecessary null check. NFC." 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... llvm-svn: 243302 --- llvm/lib/Transforms/Scalar/LoopInterchange.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/llvm/lib/Transforms/Scalar/LoopInterchange.cpp b/llvm/lib/Transforms/Scalar/LoopInterchange.cpp index 75fe0c7..9d7e57f 100644 --- a/llvm/lib/Transforms/Scalar/LoopInterchange.cpp +++ b/llvm/lib/Transforms/Scalar/LoopInterchange.cpp @@ -817,6 +817,9 @@ bool LoopInterchangeLegality::currentLimitations() { InnerIndexVarInc = dyn_cast(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. -- 2.7.4