[PATCH] D113939: [runtime-unroll] Remove restriction about unrolling multiple exit loops

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 16 11:45:07 PST 2021


lebedev.ri added a comment.

In D113939#3135607 <https://reviews.llvm.org/D113939#3135607>, @nikic wrote:

> This looks reasonable to me on a conceptual level -- we have recently removed multi-exit restrictions for non-runtime unrolling as well.
>
> Of course, this does have the potential to further exacerbate our problem with way too aggressive unrolling.

+1 to that, but i think unrolling is generally a good thing that we *should* be doing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113939



More information about the llvm-commits mailing list