<br><br><div class="gmail_quote">On Wed, Jul 11, 2012 at 9:17 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 class="im">On Wed, Jul 11, 2012 at 10:03 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote">
<div class="im">
<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>On Wed, Jul 11, 2012 at 3:06 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">
And one more question regarding ASan cmake build.<div>Currently unittests are fine, but regular "clang -faddress-sanitizer" is not:</div></blockquote><div><br></div></div><div>As a side note: I don't like that the current cmake machinery builds asan tests by explicitly passing libclang_rt.asan-x86_64.a. </div>


<div>IMO it needs to use "clang -faddress-sanitizer".</div></div></blockquote><div><br></div></div><div>Why?</div><div><br></div><div>These are internal tests, and I think it is reasonable to use internal implementation details to make them work.</div>
</div></div></blockquote><div><br></div><div>This complicates the maintenance (if we change the path where libclang_rt.asan-x86_64.a resides we'' have to change it in two places). </div><div>But as long as at least some tests actually use "clang -faddress-sanitizer", it's not that critical if other tests use libclang_rt.asan-x86_64.a</div>
<div><br></div><div>--kcc </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote">
<div><br></div><div>I'm not saying we can't get there, I think we can, but I think it may be somewhat more annoying than you might imagine to get there in the short term. </div></div></div>
</blockquote></div><br>