<html dir="ltr">

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" id="owaParaStyle"></style>

</head>

<body fpstyle="1" ocsi="0">

<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">

<div>Hi Chandler,</div>

<div><br>

</div>

<div>> What does it correct?</div>

<div><br>

</div>

<div>MCJIT functionality for MIPS32 architecture on Linux.</div>

<div>MCJIT is already supported in LLVM, but it did not work for MIPS. The old JIT has been used when needed.</div>

<div><br>

</div>

<div>> Is it fixing a bug, adding functionality?</div>

<div><br>

</div>

<div>It is fixing issues with MCJIT for MIPS32. MIPS supports static relocation model for MCJIT, and this patch enables successful execution for these cases.</div>

<div><br>

</div>

<div>> Why are there no test cases added? It seems like a lot of new code w/o tests.</div>

<div><br>

</div>

<div>No test cases have been added, because LLVM already contains tests for MCJIT.</div>

<div><br>

</div>

<div>Regards,</div>

<div>Petar</div>

<div style="font-family: Times New Roman; color: #000000; font-size: 16px">

<hr tabindex="-1">

<div id="divRpF419733" style="direction: ltr; "><font face="Tahoma" size="2" color="#000000"><b>From:</b> Chandler Carruth [chandlerc@google.com]<br>

<b>Sent:</b> Wednesday, July 25, 2012 4:27 AM<br>

<b>To:</b> Jovanovic, Petar<br>

<b>Cc:</b> llvm-commits@cs.uiuc.edu<br>

<b>Subject:</b> Re: [llvm-commits] [PATCH][MIPS] MCJIT for MIPS32<br>

</font><br>

</div>

<div></div>

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

<div><br>

</div>

<div>Why are there no test cases added? It seems like a lot of new code w/o tests.</div>

<div><br>

</div>

<div>Are all of these changes really a single patch?</div>

<div class="gmail_extra"><br>

<br>

<div class="gmail_quote">On Tue, Jul 24, 2012 at 7:04 PM, Jovanovic, Petar <span dir="ltr">

<<a href="mailto:petarj@mips.com" target="_blank">petarj@mips.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">

Hi everyone<br>

<br>

here is a patch that corrects MCJIT functionality for MIPS32 architecture.<br>

The code has been tested on little-endian MIPS32 platform.<br>

<span class="HOEnZb"><font color="#888888"><br>

Petar</font></span><br>

_______________________________________________<br>

llvm-commits mailing list<br>

<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>

<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>

<br>

</blockquote>

</div>

<br>

</div>

</div>

</div>

</div>

</body>

</html>