<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 9, 2014 at 3:15 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
On 09/01/2014 11:08, Chandler Carruth wrote:<div class="im"><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
On Thu, Jan 9, 2014 at 6:02 AM, Alp Toker <<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a> <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>>> wrote:<br>
<br>
    Hi Kostya,<br>
<br>
    This doesn't look correct and fires warnings my my strict checks<br>
    build.<br>
<br>
<br>
I'm pretty sure the change works correctly, and it was definitely what Kostya and I intended.<br>
<br>
<br>
    Double-underscore prefixed identifiers are strictly reserved for<br>
    the implementation.<br>
<br>
<br>
Yes, and LeakSanitizer is something provided by the implementation? (the toolchain's runtime libraries specifically) I don't see any problem with double underscore here, any more than with __builtin_... functions or any of the other sanitizer interfaces.<br>

</blockquote>
<br></div>
__builtin functions are introduced by the implementation (either the compiler or system header declarations) so there's no problem.<br>
<br>
Sanitizer is a system library so it's free to use the identifiers it wants.<br>
<br>
TableGen, on the other hand, is not a part of the implementation.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Can you clarify exactly what the error is and why? It's possible we could provide a macro of some kind in the sanitizer headers to bundle this functionality up if we need some special marking of this for checkers, but without more information its hard to hell what the actual problem is.<br>

</blockquote>
<br></div>
These two commits _introduced_ names in the reserved namespace.<br></blockquote><div>And that we the intention. </div><div>See the comments at <span style="font-size:13px;font-family:arial,sans-serif">compiler-rt/include/sanitizer/</span><span style="font-size:13px;font-family:arial,sans-serif">lsan_interface.h</span></div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
The checker I have is a strict warning flag I've been trying to encode the rules from the standard and in this case the diagnostic looks legitimate.</blockquote><div><br></div><div>Can it be silenced for a particular line of code (like "// NOLINT")?</div>
<div><br></div><div>--kcc </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><font color="#888888"><br>

<br>
Alp.</font></span><div class=""><div class="h5"><br>
<br>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</div></div></blockquote></div><br></div></div>