[Lldb-commits] [PATCH] D55575: [NativePDB] Support local variables

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Dec 12 01:59:21 PST 2018

labath added inline comments.

Comment at: lldb/lit/SymbolFile/NativePDB/local-variables.cpp:31-37
+// CHECK-NEXT:    14   }
+// CHECK-NEXT:    15
+// CHECK-NEXT:    16   int main(int argc, char **argv) {
+// CHECK-NEXT: -> 17     int SomeLocal = argc * 2;
+// CHECK-NEXT:    18     return Function(SomeLocal, 'a');
+// CHECK-NEXT:    19   }
+// CHECK-NEXT:    20
I'm not thrilled by all of the hard-coded information (line numbers, the default format of stop-information printing) here, which is irrelevant for the tested functionality here. This will make it very hard to update this test if there is ever a need for (perhaps one would like to remove the line break in the RUN: command once the `env LLDB_USE_NATIVE_PDB_READER=1` part disappears), or the default way of printing stop information in lldb changes. It doesn't seem like a maintainable long term strategy.

Comment at: lldb/source/Plugins/SymbolFile/NativePDB/SymbolFileNativePDB.cpp:1929
+#include "lldb/Target/MemoryRegionInfo.h"

Comment at: llvm/include/llvm/Support/BinaryStreamArray.h:142
-  void drop_front() { Stream = Stream.drop_front(begin()->length()); }
It looks like this could be done (and tested, in llvm) separately.



More information about the lldb-commits mailing list