Re: [Qemu-devel] [PATCH v1 04/21] RISC-V Disassembler

[Richard Henderson]

On 01/02/2018 04:44 PM, Michael Clark wrote:
> +static const char *rv_ireg_name_sym[] = {
> +    "zero", "ra",   "sp",   "gp",   "tp",   "t0",   "t1",   "t2",
> +    "s0",   "s1",   "a0",   "a1",   "a2",   "a3",   "a4",   "a5",
> +    "a6",   "a7",   "s2",   "s3",   "s4",   "s5",   "s6",   "s7",
> +    "s8",   "s9",   "s10",  "s11",  "t3",   "t4",   "t5",   "t6",
> +    NULL
> +};

static const char * const

But maybe even better as

static const char rv_ireg_name_sym[32][4]

and without the useless NULL.

Otherwise,

Reviewed-by: Richard Henderson <address@hidden>


r~