[PATCH] D98509: [LV] Calculate max feasible scalable VF.

[Cullen Rhodes via Phabricator via llvm-commits]

c-rhodes accepted this revision.
c-rhodes added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5582-5584
+  MinBWs = computeMinimumValueSizes(TheLoop->getBlocks(), *DB, &TTI);
+  unsigned SmallestType, WidestType;
+  std::tie(SmallestType, WidestType) = getSmallestAndWidestTypes();
----------------
sdesmalen wrote:
> c-rhodes wrote:
> > should this be moved closer to where it's used?
> This is actually moved as close as possible, because the first use is WidestType (line 5591).
> This is actually moved as close as possible, because the first use is WidestType (line 5591).




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

https://reviews.llvm.org/D98509