[llvm-commits] [PATCH][MIPS] MCJIT for MIPS32

[reed kotler]

We recently turned on MCJIT in LLVM for Mips target and Linux host.

On our build bots that run on Mips linux boxes, we got lots of failures 
from the normal "make check" from the MCJIT tests but nobody else in 
LLVM land sees it because you need to run on Mips hardware or use Qemu 
simulating Mips hardware in order to get the failures.

In the near future, we will be submitting build slaves to LLVM nightly 
builds and then you would see these tests passing or not.

On 07/24/2012 07:36 PM, Jovanovic, Petar wrote:
> Hi Chandler,
>
> > What does it correct?
>
> MCJIT functionality for MIPS32 architecture on Linux.
> MCJIT is already supported in LLVM, but it did not work for MIPS. The 
> old JIT has been used when needed.
>
> > Is it fixing a bug, adding functionality?
>
> It is fixing issues with MCJIT for MIPS32. MIPS supports static 
> relocation model for MCJIT, and this patch enables successful 
> execution for these cases.
>
> > Why are there no test cases added? It seems like a lot of new code 
> w/o tests.
>
> No test cases have been added, because LLVM already contains tests for 
> MCJIT.
>
> Regards,
> Petar
> ------------------------------------------------------------------------
> *From:* Chandler Carruth [chandlerc at google.com]
> *Sent:* Wednesday, July 25, 2012 4:27 AM
> *To:* Jovanovic, Petar
> *Cc:* llvm-commits at cs.uiuc.edu
> *Subject:* Re: [llvm-commits] [PATCH][MIPS] MCJIT for MIPS32
>
> What does it correct? Is it fixing a bug, adding functionality? We 
> need more information about what you're trying to do in order to have 
> the context for reviewing a patch.
>
> Why are there no test cases added? It seems like a lot of new code w/o 
> tests.
>
> Are all of these changes really a single patch?
>
>
> On Tue, Jul 24, 2012 at 7:04 PM, Jovanovic, Petar <petarj at mips.com 
> <mailto:petarj at mips.com>> wrote:
>
>     Hi everyone
>
>     here is a patch that corrects MCJIT functionality for MIPS32
>     architecture.
>     The code has been tested on little-endian MIPS32 platform.
>
>     Petar
>     _______________________________________________
>     llvm-commits mailing list
>     llvm-commits at cs.uiuc.edu <mailto:llvm-commits at cs.uiuc.edu>
>     http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120724/74e4efdd/attachment.html>