[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