[PATCH] D134803: [LoopPeeling] Support peeling loops with non-latch exits

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 17 14:49:31 PDT 2022


tra added a comment.

This change may have caused the https://github.com/llvm/llvm-project/issues/28112 to show up again in a few of our tests (the timeout just mentioned by @bgraur) .
Unfortunately it's likely going to be hard to create a reduced reproducer as the issue tends to shows up in large code and often disappears in an unpredictable manner when code gets reduced.

Is there a way to disable this heuristic?

Another possible source of this problem is that the loop transformation didn't take into account 'convergent' attribute and may have duplicated code that should not have been. E.g. here's what we had to do in the past: https://reviews.llvm.org/D17518


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134803



More information about the llvm-commits mailing list