Fix some leaks in lld's YAML parser
    Pete Cooper via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jan 25 12:34:05 PST 2016
    
    
  
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: leaks.diff
Type: application/octet-stream
Size: 1926 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160125/887d89c6/attachment.obj>
    
    
More information about the llvm-commits
mailing list