[llvm] r193291 - Optimizing MCJIT module state tracking

Chandler Carruth chandlerc at google.com
Thu Oct 24 02:57:27 PDT 2013


On Wed, Oct 23, 2013 at 5:19 PM, Andrew Kaylor <andrew.kaylor at intel.com>wrote:

> -
> -  LoadedObjectMap::iterator it, end = LoadedObjects.end();
> -  for (it = LoadedObjects.begin(); it != end; ++it) {
> -    ObjectImage *Obj = it->second;
> -    if (Obj) {
> -      NotifyFreeingObject(*Obj);
> -      delete Obj;
> -    }
> -  }
>

Hey Andy, I've reverted this hunk in r193323 because without it we were
seeing leaks from all the users of the MCJIT interface.

But I'm not 100% sure what you wanted the memory management strategy to be,
so I wanted to follow up here in case there is a deeper problem or other
solution we should be pursuing here.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131024/8eadf278/attachment.html>


More information about the llvm-commits mailing list