<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 9, 2014 at 4:07 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">I understand that, but the question is -- do you think that usage pattern will be prevalent? Is it worth putting an external definition in the binary *always* just to catch the case where we do a link-time-only flag?<br>

<br>
Put another way, is it really too burdensome to say that LSan does require a compile time flag in order to support some usage patterns?<br>
</div></blockquote>
<br>
Who controls / defines the interface?<br></blockquote><div><br></div><div>The sanitizers.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This looks like a spectacular thinko rather than being something intentional -- is it possible to fix it to drop one or more underscores instead of devising workarounds?<br></blockquote><div><br></div><div>I have no idea what you mean. This was a well considered change, and part of a design that has been developed over quite some time on the lists. None of these are workarounds?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
There's never a valid reason to require user code to define reserved names (although it's sometimes OK for user code to _use_ reserved names).<br></blockquote><div><br></div><div>You keep making this assertion, but I still don't find any backing for it in the standard. I'm happy to check with some of my fellow members of the committee, but I would be surprised if they drew the distinction you are drawing here.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm heading out for a couple of hours, please gate this behind a macro or revert until a resolution is found.</blockquote></div><br>Alp, I really don't think this is reasonable. This change, and the lead up to the change, were discussed on the list with code review. I understand you have some high level design concerns, but there doesn't appear to be broad agreement with them and I don't think it is reasonable to block progress here while we sort them out.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I'm happy to continue this discussion, but I do not think that this needs to be reverted, and I do not think we need to block progress on actually getting leak checking into the sanitizers and the LLVM and Clang bootstrap during that discussion.</div>
</div>