[llvm-commits] [PATCH] unit tests for MCJIT

[Malea, Daniel]

Hi Jim and Danil,

I thought I'd ping you guys as this patch has been sitting on the lists for a few weeks.

No rush, but whenever you have a moment, any comments would be appreciated :)

Thanks,
Dan


On 2012-08-29, at 3:22 PM, Malea, Daniel wrote:

Hi all,

I noticed MCJIT does not have any unit tests, so I added a few. If someone has a moment, could you please review the attached patches?

Hopefully the unit tests can serve as an example of how to programatically use MCJIT for runtime code generation and execution.

Patch descriptions:

000-move-memory-manager.patch: move LLIMCJITMemoryManager (from lli.cpp) to SectionMemoryManager in public headers (so we can test it)
001-mcjit-tests-buildscripts.patch: CMake and Makefile changes required for this patchset
002-section-memory-manager-tests.patch: Tests for the section-based memory manager
003-mcjit-tests-impl.patch: Test cases for JITting empty modules, simple functions, function-calls, globals, and multiple-modules
004-memory-buffer-tests.patch: Tests for MemoryBuffer (not really related to other patches, but MemoryBuffer had no tests and is used inside MCJIT)

Not all test cases pass yet, due to either bugs or missing functionality (for example, multiple-module support) but I commented out such cases to keep "make check" happy.

Also, I posted the combined patches on the LLVM reviewboard instance (for easier reading/reviewing): http://llvm.org/reviews/r/11/

Thanks,
Dan


<000-move-memory-manager.patch><001-mcjit-tests-buildscripts.patch><002-section-memory-manager-tests.patch><003-mcjit-tests-impl.patch><004-memory-buffer-tests.patch>