r145955.<br><br><div class="gmail_quote">On Mon, Dec 5, 2011 at 3:10 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
make sense to me. <div>I can land it tomorrow if nobody objects. </div><div><br></div><div>--kcc <div><div class="h5"><br><div><br><br><div class="gmail_quote">On Mon, Dec 5, 2011 at 10:24 AM, Alexander Potapenko <span dir="ltr"><<a href="mailto:glider@google.com" target="_blank">glider@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch makes sure that only executables are linked with the ASan<br>
runtime library.<br>
Otherwise we may end up having several instances of the runtime<br>
library, which will lead to a mess.<br>
<br>
Also, when building a dynamic library or a bundle with ASan, we need<br>
to mark the undefined symbols added by the instrumentation pass as<br>
dynamic_lookup to avoid linker errors.<br>
<span><font color="#888888"><br>
--<br>
Alexander Potapenko<br>
Software Engineer<br>
Google Moscow<br>
</font></span></blockquote></div><br></div></div></div></div>
</blockquote></div><br>