[PATCH] D67764: [LV] Runtime checks with OptForSize
Ayal Zaks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 20 21:14:26 PDT 2019
Ayal added a comment.
> When we optimise for size, and need to emit runtime checks to disambiguate memory, we should nicely bail, don't vectorise, and not run in an assert like we currently do.
The logic currently in place to do so is to call LVP.plan() which calls computeMaxVF(), which does
// Bail if runtime checks are required, which are not good when optimising
// for size.
if (runtimeChecksRequired())
return None;
Does this need to change, presumably move earlier?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67764/new/
https://reviews.llvm.org/D67764
More information about the llvm-commits
mailing list