[llvm-commits] [llvm] r120849 - /llvm/trunk/lib/Target/ARM/ARMInstrThumb.td
Bill Wendling
wendling at apple.com
Fri Dec 3 14:58:19 PST 2010
On Dec 3, 2010, at 2:33 PM, Jim Grosbach wrote:
> Author: grosbach
> Date: Fri Dec 3 16:33:42 2010
> New Revision: 120849
>
> URL: http://llvm.org/viewvc/llvm-project?rev=120849&view=rev
> Log:
> Remove incorrect BL target encoding (it's similar to, but not the same as the
> ARM instruction). Add encoding of bits 13 and 11.
>
Hi Jim,
Are bits 13 and 11 always supposed to be set? It looks like they're variable in the docs:
1 1 1 1 0 | S | imm10 || 1 1 | J1 | 1 | J2 | imm11
I1 = NOT(J1 EOR S); I2 = NOT(J2 EOR S);
imm32 = SignExtend(S:I1:I2:imm10:imm11:’0’, 32);
toARM = FALSE;
if InITBlock() && !LastInITBlock() then UNPREDICTABLE;
So if we go by that, then it looks like we should check the I1 and I2 bits in imm32 and do the following:
J1 = S EOR NOT(I1)
J2 = S EOR NOT(I2)
?
-bw
More information about the llvm-commits
mailing list