diff options
author | Dale Johannesen <dalej@apple.com> | 2009-04-27 21:03:15 +0000 |
---|---|---|
committer | Dale Johannesen <dalej@apple.com> | 2009-04-27 21:03:15 +0000 |
commit | 27b4f222cf30141080fb6a69496b0155f152268c (patch) | |
tree | 0dea9c639d470bc4e650c4952c3d483230bef849 | |
parent | 093e4c578d88e8f608ca3a1143b8b0e616c35679 (diff) | |
download | bcm5719-llvm-27b4f222cf30141080fb6a69496b0155f152268c.tar.gz bcm5719-llvm-27b4f222cf30141080fb6a69496b0155f152268c.zip |
Fix PR 4086, a bug in FP IV elimination.
llvm-svn: 70247
-rw-r--r-- | llvm/lib/Transforms/Scalar/IndVarSimplify.cpp | 6 | ||||
-rw-r--r-- | llvm/test/Transforms/IndVarSimplify/2009-04-27-Floating.ll | 18 |
2 files changed, 22 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp b/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp index cc3919da846..141b1181bf9 100644 --- a/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp +++ b/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp @@ -1037,9 +1037,11 @@ void IndVarSimplify::HandleFloatingPointIV(Loop *L, PHINode *PH, Incr->getName()+".int", Incr); NewPHI->addIncoming(NewAdd, PH->getIncomingBlock(BackEdge)); + // The back edge is edge 1 of newPHI, whatever it may have been in the + // original PHI. ConstantInt *NewEV = ConstantInt::get(Type::Int32Ty, intEV); - Value *LHS = (EVIndex == 1 ? NewPHI->getIncomingValue(BackEdge) : NewEV); - Value *RHS = (EVIndex == 1 ? NewEV : NewPHI->getIncomingValue(BackEdge)); + Value *LHS = (EVIndex == 1 ? NewPHI->getIncomingValue(1) : NewEV); + Value *RHS = (EVIndex == 1 ? NewEV : NewPHI->getIncomingValue(1)); ICmpInst *NewEC = new ICmpInst(NewPred, LHS, RHS, EC->getNameStart(), EC->getParent()->getTerminator()); diff --git a/llvm/test/Transforms/IndVarSimplify/2009-04-27-Floating.ll b/llvm/test/Transforms/IndVarSimplify/2009-04-27-Floating.ll new file mode 100644 index 00000000000..700f294eb88 --- /dev/null +++ b/llvm/test/Transforms/IndVarSimplify/2009-04-27-Floating.ll @@ -0,0 +1,18 @@ +; RUN: llvm-as < %s | opt -indvars | llvm-dis | grep icmp | grep next +; PR4086 +declare void @foo() + +define void @test() { +entry: + br label %loop_body + +loop_body: + %i = phi float [ %nexti, %loop_body ], [ 0.0, %entry ] + tail call void @foo() + %nexti = add float %i, 1.0 + %less = fcmp olt float %nexti, 2.0 + br i1 %less, label %loop_body, label %done + +done: + ret void +} |