InstCombine: Propagate exact for (sdiv X, Y) -> (udiv X, Y)
authorDavid Majnemer <david.majnemer@gmail.com>
Sat, 22 Nov 2014 20:00:38 +0000 (20:00 +0000)
committerDavid Majnemer <david.majnemer@gmail.com>
Sat, 22 Nov 2014 20:00:38 +0000 (20:00 +0000)
llvm-svn: 222624

llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
llvm/test/Transforms/InstCombine/div.ll

index b79f55e..216177f 100644 (file)
@@ -1097,7 +1097,9 @@ Instruction *InstCombiner::visitSDiv(BinaryOperator &I) {
     if (MaskedValueIsZero(Op0, Mask, 0, &I)) {
       if (MaskedValueIsZero(Op1, Mask, 0, &I)) {
         // X sdiv Y -> X udiv Y, iff X and Y don't have sign bit set
-        return BinaryOperator::CreateUDiv(Op0, Op1, I.getName());
+        auto *BO = BinaryOperator::CreateUDiv(Op0, Op1, I.getName());
+        BO->setIsExact(I.isExact());
+        return BO;
       }
 
       if (match(Op1, m_Shl(m_Power2(), m_Value()))) {
index f4166ec..cfe346e 100644 (file)
@@ -304,3 +304,13 @@ define <2 x i64> @test34(<2 x i64> %x) nounwind {
 ; CHECK-NEXT: sdiv exact <2 x i64> %x, <i64 -3, i64 -4>
 ; CHECK-NEXT: ret <2 x i64>
 }
+
+define i32 @test35(i32 %A) {
+  %and = and i32 %A, 2147483647
+  %mul = sdiv exact i32 %and, 2147483647
+  ret i32 %mul
+; CHECK-LABEL: @test35(
+; CHECK-NEXT: %[[and:.*]]  = and i32 %A, 2147483647
+; CHECK-NEXT: %[[udiv:.*]] = udiv exact i32 %[[and]], 2147483647
+; CHECK-NEXT: ret i32 %[[udiv]]
+}