[Lldb-commits] [PATCH] D55142: Minidump debugging using the native PDB reader

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 11 10:39:01 PST 2018

zturner added inline comments.

Comment at: source/Plugins/SymbolFile/NativePDB/SymbolFileNativePDB.cpp:139-144
+    llvm::consumeError(expected_binary.takeError());
+    pdb_file = obj_file.GetFileSpec()
+                   .GetFileNameStrippingExtension()
+                   .GetStringRef()
+                   .str();
+    pdb_file += ".pdb";
We talked about this offline, but bringing the discussion back here.  Can you describe the use case that this is addressing?  As you mention, this is a temporary hack until we have proper symbol searching logic, but proper symbol searching logic will do more than just look up symbols in a symbol server.  It will also, for example, look in the same directory as the executable file.  If we changed this logic to do that, would your use case still be addressed?  At least that way, the logic we're adding is not temporary, even if it will eventually live in a different place (e.g. the SymbolVendor).



More information about the lldb-commits mailing list