This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 2/7] New regs_info for aarch32
- From: Pedro Alves <palves at redhat dot com>
- To: Yao Qi <qiyaoltc at gmail dot com>, gdb-patches at sourceware dot org
- Date: Mon, 03 Aug 2015 14:58:27 +0100
- Subject: Re: [PATCH 2/7] New regs_info for aarch32
- Authentication-results: sourceware.org; auth=none
- References: <1438355801-25798-1-git-send-email-yao dot qi at linaro dot org> <1438355801-25798-3-git-send-email-yao dot qi at linaro dot org>
On 07/31/2015 04:16 PM, Yao Qi wrote:
> @@ -973,9 +916,8 @@ initialize_low_arch (void)
> /* Initialize the Linux target descriptions. */
> init_registers_arm ();
> init_registers_arm_with_iwmmxt ();
Why weren't these moved as well? At first,
> - init_registers_arm_with_vfpv2 ();
> - init_registers_arm_with_vfpv3 ();
> - init_registers_arm_with_neon ();
> +
> + initialize_low_arch_aarch32 ();
I thought that this was because aarch64 doesn't
do the old iwmmxt, but then in the following patch you
have this anyway:
> void
> initialize_low_arch_aarch32 (void)
> {
> +#ifndef __aarch64__
> init_registers_arm_with_vfpv2 ();
> init_registers_arm_with_vfpv3 ();
> +#endif
> init_registers_arm_with_neon ();
>
So I don't understand when/where to initialize arm32
descriptions going forward.
Thanks,
Pedro Alves