[Lldb-commits] [PATCH] D55122: [PDB] Fix location retrieval for function local variables and arguments that are stored relative to VFRAME

[Aleksandr Urakov via lldb-commits]

Ok, thank you! I'll contact you when I will be ready to work on the new
plugin.

Am Mo., 3. Dez. 2018, 20:07 hat Zachary Turner <zturner at google.com>
geschrieben:

> Ok that makes sense. I’ll work on this soon, I’m trying to get all
> existing dia PDB tests passing with the native plugin so i can delete old
> plugin, so I’ll keep this in mind and probably do this next.
>
> I’m trying to fix some bugs in clang so that we emit compatible debug info
> to msvc, which currently blocks conversion of a few tests. So after that
> I’ll work on this
> On Mon, Dec 3, 2018 at 9:02 AM Aleksandr Urakov <
> aleksandr.urakov at jetbrains.com> wrote:
>
>> If I understand correctly, the native plugin processes for now globals
>> and constants, but it can't create locals, right? But this FPO conversion
>> is applicable to locals only, so we need to implement locals creation in
>> the new plugin first, before moving the conversion there. That's why we
>> need these functions.
>>
>> Am Mo., 3. Dez. 2018, 19:41 hat Zachary Turner <zturner at google.com>
>> geschrieben:
>>
>>> Is the only reason we need these methods so that we can use lldb-test?
>>> If so, perhaps the tests can be written in terms of lldb itself, similar to
>>> the existing NativePDB tests
>>> On Mon, Dec 3, 2018 at 8:39 AM Aleksandr Urakov via Phabricator <
>>> reviews at reviews.llvm.org> wrote:
>>>
>>>> aleksandr.urakov added a comment.
>>>>
>>>> I can do it, but unfortunately not this week... I want to join the
>>>> native plugin development some later, at the end of this month, after some
>>>> current work.
>>>>
>>>>
>>>> Repository:
>>>>   rLLDB LLDB
>>>>
>>>> CHANGES SINCE LAST ACTION
>>>>   https://reviews.llvm.org/D55122/new/
>>>>
>>>> https://reviews.llvm.org/D55122
>>>>
>>>>
>>>>
>>>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20181203/e9f848da/attachment.html>