[llvm] r224058 - LeakDetector: Simplify code and fix comments, NFC

Duncan P. N. Exon Smith dexonsmith at apple.com
Fri Dec 19 07:45:38 PST 2014


> On 2014 Dec 19, at 04:57, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> 
> On 19 December 2014 at 04:19, Alexander Potapenko <glider at google.com> wrote:
>> AFAIK neither Valgrind nor LSan are able to detect leaked objects that
>> reference each other.
>> Moreover, it is theoretically possible to get a false negative if
>> somewhere in the memory there's a value that looks like a pointer to a
>> leaked object (or a cluster of objects).
>> The leak detector built directly into IR can do a better job.
> 
> Both asan and valgrind find the leak in the attached testcase.
> 
> In practice, I have seen both asan and valgrind find bugs in llvm more
> often than the leak checker:
> 
> Cheers,
> Rafael

I'm convinced.  Let's kill it.





More information about the llvm-commits mailing list