[llvm] c7fce3f - [DAG] Rename computeOverflowKind -> computeOverflowForUnsignedAdd. NFC.
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Fri May 5 11:39:20 PDT 2023
Author: Simon Pilgrim
Date: 2023-05-05T19:38:54+01:00
New Revision: c7fce3f98b838324617bc46854f487798e75f3f5
URL: https://github.com/llvm/llvm-project/commit/c7fce3f98b838324617bc46854f487798e75f3f5
DIFF: https://github.com/llvm/llvm-project/commit/c7fce3f98b838324617bc46854f487798e75f3f5.diff
LOG: [DAG] Rename computeOverflowKind -> computeOverflowForUnsignedAdd. NFC.
Matches the naming convention for the equivalent ValueTracking helpers - further SelectionDAG computeOverflowFor*() helpers will be added soon.
Added:
Modified:
llvm/include/llvm/CodeGen/SelectionDAG.h
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
Removed:
################################################################################
diff --git a/llvm/include/llvm/CodeGen/SelectionDAG.h b/llvm/include/llvm/CodeGen/SelectionDAG.h
index fd185f0601be..233af5154a1f 100644
--- a/llvm/include/llvm/CodeGen/SelectionDAG.h
+++ b/llvm/include/llvm/CodeGen/SelectionDAG.h
@@ -1984,8 +1984,8 @@ class SelectionDAG {
OFK_Always,
};
- /// Determine if the result of the addition of 2 node can overflow.
- OverflowKind computeOverflowKind(SDValue N0, SDValue N1) const;
+ /// Determine if the result of the unsigned addition of 2 node can overflow.
+ OverflowKind computeOverflowForUnsignedAdd(SDValue N0, SDValue N1) const;
/// Test if the given value is known to have exactly one bit set. This
diff ers
/// from computeKnownBits in that it doesn't necessarily determine which bit
diff --git a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
index b489321365d5..c8140aff50da 100644
--- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -2950,7 +2950,7 @@ SDValue DAGCombiner::visitADDSAT(SDNode *N) {
// If it cannot overflow, transform into an add.
if (Opcode == ISD::UADDSAT)
- if (DAG.computeOverflowKind(N0, N1) == SelectionDAG::OFK_Never)
+ if (DAG.computeOverflowForUnsignedAdd(N0, N1) == SelectionDAG::OFK_Never)
return DAG.getNode(ISD::ADD, DL, VT, N0, N1);
return SDValue();
@@ -3140,7 +3140,7 @@ SDValue DAGCombiner::visitADDC(SDNode *N) {
DL, MVT::Glue));
// If it cannot overflow, transform into an add.
- if (DAG.computeOverflowKind(N0, N1) == SelectionDAG::OFK_Never)
+ if (DAG.computeOverflowForUnsignedAdd(N0, N1) == SelectionDAG::OFK_Never)
return CombineTo(N, DAG.getNode(ISD::ADD, DL, VT, N0, N1),
DAG.getNode(ISD::CARRY_FALSE, DL, MVT::Glue));
@@ -3214,7 +3214,7 @@ SDValue DAGCombiner::visitADDO(SDNode *N) {
if (!IsSigned) {
// If it cannot overflow, transform into an add.
- if (DAG.computeOverflowKind(N0, N1) == SelectionDAG::OFK_Never)
+ if (DAG.computeOverflowForUnsignedAdd(N0, N1) == SelectionDAG::OFK_Never)
return CombineTo(N, DAG.getNode(ISD::ADD, DL, VT, N0, N1),
DAG.getConstant(0, DL, CarryVT));
@@ -3246,7 +3246,7 @@ SDValue DAGCombiner::visitUADDOLike(SDValue N0, SDValue N1, SDNode *N) {
if (N1.getOpcode() == ISD::UADDO_CARRY && isNullConstant(N1.getOperand(1))) {
SDValue Y = N1.getOperand(0);
SDValue One = DAG.getConstant(1, SDLoc(N), Y.getValueType());
- if (DAG.computeOverflowKind(Y, One) == SelectionDAG::OFK_Never)
+ if (DAG.computeOverflowForUnsignedAdd(Y, One) == SelectionDAG::OFK_Never)
return DAG.getNode(ISD::UADDO_CARRY, SDLoc(N), N->getVTList(), N0, Y,
N1.getOperand(2));
}
diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
index faae1cfc0a34..33defdc9dde5 100644
--- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
@@ -3943,8 +3943,8 @@ KnownBits SelectionDAG::computeKnownBits(SDValue Op, const APInt &DemandedElts,
return Known;
}
-SelectionDAG::OverflowKind SelectionDAG::computeOverflowKind(SDValue N0,
- SDValue N1) const {
+SelectionDAG::OverflowKind
+SelectionDAG::computeOverflowForUnsignedAdd(SDValue N0, SDValue N1) const {
// X + 0 never overflow
if (isNullConstant(N1))
return OFK_Never;
More information about the llvm-commits
mailing list