summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenjamin Kramer <benny.kra@googlemail.com>2011-03-31 10:46:03 +0000
committerBenjamin Kramer <benny.kra@googlemail.com>2011-03-31 10:46:03 +0000
commitbe209ab8a21b70795630ff162a2daeaccca96e34 (patch)
treee0cc189d6e83ba0bf02e780732461503710d3c56
parentd159d946449def169bff0fc86e0f6b20d59cf475 (diff)
downloadbcm5719-llvm-be209ab8a21b70795630ff162a2daeaccca96e34.tar.gz
bcm5719-llvm-be209ab8a21b70795630ff162a2daeaccca96e34.zip
InstCombine: Fix transform to use the swapped predicate.
Thanks Frits! llvm-svn: 128628
-rw-r--r--llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp4
-rw-r--r--llvm/test/Transforms/InstCombine/fcmp.ll2
2 files changed, 3 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp
index cb434bda57a..3e549d94282 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp
@@ -2855,10 +2855,10 @@ Instruction *InstCombiner::visitFCmpInst(FCmpInst &I) {
}
}
- // fcmp (fneg x), (fneg y) -> fcmp x, y
+ // fcmp pred (fneg x), (fneg y) -> fcmp swap(pred) x, y
Value *X, *Y;
if (match(Op0, m_FNeg(m_Value(X))) && match(Op1, m_FNeg(m_Value(Y))))
- return new FCmpInst(I.getPredicate(), X, Y);
+ return new FCmpInst(I.getSwappedPredicate(), X, Y);
// fcmp (fpext x), (fpext y) -> fcmp x, y
if (FPExtInst *LHSExt = dyn_cast<FPExtInst>(Op0))
diff --git a/llvm/test/Transforms/InstCombine/fcmp.ll b/llvm/test/Transforms/InstCombine/fcmp.ll
index 66607ea0017..b41cdfa7cf1 100644
--- a/llvm/test/Transforms/InstCombine/fcmp.ll
+++ b/llvm/test/Transforms/InstCombine/fcmp.ll
@@ -44,7 +44,7 @@ define i1 @test5(float %a) nounwind {
define i1 @test6(float %x, float %y) nounwind {
%neg1 = fsub float -0.000000e+00, %x
%neg2 = fsub float -0.000000e+00, %y
- %cmp = fcmp ogt float %neg1, %neg2
+ %cmp = fcmp olt float %neg1, %neg2
ret i1 %cmp
; CHECK: @test6
; CHECK-NEXT: fcmp ogt float %x, %y
OpenPOWER on IntegriCloud