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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to