[PATCH] opcodes/aarch64: make aarch64_opnd_qualifiers[] static const

Richard Earnshaw (lists) Richard.Earnshaw@arm.com
Mon Jul 28 16:12:25 GMT 2025


On 28/07/2025 16:59, Jan Beulich wrote:
> There's no reason to allow the array to be modifiable, nor for it to be
> globally visible.

OK.


> ---
> Looking at other arrays in the same file: How can an array named merely
> fields[] be non-static?

agreed.  Fixing that is pre-approved.

> 
> --- a/opcodes/aarch64-opc.c
> +++ b/opcodes/aarch64-opc.c
> @@ -809,7 +809,7 @@ struct operand_qualifier_data
>  };
>  
>  /* Indexed by the operand qualifier enumerators.  */
> -struct operand_qualifier_data aarch64_opnd_qualifiers[] =
> +static const struct operand_qualifier_data aarch64_opnd_qualifiers[] =
>  {
>    {0, 0, 0, "NIL", OQK_NIL},
>  

R.


More information about the Binutils mailing list