summaryrefslogtreecommitdiffstats
path: root/lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp
diff options
context:
space:
mode:
authorZachary Turner <zturner@google.com>2019-01-02 18:33:54 +0000
committerZachary Turner <zturner@google.com>2019-01-02 18:33:54 +0000
commit225663308de1cbdd5f3624179ab7f43ce8286bfb (patch)
tree873e63675ea53b6047e805a16f19c89da0faee39 /lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp
parentd6d29a5812d6231098b20d74bbe11793fe3e7706 (diff)
downloadbcm5719-llvm-225663308de1cbdd5f3624179ab7f43ce8286bfb.tar.gz
bcm5719-llvm-225663308de1cbdd5f3624179ab7f43ce8286bfb.zip
[NativePDB] Implement ParseDeclsForContext.
This is a first step towards getting lldb-test symbols working with the native plugin. There is a remaining issue, which is that the plugin expects that ParseDeclsForContext will also create lldb symbols rather than just the decls, but the native pdb plugin doesn't currently do this. This will be addressed in a followup patch. llvm-svn: 350243
Diffstat (limited to 'lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp')
-rw-r--r--lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp10
1 files changed, 1 insertions, 9 deletions
diff --git a/lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp b/lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp
index 40a4b5ccdf4..51a3d106180 100644
--- a/lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp
+++ b/lldb/source/Plugins/SymbolFile/NativePDB/PdbIndex.cpp
@@ -136,15 +136,7 @@ void PdbIndex::BuildAddrToSymbolMap(CompilandIndexItem &cci) {
// If the debug info is incorrect, we could have multiple symbols with the
// same address. So use try_emplace instead of insert, and the first one
// will win.
- auto insert_result =
- cci.m_symbols_by_va.insert(std::make_pair(va, PdbSymUid(cu_sym_id)));
- (void)insert_result;
-
- // The odds of an error in some function such as GetSegmentAndOffset or
- // MakeVirtualAddress are much higher than the odds of encountering bad
- // debug info, so assert that this item was inserted in the map as opposed
- // to having already been there.
- lldbassert(insert_result.second);
+ cci.m_symbols_by_va.try_emplace(va, PdbSymUid(cu_sym_id));
}
}
OpenPOWER on IntegriCloud