diff --git a/lldb/source/Core/Module.cpp b/lldb/source/Core/Module.cpp --- a/lldb/source/Core/Module.cpp +++ b/lldb/source/Core/Module.cpp @@ -440,8 +440,6 @@ bool Module::ResolveFileAddress(lldb::addr_t vm_addr, Address &so_addr) { std::lock_guard guard(m_mutex); - LLDB_SCOPED_TIMERF("Module::ResolveFileAddress (vm_addr = 0x%" PRIx64 ")", - vm_addr); SectionList *section_list = GetSectionList(); if (section_list) return so_addr.ResolveAddressUsingFileSections(vm_addr, section_list); diff --git a/lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp b/lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp --- a/lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp +++ b/lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp @@ -44,6 +44,7 @@ #include "lldb/Host/SafeMachO.h" +#include "llvm/ADT/DenseSet.h" #include "llvm/Support/FormatVariadic.h" #include "llvm/Support/MemoryBuffer.h" @@ -1895,9 +1896,9 @@ filename = first_section_sp->GetObjectFile()->GetFileSpec().GetPath(); Host::SystemLog(Host::eSystemLogError, - "error: unable to find section %d for a symbol in %s, corrupt file?\n", - n_sect, - filename.c_str()); + "error: unable to find section %d for a symbol in " + "%s, corrupt file?\n", + n_sect, filename.c_str()); } } if (m_section_infos[n_sect].vm_range.Contains(file_addr)) { @@ -2183,7 +2184,16 @@ // We add symbols to the table in the order of most information (nlist // records) to least (function starts), and avoid duplicating symbols // via this set. - std::set symbols_added; + llvm::DenseSet symbols_added; + + // We are using a llvm::DenseSet for "symbols_added" so we must be sure we + // do not add the tombstone or empty keys to the set. + auto add_symbol_addr = [&symbols_added](lldb::addr_t file_addr) { + // Don't add the tombstone or empty keys. + if (file_addr == UINT64_MAX || file_addr == UINT64_MAX - 1) + return; + symbols_added.insert(file_addr); + }; FunctionStarts function_starts; lldb::offset_t offset = MachHeaderSizeFromMagic(m_header.magic); uint32_t i; @@ -3640,9 +3650,9 @@ symbol_section); sym[GSYM_sym_idx].GetAddressRef().SetOffset( symbol_value); - symbols_added.insert(sym[GSYM_sym_idx] - .GetAddress() - .GetFileAddress()); + add_symbol_addr(sym[GSYM_sym_idx] + .GetAddress() + .GetFileAddress()); // We just need the flags from the linker // symbol, so put these flags // into the N_GSYM flags to avoid duplicate @@ -3662,7 +3672,7 @@ if (set_value) { sym[sym_idx].GetAddressRef().SetSection(symbol_section); sym[sym_idx].GetAddressRef().SetOffset(symbol_value); - symbols_added.insert( + add_symbol_addr( sym[sym_idx].GetAddress().GetFileAddress()); } sym[sym_idx].SetFlags(nlist.n_type << 16 | nlist.n_desc); @@ -4475,7 +4485,7 @@ // invalid address of zero when the global is a common symbol. sym[GSYM_sym_idx].GetAddressRef().SetSection(symbol_section); sym[GSYM_sym_idx].GetAddressRef().SetOffset(symbol_value); - symbols_added.insert( + add_symbol_addr( sym[GSYM_sym_idx].GetAddress().GetFileAddress()); // We just need the flags from the linker symbol, so put these // flags into the N_GSYM flags to avoid duplicate symbols in @@ -4494,7 +4504,8 @@ if (set_value) { sym[sym_idx].GetAddressRef().SetSection(symbol_section); sym[sym_idx].GetAddressRef().SetOffset(symbol_value); - symbols_added.insert(sym[sym_idx].GetAddress().GetFileAddress()); + if (symbol_section) + add_symbol_addr(sym[sym_idx].GetAddress().GetFileAddress()); } sym[sym_idx].SetFlags(nlist.n_type << 16 | nlist.n_desc); if (nlist.n_desc & N_WEAK_REF) @@ -4590,7 +4601,7 @@ sym[sym_idx].SetIsSynthetic(true); sym[sym_idx].SetExternal(true); sym[sym_idx].GetAddressRef() = symbol_addr; - symbols_added.insert(symbol_addr.GetFileAddress()); + add_symbol_addr(symbol_addr.GetFileAddress()); if (e.entry.flags & TRIE_SYMBOL_IS_THUMB) sym[sym_idx].SetFlags(MACHO_NLIST_ARM_SYMBOL_IS_THUMB); ++sym_idx; @@ -4645,7 +4656,7 @@ sym[sym_idx].SetType(eSymbolTypeCode); sym[sym_idx].SetIsSynthetic(true); sym[sym_idx].GetAddressRef() = symbol_addr; - symbols_added.insert(symbol_addr.GetFileAddress()); + add_symbol_addr(symbol_addr.GetFileAddress()); if (symbol_flags) sym[sym_idx].SetFlags(symbol_flags); if (symbol_byte_size) @@ -4746,7 +4757,7 @@ sym[sym_idx].SetType(eSymbolTypeResolver); sym[sym_idx].SetIsSynthetic(true); sym[sym_idx].GetAddressRef() = so_addr; - symbols_added.insert(so_addr.GetFileAddress()); + add_symbol_addr(so_addr.GetFileAddress()); sym[sym_idx].SetByteSize(symbol_stub_byte_size); ++sym_idx; } diff --git a/lldb/source/Symbol/ObjectFile.cpp b/lldb/source/Symbol/ObjectFile.cpp --- a/lldb/source/Symbol/ObjectFile.cpp +++ b/lldb/source/Symbol/ObjectFile.cpp @@ -617,11 +617,13 @@ } ConstString ObjectFile::GetNextSyntheticSymbolName() { - StreamString ss; + llvm::SmallString<256> name; + llvm::raw_svector_ostream os(name); ConstString file_name = GetModule()->GetFileSpec().GetFilename(); - ss.Printf("___lldb_unnamed_symbol%u$$%s", ++m_synthetic_symbol_idx, - file_name.GetCString()); - return ConstString(ss.GetString()); + ++m_synthetic_symbol_idx; + os << "___lldb_unnamed_symbol" << m_synthetic_symbol_idx << "$$" + << file_name.GetStringRef(); + return ConstString(os.str()); } std::vector