[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