]> git.ipfire.org Git - thirdparty/qemu.git/commit
target/sh4: Fix SUBV opcode
authorPhilippe Mathieu-Daudé <philmd@linaro.org>
Tue, 30 Apr 2024 11:10:19 +0000 (13:10 +0200)
committerMichael Tokarev <mjt@tls.msk.ru>
Sat, 4 May 2024 06:37:20 +0000 (09:37 +0300)
commit07d46408cb9837c54a449d56c9af1b6a2d69ec60
treeceb56aeda7a27af620492354baec38a6777b7ce9
parentdc5390a0ca23e3811f793fe15b40ba2a47c4729b
target/sh4: Fix SUBV opcode

The documentation says:

  SUBV Rm, Rn        Rn - Rm -> Rn, underflow -> T

The overflow / underflow can be calculated as:

  T = ((Rn ^ Rm) & (Result ^ Rn)) >> 31

However we were using the incorrect:

  T = ((Rn ^ Rm) & (Result ^ Rm)) >> 31

Fix by using the Rn register instead of Rm.

Add tests provided by Paul Cercueil.

Cc: qemu-stable@nongnu.org
Fixes: ad8d25a11f ("target-sh4: implement addv and subv using TCG")
Reported-by: Paul Cercueil <paul@crapouillou.net>
Suggested-by: Paul Cercueil <paul@crapouillou.net>
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2318
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Message-Id: <20240430163125.77430-3-philmd@linaro.org>
(cherry picked from commit e88a856efd1d3c3ffa8e53da4831eff8da290808)
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
target/sh4/translate.c
tests/tcg/sh4/Makefile.target
tests/tcg/sh4/test-subv.c [new file with mode: 0644]