From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31398 invoked by alias); 8 Oct 2007 17:53:38 -0000 Received: (qmail 31389 invoked by uid 22791); 8 Oct 2007 17:53:38 -0000 X-Spam-Check-By: sourceware.org Received: from hq.tensilica.com (HELO mailapp.tensilica.com) (65.205.227.29) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 08 Oct 2007 17:53:34 +0000 Received: from localhost ([127.0.0.1]) by mailapp.tensilica.com with esmtp (Exim 4.34) id 1Iewp9-0006t7-KG; Mon, 08 Oct 2007 10:55:23 -0700 Received: from mailapp.tensilica.com ([127.0.0.1]) by localhost (mailapp [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 26267-03; Mon, 8 Oct 2007 10:55:23 -0700 (PDT) Received: from maxim_fc5.hq.tensilica.com ([192.168.11.68]) by mailapp.tensilica.com with esmtp (Exim 4.34) id 1Iewp9-0006t2-3i; Mon, 08 Oct 2007 10:55:23 -0700 Message-ID: <470A6EE5.4030104@hq.tensilica.com> Date: Mon, 08 Oct 2007 17:53:00 -0000 From: Maxim Grigoriev User-Agent: Thunderbird 1.5.0.9 (X11/20070102) MIME-Version: 1.0 To: Markus Deuling CC: GDB Patches , Ulrich Weigand , Eli Zaretskii , Joel Brobecker , Jim Blandy , rearnsha@arm.com, Mark Kettenis Subject: Re: [rfc] [00/16] Get rid of current gdbarch References: <4709E786.1070502@de.ibm.com> In-Reply-To: <4709E786.1070502@de.ibm.com> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2007-10/txt/msg00172.txt.bz2 Hello, Markus, I think you missed xtenda-tdep.c. First time, you asked for an approval but never checked the patch in. What do you want me to do : to wait until you update it and then test it, or just go ahead and do it by myself ? -- Maxim Markus Deuling wrote: > Hi, > > a while ago I sent a first patch set to get rid of current_gdbarch. I > now added some patches and deferred some other and reworked the > ChangeLog entries due to feedback from you. Now that 6.7 branch was > created this is a reworked patch set for current head. > > This patch set is another step in the direction of per-frame > architecture. This set of 16 patches replaces (trivial to replace > occurences of) current_gdbarch by appropriate methods like > get_frame_arch or get_regcache_arch etc. > > Currently get_frame_arch just returns current_gdbarch but this will be > replaced by later patches. > > I tested this patch set on x86 native and remote by compiling and > running the testsuite. I also did a gdb_mbuild with all > targets to make sure it compiles. > > Some later patches will be non-trivial and hard for me to test due to > lack of those machines. It would be great if some of you could test > patches for different archs like mips, arm, etc. but this will still > take a while. > > This patch set replaces some of the current_gdbarch's in following files: > > * aix-thread.c > * arm-tdep.c > * alpha-{linux-nat,nat,tdep}.c > * findvar.c > * amd64-{nat,tdep}.c > * i386-tdep.c > * sh[64]-tdep.c > * remote.c > * irix5-nat.c > * regcache.c > * m68k{linux-nat, linux-tdep, -tdep}.c > * dwarf2{loc, -frame}.c > * cris-tdep.c > * hppa-{hpux-nat, hpux-tdep, linux-nat, tdep}.c > > Is this ok? > > >