<div dir="ltr">I just put in Tong's patch in r218383.<div><br></div><div>It came back clean on Ubuntu 14.04 x86_64 cmake/ninja and MacOSX 10.9.5 x86_64 with Xcode 6.1 beta.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 23, 2014 at 5:28 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 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.<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: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><font color="#888888"><div><br></div><div>- Lang.</div></font></span></div><div><div><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>
</div></div><br>_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu">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></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>