[PATCH] D80114: [VPlan] Add & use VPValue operands for VPReplicateRecipe (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 07:34:10 PDT 2020


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:445
+  /// inclusive. Uses the VPValue operands from \p Operands instead of \p
+  /// Instr's operands directly.
+  void scalarizeInstruction(Instruction *Instr, VPUser &Operands,
----------------
gilr wrote:
> I'd omit "directly" to avoid implying that Operands necessarily wrap Instr's operands.
Good point, thanks! I've dropped it in the committed version


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80114





More information about the llvm-commits mailing list