Fix some leaks in lld's YAML parser

Rafael Espíndola via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 16:53:18 PST 2016


Nick, any thought on this? It would be awesome to have check-lld asan clean.

The patch looks good to me in that all that I see is an allocator
being passed to more places, but I don't know if that was the
intention.

Cheers,
Rafael


On 26 January 2016 at 10:42, Rafael Espíndola
<rafael.espindola at gmail.com> wrote:
> Nick is probably the best reviewer for the yaml code.
>
> Cheers,
> Rafael
>
>
> On 25 January 2016 at 12:34, Pete Cooper <peter_cooper at apple.com> wrote:
>> Hi Rafael
>>
>> This should fix some of the leaks seen in PR 21466.
>>
>> The cause was the MappingNormalizationHeap struct which was allocating atoms.  In the binary file parser we allocate atoms in the file::allocator() so that we can avoid the free cost later.
>>
>> This patch makes atoms use the file allocator in the YAML parser when needed.
>>
>> Note, this doesn’t fix all the leaks.  There’s still a bunch I’m trying to understand, but its a good start.
>>
>> Thanks
>> Pete
>>


More information about the llvm-commits mailing list