[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