[llvm] r356840 - Remove unused function argument. NFCI.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 23 09:20:34 PDT 2019


Author: rksimon
Date: Sat Mar 23 09:20:34 2019
New Revision: 356840

URL: http://llvm.org/viewvc/llvm-project?rev=356840&view=rev
Log:
Remove unused function argument. NFCI.

Modified:
    llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp?rev=356840&r1=356839&r2=356840&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp Sat Mar 23 09:20:34 2019
@@ -2850,10 +2850,12 @@ void BoUpSLP::reorderAltShuffleOperands(
 // The vectorizer is trying to either have all elements one side being
 // instruction with the same opcode to enable further vectorization, or having
 // a splat to lower the vectorizing cost.
-static bool shouldReorderOperands(
-    int i, unsigned Opcode, Instruction &I, ArrayRef<Value *> Left,
-    ArrayRef<Value *> Right, bool AllSameOpcodeLeft, bool AllSameOpcodeRight,
-    bool SplatLeft, bool SplatRight, Value *&VLeft, Value *&VRight) {
+static bool shouldReorderOperands(int i, Instruction &I, ArrayRef<Value *> Left,
+                                  ArrayRef<Value *> Right,
+                                  bool AllSameOpcodeLeft,
+                                  bool AllSameOpcodeRight, bool SplatLeft,
+                                  bool SplatRight, Value *&VLeft,
+                                  Value *&VRight) {
   VLeft = I.getOperand(0);
   VRight = I.getOperand(1);
   // If we have "SplatRight", try to see if commuting is needed to preserve it.
@@ -2946,7 +2948,7 @@ void BoUpSLP::reorderInputsAccordingToOp
     // one side.
     Value *VLeft;
     Value *VRight;
-    if (shouldReorderOperands(i, Opcode, *I, Left, Right, AllSameOpcodeLeft,
+    if (shouldReorderOperands(i, *I, Left, Right, AllSameOpcodeLeft,
                               AllSameOpcodeRight, SplatLeft, SplatRight, VLeft,
                               VRight)) {
       Left.push_back(VRight);




More information about the llvm-commits mailing list