Fix some leaks in lld's YAML parser

Rafael Espíndola via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 07:42:56 PST 2016


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