This is the mail archive of the gdb-patches@sources.redhat.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]
Other format: [Raw text]

Re: [patch] Add PS_REGNUM.


> I think it is the other way round.  PS_REGNUM is the only one being used 
>> correctly - when >=0, std-regs.c (new file) maps $ps onto a 
>> hardware/pseudo register.  Cf the GDB manual.
>> 
>> On the other hand FP_REGNUM, PC_REGNUM and SP_REGNUM that are being used 
>> ``incorrectly''(1).  They have no meaning outside of std-regs.c yet are 
>> used throughout GDB.
> 
> 
> So what you're saying is that you added PS_REGNUM so that it could be
> used as a standard $ps register name, not for the rest of GDB, right?

Yes.  And that is how FP_REGNUM et.al. should be used ....

> I don't really see the point; anyone who wants to look at the processor
> status register presumably knows what some of the bits in it mean,
> which is entirely architecture dependant.  But caveat implementor :)

Who am I to argue with the documentation :-)

Andrew


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