diff options
author | Craig Topper <craig.topper@gmail.com> | 2017-04-06 20:41:48 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@gmail.com> | 2017-04-06 20:41:48 +0000 |
commit | 7226d796aa406699c291adcb3e3f87d07425dcae (patch) | |
tree | 3875ecaec0298b823eb3fb436fcc436154b5b468 /llvm/lib/Transforms/InstCombine/InstCombineInternal.h | |
parent | 18ad20eab5edd6ea0dd8d8c6258ea0f4a8c82c5c (diff) | |
download | bcm5719-llvm-7226d796aa406699c291adcb3e3f87d07425dcae.tar.gz bcm5719-llvm-7226d796aa406699c291adcb3e3f87d07425dcae.zip |
[InstCombine] Remove redundant combine from visitAnd
This combine is fully handled by SimplifyDemandedInstructionBits as of r299658 where I fixed this code to ensure the Add/Sub had only a single user. Otherwise it would fire and create additional instructions. That fix resulted in an improvement to code generated for tsan which is why I committed it before deleting.
Differential Revision: https://reviews.llvm.org/D31543
llvm-svn: 299704
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstCombineInternal.h')
-rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineInternal.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineInternal.h b/llvm/lib/Transforms/InstCombine/InstCombineInternal.h index b5b48d27a7a..ab15fd84b24 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineInternal.h +++ b/llvm/lib/Transforms/InstCombine/InstCombineInternal.h @@ -653,8 +653,6 @@ private: Instruction *OptAndOp(BinaryOperator *Op, ConstantInt *OpRHS, ConstantInt *AndRHS, BinaryOperator &TheAnd); - Value *FoldLogicalPlusAnd(Value *LHS, Value *RHS, ConstantInt *Mask, - bool isSub, Instruction &I); Value *insertRangeTest(Value *V, const APInt &Lo, const APInt &Hi, bool isSigned, bool Inside); Instruction *PromoteCastOfAllocation(BitCastInst &CI, AllocaInst &AI); |