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: [RFA] W.I.P. AltiVec ppc registers support.


On Nov 29,  4:59pm, Elena Zannoni wrote:

>  > With regard to
>  > 
>  > > Index: config/powerpc/nm-linux.h
>  > [...]
>  > > +#define FETCH_INFERIOR_REGISTERS
>  > 
>  > I think this is a good thing.  It *might not* be strictly necessary
>  > for adding AltiVec support via PEEKUSER / POKEUSER, but it does give
>  > us more control.  Also doing this allows us to clean up the code in other
>  > ways.  E.g, the following bit from config/powerpc/nm-linux.h can be
>  > removed:
>  > 
>  > >  extern int ppc_register_u_addr (int, int);
>  > >  #define REGISTER_U_ADDR(addr, blockend, regno) \
>  > >          (addr) = ppc_register_u_addr ((blockend),(regno));
>  > 
>  > This in turn means that ppc_register_u_addr() can be made static
>  > and that the ``ustart'' parameter can be removed.  All calls to
>  > register_addr() (in your new code) in ppc-linux-nat.c should be
>  > changed to invoke ppc_register_u_addr() directly.
>  > 
> 
> Unfortunately not. I thought the same, until I remembered about core
> file debugging. That function is called by fetch_core_registers() in
> core-aout.c.

Hmm... I wonder if Linux/PPC even needs this function in core-aout.c.
Daniel J. is the expert on this stuff.  Daniel, doesn't Linux/PPC use
core-regset.c instead?

Kevin


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