<div dir="ltr">Thoughts? </div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Dec 26, 2013 at 11:55 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">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"><div dir="ltr">This? <div><br></div><div><div>+extern "C" {</div><div>+// Disable LeakSanitizer for this binary as it has too many leaks that are not</div>
<div>+// very interesting to fix. __lsan_is_turned_off is explained in</div>
<div>+// compiler-rt/include/sanitizer/lsan_interface.h</div><div class="im"><div>+int __lsan_is_turned_off() { return 1; }</div><div>+}  // extern "C"</div></div></div><div><br></div></div><div class="HOEnZb">
<div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Dec 26, 2013 at 11:49 AM, 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 dir="ltr"><div class="gmail_extra"><div><br><div class="gmail_quote">On Thu, Dec 26, 2013 at 2:40 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">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"><div dir="ltr">Like this? <div><br></div><div><div>+extern "C" {</div><div>+// Disable LeakSanitizer, see <a href="http://llvm.org/bugs/show_bug.cgi?id=18325" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=18325</a>.</div>


</div></div></blockquote></div><br></div>We don't often reference bugs in comments. I would give a brief summary in the text of the comment, and mention the bug in the commit log.</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>