]> granicus.if.org Git - llvm/commitdiff
[InstCombine] use AddOne helper to reduce code; NFC
authorSanjay Patel <spatel@rotateright.com>
Fri, 13 Oct 2017 17:00:47 +0000 (17:00 +0000)
committerSanjay Patel <spatel@rotateright.com>
Fri, 13 Oct 2017 17:00:47 +0000 (17:00 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@315709 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/InstCombine/InstCombineAddSub.cpp

index 75cc683b7a657fb4524a7662037a98c3b47f8c65..92452d7df73aaea0bcdad559e67d75082e138b59 100644 (file)
@@ -960,13 +960,9 @@ Instruction *InstCombiner::foldAddWithConstant(BinaryOperator &Add) {
     return NV;
 
   Value *X;
-  Type *Ty = Add.getType();
-  if (match(Op0, m_ZExt(m_Value(X))) &&
-      X->getType()->getScalarSizeInBits() == 1) {
+  if (match(Op0, m_ZExt(m_Value(X))) && X->getType()->getScalarSizeInBits() == 1)
     // zext(bool) + C -> bool ? C + 1 : C
-    Constant *One = ConstantInt::get(Ty, 1);
-    return SelectInst::Create(X, ConstantExpr::getAdd(Op1C, One), Op1);
-  }
+    return SelectInst::Create(X, AddOne(Op1C), Op1);
 
   const APInt *C;
   if (!match(Op1, m_APInt(C)))
@@ -985,6 +981,7 @@ Instruction *InstCombiner::foldAddWithConstant(BinaryOperator &Add) {
 
   // Is this add the last step in a convoluted sext?
   // add(zext(xor i16 X, -32768), -32768) --> sext X
+  Type *Ty = Add.getType();
   const APInt *C2;
   if (match(Op0, m_ZExt(m_Xor(m_Value(X), m_APInt(C2)))) &&
       C2->isMinSignedValue() && C2->sext(Ty->getScalarSizeInBits()) == *C)