Minor updates per review.

Author: JonPsson1

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

@@ -3032,10 +3032,9 @@ class BoUpSLP {
       SmallVectorImpl<SmallVector<const TreeEntry *>> &Entries,
       unsigned NumParts, bool ForOrder = false);
 
-  /// \returns the scalarization cost for this list of values. Assuming that
-  /// this subtree gets vectorized, we may need to insert the values from the
-  /// roots. This method calculates the cost of inserting the values.
-  /// \param ForPoisonSrc true if initial vector is poison, false otherwise.
+  /// \returns the cost of gathering (inserting) the values in \p VL into a
+  /// vector.  \param ForPoisonSrc true if initial vector is poison, false
+  /// otherwise.
   InstructionCost getGatherCost(ArrayRef<Value *> VL, bool ForPoisonSrc,
                                 Type *ScalarTy) const;
 

llvm/test/Transforms/SLPVectorizer/SystemZ/vec-elt-insertion.ll

@@ -27,7 +27,7 @@ define void @fun0(ptr nocapture %0, double %1) {
 
 ; This function needs the element-load to be recognized in SystemZ
 ; getVectorInstrCost().
-define void @fun1(double %0) local_unnamed_addr {
+define void @fun1(double %0) {
 ; CHECK-LABEL: define void @fun1(
 ; CHECK:    fsub <2 x double>
 ; CHECK:    fsub <2 x double>