From e727d7a08406265acb10d0c52943ef92859710f2 Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Thu, 28 May 2009 04:49:34 +0000 Subject: [PATCH] Remove a couple of useless functions. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72516 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/SelectionDAG/LegalizeDAG.cpp | 40 ++++-------------------- 1 file changed, 6 insertions(+), 34 deletions(-) diff --git a/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp b/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp index 1d57d9a62af..d6f5b9711bb 100644 --- a/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp +++ b/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp @@ -136,15 +136,8 @@ private: bool LegalizeAllNodesNotLeadingTo(SDNode *N, SDNode *Dest, SmallPtrSet &NodesLeadingTo); - void LegalizeSetCCOperands(SDValue &LHS, SDValue &RHS, SDValue &CC, - DebugLoc dl); void LegalizeSetCCCondCode(MVT VT, SDValue &LHS, SDValue &RHS, SDValue &CC, DebugLoc dl); - void LegalizeSetCC(MVT VT, SDValue &LHS, SDValue &RHS, SDValue &CC, - DebugLoc dl) { - LegalizeSetCCOperands(LHS, RHS, CC, dl); - LegalizeSetCCCondCode(VT, LHS, RHS, CC, dl); - } SDValue ExpandLibCall(RTLIB::Libcall LC, SDNode *Node, bool isSigned); SDValue ExpandFPLibCall(SDNode *Node, RTLIB::Libcall Call_F32, @@ -1624,20 +1617,6 @@ void SelectionDAGLegalize::ExpandDYNAMIC_STACKALLOC(SDNode* Node, Results.push_back(Tmp2); } -/// LegalizeSetCCOperands - Attempts to create a legal LHS and RHS for a SETCC -/// with condition CC on the current target. This usually involves legalizing -/// or promoting the arguments. In the case where LHS and RHS must be expanded, -/// there may be no choice but to create a new SetCC node to represent the -/// legalized value of setcc lhs, rhs. In this case, the value is returned in -/// LHS, and the SDValue returned in RHS has a nil SDNode value. -void SelectionDAGLegalize::LegalizeSetCCOperands(SDValue &LHS, - SDValue &RHS, - SDValue &CC, - DebugLoc dl) { - LHS = LegalizeOp(LHS); - RHS = LegalizeOp(RHS); -} - /// LegalizeSetCCCondCode - Legalize a SETCC with given LHS and RHS and /// condition code CC on the current target. This routine assumes LHS and rHS /// have already been legalized by LegalizeSetCCOperands. It expands SETCC with @@ -2897,7 +2876,7 @@ void SelectionDAGLegalize::ExpandNode(SDNode *Node, Tmp1 = Node->getOperand(0); Tmp2 = Node->getOperand(1); Tmp3 = Node->getOperand(2); - LegalizeSetCC(Node->getValueType(0), Tmp1, Tmp2, Tmp3, dl); + LegalizeSetCCCondCode(Node->getValueType(0), Tmp1, Tmp2, Tmp3, dl); // If we expanded the SETCC into an AND/OR, return the new node if (Tmp2.getNode() == 0) { @@ -2920,8 +2899,8 @@ void SelectionDAGLegalize::ExpandNode(SDNode *Node, Tmp4 = Node->getOperand(3); // False SDValue CC = Node->getOperand(4); - LegalizeSetCC(TLI.getSetCCResultType(Tmp1.getValueType()), - Tmp1, Tmp2, CC, dl); + LegalizeSetCCCondCode(TLI.getSetCCResultType(Tmp1.getValueType()), + Tmp1, Tmp2, CC, dl); assert(!Tmp2.getNode() && "Can't legalize SELECT_CC with legal condition!"); Tmp2 = DAG.getConstant(0, Tmp1.getValueType()); @@ -2937,18 +2916,11 @@ void SelectionDAGLegalize::ExpandNode(SDNode *Node, Tmp3 = Node->getOperand(3); // RHS Tmp4 = Node->getOperand(1); // CC - LegalizeSetCC(TLI.getSetCCResultType(Tmp2.getValueType()), - Tmp2, Tmp3, Tmp4, dl); + LegalizeSetCCCondCode(TLI.getSetCCResultType(Tmp2.getValueType()), + Tmp2, Tmp3, Tmp4, dl); LastCALLSEQ_END = DAG.getEntryNode(); - // If we didn't get both a LHS and RHS back from LegalizeSetCC, - // the LHS is a legal SETCC itself. In this case, we need to compare - // the result against zero to select between true and false values. - if (Tmp3.getNode() == 0) { - Tmp3 = DAG.getConstant(0, Tmp2.getValueType()); - Tmp4 = DAG.getCondCode(ISD::SETNE); - } - + assert(!Tmp2.getNode() && "Can't legalize BR_CC with legal condition!"); Tmp1 = DAG.getNode(ISD::BR_CC, dl, Node->getValueType(0), Tmp1, Tmp4, Tmp2, Tmp3, Node->getOperand(4)); Results.push_back(Tmp1);