[llvm] r222624 - InstCombine: Propagate exact for (sdiv X, Y) -> (udiv X, Y)

David Majnemer david.majnemer at gmail.com
Sat Nov 22 12:00:39 PST 2014


Author: majnemer
Date: Sat Nov 22 14:00:38 2014
New Revision: 222624

URL: http://llvm.org/viewvc/llvm-project?rev=222624&view=rev
Log:
InstCombine: Propagate exact for (sdiv X, Y) -> (udiv X, Y)

Modified:
    llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
    llvm/trunk/test/Transforms/InstCombine/div.ll

Modified: llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp?rev=222624&r1=222623&r2=222624&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp (original)
+++ llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp Sat Nov 22 14:00:38 2014
@@ -1097,7 +1097,9 @@ Instruction *InstCombiner::visitSDiv(Bin
     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()))) {

Modified: llvm/trunk/test/Transforms/InstCombine/div.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/InstCombine/div.ll?rev=222624&r1=222623&r2=222624&view=diff
==============================================================================
--- llvm/trunk/test/Transforms/InstCombine/div.ll (original)
+++ llvm/trunk/test/Transforms/InstCombine/div.ll Sat Nov 22 14:00:38 2014
@@ -304,3 +304,13 @@ define <2 x i64> @test34(<2 x i64> %x) n
 ; 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]]
+}





More information about the llvm-commits mailing list