diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-07-06 18:39:47 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-07-06 18:39:47 +0000 |
| commit | 79ab643da842b03da893a0188426450525e6ada5 (patch) | |
| tree | 6a1a91dac806eca4ea8b6b95373193626e0d94a6 /llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp | |
| parent | eb6d5d19508c5485aafb7f9a30dee819a3b64352 (diff) | |
| download | bcm5719-llvm-79ab643da842b03da893a0188426450525e6ada5.tar.gz bcm5719-llvm-79ab643da842b03da893a0188426450525e6ada5.zip | |
[Constants] If we already have a ConstantInt*, prefer to use isZero/isOne/isMinusOne instead of isNullValue/isOneValue/isAllOnesValue inherited from Constant. NFCI
Going through the Constant methods requires redetermining that the Constant is a ConstantInt and then calling isZero/isOne/isMinusOne.
llvm-svn: 307292
Diffstat (limited to 'llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp b/llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp index 9f4d9c7e398..d9294c49930 100644 --- a/llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp +++ b/llvm/lib/Transforms/Utils/CmpInstAnalysis.cpp @@ -81,7 +81,7 @@ bool llvm::decomposeBitTestICmp(const ICmpInst *I, CmpInst::Predicate &Pred, break; case ICmpInst::ICMP_SGT: // X > -1 is equivalent to (X & SignMask) == 0. - if (!C->isAllOnesValue()) + if (!C->isMinusOne()) return false; Y = ConstantInt::get(I->getContext(), APInt::getSignMask(C->getBitWidth())); Pred = ICmpInst::ICMP_EQ; |
