[PATCH] D80773: [SLP] Apply external to vectorizable tree users cost adjustment for relevant aggregate build instructions only (UserCost).

[Alexey Bataev via Phabricator via llvm-commits]

ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.

LG with a nit



================
Comment at: llvm/include/llvm/Transforms/Vectorize/SLPVectorizer.h:96
   /// Try to vectorize a list of operands.
-  /// \param UserCost Cost of the user operations of \p VL if they may affect
-  /// the cost of the vectorization.
+  /// When \param InsertUses is provided and its entries are non-zero
+  /// then users of \p VL are known to be InsertElement instructions
----------------
I think you should use `\p` here instead of `\param`. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80773/new/

https://reviews.llvm.org/D80773