Skip to content

Commit 7d659c6

Browse files
committed
[LegalizeIntegerTypes] Rename NewLHS/NewRHS arguments to DAGTypeLegalizer::PromoteSetCCOperands. NFC
The 'New' only makes sense in the context of these being output arguments, but they are also used as inputs first. Drop the 'New' and just call them LHS/RHS. Factored out of D116421.
1 parent a699b2f commit 7d659c6

File tree

1 file changed

+14
-14
lines changed

1 file changed

+14
-14
lines changed

llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1704,7 +1704,7 @@ bool DAGTypeLegalizer::PromoteIntegerOperand(SDNode *N, unsigned OpNo) {
17041704

17051705
/// PromoteSetCCOperands - Promote the operands of a comparison. This code is
17061706
/// shared among BR_CC, SELECT_CC, and SETCC handlers.
1707-
void DAGTypeLegalizer::PromoteSetCCOperands(SDValue &NewLHS,SDValue &NewRHS,
1707+
void DAGTypeLegalizer::PromoteSetCCOperands(SDValue &LHS, SDValue &RHS,
17081708
ISD::CondCode CCCode) {
17091709
// We have to insert explicit sign or zero extends. Note that we could
17101710
// insert sign extends for ALL conditions. For those operations where either
@@ -1714,38 +1714,38 @@ void DAGTypeLegalizer::PromoteSetCCOperands(SDValue &NewLHS,SDValue &NewRHS,
17141714
default: llvm_unreachable("Unknown integer comparison!");
17151715
case ISD::SETEQ:
17161716
case ISD::SETNE: {
1717-
SDValue OpL = GetPromotedInteger(NewLHS);
1718-
SDValue OpR = GetPromotedInteger(NewRHS);
1717+
SDValue OpL = GetPromotedInteger(LHS);
1718+
SDValue OpR = GetPromotedInteger(RHS);
17191719

17201720
// We would prefer to promote the comparison operand with sign extension.
17211721
// If the width of OpL/OpR excluding the duplicated sign bits is no greater
1722-
// than the width of NewLHS/NewRH, we can avoid inserting real truncate
1722+
// than the width of LHS/RHS, we can avoid inserting real truncate
17231723
// instruction, which is redundant eventually.
17241724
unsigned OpLEffectiveBits = DAG.ComputeMinSignedBits(OpL);
17251725
unsigned OpREffectiveBits = DAG.ComputeMinSignedBits(OpR);
1726-
if (OpLEffectiveBits <= NewLHS.getScalarValueSizeInBits() &&
1727-
OpREffectiveBits <= NewRHS.getScalarValueSizeInBits()) {
1728-
NewLHS = OpL;
1729-
NewRHS = OpR;
1726+
if (OpLEffectiveBits <= LHS.getScalarValueSizeInBits() &&
1727+
OpREffectiveBits <= RHS.getScalarValueSizeInBits()) {
1728+
LHS = OpL;
1729+
RHS = OpR;
17301730
} else {
1731-
NewLHS = SExtOrZExtPromotedInteger(NewLHS);
1732-
NewRHS = SExtOrZExtPromotedInteger(NewRHS);
1731+
LHS = SExtOrZExtPromotedInteger(LHS);
1732+
RHS = SExtOrZExtPromotedInteger(RHS);
17331733
}
17341734
break;
17351735
}
17361736
case ISD::SETUGE:
17371737
case ISD::SETUGT:
17381738
case ISD::SETULE:
17391739
case ISD::SETULT:
1740-
NewLHS = SExtOrZExtPromotedInteger(NewLHS);
1741-
NewRHS = SExtOrZExtPromotedInteger(NewRHS);
1740+
LHS = SExtOrZExtPromotedInteger(LHS);
1741+
RHS = SExtOrZExtPromotedInteger(RHS);
17421742
break;
17431743
case ISD::SETGE:
17441744
case ISD::SETGT:
17451745
case ISD::SETLT:
17461746
case ISD::SETLE:
1747-
NewLHS = SExtPromotedInteger(NewLHS);
1748-
NewRHS = SExtPromotedInteger(NewRHS);
1747+
LHS = SExtPromotedInteger(LHS);
1748+
RHS = SExtPromotedInteger(RHS);
17491749
break;
17501750
}
17511751
}

0 commit comments

Comments
 (0)