mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-02-09 13:33:17 +00:00
Move pattern check outside of the if-then statement. This prevents us from fiddling with constants unless we have to.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60340 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
d0f5bfc9be
commit
9c531a0f07
@ -2928,17 +2928,19 @@ Instruction *InstCombiner::visitSDiv(BinaryOperator &I) {
|
||||
if (RHS->isAllOnesValue())
|
||||
return BinaryOperator::CreateNeg(Op0);
|
||||
|
||||
ConstantInt *RHSNeg = cast<ConstantInt>(ConstantExpr::getNeg(RHS));
|
||||
APInt RHSNegAPI(RHSNeg->getValue());
|
||||
|
||||
APInt NegOne = -APInt(RHSNeg->getBitWidth(), 1, true);
|
||||
APInt TwoToExp(RHSNeg->getBitWidth(), 1 << (RHSNeg->getBitWidth() - 1));
|
||||
|
||||
// -X/C -> X/-C, if and only if negation doesn't overflow.
|
||||
if ((RHS->getValue().isNegative() && RHSNegAPI.slt(TwoToExp - 1)) ||
|
||||
(RHS->getValue().isNonNegative() && RHSNegAPI.sgt(TwoToExp * NegOne))) {
|
||||
if (Value *LHSNeg = dyn_castNegVal(Op0)) {
|
||||
if (ConstantInt *CI = dyn_cast<ConstantInt>(LHSNeg)) {
|
||||
if (Value *LHSNeg = dyn_castNegVal(Op0)) {
|
||||
if (ConstantInt *CI = dyn_cast<ConstantInt>(LHSNeg)) {
|
||||
ConstantInt *RHSNeg = cast<ConstantInt>(ConstantExpr::getNeg(RHS));
|
||||
APInt RHSNegAPI(RHSNeg->getValue());
|
||||
|
||||
APInt NegOne = -APInt(RHSNeg->getBitWidth(), 1, true);
|
||||
APInt TwoToExp(RHSNeg->getBitWidth(), 1 << (RHSNeg->getBitWidth() - 1));
|
||||
|
||||
if ((RHS->getValue().isNegative() &&
|
||||
RHSNegAPI.slt(TwoToExp - 1)) ||
|
||||
(RHS->getValue().isNonNegative() &&
|
||||
RHSNegAPI.sgt(TwoToExp * NegOne))) {
|
||||
ConstantInt *CINeg = cast<ConstantInt>(ConstantExpr::getNeg(CI));
|
||||
APInt CINegAPI(CINeg->getValue());
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user