<div dir="ltr">Hi Pete,<div><br></div><div>I managed to catch Nick quickly, and his take matched my intuition: It's fine to leak atom memory but we really should run the atom destructors. I'll look into that, but it may be non-trivial. Your fix should stay in for now.<br></div><div><br></div><div>Cheers,</div><div>Lang.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 16, 2016 at 4:36 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Thanks.  <span style="font-family:'Helvetica Neue'">r263676/</span><span style="font-family:'Helvetica Neue'">r263677.</span><div><font face="Helvetica Neue"><br></font><div><span class=""><blockquote type="cite"><div><div dir="ltr"><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></div></blockquote></span>Yeah.  I’ll say that i’m fine with the current model of leaking atoms in an allocator, but it should be documented.</div><span class="HOEnZb"><font color="#888888"><div><br></div></font></span><div><span class="HOEnZb"><font color="#888888">Pete</font></span><div><div class="h5"><br><blockquote type="cite"><div><div dir="ltr"><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><font color="#888888"><br>
Pete<br>
</font></span><div><div>> On Feb 4, 2016, at 4:57 PM, Pete Cooper via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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>
</div></blockquote></div></div></div><br></div></div></blockquote></div><br></div>