From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23516 invoked by alias); 13 Oct 2005 17:17:43 -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 23496 invoked by uid 22791); 13 Oct 2005 17:17:35 -0000 Received: from mx1.redhat.com (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Thu, 13 Oct 2005 17:17:35 +0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.12.11/8.12.11) with ESMTP id j9DHHVK1022844; Thu, 13 Oct 2005 13:17:31 -0400 Received: from devserv.devel.redhat.com (devserv.devel.redhat.com [172.16.58.1]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id j9DHHVV30957; Thu, 13 Oct 2005 13:17:31 -0400 Received: from theseus.home..redhat.com (vpn26-3.sfbay.redhat.com [172.16.26.3]) by devserv.devel.redhat.com (8.12.11/8.12.11) with ESMTP id j9DHHTgA032121; Thu, 13 Oct 2005 13:17:30 -0400 To: Ulrich Weigand Cc: gdb-patches@sourceware.org Subject: Re: RFA: general prologue analysis framework References: <200510131349.j9DDnm4F002393@53v30g15.boeblingen.de.ibm.com> From: Jim Blandy Date: Thu, 13 Oct 2005 17:17:00 -0000 In-Reply-To: <200510131349.j9DDnm4F002393@53v30g15.boeblingen.de.ibm.com> (Ulrich Weigand's message of "Thu, 13 Oct 2005 15:49:47 +0200 (CEST)") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2005-10/txt/msg00118.txt.bz2 Ulrich Weigand writes: > I've removed the pv_is_array_ref code from the s390 analyzer, because > recent GCC (optionally) no longer uses fixed-size register save areas > in order to reduce the amount of wasted stack space. > > The analyser now assumes the top-most stack slot holding an incoming > (call-saved) register value is in fact the save area for that register. > I don't need any 'area' functions at all any more ... It looks like your data->gpr_slot[i] array effectively serves the same purpose as an area. If we had generic code to scan an area and populate a trad_frame_cache, areas might save you code.