Index: lib/Analysis/ScalarEvolution.cpp =================================================================== --- lib/Analysis/ScalarEvolution.cpp +++ lib/Analysis/ScalarEvolution.cpp @@ -5737,22 +5737,33 @@ BasicBlock *Header = L->getHeader(); assert(PN->getParent() == Header && "Can't evaluate PHI not in loop header!"); - // Since the loop is canonicalized, the PHI node must have two entries. One + BasicBlock *Latch = L->getLoopLatch(); + if (!Latch) + return nullptr; + + // NonLatch is the preheader, or something equivalent. + BasicBlock *NonLatch = Latch == PN->getIncomingBlock(0) + ? PN->getIncomingBlock(1) + : PN->getIncomingBlock(0); + + // Since the loop has one latch, the PHI node must have two entries. One // entry must be a constant (coming in from outside of the loop), and the // second must be derived from the same PHI. - bool SecondIsBackedge = L->contains(PN->getIncomingBlock(1)); + + assert(PN->getNumIncomingValues() == 2 && "Follows from having one latch!"); + for (auto &I : *Header) { PHINode *PHI = dyn_cast(&I); if (!PHI) break; auto *StartCST = - dyn_cast(PHI->getIncomingValue(!SecondIsBackedge)); + dyn_cast(PHI->getIncomingValueForBlock(NonLatch)); if (!StartCST) continue; CurrentIterVals[PHI] = StartCST; } if (!CurrentIterVals.count(PN)) return RetVal = nullptr; - Value *BEValue = PN->getIncomingValue(SecondIsBackedge); + Value *BEValue = PN->getIncomingValueForBlock(Latch); // Execute the loop symbolically to determine the exit value. if (BEs.getActiveBits() >= 32) @@ -5791,7 +5802,7 @@ PHINode *PHI = I.first; Constant *&NextPHI = NextIterVals[PHI]; if (!NextPHI) { // Not already computed. - Value *BEValue = PHI->getIncomingValue(SecondIsBackedge); + Value *BEValue = PHI->getIncomingValueForBlock(Latch); NextPHI = EvaluateExpression(BEValue, L, CurrentIterVals, DL, &TLI); } if (NextPHI != I.second) @@ -5826,15 +5837,20 @@ BasicBlock *Header = L->getHeader(); assert(PN->getParent() == Header && "Can't evaluate PHI not in loop header!"); - // One entry must be a constant (coming in from outside of the loop), and the - // second must be derived from the same PHI. - bool SecondIsBackedge = L->contains(PN->getIncomingBlock(1)); + BasicBlock *Latch = L->getLoopLatch(); + assert(Latch && "Should follow from NumIncomingValues == 2!"); + + // NonLatch is the preheader, or something equivalent. + BasicBlock *NonLatch = Latch == PN->getIncomingBlock(0) + ? PN->getIncomingBlock(1) + : PN->getIncomingBlock(0); + for (auto &I : *Header) { PHINode *PHI = dyn_cast(&I); if (!PHI) break; auto *StartCST = - dyn_cast(PHI->getIncomingValue(!SecondIsBackedge)); + dyn_cast(PHI->getIncomingValueForBlock(NonLatch)); if (!StartCST) continue; CurrentIterVals[PHI] = StartCST; } @@ -5874,7 +5890,7 @@ Constant *&NextPHI = NextIterVals[PHI]; if (NextPHI) continue; // Already computed! - Value *BEValue = PHI->getIncomingValue(SecondIsBackedge); + Value *BEValue = PHI->getIncomingValueForBlock(Latch); NextPHI = EvaluateExpression(BEValue, L, CurrentIterVals, DL, &TLI); } CurrentIterVals.swap(NextIterVals); Index: test/Transforms/IndVarSimplify/pr25060.ll =================================================================== --- /dev/null +++ test/Transforms/IndVarSimplify/pr25060.ll @@ -0,0 +1,37 @@ +; RUN: opt -S -indvars < %s | FileCheck %s + +define i16 @fn1() { +; CHECK-LABEL: @fn1( +entry: + br label %bb1 + +bb1: + %i = phi i16 [ 0, %entry ], [ 1, %bb1 ] + %storemerge = phi i16 [ %storemerge2, %bb1 ], [ 0, %entry ] + %storemerge2 = phi i16 [ 10, %entry ], [ 200, %bb1 ] + %tmp10 = icmp eq i16 %i, 1 + br i1 %tmp10, label %bb5, label %bb1 + +bb5: + %storemerge.lcssa = phi i16 [ %storemerge, %bb1 ] +; CHECK: ret i16 10 + ret i16 %storemerge.lcssa +} + +define i16 @fn2() { +; CHECK-LABEL: @fn2( +entry: + br label %bb1 + +bb1: + %canary = phi i16 [ 0, %entry ], [ %canary.inc, %bb1 ] + %i = phi i16 [ 0, %entry ], [ %storemerge, %bb1 ] + %storemerge = phi i16 [ 0, %bb1 ], [ 10, %entry ] + %canary.inc = add i16 %canary, 1 + %_tmp10 = icmp eq i16 %i, 10 + br i1 %_tmp10, label %bb5, label %bb1 + +bb5: +; CHECK: ret i16 1 + ret i16 %canary +}