From d55d159d09912b72728d9767d6cdaac36bade7dc Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Thu, 24 Oct 2013 09:52:56 +0000 Subject: Revert part of r193291, restoring the deletion of loaded objects. Without this, customers of the MCJIT were leaking memory like crazy. It's not really clear what the *right* memory management is here, so I'm not trying to add lots of tests or other logic, just trying to get us back to a better baseline. I'll follow up on the original commit to figure out the right path forward. llvm-svn: 193323 --- llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'llvm/lib/ExecutionEngine/MCJIT') diff --git a/llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp b/llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp index a6405947aba..da4164e6476 100644 --- a/llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp +++ b/llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp @@ -76,6 +76,15 @@ MCJIT::~MCJIT() { // Modules.clear(); Dyld.deregisterEHFrames(); + + LoadedObjectMap::iterator it, end = LoadedObjects.end(); + for (it = LoadedObjects.begin(); it != end; ++it) { + ObjectImage *Obj = it->second; + if (Obj) { + NotifyFreeingObject(*Obj); + delete Obj; + } + } LoadedObjects.clear(); delete TM; } -- cgit v1.2.3