On 10/14/22 22:23, Philippe Mathieu-Daudé wrote:
From: David Daney<david.da...@fungible.com>

disas/mips.c got added in commit 6643d27ea0 ("MIPS disas support")
apparently based on binutils tag 'gdb_6_1-branchpoint' [1].
Back then, MIPSr6 was not supported (added in binutils commit
7361da2c952 during 2014 [2]).

Binutils codebase diverged so much over the last 18 years, it is
not possible to simply cherry-pick their changes, so fix it BEQZC /
BNEZC 21-bit signed branch displacement locally.

[1]https://sourceware.org/git/?p=binutils-gdb.git;a=blob;f=opcodes/mips-dis.c;hb=refs/tags/gdb_6_1-branchpoint
[2]https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=7361da2c952

Fixes: 31837be3ee ("target-mips: add compact and CP1 branches")
Signed-off-by: David Daney<david.da...@fungible.com>
Reviewed-by: Marcin Nowakowski<marcin.nowakow...@fungible.com>
[PMD: Added commit description]
Signed-off-by: Philippe Mathieu-Daudé<phi...@fungible.com>
---
v2: Use sextract32 (rth)
---
  disas/mips.c | 12 ++++++++++--
  1 file changed, 10 insertions(+), 2 deletions(-)

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

r~

Reply via email to