[PATCH] D123231: [StructurizeCFG] Improve basic block ordering

Sameer Sahasrabuddhe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 12 03:47:34 PDT 2022


sameerds added a comment.

In D123231#3445000 <https://reviews.llvm.org/D123231#3445000>, @foad wrote:

>> In specific cases, this patch reduces the number of values needed
>> after loop by moving the exit block to the loop.
>
> I think "//into// the loop" would be clearer if that's what you mean.

Good points, all of them. Additionally, note that loops are different before and after structurization. So moving an exit block into a loop is not as crazy as it seems at first glance!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123231



More information about the llvm-commits mailing list