[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