[PATCH] Fix recompileAndRelinkFunction

Hello,

JIT::recompileAndRelinkFunction's runJITOnFunction does not use the
MCI arg, causing null pointer deferences at every call.

Attached patch makes runJITOnFunction more reliable.

Thanks,
Gianluca

fix-recompileAndRelinkFunction.patch (504 Bytes)

When would MCI be null?

-Chris

Everytime you call recompileAndRelinkFunction. It calls
runJITOnFunction without specifying the MCI argument, which get
defaulted to NULL.

Gianluca

Please add a test to
http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/ExecutionEngine/JIT/JITTest.cpp?view=markup
verifying that recompileAndRelinkFunction() keeps working. Then this
fix will look fine to me.

Here it is. Unfortunately I'm not exactly a C++ guy so I'm open to
change this code after any comment you might want to have.

Gianluca

recompileandrelinktest.patch (1.31 KB)

The test looks great. Thanks for the patches! I've added a couple
comments and committed them:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20091221/093013.html

Jeffrey