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

[Zachary Turner via lldb-commits]

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/1cfab539/attachment-0001.html>