<html>
  <head>
    
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <pre>Hi,

The updated patch with addressed notes is attached:
- emulation name changed to "armelf_linux_eabi"
- corrected comment referencing x86_64 instead of ARM
- TODO list is added
- test case written with elfyaml is added

- Denis.
</pre>
    <div class="moz-cite-prefix">On 11/26/2014 05:01 PM, Will Newton
      wrote:<br>
    </div>
    <blockquote
cite="mid:CANu=Dmgf=xiZnjvGC2yFVmn=dV+Lp=+yJZMMJqsCb-KasqBk9g@mail.gmail.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      <meta name="Generator" content="MS Exchange Server version
        08.03.0377.000">
      <title>Re: [PATCH] [lld] ELF: Initial implementation for ARM
        static linking</title>
      <!-- Converted from text/plain format -->
      <p><font size="2">On 26 November 2014 at 12:17, Denis Protivensky<br>
          <a class="moz-txt-link-rfc2396E" href="mailto:dprotivensky@accesssoftek.com"><dprotivensky@accesssoftek.com></a> wrote:<br>
          > Hi,<br>
          ><br>
          > The patch contains the skeleton code for ELF binary
          generation for ARM platform.<br>
          > The code is "almost" able to statically link the simplest
          case of void main() { return 0; } sample.<br>
          <br>
          Do you know what functionality is missing to enable that?<br>
          <br>
          > Added all needed relocations, GOT and TLS handling.<br>
          > Some of these may still be improper which causes
          segfaults.<br>
          ><br>
          > NOTE: the tests are absent because there's no simple way
          to generate diff for binary files (which are used in tests by
          the linker).<br>
          <br>
          Is it possible to use ELFYAML for the tests similarly to e.g.
          the MIPS<br>
          backend? I recently posted a patch to add ARM support to
          ELFYAML.<br>
          <br>
          A couple of things I noticed:<br>
          <br>
          1. GNU ld seems to use "armelf_linux_eabi" for the emulation
          name<br>
          rather than "armlinux"<br>
          2. One comment reference to x86-64 that should be ARM<br>
          3. It would be useful to add a TODO list that lists all the
          known<br>
          unimplemented parts<br>
          <br>
          Otherwise this looks like a good start!<br>
          <br>
          --<br>
          Will Newton<br>
          Toolchain Working Group, Linaro<br>
        </font>
      </p>
    </blockquote>
    <br>
  </body>
</html>