[LLVMdev] [cfe-dev] lsan for LLVM bootstrap; leaks in TableGen

Kostya Serebryany kcc at google.com
Thu Jan 9 00:43:38 PST 2014


Thoughts?


On Thu, Dec 26, 2013 at 11:55 AM, Kostya Serebryany <kcc at google.com> wrote:

> This?
>
> +extern "C" {
> +// Disable LeakSanitizer for this binary as it has too many leaks that
> are not
> +// very interesting to fix. __lsan_is_turned_off is explained in
> +// compiler-rt/include/sanitizer/lsan_interface.h
> +int __lsan_is_turned_off() { return 1; }
> +}  // extern "C"
>
>
>
> On Thu, Dec 26, 2013 at 11:49 AM, Chandler Carruth <chandlerc at google.com>wrote:
>
>>
>> On Thu, Dec 26, 2013 at 2:40 AM, Kostya Serebryany <kcc at google.com>wrote:
>>
>>> Like this?
>>>
>>> +extern "C" {
>>> +// Disable LeakSanitizer, see
>>> http://llvm.org/bugs/show_bug.cgi?id=18325.
>>>
>>
>> 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.
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140109/02302a84/attachment.html>


More information about the llvm-dev mailing list