From: Craig Topper <craig.topper@intel.com>
Date: Sat, 16 Sep 2017 17:51:55 +0000 (+0000)
Subject: [X86] Remove some extra code that snuck into r313450.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=af30292211ccf2573892f0945dbbb60db2041b04;p=llvm

[X86] Remove some extra code that snuck into r313450.

The same code appears earlier in the function. This represents an earlier version of what became r313373 that I still had sitting in my local repo.

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

diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp
index da4aa959ea2..4df7621bee2 100644
--- a/lib/Target/X86/X86ISelLowering.cpp
+++ b/lib/Target/X86/X86ISelLowering.cpp
@@ -12174,10 +12174,6 @@ static SDValue lowerV2X128VectorShuffle(const SDLoc &DL, MVT VT, SDValue V1,
   // If either input operand is a zero vector, use VPERM2X128 because its mask
   // allows us to replace the zero input with an implicit zero.
   if (!IsV1Zero && !IsV2Zero) {
-    // With AVX2, use VPERMQ/VPERMPD to allow memory folding.
-    if (Subtarget.hasAVX2() && V2.isUndef())
-      return SDValue();
-
     // Check for patterns which can be matched with a single insert of a 128-bit
     // subvector.
     bool OnlyUsesV1 = isShuffleEquivalent(V1, V2, Mask, {0, 1, 0, 1});