[PATCH v1 1/8] aarch64: make comment clearer about the location

Richard Earnshaw (lists) Richard.Earnshaw@arm.com
Tue Nov 5 17:11:36 GMT 2024


On 23/10/2024 11:48, Matthieu Longo wrote:
> The enum aarch64_opnd_qualifiers in include/opcode/aarch64.h needs to
> stay in sync with the array of struct operand_qualifier_data which
> defines various properties for the different type of operands. For
> instance, for:
> - registers: the size of the register, the number of elements.
> - immediates: lower and upper bits to determine the range of values.
> ---
>  include/opcode/aarch64.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/include/opcode/aarch64.h b/include/opcode/aarch64.h
> index bc779c973d6..2858e2aeb82 100644
> --- a/include/opcode/aarch64.h
> +++ b/include/opcode/aarch64.h
> @@ -899,7 +899,8 @@ enum aarch64_opnd
>  /* Qualifier constrains an operand.  It either specifies a variant of an
>     operand type or limits values available to an operand type.
>  
> -   N.B. Order is important; keep aarch64_opnd_qualifiers synced.  */
> +   N.B. Order is important.
> +   Keep aarch64_opnd_qualifiers (opcodes/aarch64-opc.c) synced.  */
>  
>  enum aarch64_opnd_qualifier
>  {

OK.

R.


More information about the Binutils mailing list