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

[Jim Grosbach]

Hi Daniel,

I just got back from vacation, but this is high on my queue to look at as I unbury myself.

-Jim

On Sep 11, 2012, at 11:59 AM, "Malea, Daniel" <daniel.malea at intel.com> wrote:

> 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>
>