[PATCH] Add range checking for Thumb2 PC-relative loads

Renato Golin renato.golin at linaro.org
Mon Jul 22 05:45:39 PDT 2013


Hi Mihail,

Patch looks good, but you don't have any ldr.n tests, making sure they're
still accepted and converted to plain ldr.

cheers,
--renato


On 22 July 2013 11:51, Mihail Popa <mihail.popa at arm.com> wrote:

>  Hi.
>
> Kindly review the attached patch. It adds range checking for "ldr Rn, [pc,
> #imm]" Thumb
> instructions. With this patch:
>
> 1. ldr.n is recognized as mnemonic for the short encoding
> 2. ldr.w is recognized as menmonic for the long encoding
> 3. ldr will map to either short or long encodings depending on the size of
> the offset
>
> Regards,
> Mihai
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130722/ed57b5d3/attachment.html>


More information about the llvm-commits mailing list