Remove the hack that turned an {S,U}MUL_LOHI with an unused high
authorDan Gohman <gohman@apple.com>
Mon, 25 Feb 2008 21:57:04 +0000 (21:57 +0000)
committerDan Gohman <gohman@apple.com>
Mon, 25 Feb 2008 21:57:04 +0000 (21:57 +0000)
result into a MUL late in the X86 codegen process. ISD::MUL is
once again Legal on X86, so this is no longer needed. And, the
hack was suboptimal; see PR1874 for details.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47567 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelDAGToDAG.cpp

index c8b341406ca2dd63495918bb9ea429b7083cfff3..75e9faecb290b3f72e95c2b170acf1e94153e1c1 100644 (file)
@@ -1224,15 +1224,6 @@ SDNode *X86DAGToDAGISel::Select(SDOperand N) {
       SDOperand N0 = Node->getOperand(0);
       SDOperand N1 = Node->getOperand(1);
 
-      // There are several forms of IMUL that just return the low part and
-      // don't have fixed-register operands. If we don't need the high part,
-      // use these instead. They can be selected with the generated ISel code.
-      if (NVT != MVT::i8 &&
-          N.getValue(1).use_empty()) {
-        N = CurDAG->getNode(ISD::MUL, NVT, N0, N1);
-        break;
-      }
-
       bool isSigned = Opcode == ISD::SMUL_LOHI;
       if (!isSigned)
         switch (NVT) {