This revision was automatically updated to reflect the committed changes.
Closed by commit rG583223cd5ec4: [LLDB][NativePDB] Dont complete static
members types when completing a record… (authored by zequanwu).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.
awesome
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121030/new/
https://reviews.llvm.org/D121030
zequanwu added inline comments.
Comment at: lldb/test/Shell/SymbolFile/NativePDB/local-variables.cpp:8-26
+class B;
+class A {
+public:
+static const A constA;
+static A a;
+static B b;
+int val = 1;
labath wrote:
> zequanwu wrote:
> > labath
zequanwu updated this revision to Diff 415501.
zequanwu marked an inline comment as done.
zequanwu added a comment.
Remove `REQUIRES: system-windows`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121030/new/
https://reviews.llvm.org/D121030
labath added inline comments.
Comment at: lldb/test/Shell/SymbolFile/NativePDB/local-variables.cpp:8-26
+class B;
+class A {
+public:
+static const A constA;
+static A a;
+static B b;
+int val = 1;
zequanwu wrote:
> labath wrote:
> > Would it be
zequanwu added inline comments.
Comment at: lldb/test/Shell/SymbolFile/NativePDB/local-variables.cpp:8-26
+class B;
+class A {
+public:
+static const A constA;
+static A a;
+static B b;
+int val = 1;
labath wrote:
> Would it be possible to put
zequanwu updated this revision to Diff 415213.
zequanwu added a comment.
Testing with `image lookup -type`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121030/new/
https://reviews.llvm.org/D121030
Files:
labath added a comment.
This is cool, just one question about the test.
Comment at: lldb/test/Shell/SymbolFile/NativePDB/local-variables.cpp:8-26
+class B;
+class A {
+public:
+static const A constA;
+static A a;
+static B b;
+int val = 1;