<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 13, 2016 at 3:16 AM, Dmitry Vyukov <span dir="ltr"><<a href="mailto:dvyukov@google.com" target="_blank">dvyukov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, May 13, 2016 at 12:04 PM, Kuba Brecka <<a href="mailto:jbrecka@apple.com">jbrecka@apple.com</a>> wrote:<br>
> Any specific suggestions how to do that for SANITIZER_GO?  Currently,<br>
> SANITIZER_GO isn’t even guaranteed to be “0” or “1”, instead it’s either<br>
> defined or not.  That would be a simple change, but we’ll need to change all<br>
> uses of SANITIZER_GO from #ifdef to #if.<br>
<br>
</span>There is kGoMode constant.<br>
<span class=""><br>
> Then it’s still not easy to just “if (SANITIZER_GO)”, because in this case<br>
> (and plenty of others), we’ll just fail to link.  The symbolizer stuff isn't<br>
> built in a Go build.  We *might* be able to link fine with -O<something><br>
> because the if(0) will be dead-code-removed, but that just makes the<br>
> situation even worse (some configurations build, some don’t).<br>
<br>
</span>I don't have a good answer for this one.<br>
</blockquote></div><br></div><div class="gmail_extra">Define an empty function. </div><div class="gmail_extra">This is way better than ifdefs all over the code. </div></div>