Revert 147426 because it caused pr11696.
authorNadav Rotem <nadav.rotem@intel.com>
Tue, 3 Jan 2012 22:19:42 +0000 (22:19 +0000)
committerNadav Rotem <nadav.rotem@intel.com>
Tue, 3 Jan 2012 22:19:42 +0000 (22:19 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@147485 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelLowering.cpp
test/CodeGen/X86/2011-12-28-vselecti8.ll
test/CodeGen/X86/sext-blend.ll [deleted file]
test/CodeGen/X86/sse2-blend.ll
test/CodeGen/X86/sse41-blend.ll

index 47b80d036e5da312cbbe44aa22ff697e11c70d79..74ae8a85bda31648673683e247a72d7e91418426 100644 (file)
@@ -13139,24 +13139,6 @@ static SDValue PerformSELECTCombine(SDNode *N, SelectionDAG &DAG,
       }
   }
 
-  // The VSELECT instruction is lowered to SSE blend instructions. In many cases
-  // the mask is sign-extended to fill the entire lane. However, we only care
-  // for the highest bit. Convert sign_extend to srl because it is cheaper.
-  // (vselect(sign_extend(x))) ->  vselect(srl(x))
-  if (N->getOpcode() == ISD::VSELECT &&
-      Cond.getOpcode() == ISD::SIGN_EXTEND_INREG && Cond.hasOneUse()) {
-    EVT CondVT = Cond.getValueType();
-    EVT SExtTy = cast<VTSDNode>(Cond.getOperand(1))->getVT();
-    unsigned BitsDiff = CondVT.getScalarType().getSizeInBits() -
-                        SExtTy.getScalarType().getSizeInBits();
-
-    EVT ShiftType = EVT::getVectorVT(*DAG.getContext(),
-                MVT::i32, CondVT.getVectorNumElements());
-    SDValue SHL = DAG.getNode(ISD::SHL, DL, CondVT, Cond.getOperand(0),
-                DAG.getConstant(BitsDiff, ShiftType));
-    return DAG.getNode(ISD::VSELECT, DL, VT, SHL, LHS, RHS);
-  }
-
   return SDValue();
 }
 
index fc1b83b2224934c2a49e4424892a05c09daf8790..dbc122ac6e40950f3bc48035dab99b94a7bbdcc0 100644 (file)
@@ -5,10 +5,8 @@ target triple = "x86_64-apple-darwin11.2.0"
 
 ; CHECK: @foo8
 ; CHECK: psll
-; CHECK-NOT: sra
-; CHECK: pandn
-; CHECK: pand
-; CHECK: or
+; CHECK: psraw
+; CHECK: pblendvb
 ; CHECK: ret
 define void @foo8(float* nocapture %RET) nounwind {
 allocas:
diff --git a/test/CodeGen/X86/sext-blend.ll b/test/CodeGen/X86/sext-blend.ll
deleted file mode 100644 (file)
index b1f9573..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-; RUN: llc < %s -mtriple=x86_64-apple-darwin -mcpu=corei7 -promote-elements -mattr=+sse41 | FileCheck %s
-
-; CHECK: foo
-define <4 x double> @foo(<4 x double> %x, <4 x double> %y) {
-  ; CHECK: cmpnlepd
-  ; CHECK: psllq
-  ; CHECK-NEXT: blendvpd
-  ; CHECK: psllq
-  ; CHECK-NEXT: blendvpd
-  ; CHECK: ret
-  %min_is_x = fcmp ult <4 x double> %x, %y
-  %min = select <4 x i1> %min_is_x, <4 x double> %x, <4 x double> %y
-  ret <4 x double> %min
-}
-
index c6602d3a3b422a2586c3598bd911debe096d8b60..2f4317bf294c252f278745026bc209e5c74682b0 100644 (file)
@@ -28,10 +28,10 @@ define void@vsel_i32(<4 x i32>* %v1, <4 x i32>* %v2) {
 
 ; Without forcing instructions, fall back to the preferred PS domain.
 ; CHECK: vsel_i64
-; CHECK: pxor
-; CHECK: and
-; CHECK: andn
-; CHECK: or
+; CHECK: xorps
+; CHECK: andps
+; CHECK: andnps
+; CHECK: orps
 ; CHECK: ret
 
 define void@vsel_i64(<4 x i64>* %v1, <4 x i64>* %v2) {
@@ -44,10 +44,10 @@ define void@vsel_i64(<4 x i64>* %v1, <4 x i64>* %v2) {
 
 ; Without forcing instructions, fall back to the preferred PS domain.
 ; CHECK: vsel_double
-; CHECK: xor
-; CHECK: and
-; CHECK: andn
-; CHECK: or
+; CHECK: xorps
+; CHECK: andps
+; CHECK: andnps
+; CHECK: orps
 ; CHECK: ret
 
 define void@vsel_double(<4 x double>* %v1, <4 x double>* %v2) {
index 0a71dd0d04d44804ea45c5222ba80a9a44496ed3..78604a0e96345ce0a5477ab5ec795c986e1885b9 100644 (file)
@@ -36,7 +36,6 @@ define <4 x i32> @vsel_i32(<4 x i32> %v1, <4 x i32> %v2) {
 
 
 ;CHECK: vsel_double
-;CHECK-NOT: sra
 ;CHECK: blendvpd
 ;CHECK: ret
 define <4 x double> @vsel_double(<4 x double> %v1, <4 x double> %v2) {
@@ -55,7 +54,6 @@ define <4 x i64> @vsel_i64(<4 x i64> %v1, <4 x i64> %v2) {
 
 
 ;CHECK: vsel_i8
-;CHECK-NOT: sra
 ;CHECK: pblendvb
 ;CHECK: ret
 define <16 x i8> @vsel_i8(<16 x i8> %v1, <16 x i8> %v2) {
@@ -67,7 +65,6 @@ define <16 x i8> @vsel_i8(<16 x i8> %v1, <16 x i8> %v2) {
 ; CHECK: A
 define <2 x double> @A(<2 x double> %x, <2 x double> %y) {
   ; CHECK: cmplepd
-  ; CHECK-NOT: sra
   ; CHECK: blendvpd
   %max_is_x = fcmp oge <2 x double> %x, %y
   %max = select <2 x i1> %max_is_x, <2 x double> %x, <2 x double> %y
@@ -77,7 +74,6 @@ define <2 x double> @A(<2 x double> %x, <2 x double> %y) {
 ; CHECK: B
 define <2 x double> @B(<2 x double> %x, <2 x double> %y) {
   ; CHECK: cmpnlepd
-  ; CHECK-NOT: sra
   ; CHECK: blendvpd
   %min_is_x = fcmp ult <2 x double> %x, %y
   %min = select <2 x i1> %min_is_x, <2 x double> %x, <2 x double> %y