[PATCH] D99774: [LoopUtils] Populate sibling loops in reverse program order on new pass manager

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

Traversing forward for the loops can cause a value in an earlier loop to be evaluated, which could help a later loop, right? I'm now not sure that it's such a clear tradeoff.
For example, scev-expander-preserve-lcssa.ll looks worse now.

It could be interesting to revisit a previous loop if a loop was deleted.

I think there are more knowledgeable people about this sort of stuff on llvm-dev, could you make a post there?


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

https://reviews.llvm.org/D99774