<div dir="ltr">Hi guys.<div><br></div><div>I fixed the coding style and removed the RTTI usage. Also added two more tests.</div><div>Flipping the emit bit doesn't seem to make any difference for some reason, so </div><div>
I propose to leave as is.</div><div><br></div><div>New patch attached.</div><div><br>Regards,</div><div>Mihai</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 22, 2013 at 1:45 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"><div><div class="h5">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>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">

  

    
  
  <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></div></div><div class="im">_______________________________________________<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></div></blockquote></div><br></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>