diff options
author | Peter Maydell <peter.maydell@linaro.org> | 2024-07-16 11:30:32 +0100 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2024-07-18 13:49:28 +0100 |
commit | 5669d26ec614b3f4c56cf1489b9095ed327938b1 (patch) | |
tree | e34c06cec3f1e99b1c13308442c2d4f8754ad09d /target/arm/tcg/mve.decode | |
parent | d74ec4d7dda6322bcc51d1b13ccbd993d3574795 (diff) |
target/arm: Fix handling of LDAPR/STLR with negative offset
When we converted the LDAPR/STLR instructions to decodetree we
accidentally introduced a regression where the offset is negative.
The 9-bit immediate field is signed, and the old hand decoder
correctly used sextract32() to get it out of the insn word,
but the ldapr_stlr_i pattern in the decode file used "imm:9"
instead of "imm:s9", so it treated the field as unsigned.
Fix the pattern to treat the field as a signed immediate.
Cc: qemu-stable@nongnu.org
Fixes: 2521b6073b7 ("target/arm: Convert LDAPR/STLR (imm) to decodetree")
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2419
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20240709134504.3500007-2-peter.maydell@linaro.org
Diffstat (limited to 'target/arm/tcg/mve.decode')
0 files changed, 0 insertions, 0 deletions