[PATCH] D28452: [PM] Move the LoopPassManager to the transforms library.

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 7 21:29:59 PST 2017


sanjoy added a comment.

In https://reviews.llvm.org/D28452#639182, @chandlerc wrote:

> In https://reviews.llvm.org/D28452#639167, @sanjoy wrote:
>
> > Any reason to prefer `Transforms/Scalar` over `Transforms/Utils`?  I'd have expected the latter.
>
>
> Because Utils is mostly non-passes. I'd like eventually to make that a reality.
>
> And I'd like the loop PM to be able to use the *pass* formulations of LCSSA and LoopSimplify so that it can work (and be debugged) very much as a less customizable pass pipeline. Does that make sense?


sgtm!


https://reviews.llvm.org/D28452





More information about the llvm-commits mailing list