Revert to pacify a buildbot. rdar://problem/6501862

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@132351 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Stuart Hastings 2011-05-31 19:56:35 +00:00
parent 471fa149d1
commit 3200c4b53c
2 changed files with 15 additions and 55 deletions

View File

@ -136,26 +136,19 @@ Instruction *InstCombiner::visitMul(BinaryOperator &I) {
} }
} }
// (Y - X) * (-(2**n)) -> (X - Y) * (2**n), for positive nonzero n // (1 - X) * (-2) -> (x - 1) * 2, for all positive nonzero powers of 2
// (Y + const) * (-(2**n)) -> (-constY) * (2**n), for positive nonzero n // The "* 2" thus becomes a potential shifting opportunity.
// The "* (2**n)" thus becomes a potential shifting opportunity.
{ {
const APInt & Val = CI->getValue(); const APInt & Val = CI->getValue();
const APInt &PosVal = Val.abs(); const APInt &PosVal = Val.abs();
if (Val.isNegative() && PosVal.isPowerOf2()) { if (Val.isNegative() && PosVal.isPowerOf2()) {
Value *X = 0, *Y = 0; Value *X = 0;
ConstantInt *C1 = 0; if (match(Op0, m_Sub(m_One(), m_Value(X)))) {
if (Op0->hasOneUse() && // ConstantInt::get(Op0->getType(), 2);
(match(Op0, m_Sub(m_Value(Y), m_Value(X)))) || Value *Sub = Builder->CreateSub(X, ConstantInt::get(X->getType(), 1),
(match(Op0, m_Add(m_Value(Y), m_ConstantInt(C1))))) { "dec1");
Value *Sub; return BinaryOperator::CreateMul(Sub, ConstantInt::get(X->getType(),
if (C1) // Matched ADD of constant, negate both operands: PosVal));
Sub = Builder->CreateSub(Builder->CreateNeg(C1), Y, "subc");
else // Matched SUB, swap operands:
Sub = Builder->CreateSub(X, Y, "suba");
return
BinaryOperator::CreateMul(Sub,
ConstantInt::get(X->getType(), PosVal));
} }
} }
} }

View File

@ -2,14 +2,18 @@
; RUN: opt -S -instcombine < %s | FileCheck %s ; RUN: opt -S -instcombine < %s | FileCheck %s
target triple = "x86_64-apple-macosx10.6.6" target triple = "x86_64-apple-macosx10.6.6"
define zeroext i16 @foo1(i32 %on_off) nounwind uwtable ssp { define zeroext i16 @foo(i32 %on_off, i16* %puls) nounwind uwtable ssp {
entry: entry:
%on_off.addr = alloca i32, align 4 %on_off.addr = alloca i32, align 4
%puls.addr = alloca i16*, align 8
%a = alloca i32, align 4 %a = alloca i32, align 4
store i32 %on_off, i32* %on_off.addr, align 4 store i32 %on_off, i32* %on_off.addr, align 4
store i16* %puls, i16** %puls.addr, align 8
%tmp = load i32* %on_off.addr, align 4 %tmp = load i32* %on_off.addr, align 4
; CHECK-NOT: sub
; CHECK-NOT: mul
; (1 - %tmp) * (-2) -> (%tmp - 1) * 2
%sub = sub i32 1, %tmp %sub = sub i32 1, %tmp
; CHECK-NOT: mul i32
%mul = mul i32 %sub, -2 %mul = mul i32 %sub, -2
; CHECK: shl ; CHECK: shl
; CHECK-NEXT: add ; CHECK-NEXT: add
@ -18,40 +22,3 @@ entry:
%conv = trunc i32 %tmp1 to i16 %conv = trunc i32 %tmp1 to i16
ret i16 %conv ret i16 %conv
} }
define zeroext i16 @foo2(i32 %on_off, i32 %q) nounwind uwtable ssp {
entry:
%on_off.addr = alloca i32, align 4
%q.addr = alloca i32, align 4
%a = alloca i32, align 4
store i32 %on_off, i32* %on_off.addr, align 4
store i32 %q, i32* %q.addr, align 4
%tmp = load i32* %q.addr, align 4
%tmp1 = load i32* %on_off.addr, align 4
%sub = sub i32 %tmp, %tmp1
; CHECK-NOT: mul i32
%mul = mul i32 %sub, -4
; CHECK: sub i32
; CHECK-NEXT: shl
store i32 %mul, i32* %a, align 4
%tmp2 = load i32* %a, align 4
%conv = trunc i32 %tmp2 to i16
ret i16 %conv
}
define zeroext i16 @foo3(i32 %on_off) nounwind uwtable ssp {
entry:
%on_off.addr = alloca i32, align 4
%a = alloca i32, align 4
store i32 %on_off, i32* %on_off.addr, align 4
%tmp = load i32* %on_off.addr, align 4
%sub = sub i32 7, %tmp
; CHECK-NOT: mul i32
%mul = mul i32 %sub, -4
; CHECK: shl
; CHECK-NEXT: add
store i32 %mul, i32* %a, align 4
%tmp1 = load i32* %a, align 4
%conv = trunc i32 %tmp1 to i16
ret i16 %conv
}