forked from OSchip/llvm-project
[Orc] Make the ObjectLinkingLayer take ownership of object files until
finalization time. As currently implemented, RuntimeDyldELF requires the original object file to be avaible when relocations are being resolved. This patch ensures that the ObjectLinkingLayer preserves it until then. In the future RuntimeDyldELF should be rewritten to remove this requirement, at which point this patch can be reverted. Regression test cases for Orc (which include coverage of this bug) will be committed shortly. llvm-svn: 227778
This commit is contained in:
parent
10d02b5604
commit
29dff9837a
|
@ -76,7 +76,12 @@ public:
|
|||
Buffers.push_back(std::move(Buffer));
|
||||
}
|
||||
|
||||
return BaseLayer.addObjectSet(std::move(Objects), std::move(MM));
|
||||
ModuleSetHandleT H =
|
||||
BaseLayer.addObjectSet(Objects, std::move(MM));
|
||||
|
||||
BaseLayer.takeOwnershipOfBuffers(H, std::move(Buffers));
|
||||
|
||||
return H;
|
||||
}
|
||||
|
||||
/// @brief Remove the module set associated with the handle H.
|
||||
|
|
|
@ -61,6 +61,7 @@ protected:
|
|||
RTDyld->resolveRelocations();
|
||||
RTDyld->registerEHFrames();
|
||||
MM->finalizeMemory();
|
||||
OwnedBuffers.clear();
|
||||
State = Finalized;
|
||||
}
|
||||
|
||||
|
@ -70,10 +71,19 @@ protected:
|
|||
RTDyld->mapSectionAddress(LocalAddress, TargetAddress);
|
||||
}
|
||||
|
||||
void takeOwnershipOfBuffer(std::unique_ptr<MemoryBuffer> B) {
|
||||
OwnedBuffers.push_back(std::move(B));
|
||||
}
|
||||
|
||||
private:
|
||||
std::unique_ptr<RTDyldMemoryManager> MM;
|
||||
std::unique_ptr<RuntimeDyld> RTDyld;
|
||||
enum { Raw, Finalizing, Finalized } State;
|
||||
|
||||
// FIXME: This ownership hack only exists because RuntimeDyldELF still
|
||||
// wants to be able to inspect the original object when resolving
|
||||
// relocations. As soon as that can be fixed this should be removed.
|
||||
std::vector<std::unique_ptr<MemoryBuffer>> OwnedBuffers;
|
||||
};
|
||||
|
||||
typedef std::list<LinkedObjectSet> LinkedObjectSetListT;
|
||||
|
@ -81,6 +91,16 @@ protected:
|
|||
public:
|
||||
/// @brief Handle to a set of loaded objects.
|
||||
typedef LinkedObjectSetListT::iterator ObjSetHandleT;
|
||||
|
||||
// Ownership hack.
|
||||
// FIXME: Remove this as soon as RuntimeDyldELF can apply relocations without
|
||||
// referencing the original object.
|
||||
template <typename OwningMBSet>
|
||||
void takeOwnershipOfBuffers(ObjSetHandleT H, OwningMBSet MBs) {
|
||||
for (auto &MB : MBs)
|
||||
H->takeOwnershipOfBuffer(std::move(MB));
|
||||
}
|
||||
|
||||
};
|
||||
|
||||
/// @brief Default (no-op) action to perform when loading objects.
|
||||
|
|
Loading…
Reference in New Issue