[PATCH] D79037: [StructurizeCFG] Fix region nodes ordering

Sameer Sahasrabuddhe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 6 08:35:29 PDT 2020


sameerds added a comment.

In D79037#2022612 <https://reviews.llvm.org/D79037#2022612>, @ekatz wrote:

> The problem with those tests is something else, and not produced by the wrong ordering. You may refer to PR25378 for an example.
>  Still, this fix the ordering issue and fixes PR41509 (and its duplicates).
>  I think we should insert this patch, and attend to the other bug as a separate issue.


I see. It sure does look like PR25378. Then the only thing left is one new test with two sibling loops. The intention is to cover the case where CurrentLoop is not the parent of L.


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

https://reviews.llvm.org/D79037





More information about the llvm-commits mailing list