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

Jovanovic, Petar petarj at mips.com
Tue Jul 24 19:36:25 PDT 2012


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


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120725/713305c6/attachment.html>


More information about the llvm-commits mailing list