diff options
| author | Jay Foad <jay.foad@amd.com> | 2020-03-06 15:22:11 +0000 |
|---|---|---|
| committer | Jay Foad <jay.foad@amd.com> | 2020-03-06 16:42:53 +0000 |
| commit | 11d1573bb6688d71c7f9183e084776cf8067c0fd (patch) | |
| tree | 95f4b384213c74a0fe8cc0d1d4d3aceffa9db00f /llvm/lib/Analysis/ConstantFolding.cpp | |
| parent | 6c61edcbab1ccfe456a1cade499a3c48d43c616f (diff) | |
| download | llvm-11d1573bb6688d71c7f9183e084776cf8067c0fd.zip llvm-11d1573bb6688d71c7f9183e084776cf8067c0fd.tar.gz llvm-11d1573bb6688d71c7f9183e084776cf8067c0fd.tar.bz2 | |
[APFloat] Make use of new overloaded comparison operators. NFC.
Reviewers: ekatz, spatel, jfb, tlively, craig.topper, RKSimon, nikic, scanon
Subscribers: arsenm, jvesely, nhaehnle, hiraditya, dexonsmith, kerbowa, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D75744
Diffstat (limited to 'llvm/lib/Analysis/ConstantFolding.cpp')
| -rw-r--r-- | llvm/lib/Analysis/ConstantFolding.cpp | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/llvm/lib/Analysis/ConstantFolding.cpp b/llvm/lib/Analysis/ConstantFolding.cpp index 0998f50..b73751a 100644 --- a/llvm/lib/Analysis/ConstantFolding.cpp +++ b/llvm/lib/Analysis/ConstantFolding.cpp @@ -2555,11 +2555,9 @@ bool llvm::isMathLibCallNoop(const CallBase *Call, case LibFunc_expf: // FIXME: These boundaries are slightly conservative. if (OpC->getType()->isDoubleTy()) - return Op.compare(APFloat(-745.0)) != APFloat::cmpLessThan && - Op.compare(APFloat(709.0)) != APFloat::cmpGreaterThan; + return !(Op < APFloat(-745.0) || Op > APFloat(709.0)); if (OpC->getType()->isFloatTy()) - return Op.compare(APFloat(-103.0f)) != APFloat::cmpLessThan && - Op.compare(APFloat(88.0f)) != APFloat::cmpGreaterThan; + return !(Op < APFloat(-103.0f) || Op > APFloat(88.0f)); break; case LibFunc_exp2l: @@ -2567,11 +2565,9 @@ bool llvm::isMathLibCallNoop(const CallBase *Call, case LibFunc_exp2f: // FIXME: These boundaries are slightly conservative. if (OpC->getType()->isDoubleTy()) - return Op.compare(APFloat(-1074.0)) != APFloat::cmpLessThan && - Op.compare(APFloat(1023.0)) != APFloat::cmpGreaterThan; + return !(Op < APFloat(-1074.0) || Op > APFloat(1023.0)); if (OpC->getType()->isFloatTy()) - return Op.compare(APFloat(-149.0f)) != APFloat::cmpLessThan && - Op.compare(APFloat(127.0f)) != APFloat::cmpGreaterThan; + return !(Op < APFloat(-149.0f) || Op > APFloat(127.0f)); break; case LibFunc_sinl: @@ -2601,10 +2597,8 @@ bool llvm::isMathLibCallNoop(const CallBase *Call, case LibFunc_acosl: case LibFunc_acos: case LibFunc_acosf: - return Op.compare(APFloat(Op.getSemantics(), "-1")) != - APFloat::cmpLessThan && - Op.compare(APFloat(Op.getSemantics(), "1")) != - APFloat::cmpGreaterThan; + return !(Op < APFloat(Op.getSemantics(), "-1") || + Op > APFloat(Op.getSemantics(), "1")); case LibFunc_sinh: case LibFunc_cosh: @@ -2614,11 +2608,9 @@ bool llvm::isMathLibCallNoop(const CallBase *Call, case LibFunc_coshl: // FIXME: These boundaries are slightly conservative. if (OpC->getType()->isDoubleTy()) - return Op.compare(APFloat(-710.0)) != APFloat::cmpLessThan && - Op.compare(APFloat(710.0)) != APFloat::cmpGreaterThan; + return !(Op < APFloat(-710.0) || Op > APFloat(710.0)); if (OpC->getType()->isFloatTy()) - return Op.compare(APFloat(-89.0f)) != APFloat::cmpLessThan && - Op.compare(APFloat(89.0f)) != APFloat::cmpGreaterThan; + return !(Op < APFloat(-89.0f) || Op > APFloat(89.0f)); break; case LibFunc_sqrtl: |