[PATCH] D109368: [LV] Vectorize cases with larger number of RT checks, execute only if profitable.

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.

In D109368#3653703 <https://reviews.llvm.org/D109368#3653703>, @fhahn wrote:

> In D109368#3653700 <https://reviews.llvm.org/D109368#3653700>, @paulwalker-arm wrote:
>
>> Hi @fhahn, I believe this patch breaks the AArch64/SVE buildbots.  You can see the affect on https://lab.llvm.org/buildbot/#/builders/197/builds/2185.  It looks like we were unlucky with the buildbot run that actually contained your patch because this failed for what looks like a temporary CI issue, but all the runs after this point show functional regressions when running LNT.  I've manually checked your commit and the one before yours to confirm this is when things start to fail.
>
> Thanks for the heads-up! Would it be possible to share a preprocessed file that gets miscompiled or an LLVM IR function that shows the issue? Otherwise I won't be able to reproduce/investigate the failure as I do not have accesses to SVE hardware.

The issue is easily reproduced via user mode qemu (in my case I did this on x86) if that helps?  Otherwise I can see about getting a reproducer but that'll take some time so can you revert the patch in the interim?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109368