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

Chandler Carruth chandlerc at google.com
Thu Jan 9 01:14:42 PST 2014


LGTM


On Thu, Jan 9, 2014 at 3:43 AM, Kostya Serebryany <kcc at google.com> wrote:

> 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/ba768759/attachment.html>


More information about the llvm-dev mailing list