<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 5, 2015 at 2:46 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Yeah, I agree that layering is an issue.</div><div><br></div><div>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.</div></blockquote><div><br></div><div>Wait, you've lost me.</div><div><br></div><div>Target depends on Core. Passes depends on Target. So this can't live in Core?</div><div><br></div><div>Same relation ship for Analysis and Transforms?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br></div><div>Then the question becomes:</div><div>- Is that gross putting the pass manager in there?</div><div>- Will there come a time when you actually want passes and don’t have IR?</div><div><br></div><div>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.</div><div><br></div><div>Thanks,</div><div>Pete</div></blockquote></div><br><br></div></div>