[PATCH] D87045: [LoopNest] Handle loop-nest passes in LoopPassManager
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 5 19:11:18 PDT 2020
asbirlea added a comment.
The design looks good, but I'd like to see more comments documenting some of the details for future code readers.
e.g. document the Optional<PreservedAnalysis> as including the test for skipped passes in that single function call (this is new behavior vs other implementations), document the addPass specializations and populating IsLoopNestPass.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87045/new/
https://reviews.llvm.org/D87045
More information about the llvm-commits
mailing list