<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#ffffff" text="#000000">
    We recently turned on MCJIT in LLVM for Mips target and Linux host.<br>
    <br>
    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.<br>
    <br>
    In the near future, we will be submitting build slaves to LLVM
    nightly builds and then you would see these tests passing or not.<br>
    <br>
    On 07/24/2012 07:36 PM, Jovanovic, Petar wrote:
    <blockquote
      cite="mid:874D219413C17C42B1D2E0432B92BE5CBBE1C7FE@exchdb03.mips.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=ISO-8859-1">
      <style type="text/css" id="owaParaStyle"></style>
      <div style="direction: ltr; font-family: Tahoma; color: rgb(0, 0,
        0); 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: rgb(0, 0, 0);
          font-size: 16px;">
          <hr tabindex="-1">
          <div id="divRpF419733" style="direction: ltr;"><font
              color="#000000" face="Tahoma" size="2"><b>From:</b>
              Chandler Carruth [<a class="moz-txt-link-abbreviated" href="mailto:chandlerc@google.com">chandlerc@google.com</a>]<br>
              <b>Sent:</b> Wednesday, July 25, 2012 4:27 AM<br>
              <b>To:</b> Jovanovic, Petar<br>
              <b>Cc:</b> <a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
              <b>Subject:</b> Re: [llvm-commits] [PATCH][MIPS] MCJIT for
              MIPS32<br>
            </font><br>
          </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 moz-do-not-send="true"
                    href="mailto:petarj@mips.com" target="_blank">petarj@mips.com</a>></span>
                wrote:<br>
                <blockquote class="gmail_quote" style="margin: 0pt 0pt
                  0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204);
                  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 moz-do-not-send="true"
                    href="mailto:llvm-commits@cs.uiuc.edu"
                    target="_blank">llvm-commits@cs.uiuc.edu</a><br>
                  <a moz-do-not-send="true"
                    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>
      <pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
llvm-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>
<a class="moz-txt-link-freetext" href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>