From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28226 invoked by alias); 1 Jun 2003 19:08:12 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 28214 invoked from network); 1 Jun 2003 19:08:11 -0000 Received: from unknown (HELO localhost.redhat.com) (24.157.166.107) by sources.redhat.com with SMTP; 1 Jun 2003 19:08:11 -0000 Received: from redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id CDC7D2B2F; Sun, 1 Jun 2003 15:08:06 -0400 (EDT) Message-ID: <3EDA4F16.30207@redhat.com> Date: Sun, 01 Jun 2003 19:08:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.2) Gecko/20030223 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Richard Henderson Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA] remove ALPHA_FP_REGNUM References: <20030601185441.GA4652@twiddle.net> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-06/txt/msg00031.txt.bz2 > Even worse than other ports, this was some bogus non-architectural > register that sort of corresponds to the dwarf2 CFA concept, but > hasn't actually been set to anything useful in who knows how long. > > So I've removed it entirely. Ok? Almost. > * alpha-tdep.h (ALPHA_NUM_REGS): Decrement. > (ALPHA_FP_REGNUM): Remove. > (ALPHA_UNIQUE_REGNUM): Decrement. > * alpha-tdep.c (alpha_register_name): Remove vfp entry. > (alpha_cannot_fetch_register): Remove ALPHA_FP_REGNUM. > (alpha_cannot_store_register): Likewise. > * alphabsd-nat.c (fetch_inferior_registers): Don't set FP_REGNUM. > * alpha-nat.c (supply_gregset): Likewise. > * alphanbsd-tdep.c (fetch_core_registers): Likewise. > Index: alpha-tdep.c > =================================================================== > RCS file: /cvs/src/src/gdb/alpha-tdep.c,v > retrieving revision 1.91 > diff -c -p -d -u -r1.91 alpha-tdep.c > --- alpha-tdep.c 1 Jun 2003 18:44:13 -0000 1.91 > +++ alpha-tdep.c 1 Jun 2003 18:45:24 -0000 > @@ -57,7 +57,7 @@ alpha_register_name (int regno) > "f8", "f9", "f10", "f11", "f12", "f13", "f14", "f15", > "f16", "f17", "f18", "f19", "f20", "f21", "f22", "f23", > "f24", "f25", "f26", "f27", "f28", "f29", "f30", "fpcr", > - "pc", "vfp", "unique", > + "pc", "unique" > }; "vfp" should be replaced with "" leaving a hole in the register space. Otherwize the layout of the remote protocol buffer gets changed and doing that breaks compatibility with existing remote alpha targets (yes, this shouldn't happen). Otherwize ok. Andrew