Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: Richard Henderson <rth@redhat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] cleanup alpha core read/write registers
Date: Mon, 02 Jun 2003 20:40:00 -0000	[thread overview]
Message-ID: <20030602204008.GA16342@nevyn.them.org> (raw)
In-Reply-To: <20030602202949.GA25944@redhat.com>

On Mon, Jun 02, 2003 at 01:29:49PM -0700, Richard Henderson wrote:
> On Mon, Jun 02, 2003 at 11:19:37AM -0700, Richard Henderson wrote:
> > RFA because I'm touching systems I can't test.
> > 
> > As far as alpha-nat is concerned, this gets rid of the use
> > of the deprecated register array.  Indeed, there are no
> > longer *any* instances of "deprecated" in the alpha port.
> > 
> > As far as Linux is concerned, this fixes the gcore test
> > because we now properly dump the FPCR and UNIQUE registers.
> > 
> > As far as the BSD's are concerned, this should be a no-op,
> > just avoiding some amount of code duplication.
> > 
> > Ok?
> 
> Gah.  This is the patch I meant to send.

I don't have anywhere to test it either, but it looks good.  This
touches some code which would be triggered on Tru64, right?  If so and
you're particularly nervous about it, ask Joel Brobecker to test it.
Otherwise I think it's OK to check in.

> 
> 
> r~
> 
> 
> 
> 	* alpha-tdep.c (alpha_supply_int_regs, alpha_fill_int_regs): New.
> 	(alpha_supply_fp_regs, alpha_fill_fp_regs): New.
> 	* alpha-tdep.h: Declare them.
> 
> 	* alpha-nat.c (fetch_osf_core_registers): Constify core_reg_mapping.
> 	Remove zerobuf.  Don't error on UNIQUE.
> 	(fetch_elf_core_registers): Use alpha_supply_{int,fp}_regs.
> 	(ALPHA_REGSET_UNIQUE): Provide default.
> 	(supply_gregset): Use alpha_supply_int_regs.
> 	(fill_gregset): Use alpha_fill_int_regs.
> 	(supply_fpregset): Use alpha_supply_fp_regs.
> 	(fill_fpregset): Use alpha_fill_fp_regs.
> 	* alphabsd-tdep.c (NUM_GREGS, NUM_FPREGS): Remove.
> 	(alphabsd_supply_reg): Use alpha_supply_int_regs.
> 	(alphabsd_fill_reg): Use alpha_fill_int_regs.
> 	(alphabsd_supply_fpreg): Use alpha_supply_fp_regs.
> 	(alphabsd_fill_fpreg): Use alpha_fill_fp_regs.
> 	* config/alpha/nm-linux.h (ALPHA_REGSET_UNIQUE): New.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer


      reply	other threads:[~2003-06-02 20:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-02 18:27 Richard Henderson
2003-06-02 20:33 ` Richard Henderson
2003-06-02 20:40   ` Daniel Jacobowitz [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030602204008.GA16342@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=rth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox