[Lldb-commits] [PATCH] D41092: Enable more abilities in SymbolFilePDB

2017-12-19 Thread Aaron Smith via Phabricator via lldb-commits
asmith updated this revision to Diff 127658. asmith added a comment. If a `Symbols` table is present then lldb can retrieve symbols for the types listed in PDBSym_Type. In other words, if we want symbolic information for a function then checking for the `Symbols` table is sufficient.

[Lldb-commits] [PATCH] D41092: Enable more abilities in SymbolFilePDB

2017-12-13 Thread Aaron Smith via Phabricator via lldb-commits
asmith added inline comments. Comment at: source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp:127 + } + lldbassert(m_session_up.get()); + if (auto enum_tables_up = m_session_up->getEnumTables()) { clayborg wrote: > I am assuming this assert won't fire if we give

[Lldb-commits] [PATCH] D41092: Enable more abilities in SymbolFilePDB

2017-12-11 Thread Zachary Turner via Phabricator via lldb-commits
zturner added inline comments. Comment at: source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp:108-125 + if (auto module_sp = m_obj_file->GetModule()) { +// See if a symbol file was specified through the `--symfile` option. +FileSpec symfile =

[Lldb-commits] [PATCH] D41092: Enable more abilities in SymbolFilePDB

2017-12-11 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added inline comments. Comment at: source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp:127 + } + lldbassert(m_session_up.get()); + if (auto enum_tables_up = m_session_up->getEnumTables()) { I am assuming this assert won't fire if we give this an ELF or

[Lldb-commits] [PATCH] D41092: Enable more abilities in SymbolFilePDB

2017-12-11 Thread Aaron Smith via Phabricator via lldb-commits
asmith created this revision. 1. Finding symbols through --symfile 2. More abilities: Functions, Blocks, GlobalVariables, LocalVariables, VariableTypes Repository: rL LLVM https://reviews.llvm.org/D41092 Files: source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp