<div dir="ltr">Hi Mihail,<div><br></div><div>Patch looks good, but you don't have any ldr.n tests, making sure they're still accepted and converted to plain ldr.</div><div><br></div><div>cheers,</div><div>--renato</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 22 July 2013 11:51, Mihail Popa <span dir="ltr"><<a href="mailto:mihail.popa@arm.com" target="_blank">mihail.popa@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  

    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <font face="monospace">Hi. <br>
      <br>
      Kindly review the attached patch. It adds range checking for "ldr
      Rn, [pc, #imm]" Thumb <br>
      instructions. With this patch:<br>
      <br>
      1. ldr.n is recognized as mnemonic for the short encoding<br>
      2. ldr.w is recognized as menmonic for the long encoding<br>
      3. ldr will map to either short or long encodings depending on the
      size of the offset<br>
      <br>
      Regards,<br>
      Mihai<br>
    </font>
  </div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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>