[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