<div dir="ltr">Looks (pretty) good to me. Lets put this in for now.<div><br></div><div>Tomorrow I'm going to go and pick Nick's brain about the intended memory model for lld - we should either properly delete atoms, or document the policy that anything they contain be allocated on the same allocator.</div><div><br></div><div>- Lang. </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 16, 2016 at 3:57 PM, Pete Cooper via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping.  @Lang, does this look ok?<br>
<span class="HOEnZb"><font color="#888888"><br>
Pete<br>
</font></span><div class="HOEnZb"><div class="h5">> On Feb 4, 2016, at 4:57 PM, Pete Cooper via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
> Thanks for pinging this.  I’d forgotten about it.<br>
><br>
> Spoke to Nick in person.  He said he was happy to defer to Lang on this one.<br>
><br>
> And thanks for your LGTM.  Assuming Lang is ok with it i’ll commit it.<br>
><br>
> Cheers,<br>
> Pete<br>
>> On Feb 4, 2016, at 4:53 PM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> Nick, any thought on this? It would be awesome to have check-lld asan clean.<br>
>><br>
>> The patch looks good to me in that all that I see is an allocator<br>
>> being passed to more places, but I don't know if that was the<br>
>> intention.<br>
>><br>
>> Cheers,<br>
>> Rafael<br>
>><br>
>><br>
>> On 26 January 2016 at 10:42, Rafael Espíndola<br>
>> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>>> Nick is probably the best reviewer for the yaml code.<br>
>>><br>
>>> Cheers,<br>
>>> Rafael<br>
>>><br>
>>><br>
>>> On 25 January 2016 at 12:34, Pete Cooper <<a href="mailto:peter_cooper@apple.com">peter_cooper@apple.com</a>> wrote:<br>
>>>> Hi Rafael<br>
>>>><br>
>>>> This should fix some of the leaks seen in PR 21466.<br>
>>>><br>
>>>> 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.<br>
>>>><br>
>>>> This patch makes atoms use the file allocator in the YAML parser when needed.<br>
>>>><br>
>>>> Note, this doesn’t fix all the leaks.  There’s still a bunch I’m trying to understand, but its a good start.<br>
>>>><br>
>>>> Thanks<br>
>>>> Pete<br>
>>>><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>