[PATCH] D84886: Create LoopNestPass

[Ta-Wei Tu via Phabricator via cfe-commits]

TaWeiTu added a comment.

Hi, thanks for your comments and suggestions!

I've thought about the suggestion by @ychen to extend the existing `LoopPassManager` to handle loop-nest passes instead of having a separate `LoopNestPassManager`, and I've uploaded a new patch D87045 <https://reviews.llvm.org/D87045> that implements the basic functionalities of running loop-nest passes in `LoopPassManager`.
I've also added you as reviewers to the new patch, and I will work on the follow-up patches if you think that direction is better.

I'll also be working on evaluating the benefits of using loop-nest passes, as @fhahn mentioned. But as far as I understand, most passes that may potentially benefit from loop-nest passes require certain algorithmic changes that incorporate the information of perfectly-nested loops provided by the `LoopNest` object first. So it will probably take more time to investigate those passes and convert those passes using the new facilities.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84886