<div dir="ltr">I just had a look at Tong's patch - that definitely looks like a better way to go. I had assumed that LLDB was actually using JITMemoryManager's functionality (given that it was being forwarded to), but since that's not the case it makes much more sense to just switch to SectionMemoryManager.<div><br></div><div>- Lang.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 23, 2014 at 5:24 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I should have checked lldb-commits first. That sounds like a much better option. :)<span class="HOEnZb"><font color="#888888"><div><br></div><div>- Lang.</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 23, 2014 at 5:13 PM, Tong Shen <span dir="ltr"><<a href="mailto:endlessroad@google.com" target="_blank">endlessroad@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Oops, sorry Lang, didn't see your patch.<br>
<br>
The old JIT has been completely removed by upstream LLVM :-(<br>
So I think it's better to avoid maintaining that ourselves...<br>
<br>
Also, the functions specific to the old JITMemoryManager seems not<br>
used by lldb (except the wrapper functions).<br>
<div><div><br>
On Tue, Sep 23, 2014 at 5:09 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> See also Tong's patch to just move to SectionMemoryManager on lldb-commits :)<br>
><br>
> -eric<br>
><br>
> On Tue, Sep 23, 2014 at 4:53 PM, Lang Hames <<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>> wrote:<br>
>> Hi All,<br>
>><br>
>> The JITMemoryManager class was removed from MCJIT in r218316, but LLDB is<br>
>> still relying on it.<br>
>><br>
>> The attached patch moves the JITMemoryManager in to LLDB (minus the AtExit<br>
>> handler functionality, which I don't believe LLDB was using, and which was<br>
>> causing build warnings for me locally).<br>
>><br>
>> I'm still running the test suite locally, so I'm sure whether this passes<br>
>> all the tests, but it does at least seem to get LLDB building again.<br>
>><br>
>> I'll update with test results as soon as they come through.<br>
>><br>
>> - Lang.<br>
>><br>
>> _______________________________________________<br>
>> lldb-dev mailing list<br>
>> <a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
>><br>
<br>
<br>
<br>
</div></div><span><font color="#888888">--<br>
Best Regards, Tong Shen<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>