[llvm-commits] [llvm] r128171 - /llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp

[Jim Grosbach]

Author: grosbach
Date: Wed Mar 23 16:35:02 2011
New Revision: 128171

URL: http://llvm.org/viewvc/llvm-project?rev=128171&view=rev
Log:
Fix double-free of Module.

The ExecutionEngine constructor already added the module, so there's no
need to call addModule() directly. Doing so causes a double-free of the
Module at program termination.

Modified:
    llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp

Modified: llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp?rev=128171&r1=128170&r2=128171&view=diff
==============================================================================
--- llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp Wed Mar 23 16:35:02 2011
@@ -78,7 +78,6 @@
   }
 
   // Initialize passes.
-  ExecutionEngine::addModule(M);
   // FIXME: When we support multiple modules, we'll want to move the code
   // gen and finalization out of the constructor here and do it more
   // on-demand as part of getPointerToFunction().