This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] xtensa: fix access to the last pseudo register
- From: Max Filippov <jcmvbkbc at gmail dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: gdb-patches at sourceware dot org, Maxim Grigoriev <maxim2405 at gmail dot com>, Woody LaRue <larue at cadence dot com>, Marc Gauthier <marc at cadence dot com>
- Date: Wed, 27 May 2015 13:55:19 +0300
- Subject: Re: [PATCH] xtensa: fix access to the last pseudo register
- Authentication-results: sourceware.org; auth=none
- References: <1431904329-13965-1-git-send-email-jcmvbkbc at gmail dot com> <55659984 dot 90103 at redhat dot com>
On Wed, May 27, 2015 at 1:16 PM, Pedro Alves <palves@redhat.com> wrote:
> On 05/18/2015 12:12 AM, Max Filippov wrote:
>> Currently access to the last pseudo register is aliased to a1. This is
>> done by little snippets in the beginning of xtensa_pseudo_register_read
>> and xtensa_pseudo_register_write that used to do such aliasing for FP
>> register since bdb4c075a29dd086f0868b394b488b1c94666be6, but then
>> FP_ALIAS was expanded into gdbarch_num_regs (current_gdbarch) +
>> gdbarch_num_pseudo_regs (current_gdbarch) (one register past the last
>> pseudo register) in 304fe2552d6e0821e8fdb7575f8e7ba6607a076d, which
>> then was changed to the last pseudo register in
>> 94a0e877111421d300d26b858bd3a0a27078d1e8.
>
> IIUC, the original intention was for FP to alias a1, and then through
> that series of patches (part of old current_gdbarch elimination)
> we ended up aliasing the wrong register. Instead of fixing the
> aliasing, you're just removing it altogether. Correct?
I'm removing that aliasing to expose the last pseudoregister.
After that patch fp pseudo register still exists and it is a read only alias
for a1. I don't think that's right, because normally frame pointer is a7 or
a15, depending on ABI, but that's definitely a separate question.
--
Thanks.
-- Max