[Lldb-commits] [lldb] r349175 - [NativePDB] Fix local-variables.cpp test.

[Zachary Turner via lldb-commits]

I don't know if there's a good way to do that.  Because debuggers need to
be able to see across multiple translation units, this is implemented
internally as constructing one giant AST that contains everything for the
entire program.  LLDB has no notion of a per-module AST, so it would be
hard to make this work.


On Fri, Dec 14, 2018 at 10:51 AM Pavel Labath <pavel at labath.sk> wrote:

> On 14/12/2018 19:43, Zachary Turner via lldb-commits wrote:
> > Author: zturner
> > Date: Fri Dec 14 10:43:42 2018
> > New Revision: 349175
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=349175&view=rev
> > Log:
> > [NativePDB] Fix local-variables.cpp test.
> >
> > Since we're actually running an executable on the host now, different
> > versions of Windows could load different system libraries, so we need
> > to regex out the number of loaded modules.
> >
> > Modified:
> >      lldb/trunk/lit/SymbolFile/NativePDB/local-variables.cpp
> >
> > Modified: lldb/trunk/lit/SymbolFile/NativePDB/local-variables.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/lldb/trunk/lit/SymbolFile/NativePDB/local-variables.cpp?rev=349175&r1=349174&r2=349175&view=diff
> >
> ==============================================================================
> > --- lldb/trunk/lit/SymbolFile/NativePDB/local-variables.cpp (original)
> > +++ lldb/trunk/lit/SymbolFile/NativePDB/local-variables.cpp Fri Dec 14
> 10:43:42 2018
> > @@ -151,7 +151,7 @@ int main(int argc, char **argv) {
> >   // CHECK-NEXT: Process {{.*}} exited with status = 18 (0x00000012)
> >
> >   // CHECK:      (lldb) target modules dump ast
> > -// CHECK-NEXT: Dumping clang ast for 7 modules.
> > +// CHECK-NEXT: Dumping clang ast for {{.*}} modules.
> >   // CHECK-NEXT: TranslationUnitDecl
> >   // CHECK-NEXT: |-FunctionDecl {{.*}} main 'int (int, char **)'
> >   // CHECK-NEXT: | |-ParmVarDecl {{.*}} argc 'int'
> >
> >
> > _______________________________________________
> > lldb-commits mailing list
> > lldb-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
> >
>
> It looks like it would be nice to be able to specify the module whose
> clang ast one wishes to dump. This would reduce the amount of output,
> and with it, the chance of FileCheck picking up some false positives.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20181214/4198cc2e/attachment.html>