[PATCH] [PM] Create a separate library for high-level pass management code.

Chandler Carruth chandlerc at google.com
Thu Mar 5 14:47:37 PST 2015


On Thu, Mar 5, 2015 at 2:46 PM, Pete Cooper <peter_cooper at apple.com> wrote:

> Yeah, I agree that layering is an issue.
>
> The current pass code is in lib/IR, which I think is libCore.  There’s no
> fundamental reason why your code couldn’t also live there right now.  Its
> linked against everything you mentioned already.
>

Wait, you've lost me.

Target depends on Core. Passes depends on Target. So this can't live in
Core?

Same relation ship for Analysis and Transforms?


>
> Then the question becomes:
> - Is that gross putting the pass manager in there?
> - Will there come a time when you actually want passes and don’t have IR?
>
> I think if the answer to either of those questions is ‘yes’, then go for
> it.  Otherwise i’m not opposed to the library (i.e., go for it if no-one
> else is strongly opposed), but i’m also not seeing a big win either.
>
> Thanks,
> Pete
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150305/82894344/attachment.html>


More information about the llvm-commits mailing list