<div class="gmail_quote">On Wed, Jul 25, 2012 at 12:31 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think fundamentally this is the right approach, but figuring out Clang's version number is pretty gross.<div><br></div><div>Can you add a clear FIXME that we should switch to re-use the Clang cmake build's information?</div>

<div><br></div><div>Also, you should document the use case and behavior for the other generic bits of the patch to support building clang runtime libraries and dropping them into the appropriate location.</div></blockquote>
<div><br></div><div>r160721</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><div class="gmail_extra">

<br><br><div class="gmail_quote">On Tue, Jul 24, 2012 at 7:06 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br><br><div class="gmail_quote"><div><div>On Tue, Jul 24, 2012 at 3:59 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div><br><br><div class="gmail_quote">On Tue, Jul 24, 2012 at 3:53 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>On Tue, Jul 24, 2012 at 4:49 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote">



<div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: samsonov<br>
Date: Tue Jul 24 06:49:55 2012<br>
New Revision: 160669<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=160669&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=160669&view=rev</a><br>
Log:<br>
[ASan] hacking cmake build</blockquote><div><br></div></div><div>No, this is completely incorrect. Please get review beforehand if you're not confident in a patch.</div><div><br></div><div>CMake has specific infrastructure to manage installing files if you want this to happen during install.</div>




<div><br></div><div>If you just want the directory layout of the build tree to Just Work (which makes sense) you should setup the paths so the libraries go into the correct place to begin with, not copy them there after-the-fact.</div>




</div></div>
</blockquote></div><br></div></div></blockquote></div></div><div>Should I setup the build path for static runtime libs like this: <a href="http://codereview.appspot.com/6436051/" target="_blank">http://codereview.appspot.com/6436051/</a> ?</div>

<div>
All runtimes would have to be placed in lib/clang/3.2/... (not only ASan runtime).</div></div><span><font color="#888888"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>