This is the mail archive of the gdb-patches@sourceware.cygnus.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: RFC: Patch to arm-tdep.c -- Register flavors


Scott Bambrough wrote:
> 
> Richard Earnshaw wrote:
> >
> > On the other hand, coming from a RISC iX/NetBSD background I've never used
> > the A{T}PCS names, and I object to having them imposed on me by default
> > (mainly because it means that the names keep changing).  I don't mind
> > having an option to change them but I do think that a particular flavour
> > of the ARM debugger should be able to select which default we see.
> 
> I'm in agreement here.  I don't object to the ATPCS names being used, but I
> would like to set them as default in the Linux port.
> 
Gentlemen, please, this is clearly a question of preference.  Of course
we should try to set the default to the most liked option, but this is
sometimes difficult to determine (lots of people use arm gdb but they
are not on the gdb lists).

Remember that you can have a .gdbinit file on your home directory just
to make gdb behave as you, personally, prefer it to do.  You can have
another one at your project working directory for things specific to
your program (setting breakpoints etc.).

Anyway, if everybody in the list can agree with what the default should
be I will change it and wait for the rest of the world to complain (and
send them the above paragraph :-).  I left the default decoupled from
the disassembly on purpose  --  gdb can choose its own default and tell
the disassembly to use the same.


> > Further, why have several of the register names recently been made upper
> > case?  They never used to be.
> 
> This seems to be a convention adopted in the ATPCS docs.  See section 4.1.
> 
> > And finally, the ps register name is wrong, especially in the A{T}PCS area
> > but also more generally; the name is "cpsr" in all ARM documentation (the
> > "ps" name comes from when the processor flags weren't really a separate
> > register -- they were in the unused parts of r15 -- so there was no name).
> 
> Richard is correct here, in arm-linux-nat.c, I initialize the PS register with
> the contents of CPSR if arm_apcs_32 is set, or with the contents of the PC if
> not.  Perhaps we should set the register name based on this information as well.
> 
> Scott
> 
> --
> Scott Bambrough - Software Engineer
> REBEL.COM    http://www.rebel.com
> NetWinder    http://www.netwinder.org

-- 
Fernando Nasser
Red Hat - Toronto                       E-Mail:  fnasser@cygnus.com
2323 Yonge Street, Suite #300           Tel:  416-482-2661 ext. 311
Toronto, Ontario   M4P 2C9              Fax:  416-482-6299

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]