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

2018-12-14 Thread Zachary Turner via lldb-commits
Ahh, so I misinterpreted what you originally said. There is one Clang AST *per SymboFile*. In PDB land, compile units are often called "modules" so it's easy to get confused. So in my original response, what I actually meant was that there's no notion of a per compile-unit AST. But that also ha

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

2018-12-14 Thread Pavel Labath via lldb-commits
On 14/12/2018 19:52, Zachary Turner wrote: 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 notio

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

2018-12-14 Thread 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 m

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

2018-12-14 Thread Pavel Labath via lldb-commits
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,

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

2018-12-14 Thread Zachary Turner via lldb-commits
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 lib