Re: [PATCH-ppc 1/5] Add basic support for new VSX register set

["Ulrich Weigand" <uweigand at de dot ibm dot com>]

Luis Machado wrote:

> > Please use builtin_type (gdbarch)->builtin_... instead of builtin_type_...
> > I'm just trying to get rid of those ...
> 
> Like the following updated patch?

Yes, thanks.

> 2008-08-15  Luis Machado  <luisgpm@br.ibm.com>
> 
> 	* rs6000-tdep: Include "features/rs6000/powerpc-vsx32.c".
> 	Include "features/rs6000/powerpc-vsx64.c".
> 	(ppc_supply_vsxregset): New function.
> 	(ppc_collect_vsxregset): New function.
> 	(IS_VSX_PSEUDOREG): New macro.
> 	(IS_EFP_PSEUDOREG): New macro.
> 	(vsx_register_p): New function.
> 	(ppc_vsx_support_p): New function.
> 	(rs6000_builtin_type_vec128): New function.
> 	(rs6000_register_name): Hide upper halves of vs0~vs31.  Return
> 	correct names for VSX registers and EFPR registers.
> 	(rs6000_pseudo_register_type): Return correct types for VSX
> 	and EFPR registers.
> 	(rs6000_pseudo_register_reggroup_p): Return correct group for
> 	VSX and EFPR registers.
> 	(ppc_pseudo_register_read): Rename to dfp_pseudo_register_read.
> 	(ppc_pseudo_register_write): Rename to dfp_pseudo_register_write.
> 	(vsx_pseudo_register_read): New function.
> 	(vsx_pseudo_register_write): New function.
> 	(efpr_pseudo_register_read): New function.
> 	(efpr_pseudo_register_write): New function.
> 	(rs6000_pseudo_register_read): Call new VSX and EFPR read functions.
> 	(rs6000_pseudo_register_write): Call new VSX and EFPR write functions.
> 	(rs6000_gdbarch_init): Declare have_vsx.
> 	Initialize new upper half VSX registers.
> 	Initialize VSX-related and EFPR-related pseudo-registers variables.
> 	Adjust the number of pseudo registers accordingly.
> 	
> 	* ppc-linux-nat.c: Define PTRACE_GETVSXREGS, PTRACE_SETVSXREGS
> 	and SIZEOF_VSRREGS.
> 	(gdb_vsxregset_t): New type.
> 	(have_ptrace_getsetvsxregs): New variable.
> 	(fetch_vsx_register): New function.	
> 	(fetch_register): Handle VSX registers.
> 	(fetch_vsx_registers): New function.
> 	(fetch_ppc_registers): Handle VSX registers.
> 	(store_ppc_registers): Handle VSX registers.
> 	(store_vsx_register): New function.	
> 	(store_register): Handle VSX registers.
> 	(store_vsx_registers): New function.
> 	(ppc_linux_read_description): Handle VSX-enabled inferiors.
> 	(gdb_vsxregset_t): New type.
> 	(supply_vsxregset): New function.
> 	(fill_vsxregset): New function.
> 
> 	* ppc-tdep.h (vsx_register_p): New prototype.
> 	(vsx_support_p): New prototype.
> 	(ppc_vsr0_regnum): New variable.
> 	(ppc_vsr0_upper_regnum): Likewise.
> 	(ppc_efpr0_regnum): Likewise.
> 	(ppc_builtin_type_vec128): New type.
> 	(ppc_num_vsrs): New constant.
> 	(ppc_num_vshrs): New constant.
> 	(ppc_num_efprs): Likewise.
> 	Define POWERPC_VEC_VSX PPC_VSR0_UPPER_REGNUM and PPC_VSR31_UPPER_REGNUM.
> 	(ppc_supply_vsxregset): New prototype.
> 	(ppc_collect_vsxregset): New prototype.
> 
> 	* ppc-linux-tdep.c: Include "features/rs6000/powerpc-vsx32l.c"
> 	Include "features/rs6000/powerpc-vsx64l.c".
> 	(_initialize_ppc_linux_tdep): Initialize VSX-enabled targets.
> 
> 	* ppc-linux-tdep.h: Declare *tdesc_powerpc_vsx32l
> 	Declare tdesc_powerpc_vsx64l

This is OK, except:

> @@ -3357,9 +3673,38 @@
>    /* Set the register number for _Decimal128 pseudo-registers.  */
>    tdep->ppc_dl0_regnum = have_dfp? gdbarch_num_regs (gdbarch) : -1;
>  
> -  if (have_dfp && have_spe)
> -    /* Put the _Decimal128 pseudo-registers after the SPE registers.  */
> -    tdep->ppc_dl0_regnum += 32;
> +  /* Set the register number for VSX pseudo-registers.  */
> +  tdep->ppc_vsr0_regnum = have_vsx? gdbarch_num_regs (gdbarch) : -1;
> +  tdep->ppc_efpr0_regnum = have_vsx? gdbarch_num_regs (gdbarch) : -1;

All the above is now obsolete if you use the new code below:

> +  /* Choose register numbers for all supported pseudo-registers.  */
> +  tdep->ppc_ev0_regnum = -1;
> +  tdep->ppc_dl0_regnum = -1;
> +  tdep->ppc_vsr0_regnum = -1;
> +  tdep->ppc_efpr0_regnum = -1;

...

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com