[PATCH] D91718: [LV] Legalize scalable VF hints
Sander de Smalen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 11 03:57:00 PST 2020
sdesmalen added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5609
+ return computeFeasibleMaxVF(
+ ConstTripCount, ElementCount::getFixed(UserVF.getKnownMinValue()));
+ }
----------------
sdesmalen wrote:
> Can it know that `ElementCount::getFixed(UserVF.getKnownMinValue())` would be a valid VF? (that check is below on line 5614).
> Probably good to have a test for that as well.
It seems I misread this line/statement, so please ignore this comment.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91718/new/
https://reviews.llvm.org/D91718
More information about the llvm-commits
mailing list