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

[Cullen Rhodes via Phabricator via llvm-commits]

c-rhodes added inline comments.


================
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.";
----------------
sdesmalen wrote:
> nit: is it useful to print MaxSafeElements or MaxSafeVectorWidthInBits here?
> nit: is it useful to print MaxSafeElements or MaxSafeVectorWidthInBits here?

`MaxSafeElements` is undefined here but LAA emits the max VF anyway which should be sufficient?


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

https://reviews.llvm.org/D91718