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

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 16 10:51:55 PDT 2017


Author: ctopper
Date: Sat Sep 16 10:51:55 2017
New Revision: 313465

URL: http://llvm.org/viewvc/llvm-project?rev=313465&view=rev
Log:
[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.

Modified:
    llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=313465&r1=313464&r2=313465&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Sat Sep 16 10:51:55 2017
@@ -12174,10 +12174,6 @@ static SDValue lowerV2X128VectorShuffle(
   // 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});




More information about the llvm-commits mailing list