<div dir="ltr">Thanks. Please go ahead and submit.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 1, 2016 at 1:42 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Fixed the problem and a rebased patch is attached.<br>
<br>
Cheers,<br>
Rafael<br>
<br>
<br>
On 1 November 2016 at 16:01, Rafael Espíndola<br>
<div class="HOEnZb"><div class="h5"><<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> I see. I will fix that.<br>
><br>
> Cheers,<br>
> Rafael<br>
><br>
><br>
> On 1 November 2016 at 13:12, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
>> One test is failing with this patch because of the reason I described in the<br>
>> review thread for <a href="https://reviews.llvm.org/D25516" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25516</a>.<br>
>><br>
>> On Tue, Nov 1, 2016 at 5:09 AM, Rafael Espíndola<br>
>> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>>><br>
>>> Ping on the commit :-)<br>
>>><br>
>>> Cheers,<br>
>>> Rafael<br>
>>><br>
>>><br>
>>> On 28 October 2016 at 18:22, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
>>> > ruiu added inline comments.<br>
>>> ><br>
>>> ><br>
>>> > ================<br>
>>> > Comment at: ELF/Memory.cpp:23<br>
>>> > +<br>
>>> > +void elf::freeArena() {<br>
>>> > +  for (SpecificAllocBase *Alloc : SpecificAllocBase::Instances)<br>
>>> > ----------------<br>
>>> > rafael wrote:<br>
>>> >> Do we still need to free this at a specific point in time? If not just<br>
>>> >> letting the destructors take care of it should simplify the code.<br>
>>> > This is for the use case in which you directly call lld::elf::link from<br>
>>> > your program. In that use case, we want to free all memories before<br>
>>> > returning from link().<br>
>>> ><br>
>>> ><br>
>>> > <a href="https://reviews.llvm.org/D26043" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26043</a><br>
>>> ><br>
>>> ><br>
>>> ><br>
>><br>
>><br>
</div></div></blockquote></div><br></div>