mirror of
https://github.com/RPCSX/llvm.git
synced 2024-11-26 13:10:34 +00:00
Transform -X/C to X/-C, implementing a README.txt entry.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@78812 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
fa94b948d9
commit
9ca9daad21
@ -1111,16 +1111,6 @@ optimized with "clang -emit-llvm-bc | opt -std-compile-opts".
|
||||
|
||||
//===---------------------------------------------------------------------===//
|
||||
|
||||
We would like to do the following transform in the instcombiner:
|
||||
|
||||
-X/C -> X/-C
|
||||
|
||||
However, this isn't valid if (-X) overflows. We can implement this when we
|
||||
have the concept of a "C signed subtraction" operator that which is undefined
|
||||
on overflow.
|
||||
|
||||
//===---------------------------------------------------------------------===//
|
||||
|
||||
This was noticed in the entryblock for grokdeclarator in 403.gcc:
|
||||
|
||||
%tmp = icmp eq i32 %decl_context, 4
|
||||
|
@ -3077,6 +3077,14 @@ Instruction *InstCombiner::visitSDiv(BinaryOperator &I) {
|
||||
RHS->getValue().exactLogBase2());
|
||||
return BinaryOperator::CreateAShr(Op0, ShAmt, I.getName());
|
||||
}
|
||||
|
||||
// -X/C --> X/-C provided the negation doesn't overflow.
|
||||
if (SubOperator *Sub = dyn_cast<SubOperator>(Op0))
|
||||
if (isa<Constant>(Sub->getOperand(0)) &&
|
||||
cast<Constant>(Sub->getOperand(0))->isNullValue() &&
|
||||
Sub->hasNoSignedOverflow())
|
||||
return BinaryOperator::CreateSDiv(Sub->getOperand(1),
|
||||
ConstantExpr::getNeg(RHS));
|
||||
}
|
||||
|
||||
// If the sign bits of both operands are zero (i.e. we can prove they are
|
||||
|
20
test/Transforms/InstCombine/nsw.ll
Normal file
20
test/Transforms/InstCombine/nsw.ll
Normal file
@ -0,0 +1,20 @@
|
||||
; RUN: llvm-as < %s | opt -instcombine | llvm-dis | FileCheck %s
|
||||
|
||||
; CHECK: define i32 @foo
|
||||
; %y = sub i32 0, %x
|
||||
; %z = sdiv i32 %y, 337
|
||||
; ret i32 %y
|
||||
define i32 @foo(i32 %x) {
|
||||
%y = sub i32 0, %x
|
||||
%z = sdiv i32 %y, 337
|
||||
ret i32 %y
|
||||
}
|
||||
|
||||
; CHECK: define i32 @bar
|
||||
; %y = sdiv i32 %x, -337
|
||||
; ret i32 %y
|
||||
define i32 @bar(i32 %x) {
|
||||
%y = sub nsw i32 0, %x
|
||||
%z = sdiv i32 %y, 337
|
||||
ret i32 %y
|
||||
}
|
Loading…
Reference in New Issue
Block a user