<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi,
    <p>  This issue is caused by a recent change in ARM assembler
      included since Binutils 2.29.</p>
    <p>  The details of that change can be found at
      <a class="moz-txt-link-freetext" href="https://sourceware.org/bugzilla/show_bug.cgi?id=21458">https://sourceware.org/bugzilla/show_bug.cgi?id=21458</a><br>
    </p>
    <p>  The semantics of ADR has changed.  In general, the address
      generated by ADR will guarantee the LSB be set if it's a thumb
      function address.</p>
    <p>   I noticed h264idct_neon.S is using something like:</p>
    <p>     <span class="pl-en">adreq lr</span><span class="pl-s1">,</span><span
        class="pl-en"> X(ff_h264_idct_add_neon) </span><span
        class="pl-s1">+</span><span class="pl-en"> CONFIG_THUMB<br>
      </span></p>
    <p>   As ADR now will set the LSB automatically, you don't need
      CONFIG_THUMB any more.</p>
    <p>   I think h264idct_neon.S needs to be updated, and the
      modification should make sure it works with both old Binutils and
      the new one.</p>
    Regards,<br>
    Jiong<br>
    <br>
  </body>
</html>