[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