[PATCH] D152366: [LoopVectorize] Allow inner loop runtime checks to be hoisted above an outer loop

[David Sherwood via Phabricator via llvm-commits]

david-arm added a comment.

In D152366#4450222 <https://reviews.llvm.org/D152366#4450222>, @fhahn wrote:

> Does this require the `extra-vectorizer-passes` to hoist the checks out of the loop in the full pipeline?

Hi @fhahn, I just realised I forgot to reply to this comment. It does require follow-on passes like LICM to hoist the checks out of the loop, but this does seem to be happening for x264. For both normal and LTO pipelines we do run LICM after the vectoriser, so I think it's fine without the `extra-vectorizer-passes` flag.


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

https://reviews.llvm.org/D152366