Re: [PATCH] [Arm] Update the list of recognized m-profile TAG_CPU_ARCH_*

[Alan Hayward <Alan.Hayward@arm.com>]

> On 23 Mar 2022, at 09:21, Luis Machado via Gdb-patches <gdb-patches@sourceware.org> wrote:
> 
> Check 3 additional variants previously not recognized:
> 
> - TAG_CPU_ARCH_V7E_M
> - TAG_CPU_ARCH_V8M_BASE
> - TAG_CPU_ARCH_V8M_MAIN
> ---
> gdb/arm-tdep.c | 3 +++
> 1 file changed, 3 insertions(+)
> 
> diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
> index 7903069c076..9a10b4d1497 100644
> --- a/gdb/arm-tdep.c
> +++ b/gdb/arm-tdep.c
> @@ -9342,6 +9342,9 @@ arm_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
> 	      if (!tdesc_has_registers (tdesc)
> 		  && (attr_arch == TAG_CPU_ARCH_V6_M
> 		      || attr_arch == TAG_CPU_ARCH_V6S_M
> +		      || attr_arch == TAG_CPU_ARCH_V7E_M
> +		      || attr_arch == TAG_CPU_ARCH_V8M_BASE
> +		      || attr_arch == TAG_CPU_ARCH_V8M_MAIN
> 		      || attr_arch == TAG_CPU_ARCH_V8_1M_MAIN
> 		      || attr_profile == 'M'))

This looks ok to me.
I double checked to make sure there aren’t any more mprofile defines not being checked.


Alan.

> 		is_m = true;
> -- 
> 2.25.1
>