]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
arm: Fix bugs with MVE vmov from two GPRs to vector lanes
authorAlex Coplan <alex.coplan@arm.com>
Wed, 19 May 2021 09:38:23 +0000 (10:38 +0100)
committerAlex Coplan <alex.coplan@arm.com>
Wed, 19 May 2021 09:38:23 +0000 (10:38 +0100)
commitb8f4b69aaaade0e3872597f1597506824f25434d
treea116071dd61646a0810be94a60d0bb76d631cbef
parent99c9bb313ead94889a54824e5864a2e5b7dd269c
arm: Fix bugs with MVE vmov from two GPRs to vector lanes

The initial problem I wanted to fix here is that GAS was rejecting MVE
instructions such as:

vmov q3[2], q3[0], r2, r2

with:

Error: General purpose registers may not be the same -- `vmov q3[2],q3[0],r2,r2'

which is incorrect; such instructions are valid. Note that for moves in
the other direction, e.g.:

vmov r2, r2, q3[2], q3[0]

GAS is correct in rejecting this as it does not make sense to move both
lanes into the same register (the Arm ARM says this is CONSTRAINED
UNPREDICTABLE).

After fixing this issue, I added assembly/disassembly tests for these
vmovs. This revealed several disassembly issues, including incorrectly
marking the moves into vector lanes as UNPREDICTABLE, and disassembling
many of the vmovs as vector loads. These are now fixed.

Regtested on arm-eabi, no regressions.

OK for trunk? What about backports?

Thanks,
Alex

gas/ChangeLog:

* config/tc-arm.c (do_mve_mov): Only reject vmov if we're moving
into the same GPR twice.
* testsuite/gas/arm/mve-vmov-bad-2.l: Tweak error message.
* testsuite/gas/arm/mve-vmov-3.d: New test.
* testsuite/gas/arm/mve-vmov-3.s: New test.

opcodes/ChangeLog:

* arm-dis.c (mve_opcodes): Fix disassembly of
MVE_VMOV2_GP_TO_VEC_LANE when idx == 1.
(is_mve_encoding_conflict): MVE vector loads should not match
when P = W = 0.
(is_mve_unpredictable): It's not unpredictable to use the same
source register twice (for MVE_VMOV2_GP_TO_VEC_LANE).
gas/ChangeLog
gas/config/tc-arm.c
gas/testsuite/gas/arm/mve-vmov-3.d [new file with mode: 0644]
gas/testsuite/gas/arm/mve-vmov-3.s [new file with mode: 0644]
gas/testsuite/gas/arm/mve-vmov-bad-2.l
opcodes/ChangeLog
opcodes/arm-dis.c