[Lldb-commits] [lldb] [lldb] Avoid repeated map lookups (NFC) (PR #113121)
via lldb-commits
lldb-commits at lists.llvm.org
Sun Oct 20 18:48:24 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lldb
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/113121.diff
1 Files Affected:
- (modified) lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp (+4-5)
``````````diff
diff --git a/lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp b/lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp
index 584c2115459c66..4fc48b4d133382 100644
--- a/lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp
+++ b/lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp
@@ -1295,12 +1295,11 @@ void SymbolFilePDB::CacheFunctionNames() {
continue;
if (CPlusPlusLanguage::IsCPPMangledName(name.c_str())) {
- auto vm_addr = pub_sym_up->getVirtualAddress();
-
// PDB public symbol has mangled name for its associated function.
- if (vm_addr && addr_ids.find(vm_addr) != addr_ids.end()) {
- // Cache mangled name.
- m_func_full_names.Append(ConstString(name), addr_ids[vm_addr]);
+ if (auto vm_addr = pub_sym_up->getVirtualAddress()) {
+ if (auto it = addr_ids.find(vm_addr); it != addr_ids.end())
+ // Cache mangled name.
+ m_func_full_names.Append(ConstString(name), it->second);
}
}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/113121
More information about the lldb-commits
mailing list