]> git.ipfire.org Git - thirdparty/qemu.git/commit
target/sh4: mac.w: memory accesses are 16-bit words
authorZack Buhman <zack@buhman.org>
Tue, 2 Apr 2024 09:37:49 +0000 (17:37 +0800)
committerMichael Tokarev <mjt@tls.msk.ru>
Wed, 10 Apr 2024 17:32:12 +0000 (20:32 +0300)
commitbe88ed87a6ff7ac2cb9dd60690e9c204dc5b2d17
tree24af00d40c28ef0f70cac3c190bd1f38ba3c589c
parentc02844b8e8f01239681e96ad663db86ac907bb77
target/sh4: mac.w: memory accesses are 16-bit words

Before this change, executing a code sequence such as:

           mova   tblm,r0
           mov    r0,r1
           mova   tbln,r0
           clrs
           clrmac
           mac.w  @r0+,@r1+
           mac.w  @r0+,@r1+

           .align 4
  tblm:    .word  0x1234
           .word  0x5678
  tbln:    .word  0x9abc
           .word  0xdefg

Does not result in correct behavior:

Expected behavior:
  first macw : macl = 0x1234 * 0x9abc + 0x0
               mach = 0x0

  second macw: macl = 0x5678 * 0xdefg + 0xb00a630
               mach = 0x0

Observed behavior (qemu-sh4eb, prior to this commit):

  first macw : macl = 0x5678 * 0xdefg + 0x0
               mach = 0x0

  second macw: (unaligned longword memory access, SIGBUS)

Various SH-4 ISA manuals also confirm that `mac.w` is a 16-bit word memory
access, not a 32-bit longword memory access.

Signed-off-by: Zack Buhman <zack@buhman.org>
Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20240402093756.27466-1-zack@buhman.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
(cherry picked from commit b0f2f2976b4db05351117b0440b32bf0aac2c5c6)
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
target/sh4/translate.c