RenderScript lookup bug fix

After the std::move operation the unique pointer is null. 
So this statement always returns a null pointer. 

Also remove unnecessary call to Module::ParseAllDebugSymbols(), 
which spews errors due to how it incorrectly tries to parse DWARF DIE types.

llvm-svn: 248274
This commit is contained in:
Ewan Crawford
2015-09-22 13:36:35 +00:00
parent 86cce70010
commit d10ca9deee

View File

@@ -1225,7 +1225,7 @@ RenderScriptRuntime::LookUpScript(addr_t address, bool create)
std::unique_ptr<ScriptDetails> s(new ScriptDetails);
s->script = address;
m_scripts.push_back(std::move(s));
return s.get();
return m_scripts.back().get();
}
return nullptr;
}
@@ -1244,7 +1244,7 @@ RenderScriptRuntime::LookUpAllocation(addr_t address, bool create)
std::unique_ptr<AllocationDetails> a(new AllocationDetails);
a->address = address;
m_allocations.push_back(std::move(a));
return a.get();
return m_allocations.back().get();
}
return nullptr;
}
@@ -1254,7 +1254,6 @@ RSModuleDescriptor::Dump(Stream &strm) const
{
strm.Indent();
m_module->GetFileSpec().Dump(&strm);
m_module->ParseAllDebugSymbols();
if(m_module->GetNumCompileUnits())
{
strm.Indent("Debug info loaded.");