<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Thanks all for your answers!<div><br></div><div>Seems like we agree that CodeGenPrepare should be moved into lib/CodeGen.</div><div><br></div><div>Following Hal’s advice to do that for every IR pass that depends directly on TargetLowering, it seems we would have to do the same thing for GlobalMerge and LowerInvoke. For the latter, I do not really know what it implies since this is part of Transforms/Utils.</div><div><br></div><div>Let me see what I can do with CodeGenPrepare first.</div><div><br><div apple-content-edited="true">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">-Quentin</div>
</div>
<br><div><div>On Feb 19, 2014, at 1:52 PM, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><blockquote type="cite">or we must move CGP into lib/CodeGen...[snip]<br></blockquote><br><blockquote type="cite">It solves all of the horrible layering problems that are currently blocking progress<br>and creates no new ones I'm aware of...<br></blockquote><br>Seems totally sane to me.<br><br>- Lang.<br></blockquote></div><br></div></body></html>