[PATCH] [PM] Create a separate library for high-level pass management code.
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
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits