mbox series

[0/6] disas/cris: Use GString instead of sprintf

Message ID 20240413052333.688151-1-richard.henderson@linaro.org
Headers show
Series disas/cris: Use GString instead of sprintf | expand

Message

Richard Henderson April 13, 2024, 5:23 a.m. UTC 
More sprintf cleanup encouraged by the Apple deprecation.
Probably there's a more minimal patch.  On the other hand,
there's certainly a larger cleanup possible.


r~


Richard Henderson (6):
  disas/cris: Untabify
  disas/cris: Remove TRACE_CASE and related code
  disas/cris: Drop with_reg_prefix
  disas/cris: Use GString in print_with_operands and subroutines
  disas/cris: Remove struct cris_disasm_data
  disas/cris: Improve output of register names

 disas/cris.c | 2498 +++++++++++++++++++++-----------------------------
 1 file changed, 1070 insertions(+), 1428 deletions(-)

Comments

Peter Maydell April 13, 2024, 8:45 a.m. UTC | #1 
On Sat, 13 Apr 2024 at 06:25, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> More sprintf cleanup encouraged by the Apple deprecation.
> Probably there's a more minimal patch.  On the other hand,
> there's certainly a larger cleanup possible.

The CRIS architecture is deprecated, so all this code will
be deleted in 9.2; that seems to me to limit the utility
of doing big cleanups on it.

thanks
-- PMM
Edgar E. Iglesias April 16, 2024, 8:35 a.m. UTC | #2 
On Sat, Apr 13, 2024 at 7:23 AM Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> More sprintf cleanup encouraged by the Apple deprecation.
> Probably there's a more minimal patch.  On the other hand,
> there's certainly a larger cleanup possible.
>
>
> r~


On the series:
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@amd.com>


>
>
> Richard Henderson (6):
>   disas/cris: Untabify
>   disas/cris: Remove TRACE_CASE and related code
>   disas/cris: Drop with_reg_prefix
>   disas/cris: Use GString in print_with_operands and subroutines
>   disas/cris: Remove struct cris_disasm_data
>   disas/cris: Improve output of register names
>
>  disas/cris.c | 2498 +++++++++++++++++++++-----------------------------
>  1 file changed, 1070 insertions(+), 1428 deletions(-)
>
> --
> 2.34.1
>