diff options
author | phyBrackets <singh.shivamsingh2003@gmail.com> | 2022-02-11 22:34:05 +0530 |
---|---|---|
committer | phyBrackets <singh.shivamsingh2003@gmail.com> | 2022-02-11 22:34:05 +0530 |
commit | 4f72fbab3d1456be8e14185443b5271df21eb995 (patch) | |
tree | d92843f5e773bbad228708b432b802a40e1a05a1 | |
parent | 3b470d1ce992407df81662c04cf0137850338af6 (diff) |
Refactor nested if else with ternary operator in CGExprScalar.cpp
Reviewed By: rjmccall
Differential Revision: https://reviews.llvm.org/D119364
-rw-r--r-- | clang/lib/CodeGen/CGExprScalar.cpp | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/clang/lib/CodeGen/CGExprScalar.cpp b/clang/lib/CodeGen/CGExprScalar.cpp index 59d0bd08d33d..a23563905493 100644 --- a/clang/lib/CodeGen/CGExprScalar.cpp +++ b/clang/lib/CodeGen/CGExprScalar.cpp @@ -65,20 +65,14 @@ bool mayHaveIntegerOverflow(llvm::ConstantInt *LHS, llvm::ConstantInt *RHS, const auto &LHSAP = LHS->getValue(); const auto &RHSAP = RHS->getValue(); if (Opcode == BO_Add) { - if (Signed) - Result = LHSAP.sadd_ov(RHSAP, Overflow); - else - Result = LHSAP.uadd_ov(RHSAP, Overflow); + Result = Signed ? LHSAP.sadd_ov(RHSAP, Overflow) + : LHSAP.uadd_ov(RHSAP, Overflow); } else if (Opcode == BO_Sub) { - if (Signed) - Result = LHSAP.ssub_ov(RHSAP, Overflow); - else - Result = LHSAP.usub_ov(RHSAP, Overflow); + Result = Signed ? LHSAP.ssub_ov(RHSAP, Overflow) + : LHSAP.usub_ov(RHSAP, Overflow); } else if (Opcode == BO_Mul) { - if (Signed) - Result = LHSAP.smul_ov(RHSAP, Overflow); - else - Result = LHSAP.umul_ov(RHSAP, Overflow); + Result = Signed ? LHSAP.smul_ov(RHSAP, Overflow) + : LHSAP.umul_ov(RHSAP, Overflow); } else if (Opcode == BO_Div || Opcode == BO_Rem) { if (Signed && !RHS->isZero()) Result = LHSAP.sdiv_ov(RHSAP, Overflow); |