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

Wed Apr 7 12:06:39 PDT 2021

jaykang10 added a comment.

In D99774#2674608 <https://reviews.llvm.org/D99774#2674608>, @aeubanks wrote:

> 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?

I agree with you. It could be better to discuss this issue with llvm-dev. Let me make a post to llvm-dev.

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

https://reviews.llvm.org/D99774