6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-04-05 01:31:05 +00:00
Code Issues Projects Releases Wiki Activity

[RuntimeDyld] Make RuntimeDyldImpl::resolveExternalSymbols preserve the

Browse Source 
relocation entries it applies.

Prior to this patch, RuntimeDyldImpl::resolveExternalSymbols discarded
relocations for external symbols once they had been applied. This causes issues
if the client calls MCJIT::finalizeLoadedModules more than once, and updates the
location of any symbols in between (e.g. by calling MCJIT::mapSectionAddress).

No test case yet: None of our in-tree memory managers support moving sections
around. I'll have to hack up a dummy memory manager before I can write a unit
test.

Fixes <rdar://problem/16764378>



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@208257 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Lang Hames 2014-05-07 22:34:08 +00:00
parent df60e43e05
commit 3186597423
1 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp
View File

@ -620,6 +620,8 @@ void RuntimeDyldImpl::resolveRelocationList(const RelocationList &Relocs,
}
void RuntimeDyldImpl::resolveExternalSymbols() {
StringMap<RelocationList> ProcessedSymbols;
while (!ExternalSymbolRelocations.empty()) {
StringMap<RelocationList>::iterator i = ExternalSymbolRelocations.begin();
@ -665,8 +667,20 @@ void RuntimeDyldImpl::resolveExternalSymbols() {
resolveRelocationList(Relocs, Addr);
}
ProcessedSymbols[i->first()] = i->second;
ExternalSymbolRelocations.erase(i);
}
// Restore the relocation entries that were consumed in the loop above:
//
// FIXME: Replace the following loop with:
// std::swap(ProcessedSymbols, ExternalSymbolRelocations)
// once StringMap has copy and move construction.
for (StringMap<RelocationList>::iterator I = ProcessedSymbols.begin(),
E = ProcessedSymbols.end();
I != E; ++I) {
ExternalSymbolRelocations[I->first()] = I->second;
}
}
//===----------------------------------------------------------------------===//