[PATCH] D84886: Create LoopNestPass

[Yuanfang Chen via Phabricator via cfe-commits]

ychen added a comment.

Hello, sorry for the late reply. I have some high-level questions

- What are existing and future loop passes/analyses that could benefit from using LoopNestPass?
- I'd feel more comfortable if existing loop pass/analysis managers could be extended to handle LoopNestPass, rather than have separate full-fledged managers. Implementation aside, this models the idea that loopnest is just a special kind of loop, not a fundamental different IR unit. Have you thought about the alternative design of adding new adaptors or extending existing loop managers to run/query/invalidate loopnest passes?
- I'm not sure how common loopnest analyses are or will be? What's your opinion?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84886