[llvm-commits] [llvm] r59788 - /llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp

Bill Wendling isanbard at gmail.com
Thu Nov 20 18:38:44 PST 2008


Author: void
Date: Thu Nov 20 20:38:44 2008
New Revision: 59788

URL: http://llvm.org/viewvc/llvm-project?rev=59788&view=rev
Log:
Combine the two add with overflow intrinsics lowerings. They differ only in DAG node type.

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp?rev=59788&r1=59787&r2=59788&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp Thu Nov 20 20:38:44 2008
@@ -4093,8 +4093,8 @@
     return 0;
   }
 
+  case Intrinsic::uadd_with_overflow:
   case Intrinsic::sadd_with_overflow: {
-    // Convert to "ISD::SADDO" instruction.
     SDValue Op1 = getValue(I.getOperand(1));
     SDValue Op2 = getValue(I.getOperand(2));
     MVT Ty = Op1.getValueType();
@@ -4102,23 +4102,10 @@
     MVT ValueVTs[] = { Ty, MVT::i1 };
     SDValue Ops[] = { Op1, Op2 };
 
-    SDValue Result = DAG.getNode(ISD::SADDO, DAG.getVTList(&ValueVTs[0], 2),
-                                 &Ops[0], 2);
-
-    setValue(&I, Result);
-    return 0;
-  }
-  case Intrinsic::uadd_with_overflow: {
-    // Convert to "ISD::UADDO" instruction.
-    SDValue Op1 = getValue(I.getOperand(1));
-    SDValue Op2 = getValue(I.getOperand(2));
-    MVT Ty = Op1.getValueType();
-
-    MVT ValueVTs[] = { Ty, MVT::i1 };
-    SDValue Ops[] = { Op1, Op2 };
-
-    SDValue Result = DAG.getNode(ISD::UADDO, DAG.getVTList(&ValueVTs[0], 2),
-                                 &Ops[0], 2);
+    SDValue Result =
+      DAG.getNode((Intrinsic == Intrinsic::sadd_with_overflow) ?
+                    ISD::SADDO : ISD::UADDO,
+                  DAG.getVTList(&ValueVTs[0], 2), &Ops[0], 2);
 
     setValue(&I, Result);
     return 0;





More information about the llvm-commits mailing list