[llvm-commits] FIX for PR13537: incorrect ADR encoding

Stepan Dyatkovskiy stpworld at narod.ru
Fri Aug 17 03:28:23 PDT 2012


Anton Korobeynikov wrote:
> Stepan,
>
>> T1 encoding implemented incorrectly for ADR. We should store high 8 bits of
>> immediate, that is 10 bit width value. Currently imm interpreted as 8 bit
>> value and low 8 bits are encoded instead.
> Am I right to understand that the encoding of "adr	r2, #4" was
> incorrect previously?
>
Yes. Low two bits of imm operand shouldn't be encoded in T1, since it 
should be multiple of 4.
For reference I'm using ARM Architecture Reference Manual ARMv7-A and 
ARMv7-R edition, Chapter A8.6.10, T1 encoding.




More information about the llvm-commits mailing list