<p dir="ltr"><br>
On 13 Jan 2014 23:25, "Kostya Serebryany" <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>
><br>
><br>
><br>
><br>
> On Tue, Jan 14, 2014 at 1:47 AM, Richard Smith <<a href="mailto:richardsmith@google.com">richardsmith@google.com</a>> wrote:<br>
>><br>
>> On Mon, Jan 13, 2014 at 8:03 AM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:<br>
>>><br>
>>><br>
>>> On 13/01/2014 15:02, Kostya Serebryany wrote:<br>
>>>><br>
>>>> Alp, others,<br>
>>>><br>
>>>> Now that there is no urgent issue I'd like to return to the discussion about names with "__" prefix in sanitizer run-times.<br>
>>><br>
>>><br>
>>> If you're OK to conditionalize the code so it's only defined in sanitizer builds, that's fine and as Sean pointed out there's already plenty of precedent -- it's what every software project does to define "__" prefixed weak link functions like libc malloc hooks.<br>

>><br>
>>  <br>
>> This discussion is so vastly far from the best use of any of our time. Let's just restore the name we had before and wrap the definition in __has_feature(leak_sanitizer). I think that will make everyone happy. OK?<br>

><br>
><br>
> There is no __has_feature(leak_sanitizer) and it can't be implemented with reasonable semantics (discussed here before).<br>
> Unless someone objects, I'll change the code back to use __lsan_is_turned_off and hide it under __has_feature(address_sanitizer),<br>
> then I will revert my change that added an alternative to __lsan_is_turned_off (LeakSanitizerIsTurnedOffForTheCurrentProcess)</p>
<p dir="ltr">Sounds great, thanks.</p>
<p dir="ltr">> This means that  -Wreserved, -Wreserved-macros, -Wreserved-identifiers (once implemented) <br>
> will not work on clang bootstrap in AddressSanitizer mode, <br>
> but let Alp handle this if it ever becomes a problem.<br>
>  <br>
> --kcc <br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</p>