Don't modify constant in-place.
authorJim Grosbach <grosbach@apple.com>
Fri, 30 Sep 2011 19:58:46 +0000 (19:58 +0000)
committerJim Grosbach <grosbach@apple.com>
Fri, 30 Sep 2011 19:58:46 +0000 (19:58 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@140875 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/InstCombine/InstCombineCompares.cpp

index 26432d2eed5e10b4edfa1c90719cbabec63e8574..bb1cbfade34d1a4701ed7f86ffa9a2e465db3629 100644 (file)
@@ -2839,10 +2839,11 @@ Instruction *InstCombiner::visitFCmpInst(FCmpInst &I) {
 
         // Avoid lossy conversions and denormals. Zero is a special case
         // that's OK to convert.
-        F.clearSign();
+        APFloat Fabs = F;
+        Fabs.clearSign();
         if (!Lossy &&
-            ((F.compare(APFloat::getSmallestNormalized(*Sem)) !=
-                 APFloat::cmpLessThan) || F.isZero()))
+            ((Fabs.compare(APFloat::getSmallestNormalized(*Sem)) !=
+                 APFloat::cmpLessThan) || Fabs.isZero()))
 
           return new FCmpInst(I.getPredicate(), LHSExt->getOperand(0),
                               ConstantFP::get(RHSC->getContext(), F));