mbox series

[v2,0/6] Two -Wclobbered fixes, plus other cleanup

Message ID 20221106212852.152384-1-richard.henderson@linaro.org
Headers show
Series Two -Wclobbered fixes, plus other cleanup | expand

Message

Richard Henderson Nov. 6, 2022, 9:28 p.m. UTC 
Stefan reported for accel/tcg, and I reproduced on Ubuntu 22.04.

Changes for v2:
  * Incorporate suggested changes to nanomips.c (phil, balaton).


r~


Richard Henderson (6):
  disas/nanomips: Move setjmp into nanomips_dis
  disas/nanomips: Merge insn{1,2,3} into words[3]
  disas/nanomips: Split out read_u16
  disas/nanomips: Tidy read for 48-bit opcodes
  tcg: Move TCG_TARGET_HAS_direct_jump init to tb_gen_code
  accel/tcg: Split out setjmp_gen_code

 accel/tcg/translate-all.c |  68 ++++++++++++-----------
 disas/nanomips.c          | 110 ++++++++++++++++----------------------
 tcg/tcg.c                 |  12 +++++
 3 files changed, 90 insertions(+), 100 deletions(-)

Comments

Philippe Mathieu-Daudé Nov. 7, 2022, 10:30 a.m. UTC | #1 
On 6/11/22 22:28, Richard Henderson wrote:
> Stefan reported for accel/tcg, and I reproduced on Ubuntu 22.04.
> 
> Changes for v2:
>    * Incorporate suggested changes to nanomips.c (phil, balaton).

Thanks for v2, it was an optional review change request from my
side, but I appreciate the change, the result is cleaner.

> Richard Henderson (6):
>    disas/nanomips: Move setjmp into nanomips_dis
>    disas/nanomips: Merge insn{1,2,3} into words[3]
>    disas/nanomips: Split out read_u16
>    disas/nanomips: Tidy read for 48-bit opcodes

I'm queuing the nanomips patches via mips-fixes.

Regards,

Phil.