This is the mail archive of the gdb-patches@sourceware.org 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: [RFC 02/23] Remove 'arch' field from regset structure


> From: Andreas Arnez <arnez@linux.vnet.ibm.com>
> 
> Removes the 'arch' field from the regset structure, since it
> represents the only "dynamic" data in a regset.  It was referenced in
> some regset supply- and collect routines, to get access to the gdbarch
> associated with the regset.  Naturally, the affected routines always
> have access to the regcache to be supplied to or collected from.  Thus
> the gdbarch associated with that regcache can be used instead.
> 
> gdb/
> 	* regset.h (struct regset): Remove gdbarch field.
> 	* regset.c (regset_alloc): Drop initialization of gdbarch field.
> 	* nios2-linux-tdep.c (nios2_core_regset): Likewise.
> 	* ppcfbsd-tdep.c (ppc32_fbsd_gregset, ppc64_fbsd_gregset):
> 	Likewise.
> 	* ppc-linux-tdep.c (ppc32_linux_gregset, ppc64_linux_gregset)
> 	(ppc32_linux_fpregset, ppc32_linux_vrregset)
> 	(ppc32_linux_vsxregset): Likewise.
> 	* i386obsd-tdep.c (i386obsd_aout_supply_regset): Get the gdbarch
> 	via the regcache instead of the regset.
> 	* i386-tdep.c (i386_supply_gregset, i386_collect_gregset)
> 	(i386_supply_fpregset, i386_collect_fpregset): Likewise.
> 	* amd64obsd-tdep.c (amd64obsd_supply_regset): Likewise.
> 	* amd64-tdep.c (amd64_supply_fpregset, amd64_collect_fpregset):
> 	Likewise.


The

> -  const struct gdbarch_tdep *tdep = gdbarch_tdep (regset->arch);
> +  const struct gdbarch_tdep *tdep = gdbarch_tdep (get_regcache_arch (regcache));

changes make the lines too long.  I suggest changing them to:

  struct gdbarch *gdbarch = get_regcache_arch (regcache);
  const struct gdbarch_tdep *tdep = gdbarch_tdep(gdbarch);

Otherwise, this change is fine with me.


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