[PATCH] D93317: [LV] Vectorize (some) early and multiple exit loops

Sun Dec 27 10:25:49 PST 2020

fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks! I pushed a small fix for the crash discussed earlier in 0ea3749b3cde <https://reviews.llvm.org/rG0ea3749b3cde16d70c5f66357b623c8edf521f2b>. It should be a very safe/straight-forward fix, but I would appreciate taking a look post-review.

I also did some additional testing with loop rotation disabled, which should stress test parts of the multi-exit support, on SPEC2006 & MultiSource. That didn't surface any further issues, so I think it should be fine to land this without a flag for now.

There appears to be some training whitespace in the diff, it would be good to re-format before landing.

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

https://reviews.llvm.org/D93317