SelectionDAG: Don't do libcall on div/rem if divrem is custom

[jvesely]

Summary

This removes the need for backends to set DIV/REM pairs to custom if they implement divrem

[majnemer]

This needs a test case.

[jvesely]

the following patch connects tests for r600 that exercise this codepath

http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20150406/270202.html

[jvesely]

ping

[jvesely]

ping. are the r600 tests not good enough?

[ab]

lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp

2185

TargetLoweringBase:: -> TargetLowering::

2186

RES -> Res

Also, what about 'SDNode *Res'? Doing an explicit SDValue(Res, 0/1) is clearer, I think.

2187

I see we have Ops[2] a few lines down, can you move that above and just pass it here?

[jvesely]

v2:
TargetLoweringBase:: -> TargetLowering::
Use Ops array

i considered using SDNode, but the line
SDNode *Res = DAG.getNode(..................).getNode();
looked a bit long/awkward.
I can add the change in v3 if you really prefer it that way.

[ab]

I'm not a big fan, but I don't have a better alternative, so, LGTM, thanks!

In D7803#177402, @jvesely wrote:

i considered using SDNode, but the line
SDNode *Res = DAG.getNode(..................).getNode();
looked a bit long/awkward.

Fair enough. How about using an explicit Res.getValue(0) for DIV instead?

[ab]

lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp

2551

Also, unnecessary newline.

lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp

void DAGTypeLegalizer::ExpandIntRes_SADDSUBO(SDNode *Node,
   ReplaceValueWith(SDValue(Node, 1), Cmp);
 }
 
 void DAGTypeLegalizer::ExpandIntRes_SDIV(SDNode *N,
                                          SDValue &Lo, SDValue &Hi) {
   EVT VT = N->getValueType(0);
   SDLoc dl(N);
 
+  if (TLI.getOperationAction(ISD::SDIVREM, VT) == TargetLoweringBase::Custom) {
+    SDValue RES = DAG.getNode(ISD::SDIVREM, dl, DAG.getVTList(VT, VT),
+                              N->getOperand(0), N->getOperand(1));
+    SplitInteger(RES, Lo, Hi);
+    return;
+  }
+
   RTLIB::Libcall LC = RTLIB::UNKNOWN_LIBCALL;
   if (VT == MVT::i16)
     LC = RTLIB::SDIV_I16;
   else if (VT == MVT::i32)
     LC = RTLIB::SDIV_I32;
   else if (VT == MVT::i64)
     LC = RTLIB::SDIV_I64;
   else if (VT == MVT::i128)
ExpandIntRes_SIGN_EXTEND_INREG(SDNode *N, SDValue &Lo, SDValue &Hi) {
   }
 }
 
 void DAGTypeLegalizer::ExpandIntRes_SREM(SDNode *N,
                                          SDValue &Lo, SDValue &Hi) {
   EVT VT = N->getValueType(0);
   SDLoc dl(N);
 
+  if (TLI.getOperationAction(ISD::SDIVREM, VT) == TargetLoweringBase::Custom) {
+    SDValue RES = DAG.getNode(ISD::SDIVREM, dl, DAG.getVTList(VT, VT),
+                              N->getOperand(0), N->getOperand(1));
+    SplitInteger(RES.getValue(1), Lo, Hi);
+    return;
+  }
+
   RTLIB::Libcall LC = RTLIB::UNKNOWN_LIBCALL;
   if (VT == MVT::i16)
     LC = RTLIB::SREM_I16;
   else if (VT == MVT::i32)
     LC = RTLIB::SREM_I32;
   else if (VT == MVT::i64)
     LC = RTLIB::SREM_I64;
   else if (VT == MVT::i128)
void DAGTypeLegalizer::ExpandIntRes_XMULO(SDNode *N,
   ReplaceValueWith(SDValue(N, 1), Ofl);
 }
 
 void DAGTypeLegalizer::ExpandIntRes_UDIV(SDNode *N,
                                          SDValue &Lo, SDValue &Hi) {
   EVT VT = N->getValueType(0);
   SDLoc dl(N);
 
+  if (TLI.getOperationAction(ISD::UDIVREM, VT) == TargetLoweringBase::Custom) {
+    SDValue RES = DAG.getNode(ISD::UDIVREM, dl, DAG.getVTList(VT, VT),
+                              N->getOperand(0), N->getOperand(1));
+    SplitInteger(RES, Lo, Hi);
+    return;
+  }
+
   RTLIB::Libcall LC = RTLIB::UNKNOWN_LIBCALL;
   if (VT == MVT::i16)
     LC = RTLIB::UDIV_I16;
   else if (VT == MVT::i32)
     LC = RTLIB::UDIV_I32;
   else if (VT == MVT::i64)
     LC = RTLIB::UDIV_I64;
   else if (VT == MVT::i128)
     LC = RTLIB::UDIV_I128;
   assert(LC != RTLIB::UNKNOWN_LIBCALL && "Unsupported UDIV!");
 
   SDValue Ops[2] = { N->getOperand(0), N->getOperand(1) };
   SplitInteger(TLI.makeLibCall(DAG, LC, VT, Ops, 2, false, dl).first, Lo, Hi);
 }
 
 void DAGTypeLegalizer::ExpandIntRes_UREM(SDNode *N,
                                          SDValue &Lo, SDValue &Hi) {
   EVT VT = N->getValueType(0);
   SDLoc dl(N);
 
+  if (TLI.getOperationAction(ISD::UDIVREM, VT) == TargetLoweringBase::Custom) {
+    SDValue RES = DAG.getNode(ISD::UDIVREM, dl, DAG.getVTList(VT, VT),
+                              N->getOperand(0), N->getOperand(1));
+    SplitInteger(RES.getValue(1), Lo, Hi);
+    return;
+  }
+
+
   RTLIB::Libcall LC = RTLIB::UNKNOWN_LIBCALL;
   if (VT == MVT::i16)
     LC = RTLIB::UREM_I16;
   else if (VT == MVT::i32)
     LC = RTLIB::UREM_I32;
   else if (VT == MVT::i64)
     LC = RTLIB::UREM_I64;
   else if (VT == MVT::i128)