[PATCH] D67764: [LV] Runtime checks with OptForSize

[Ayal Zaks via Phabricator via llvm-commits]

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