Redo previous patch so optimization only done for i1.

Simpler and safer.

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

diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
index fd0df3a0a1174c8dfe0c8ecb4a7af7bdf3dc91d1..1400c3ee4f92428abedcd8a80931315d6ca29709 100644 (file)
--- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -2455,10 +2455,6 @@ SDOperand DAGCombiner::visitSELECT(SDNode *N) {
   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 @@ SDOperand DAGCombiner::visitSELECT(SDNode *N) {
     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);
   }