[lldb-dev] [PATCH] Tentative patch to move JITMemoryManager in to LLDB.

[Lang Hames]

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.

- Lang.

On Tue, Sep 23, 2014 at 5:24 PM, Lang Hames <lhames at gmail.com> wrote:

> I should have checked lldb-commits first. That sounds like a much better
> option. :)
>
> - Lang.
>
> On Tue, Sep 23, 2014 at 5:13 PM, Tong Shen <endlessroad at google.com> wrote:
>
>> Oops, sorry Lang, didn't see your patch.
>>
>> The old JIT has been completely removed by upstream LLVM :-(
>> So I think it's better to avoid maintaining that ourselves...
>>
>> Also, the functions specific to the old JITMemoryManager seems not
>> used by lldb (except the wrapper functions).
>>
>> On Tue, Sep 23, 2014 at 5:09 PM, Eric Christopher <echristo at gmail.com>
>> wrote:
>> > See also Tong's patch to just move to SectionMemoryManager on
>> lldb-commits :)
>> >
>> > -eric
>> >
>> > On Tue, Sep 23, 2014 at 4:53 PM, Lang Hames <lhames at gmail.com> wrote:
>> >> Hi All,
>> >>
>> >> The JITMemoryManager class was removed from MCJIT in r218316, but LLDB
>> is
>> >> still relying on it.
>> >>
>> >> The attached patch moves the JITMemoryManager in to LLDB (minus the
>> AtExit
>> >> handler functionality, which I don't believe LLDB was using, and which
>> was
>> >> causing build warnings for me locally).
>> >>
>> >> I'm still running the test suite locally, so I'm sure whether this
>> passes
>> >> all the tests, but it does at least seem to get LLDB building again.
>> >>
>> >> I'll update with test results as soon as they come through.
>> >>
>> >> - Lang.
>> >>
>> >> _______________________________________________
>> >> lldb-dev mailing list
>> >> lldb-dev at cs.uiuc.edu
>> >> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
>> >>
>>
>>
>>
>> --
>> Best Regards, Tong Shen
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140923/41b65667/attachment.html>