[PATCH] D57382: [LV] Move interleave count computation to LVP::plan().

[Florian Hahn via Phabricator via llvm-commits]

fhahn updated this revision to Diff 185415.
fhahn retitled this revision from "[LV] Add planForInterleaving helper, to create VPlan if they are missing." to "[LV] Move interleave count computation to LVP::plan().".
fhahn edited the summary of this revision.
fhahn added a comment.

Update the patch to move interleave count computation into LVP::plan(). The early exit if computeMaxVF == None made it a bit tricky.

As it is at the moment, we need to know the selected vectorization factor to compute the interleave count. And we need to know the interleave count to know if we need to generate VPlans. Maybe with this refactor it would also make sense to only generate a VPlan for the selected vectorization factor for now? There is no need to build VPlans for multiple vectorization factors in the legacy planning and we have the VPlan native path which builds the VPlans up front for planning.


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

https://reviews.llvm.org/D57382

Files:
  llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/interleave-need-vplan.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57382.185415.patch
Type: text/x-patch
Size: 9102 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190205/51653322/attachment.bin>