diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2019-05-28 18:08:06 +0000 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2019-05-28 18:08:06 +0000 |
commit | 2fb0a820df9c9884b9f42efdd0919b309e2b1204 (patch) | |
tree | c6b6e7bccdf4b9d9ebb7a49719c1e657c2d2ff3f /llvm/lib/IR/IntrinsicInst.cpp | |
parent | 76696654322bd502cfd4592e48fc6a1782de5694 (diff) | |
download | bcm5719-llvm-2fb0a820df9c9884b9f42efdd0919b309e2b1204.tar.gz bcm5719-llvm-2fb0a820df9c9884b9f42efdd0919b309e2b1204.zip |
[IR] Add SaturatingInst and BinaryOpIntrinsic classes
Based on the suggestion in D62447, this adds a SaturatingInst class
that represents the saturating add/sub family of intrinsics. It
exposes the same interface as WithOverflowInst, for this reason I
have also added a common base class BinaryOpIntrinsic that holds the
actual implementation code and will be useful in some places handling
both overflowing and saturating math.
Differential Revision: https://reviews.llvm.org/D62466
llvm-svn: 361857
Diffstat (limited to 'llvm/lib/IR/IntrinsicInst.cpp')
-rw-r--r-- | llvm/lib/IR/IntrinsicInst.cpp | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/llvm/lib/IR/IntrinsicInst.cpp b/llvm/lib/IR/IntrinsicInst.cpp index 7ff8631c76f..793e2895dce 100644 --- a/llvm/lib/IR/IntrinsicInst.cpp +++ b/llvm/lib/IR/IntrinsicInst.cpp @@ -171,13 +171,17 @@ bool ConstrainedFPIntrinsic::isTernaryOp() const { } } -Instruction::BinaryOps WithOverflowInst::getBinaryOp() const { +Instruction::BinaryOps BinaryOpIntrinsic::getBinaryOp() const { switch (getIntrinsicID()) { case Intrinsic::uadd_with_overflow: case Intrinsic::sadd_with_overflow: + case Intrinsic::uadd_sat: + case Intrinsic::sadd_sat: return Instruction::Add; case Intrinsic::usub_with_overflow: case Intrinsic::ssub_with_overflow: + case Intrinsic::usub_sat: + case Intrinsic::ssub_sat: return Instruction::Sub; case Intrinsic::umul_with_overflow: case Intrinsic::smul_with_overflow: @@ -187,18 +191,20 @@ Instruction::BinaryOps WithOverflowInst::getBinaryOp() const { } } -bool WithOverflowInst::isSigned() const { +bool BinaryOpIntrinsic::isSigned() const { switch (getIntrinsicID()) { case Intrinsic::sadd_with_overflow: case Intrinsic::ssub_with_overflow: case Intrinsic::smul_with_overflow: + case Intrinsic::sadd_sat: + case Intrinsic::ssub_sat: return true; default: return false; } } -unsigned WithOverflowInst::getNoWrapKind() const { +unsigned BinaryOpIntrinsic::getNoWrapKind() const { if (isSigned()) return OverflowingBinaryOperator::NoSignedWrap; else |