Removed the function:
ModuleSP
Module::GetSP();
Since we are now using intrusive ref counts, we can easily turn any
pointer to a module into a shared pointer just by assigning it.
llvm-svn: 139984
diff --git a/lldb/source/API/SBAddress.cpp b/lldb/source/API/SBAddress.cpp
index 110ef2d..e4161a2 100644
--- a/lldb/source/API/SBAddress.cpp
+++ b/lldb/source/API/SBAddress.cpp
@@ -221,9 +221,9 @@
SBModule sb_module;
if (m_opaque_ap.get())
{
- const Module *module = m_opaque_ap->GetModule();
+ Module *module = m_opaque_ap->GetModule();
if (module)
- *sb_module = module->GetSP();
+ *sb_module = module;
}
return sb_module;
}
diff --git a/lldb/source/Commands/CommandObjectTarget.cpp b/lldb/source/Commands/CommandObjectTarget.cpp
index 93ed631..dd5f166 100644
--- a/lldb/source/Commands/CommandObjectTarget.cpp
+++ b/lldb/source/Commands/CommandObjectTarget.cpp
@@ -2725,7 +2725,7 @@
if (use_global_module_list)
{
module = Module::GetAllocatedModuleAtIndex(image_idx);
- module_sp = module->GetSP();
+ module_sp = module;
}
else
{
diff --git a/lldb/source/Core/Address.cpp b/lldb/source/Core/Address.cpp
index d6e851c..c07901c 100644
--- a/lldb/source/Core/Address.cpp
+++ b/lldb/source/Core/Address.cpp
@@ -737,7 +737,7 @@
Module *address_module = m_section->GetModule();
if (address_module)
{
- sc->module_sp = address_module->GetSP();
+ sc->module_sp = address_module;
if (sc->module_sp)
return sc->module_sp->ResolveSymbolContextForAddress (*this, resolve_scope, *sc);
}
@@ -759,7 +759,7 @@
if (m_section)
{
SymbolContext sc;
- sc.module_sp = m_section->GetModule()->GetSP();
+ sc.module_sp = m_section->GetModule();
if (sc.module_sp)
{
sc.module_sp->ResolveSymbolContextForAddress (*this, eSymbolContextCompUnit, sc);
@@ -775,7 +775,7 @@
if (m_section)
{
SymbolContext sc;
- sc.module_sp = m_section->GetModule()->GetSP();
+ sc.module_sp = m_section->GetModule();
if (sc.module_sp)
{
sc.module_sp->ResolveSymbolContextForAddress (*this, eSymbolContextFunction, sc);
@@ -791,7 +791,7 @@
if (m_section)
{
SymbolContext sc;
- sc.module_sp = m_section->GetModule()->GetSP();
+ sc.module_sp = m_section->GetModule();
if (sc.module_sp)
{
sc.module_sp->ResolveSymbolContextForAddress (*this, eSymbolContextBlock, sc);
@@ -807,7 +807,7 @@
if (m_section)
{
SymbolContext sc;
- sc.module_sp = m_section->GetModule()->GetSP();
+ sc.module_sp = m_section->GetModule();
if (sc.module_sp)
{
sc.module_sp->ResolveSymbolContextForAddress (*this, eSymbolContextSymbol, sc);
@@ -823,7 +823,7 @@
if (m_section)
{
SymbolContext sc;
- sc.module_sp = m_section->GetModule()->GetSP();
+ sc.module_sp = m_section->GetModule();
if (sc.module_sp)
{
sc.module_sp->ResolveSymbolContextForAddress (*this, eSymbolContextLineEntry, sc);
diff --git a/lldb/source/Core/Module.cpp b/lldb/source/Core/Module.cpp
index 2786600..6619578 100644
--- a/lldb/source/Core/Module.cpp
+++ b/lldb/source/Core/Module.cpp
@@ -128,13 +128,6 @@
}
-ModuleSP
-Module::GetSP () const
-{
- ModuleSP module_sp(const_cast<Module*>(this));
- return module_sp;
-}
-
const lldb_private::UUID&
Module::GetUUID()
{
@@ -177,8 +170,8 @@
if (num_comp_units == 0)
return;
- TargetSP null_target;
- SymbolContext sc(null_target, GetSP());
+ SymbolContext sc;
+ sc.module_sp = this;
uint32_t cu_idx;
SymbolVendor *symbols = GetSymbolVendor ();
@@ -212,7 +205,7 @@
void
Module::CalculateSymbolContext(SymbolContext* sc)
{
- sc->module_sp = GetSP();
+ sc->module_sp = this;
}
Module *
@@ -282,7 +275,7 @@
{
// If the section offset based address resolved itself, then this
// is the right module.
- sc.module_sp = GetSP();
+ sc.module_sp = this;
resolved_flags |= eSymbolContextModule;
// Resolve the compile unit, function, block, line table or line
@@ -384,7 +377,7 @@
const uint32_t start_size = sc_list.GetSize();
const uint32_t num_compile_units = GetNumCompileUnits();
SymbolContext sc;
- sc.module_sp = GetSP();
+ sc.module_sp = this;
const bool compare_directory = path.GetDirectory();
for (uint32_t i=0; i<num_compile_units; ++i)
{
diff --git a/lldb/source/Core/SearchFilter.cpp b/lldb/source/Core/SearchFilter.cpp
index b0934b4..adcc1ca 100644
--- a/lldb/source/Core/SearchFilter.cpp
+++ b/lldb/source/Core/SearchFilter.cpp
@@ -401,7 +401,7 @@
Module* module = m_target_sp->GetImages().GetModulePointerAtIndex(i);
if (FileSpec::Compare (m_module_spec, module->GetFileSpec(), false) == 0)
{
- SymbolContext matchingContext(m_target_sp, module->GetSP());
+ SymbolContext matchingContext(m_target_sp, ModuleSP(module));
Searcher::CallbackReturn shouldContinue;
shouldContinue = DoModuleIteration(matchingContext, searcher);
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp b/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
index 2e0b956..9a2c10e 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
+++ b/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
@@ -1664,7 +1664,7 @@
{
sc.Clear();
// Check if the symbol vendor already knows about this compile unit?
- sc.module_sp = m_obj_file->GetModule()->GetSP();
+ sc.module_sp = m_obj_file->GetModule();
sc.comp_unit = GetCompUnitForDWARFCompUnit(curr_cu, UINT32_MAX);
sc.function = sc.comp_unit->FindFunctionByUID (func_die->GetOffset()).get();
@@ -1969,7 +1969,7 @@
Index ();
SymbolContext sc;
- sc.module_sp = m_obj_file->GetModule()->GetSP();
+ sc.module_sp = m_obj_file->GetModule();
assert (sc.module_sp);
DWARFCompileUnit* dwarf_cu = NULL;
@@ -2018,7 +2018,7 @@
Index ();
SymbolContext sc;
- sc.module_sp = m_obj_file->GetModule()->GetSP();
+ sc.module_sp = m_obj_file->GetModule();
assert (sc.module_sp);
DWARFCompileUnit* dwarf_cu = NULL;
@@ -2057,7 +2057,7 @@
const uint32_t sc_list_initial_size = sc_list.GetSize();
SymbolContext sc;
- sc.module_sp = m_obj_file->GetModule()->GetSP();
+ sc.module_sp = m_obj_file->GetModule();
assert (sc.module_sp);
DWARFCompileUnit* dwarf_cu = NULL;
@@ -2130,7 +2130,7 @@
return;
SymbolContext sc;
- sc.module_sp = m_obj_file->GetModule()->GetSP();
+ sc.module_sp = m_obj_file->GetModule();
assert (sc.module_sp);
DWARFCompileUnit* dwarf_cu = NULL;
@@ -2208,7 +2208,7 @@
return;
SymbolContext sc;
- sc.module_sp = m_obj_file->GetModule()->GetSP();
+ sc.module_sp = m_obj_file->GetModule();
assert (sc.module_sp);
DWARFCompileUnit* dwarf_cu = NULL;
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDebugMap.cpp b/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDebugMap.cpp
index cc7ff1b0..f615dc4 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDebugMap.cpp
+++ b/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDebugMap.cpp
@@ -168,8 +168,7 @@
FileSpec oso_file_spec(oso_symbol->GetMangled().GetName().AsCString(), true);
// Don't allow cached .o files since we dress up each .o file with
// new sections. We want them to be in the module list so we can
- // always find a shared pointer to the module (in Module::GetSP()),
- // but just don't share them.
+ // always find a shared pointer to the module but just don't share them.
const bool always_create = true;
ModuleList::GetSharedModule (oso_file_spec,
m_obj_file->GetModule()->GetArchitecture(),
diff --git a/lldb/source/Symbol/Symbol.cpp b/lldb/source/Symbol/Symbol.cpp
index 7b44d77..71caa5e 100644
--- a/lldb/source/Symbol/Symbol.cpp
+++ b/lldb/source/Symbol/Symbol.cpp
@@ -359,7 +359,7 @@
Module *module = range->GetBaseAddress().GetModule ();
if (module)
{
- sc->module_sp = module->GetSP();
+ sc->module_sp = module;
return;
}
}
diff --git a/lldb/source/Target/StackFrame.cpp b/lldb/source/Target/StackFrame.cpp
index 862e4c9..2b13fe8 100644
--- a/lldb/source/Target/StackFrame.cpp
+++ b/lldb/source/Target/StackFrame.cpp
@@ -147,7 +147,7 @@
{
if (pc_module)
{
- m_sc.module_sp = pc_module->GetSP();
+ m_sc.module_sp = pc_module;
m_flags.Set (eSymbolContextModule);
}
else
@@ -227,7 +227,7 @@
Module *module = section->GetModule();
if (module)
{
- m_sc.module_sp = module->GetSP();
+ m_sc.module_sp = module;
if (m_sc.module_sp)
m_flags.Set(eSymbolContextModule);
}