[LLVMbugs] [Bug 4762] [2.6] AssertingVH abort in Kaleidoscope example

[bugzilla-daemon at cs.uiuc.edu]

http://llvm.org/bugs/show_bug.cgi?id=4762


reid.kleckner at gmail.com <reid.kleckner at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED




--- Comment #5 from reid.kleckner at gmail.com  2009-08-26 16:03:10 ---
The module provider needed to be allocated on the heap, not the stack, so it
could be properly deleted, as identified by Chris.  I committed the fix as
r80138.

I think the double-delete bug has been in the code since before it was added to
the repository, but when Jeff added the AssertingVH in the JIT it became a
problem.  Now that the EE owns the MP, the JIT can properly free the machine
code before it deletes the module and all associated functions.


-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.