[ValueTracking] Make the code in lookThroughCast
authorDavid Majnemer <david.majnemer@gmail.com>
Fri, 29 Apr 2016 21:22:04 +0000 (21:22 +0000)
committerDavid Majnemer <david.majnemer@gmail.com>
Fri, 29 Apr 2016 21:22:04 +0000 (21:22 +0000)
No functionality change is intended.

llvm-svn: 268108

llvm/lib/Analysis/ValueTracking.cpp

index 4097322..a0a294d 100644 (file)
@@ -3714,29 +3714,22 @@ static Value *lookThroughCast(CmpInst *CmpI, Value *V1, Value *V2,
     return nullptr;
   }
 
+  Constant *CastedTo = nullptr;
+
   if (isa<ZExtInst>(CI) && CmpI->isUnsigned())
-    return ConstantExpr::getTrunc(C, CI->getSrcTy());
+    CastedTo = ConstantExpr::getTrunc(C, CI->getSrcTy());
+
+  if (isa<SExtInst>(CI) && CmpI->isSigned())
+    CastedTo = ConstantExpr::getTrunc(C, CI->getSrcTy(), true);
 
   if (isa<TruncInst>(CI))
-    return ConstantExpr::getIntegerCast(C, CI->getSrcTy(), CmpI->isSigned());
+    CastedTo = ConstantExpr::getIntegerCast(C, CI->getSrcTy(), CmpI->isSigned());
 
   if (isa<FPTruncInst>(CI))
-    return ConstantExpr::getFPExtend(C, CI->getSrcTy(), true);
+    CastedTo = ConstantExpr::getFPExtend(C, CI->getSrcTy(), true);
 
   if (isa<FPExtInst>(CI))
-    return ConstantExpr::getFPTrunc(C, CI->getSrcTy(), true);
-
-  // Sophisticated constants can have values which we cannot easily reason
-  // about.  Skip them for the fp<->int case.
-  if (isa<ConstantExpr>(C))
-    return nullptr;
-
-  Constant *CastedTo = nullptr;
-
-  // This is only valid if the truncated value can be sign-extended
-  // back to the original value.
-  if (isa<SExtInst>(CI) && CmpI->isSigned())
-    CastedTo = ConstantExpr::getTrunc(C, CI->getSrcTy(), true);
+    CastedTo = ConstantExpr::getFPTrunc(C, CI->getSrcTy(), true);
 
   if (isa<FPToUIInst>(CI))
     CastedTo = ConstantExpr::getUIToFP(C, CI->getSrcTy(), true);