Fix rev 96389 by restricting the xform to mask that's either signbit or max signed...
authorEvan Cheng <evan.cheng@apple.com>
Tue, 23 Feb 2010 21:51:54 +0000 (21:51 +0000)
committerEvan Cheng <evan.cheng@apple.com>
Tue, 23 Feb 2010 21:51:54 +0000 (21:51 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@96990 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelLowering.cpp

index 972773176043d902a67eef720b8da660298070ce..d59c1a82e2f5a180ab5a0b54f7283482bdbbbeac 100644 (file)
@@ -9092,9 +9092,10 @@ static SDValue PerformCMOVCombine(SDNode *N, SelectionDAG &DAG,
 }
 
 /// PerformANDCombine - Look for SSE and instructions of this form:
-/// (and x, (build_vector c1,c2,c3,c4)). If there exists a use of a build_vector
-/// that's the bitwise complement of the mask, then transform the node to
-/// (and (xor x, (build_vector -1,-1,-1,-1)), (build_vector ~c1,~c2,~c3,~c4)).
+/// (and x, (build_vector signbit,signbit,signbit,signbit)). If there
+/// exists a use of a build_vector that's the bitwise complement of the mask,
+/// then transform the node to
+/// (and (xor x, (build_vector -1,-1,-1,-1)), (build_vector ~sb,~sb,~sb,~sb)).
 static SDValue PerformANDCombine(SDNode *N, SelectionDAG &DAG,
                                  TargetLowering::DAGCombinerInfo &DCI) {
   EVT VT = N->getValueType(0);
@@ -9118,7 +9119,11 @@ static SDValue PerformANDCombine(SDNode *N, SelectionDAG &DAG,
         continue;
       }
       ConstantSDNode *C = dyn_cast<ConstantSDNode>(Arg);
-      if (!C) return SDValue();
+      if (!C)
+        return SDValue();
+      if (!C->getAPIntValue().isSignBit() &&
+          !C->getAPIntValue().isMaxSignedValue())
+        return SDValue();
       Mask.push_back(DAG.getConstant(~C->getAPIntValue(), EltVT));
     }
     N1 = DAG.getNode(ISD::BUILD_VECTOR, N1.getDebugLoc(), VT,