[llvm-commits] [llvm] r44663 - /llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

[Dale Johannesen]

Author: johannes
Date: Thu Dec  6 11:53:31 2007
New Revision: 44663

URL: http://llvm.org/viewvc/llvm-project?rev=44663&view=rev
Log:
Redo previous patch so optimization only done for i1.
Simpler and safer.


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

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

==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Thu Dec  6 11:53:31 2007
@@ -2455,10 +2455,6 @@
   MVT::ValueType VT = N->getValueType(0);
   MVT::ValueType VT0 = N0.getValueType();
 
-
-  // Some targets have SETCC types bigger than 1 bit, but do not set all the
-  // bits to 1; identified by getSetCCResultContents.  Watch out for these.
-
   // fold select C, X, X -> X
   if (N1 == N2)
     return N1;
@@ -2483,22 +2479,14 @@
     return DAG.getNode(ISD::TRUNCATE, VT, XORNode);
   }
   // fold select C, 0, X -> ~C & X
-  if (VT == VT0 && N1C && N1C->isNullValue() &&
-      (N0.Val->getOpcode()!=ISD::SETCC || VT==MVT::i1 ||
-       TLI.getSetCCResultContents()==
-          TargetLowering::ZeroOrNegativeOneSetCCResult)) {
-    SDOperand XORNode;
-    XORNode = DAG.getNode(ISD::XOR, VT, N0, DAG.getConstant(~0UL, VT));
+  if (VT == VT0 && VT == MVT::i1 && N1C && N1C->isNullValue()) {
+    SDOperand XORNode = DAG.getNode(ISD::XOR, VT, N0, DAG.getConstant(1, VT));
     AddToWorkList(XORNode.Val);
     return DAG.getNode(ISD::AND, VT, XORNode, N2);
   }
   // fold select C, X, 1 -> ~C | X
-  if (VT == VT0 && N2C && N2C->getValue() == 1 &&
-      (N0.Val->getOpcode()!=ISD::SETCC || VT==MVT::i1 ||
-       TLI.getSetCCResultContents()==
-          TargetLowering::ZeroOrNegativeOneSetCCResult)) {
-    SDOperand XORNode;
-    XORNode = DAG.getNode(ISD::XOR, VT, N0, DAG.getConstant(~0UL, VT));
+  if (VT == VT0 && VT == MVT::i1 && N2C && N2C->getValue() == 1) {
+    SDOperand XORNode = DAG.getNode(ISD::XOR, VT, N0, DAG.getConstant(1, VT));
     AddToWorkList(XORNode.Val);
     return DAG.getNode(ISD::OR, VT, XORNode, N1);
   }