search for: 379fc3df

Hi Yaron, Overall this looks great. There are a couple of remaining holes. Specifically, MCJIT inherits implementations of the FindFunctionNamed and runStaticConstructorsDestructors methods from ExecutionEngine which use the old Modules vector, so you'll need to override these in MCJIT. (The implementations are fairly trivial.) Beyond that I just have a couple of questions. First,

Hi Andy, Here is the patch. it incorporates: 1) your latest patch to SVN. 2) mcjit-module-state-optimization.patch. 3) the PtrSet changes. Other than the OwnedModules implementation there were other differences between 1) and 2), especially in the Finalize* functions, so please review that I got the right code. I got bitten by subtle bugs arising from MCJIT inheriting from EE: First, MCJIT

...ode and after fixing the above bugs > the code, debugged line by line, appear to work OK.**** > > ** ** > > Yaron**** > > ** ** > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20131023/379fc3df/attachment.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: mcjit-ptrset2.diff Type: application/octet-stream Size: 17405 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20131023/379fc3df/attachment.obj>