diff options
author | Chris Lattner <sabre@nondot.org> | 2006-09-18 04:31:40 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2006-09-18 04:31:40 +0000 |
commit | b7363791edfbbc017fd6670c20682aca9f2c543a (patch) | |
tree | 7ad8e49c073523d5ead86d81961dedc3d9a5b51f | |
parent | 2f88a1bbbed9e73411ac89684d9ebd64b26da2ee (diff) |
Implement Transforms/InstCombine/shift-sra.ll:test0
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@30450 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/Scalar/InstructionCombining.cpp | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 4d7dfe9e70..4ed827944d 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -1089,6 +1089,26 @@ bool InstCombiner::SimplifyDemandedBits(Value *V, uint64_t DemandedMask, } break; case Instruction::Shr: + // If this is an arithmetic shift right and only the low-bit is set, we can + // always convert this into a logical shr, even if the shift amount is + // variable. The low bit of the shift cannot be an input sign bit unless + // the shift amount is >= the size of the datatype, which is undefined. + if (DemandedMask == 1 && I->getType()->isSigned()) { + // Convert the input to unsigned. + Instruction *NewVal = new CastInst(I->getOperand(0), + I->getType()->getUnsignedVersion(), + I->getOperand(0)->getName()); + InsertNewInstBefore(NewVal, *I); + // Perform the unsigned shift right. + NewVal = new ShiftInst(Instruction::Shr, NewVal, I->getOperand(1), + I->getName()); + InsertNewInstBefore(NewVal, *I); + // Then cast that to the destination type. + NewVal = new CastInst(NewVal, I->getType(), I->getName()); + InsertNewInstBefore(NewVal, *I); + return UpdateValueUsesWith(I, NewVal); + } + if (ConstantUInt *SA = dyn_cast<ConstantUInt>(I->getOperand(1))) { unsigned ShAmt = SA->getValue(); |