From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22144 invoked by alias); 16 Sep 2005 13:56:14 -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 22124 invoked by uid 22791); 16 Sep 2005 13:56:08 -0000 Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Fri, 16 Sep 2005 13:56:08 +0000 Received: from drow by nevyn.them.org with local (Exim 4.52) id 1EGGh8-0000xz-JK; Fri, 16 Sep 2005 09:56:02 -0400 Date: Fri, 16 Sep 2005 13:56:00 -0000 From: Daniel Jacobowitz To: Paul Gilliam Cc: gdb-patches@sources.redhat.com Subject: Re: [PATCH]*3 Re: [RFC] "info powerpc" Message-ID: <20050916135602.GA3473@nevyn.them.org> Mail-Followup-To: Paul Gilliam , gdb-patches@sources.redhat.com References: <200509091633.02630.pgilliam@us.ibm.com> <200509141710.39794.pgilliam@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200509141710.39794.pgilliam@us.ibm.com> User-Agent: Mutt/1.5.8i X-SW-Source: 2005-09/txt/msg00114.txt.bz2 On Wed, Sep 14, 2005 at 05:10:39PM -0700, Paul Gilliam wrote: > My least favorite is the third one. While it's a clean fix to the > problem, it slams the door on any future 'info powerpc' sub-commands. > It also requires a fix so that prefix commands without any > sub-commands can be depreciated. Of course, we could just get rid of > the "info powerpc" command without depreciating it for a while > first.... If you just remove it, you don't slam the door on anything. We can add it back the moment we've got a use for it. We have a common routine for printing vector registers; I would prefer using "info vector" instead of extending "info powerpc". How on earth did these tests get added to the FSF sources without the command they're testing? Ah-hah, they didn't. This is why repository history and ChangeLogs are so useful. 2002-08-20 Elena Zannoni * rs6000-tdep.c (altivec_register_p): Delete. (rs6000_do_altivec_registers): Delete. (rs6000_altivec_registers_info): Delete. (rs6000_do_registers_info): Delete. (_initialize_rs6000_tdep): Remove command 'info powerpc altivec'. (rs6000_gdbarch_init): Remove setting of do_registers_info. Here: http://sourceware.org/ml/gdb-patches/2002-08/msg00597.html It was Elena's goal to get rid of 'info powerpc altivec'; she just didn't update the testsuite, apparently. -- Daniel Jacobowitz CodeSourcery, LLC