Re: fix "too much information" bug w/ "info vector" on PowerPC

Daniel Jacobowitz
2005-08-31T20:32:56Z

On Wed, Aug 31, 2005 at 01:23:15PM -0700, Paul Gilliam wrote:
> Here is the new patch with comments added/chanded in ppc-tdep.h
>
> If I hear no objections, I'll commit this tomorrow.

Kevin's OK'd it, so go ahead. But:

> 2005-08-30 Paul Gilliam <pgilliam@us.ibm.com>
>
> * ppc-tdep.h: better explanation of using -1 for nonexistant registers
> * rs6000-tdep.c (rs6000_register_reggroup_p): don't assume that
> tdep->ppc_vr0_regnum or tdep->ppc_ev0_regnum are not -1

* ppc-tdep.h (struct gdbarch_tdep): Better explanation of using
-1 for nonexistant registers.
* rs6000-tdep.c (rs6000_register_reggroup_p): Don't assume that
tdep->ppc_vr0_regnum and tdep->ppc_ev0_regnum are not -1.
> ! /* Not all PPC and RS6000 variants will have the registers
> ! represented below. A -1 is used to indicate that the register
> ! is not present in this variant. */

Two spaces after periods please, here and below. Also end comments
with periods where you didn't. Yes, I realize the file is inconsistent
about this.

--
Daniel Jacobowitz
CodeSourcery, LLC