replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[Simplify] Do not remove dependencies of phis within region stmts. 

These were wrongly assumed to be phi nodes that require
MemoryKind::PHI accesses.

llvm-svn: 309454
This commit is contained in:
Michael Kruse 2017-07-28 23:22:32 +00:00
parent fd7f40961b
commit 8e41d2baab
2 changed files with 52 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
polly/lib/Support/VirtualInstruction.cpp
View File

@ -355,7 +355,8 @@ static void walkReachable(Scop *S, LoopInfo *LI,
continue;
// Add all operands to the worklists.
if (PHINode *PHI = dyn_cast<PHINode>(Inst)) {
PHINode *PHI = dyn_cast<PHINode>(Inst);
if (PHI && PHI->getParent() == Stmt->getEntryBlock()) {
if (MemoryAccess *PHIRead = Stmt->lookupPHIReadOf(PHI))
WorklistAccs.push_back(PHIRead);
} else {

50
polly/test/Simplify/notdead_region_innerphi.ll Normal file
View File

@ -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