[llvm] r284913 - [X86] Remove unnecessary AVX2 check that was already covered by an assertion earlier in the function. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 23:51:50 PDT 2016


Author: ctopper
Date: Sat Oct 22 01:51:49 2016
New Revision: 284913

URL: http://llvm.org/viewvc/llvm-project?rev=284913&view=rev
Log:
[X86] Remove unnecessary AVX2 check that was already covered by an assertion earlier in the function. NFC

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=284913&r1=284912&r2=284913&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Sat Oct 22 01:51:49 2016
@@ -11459,8 +11459,8 @@ static SDValue lowerV4I64VectorShuffle(c
   // shuffle. However, if we have AVX2 and either inputs are already in place,
   // we will be able to shuffle even across lanes the other input in a single
   // instruction so skip this pattern.
-  if (!(Subtarget.hasAVX2() && (isShuffleMaskInputInPlace(0, Mask) ||
-                                 isShuffleMaskInputInPlace(1, Mask))))
+  if (!isShuffleMaskInputInPlace(0, Mask) &&
+      !isShuffleMaskInputInPlace(1, Mask))
     if (SDValue Result = lowerVectorShuffleByMerging128BitLanes(
             DL, MVT::v4i64, V1, V2, Mask, Subtarget, DAG))
       return Result;




More information about the llvm-commits mailing list