<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 23, 2013 at 5:19 PM, Andrew Kaylor <span dir="ltr"><<a href="mailto:andrew.kaylor@intel.com" target="_blank" class="cremed">andrew.kaylor@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":ff3" style="overflow:hidden">-<br>
-  LoadedObjectMap::iterator it, end = LoadedObjects.end();<br>
-  for (it = LoadedObjects.begin(); it != end; ++it) {<br>
-    ObjectImage *Obj = it->second;<br>
-    if (Obj) {<br>
-      NotifyFreeingObject(*Obj);<br>
-      delete Obj;<br>
-    }<br>
-  }</div></blockquote></div><br>Hey Andy, I've reverted this hunk in r193323 because without it we were seeing leaks from all the users of the MCJIT interface.</div><div class="gmail_extra"><br></div><div class="gmail_extra">
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.</div></div>