From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21551 invoked by alias); 8 Feb 2010 18:56:57 -0000 Received: (qmail 21542 invoked by uid 22791); 8 Feb 2010 18:56:56 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mail-ww0-f41.google.com (HELO mail-ww0-f41.google.com) (74.125.82.41) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 08 Feb 2010 18:56:49 +0000 Received: by wwb22 with SMTP id 22so1267715wwb.0 for ; Mon, 08 Feb 2010 10:56:47 -0800 (PST) MIME-Version: 1.0 Received: by 10.216.156.203 with SMTP id m53mr4192985wek.209.1265655407043; Mon, 08 Feb 2010 10:56:47 -0800 (PST) In-Reply-To: <20100208141423.GA26959@caradoc.them.org> References: <20100205011447.GA28263@lucon.org> <6dc9ffc81002071400u5bad1726v93330aeb38246143@mail.gmail.com> <20100207221444.GA5399@caradoc.them.org> <6dc9ffc81002071425y74b5bb8eha47bacc6f1d2a33e@mail.gmail.com> <6dc9ffc81002071503r79077130meb9f66a6cf51ed09@mail.gmail.com> <20100207230748.GA8766@caradoc.them.org> <6dc9ffc81002071816n44363fffpe08a964e513a10c8@mail.gmail.com> <20100208041656.GA26375@caradoc.them.org> <6dc9ffc81002072116sd0797ecnf24550199f1d12a2@mail.gmail.com> <20100208141423.GA26959@caradoc.them.org> Date: Mon, 08 Feb 2010 18:56:00 -0000 Message-ID: <6dc9ffc81002081056m444936f1hd2cc12183addaf3c@mail.gmail.com> Subject: Re: PATCH: PATCH: Add builtin floating point types From: "H.J. Lu" To: "H.J. Lu" , Mark Kettenis , gdb-patches@sourceware.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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: 2010-02/txt/msg00215.txt.bz2 On Mon, Feb 8, 2010 at 6:14 AM, Daniel Jacobowitz wr= ote: > On Sun, Feb 07, 2010 at 09:16:25PM -0800, H.J. Lu wrote: >> in gdbtypes.c. struct arch_type will have >> >> struct type *arch_ieee_single; >> struct type *arch_ieee_double; >> struct type *arch_i387_ext; >> >> They will be lazy initialized. I thought that was what you suggested. > > This hasn't changed anything; it still has to know about i387_ext. > > There are at least two options. =A0We could use a per-gdbarch hash table > to cache the result of arch_float_type (in arch_float_type, not in its > caller). =A0Or we could put the cache in tdesc_gdb_type, which is > probably easier. A patch is posted at http://sourceware.org/ml/gdb-patches/2010-02/msg00214.html Thanks. --=20 H.J.