[llvm] r375253 - [X86] combineX86ShufflesRecursively - pull out isTargetShuffleVariableMask. NFCI.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 09:39:01 PDT 2019


Author: rksimon
Date: Fri Oct 18 09:39:01 2019
New Revision: 375253

URL: http://llvm.org/viewvc/llvm-project?rev=375253&view=rev
Log:
[X86] combineX86ShufflesRecursively - pull out isTargetShuffleVariableMask. NFCI.

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=375253&r1=375252&r2=375253&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Fri Oct 18 09:39:01 2019
@@ -33035,6 +33035,7 @@ static SDValue combineX86ShufflesRecursi
   SmallVector<SDValue, 2> OpInputs;
   APInt OpUndef, OpZero;
   APInt OpDemandedElts = APInt::getAllOnesValue(VT.getVectorNumElements());
+  bool IsOpVariableMask = isTargetShuffleVariableMask(Op.getOpcode());
   if (!getTargetShuffleInputs(Op, OpDemandedElts, OpInputs, OpMask, OpUndef,
                               OpZero, DAG, Depth, false))
     return SDValue();
@@ -33156,7 +33157,7 @@ static SDValue combineX86ShufflesRecursi
                          SDLoc(Root));
 
   assert(!Ops.empty() && "Shuffle with no inputs detected");
-  HasVariableMask |= isTargetShuffleVariableMask(Op.getOpcode());
+  HasVariableMask |= IsOpVariableMask;
 
   // Update the list of shuffle nodes that have been combined so far.
   SmallVector<const SDNode *, 16> CombinedNodes(SrcNodes.begin(),




More information about the llvm-commits mailing list