From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30249 invoked by alias); 10 Jul 2002 22:48:31 -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 30241 invoked from network); 10 Jul 2002 22:48:30 -0000 Received: from unknown (HELO localhost.redhat.com) (216.138.202.10) by sources.redhat.com with SMTP; 10 Jul 2002 22:48:30 -0000 Received: from ges.redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id CABDB3D22; Wed, 10 Jul 2002 18:48:26 -0400 (EDT) Message-ID: <3D2CB9BA.1040304@ges.redhat.com> Date: Wed, 10 Jul 2002 16:04:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.0) Gecko/20020708 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Kevin Buettner Cc: Grace Sainsbury , gdb-patches@sources.redhat.com Subject: Re: [PATCH] more m68k multi-arch References: <20020624131028.A1510@tomago.toronto.redhat.com> <1020628180358.ZM26305@localhost.localdomain> <1020628182659.ZM26406@localhost.localdomain> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2002-07/txt/msg00213.txt.bz2 > On Jun 28, 11:03am, Kevin Buettner wrote: Kevin, See: http://sources.redhat.com/gdb/current/onlinedocs/gdbint_9.html#SEC82 > 9.11.4 Prepare for multi-arch level to one > > Temporally set GDB_MULTI_ARCH to GDB_MULTI_ARCH_PARTIAL and then build and start GDB (the change should not be committed). GDB may not build, and once built, it may die with an internal error listing the architecture methods that must be provided. > > Fix any build problems (patch(es)). > > Convert all the architecture methods listed, which are only macros, into functions (patch(es)). > > Update arch_gdbarch_init to set all the missing architecture methods and wrap the corresponding macros in #if !GDB_MULTI_ARCH (patch(es)). The presence of #define GDB_MULTI_ARCH 0 indicates that this is being tested. enjoy, Andrew > I've been looking at this some more... > > I've noticed the following define near the top of tm-m68k.h: > > #define GDB_MULTI_ARCH 0 > > Clearly, with this definition in place, you do need to keep the old > definitions around, because those are the ones that will be used! > > It seems to me that your testing will be more productive if you do > one of the following: > > 1) define GDB_MULTI_ARCH to be GDB_MULTI_ARCH_PARTIAL (1) instead of 0. > > or > > 2) change the macro definitions being replaced to instead refer to > the newly defined functions in m68k-tdep.c. > > As it stands now, you are only testing that the functions that you've > added to the tdep.c file build (i.e. compile), but GDB won't actually > be using them. This means that if you make a typo, it won't be > noticed until some later point when GDB_MULTI_ARCH is changed from 0 > to some other value.