[PATCH] D91718: [LV] Legalize scalable VF hints
Cullen Rhodes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 7 07:35:28 PST 2021
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
More information about the llvm-commits
mailing list