[PATCH] D102615: [LoopDeletion] Break backedge if we can prove that the loop is exited on 1st iteration

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 26 02:11:15 PDT 2021


xbolva00 added a comment.

In D102615#2781336 <https://reviews.llvm.org/D102615#2781336>, @mkazantsev wrote:

> @nikic > Is the "at" part an important part of this optimization?
>
> Unfortunately yes, without it the motivating test won't pass. I'll simply make option to enable/disable this transform.

Is it worth it? It should not be commited at all if off by default - you can have it in your fork, no?

It would just add additional code complexivity for future developers + bit rot.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102615



More information about the llvm-commits mailing list