Hello,
I noticed that the AST importer is very eager to import classes/structs
that were already completed, even if they are not needed. This is best
illustrated with an example.
struct C0 { int x = 0; };
struct C1 { int x = 1; C0* c0 = 0; };
struct C2 { int x = 2; C1* c1 = 0; };
int main() {
h (The ASTImporter
>> is more of a Clang thing, so they might not read lldb-dev). Also it makes
>> it easier to see/test your patch :)
>>
>> (And +Gabor just in case)
>>
>> > On Nov 6, 2019, at 10:25 PM, Jaroslav Sevcik via lldb-dev <
>> lldb-dev@lis
;
>> Gabor
>>
>>
>> On Wed, Nov 6, 2019 at 11:21 PM Raphael “Teemperor” Isemann via lldb-dev <
>> lldb-dev@lists.llvm.org> wrote:
>>
>>> Can you post that patch on Phabricator with an '[ASTImporter]’ as a name
>>> prefix? This way the ASTI
> On Thu, Nov 7, 2019 at 6:14 PM Gábor Márton
>>> wrote:
>>>
>>>> Hi Jaroslav,
>>>>
>>>> Thanks for working on this. Still, there are things that are unclear
>>>> for me.
>>>> I see that `LayoutRecordType` is called su
Attempt at a fix: https://reviews.llvm.org/D76650
On Mon, Mar 23, 2020 at 5:22 PM Lutz Justen via lldb-dev <
lldb-dev@lists.llvm.org> wrote:
> Sorry, the link was internal. Here's the proper on:
>
> https://github.com/llvm-mirror/lldb/blob/master/source/Core/FormatEntity.cpp#L861
>
>
> On Mon, Ma
The svr4 support seems to be off by default:
https://github.com/llvm/llvm-project/blob/2974b3c566d68f1d7c907f891137cf0292dd35aa/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemoteProperties.td#L14
It would definitely make sense to turn it on by default.
- J.
On Thu, May 14, 2020 at 10:13 AM
