[PATCH] D77655: [VPlan] Add & use VPValue operands for VPWidenCallRecipe (NFC).
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 8 10:51:48 PDT 2020
fhahn marked an inline comment as done.
fhahn added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:6935
+
+ return new VPWidenCallRecipe(*CI, Values);
}
----------------
gilr wrote:
> Have VPWidenCallRecipe() take an iterator_range instead of ArrayRef, saving the temporary SmallVector?
The way the types stack up with iterator_range and map_iterator we would need to add template version of the VPWidenCallRecipe and VPUser constructors. I think that would make sense, to provide more convenience to users of those classes. I;ve updated the patch. What do you think?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77655/new/
https://reviews.llvm.org/D77655
More information about the llvm-commits
mailing list