diff --git a/polly/lib/Support/VirtualInstruction.cpp b/polly/lib/Support/VirtualInstruction.cpp index a0fecd17a9b6..8e2ab0c3c445 100644 --- a/polly/lib/Support/VirtualInstruction.cpp +++ b/polly/lib/Support/VirtualInstruction.cpp @@ -355,7 +355,8 @@ static void walkReachable(Scop *S, LoopInfo *LI, continue; // Add all operands to the worklists. - if (PHINode *PHI = dyn_cast(Inst)) { + PHINode *PHI = dyn_cast(Inst); + if (PHI && PHI->getParent() == Stmt->getEntryBlock()) { if (MemoryAccess *PHIRead = Stmt->lookupPHIReadOf(PHI)) WorklistAccs.push_back(PHIRead); } else { diff --git a/polly/test/Simplify/notdead_region_innerphi.ll b/polly/test/Simplify/notdead_region_innerphi.ll new file mode 100644 index 000000000000..5749186f4ce5 --- /dev/null +++ b/polly/test/Simplify/notdead_region_innerphi.ll @@ -0,0 +1,50 @@ +; RUN: opt %loadPolly -polly-simplify -analyze < %s | FileCheck %s -match-full-lines +; +; Do not remove dependencies of a phi node within a region statement (%phi). +; +define void @func(i32 %n, double* noalias nonnull %A, double %alpha) { +entry: + br label %for + +for: + %j = phi i32 [0, %entry], [%j.inc, %inc] + %j.cmp = icmp slt i32 %j, %n + br i1 %j.cmp, label %body, label %exit + + body: + %val = fadd double 21.0, 21.0 + br label %region_entry + + + region_entry: + %region.cmp = fcmp ueq double %alpha, 0.0 + br i1 %region.cmp, label %region_true, label %region_exit + + region_true: + br i1 true, label %region_verytrue, label %region_mostlytrue + + region_verytrue: + br label %region_mostlytrue + + region_mostlytrue: + %phi = phi double [%val, %region_true], [0.0, %region_verytrue] + store double %phi, double* %A + br label %region_exit + + region_exit: + br label %inc + + +inc: + %j.inc = add nuw nsw i32 %j, 1 + br label %for + +exit: + br label %return + +return: + ret void +} + + +; CHECK: SCoP could not be simplified