[PATCH] D91718: [LV] Legalize scalable VF hints

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen accepted this revision.
sdesmalen added a comment.

LGTM, thanks for your work on this patch @c-rhodes!



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5654
+                                            TheLoop->getHeader())
+                 << "Max legal vector width too small, scalable vectorization "
+                 << "unfeasible. Using fixed-width vectorization instead.";
----------------
nit: is it useful to print MaxSafeElements or MaxSafeVectorWidthInBits here?


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

https://reviews.llvm.org/D91718