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


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.
 
> 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

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