From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22215 invoked by alias); 2 Apr 2004 03:47:33 -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 22202 invoked from network); 2 Apr 2004 03:47:32 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 2 Apr 2004 03:47:32 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.12.10/8.12.10) with ESMTP id i323lWjj025393 for ; Thu, 1 Apr 2004 22:47:32 -0500 Received: from zenia.home.redhat.com (porkchop.devel.redhat.com [172.16.58.2]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id i323lUj19325; Thu, 1 Apr 2004 22:47:30 -0500 To: Daniel Jacobowitz Cc: gdb-patches@sources.redhat.com, ezannoni@redhat.com Subject: Re: [rfa/dwarf] Use objfile_data mechanism for per-objfile data References: <20040401171557.GA17948@nevyn.them.org> <20040401210608.GA9848@nevyn.them.org> <20040401220136.GA30848@nevyn.them.org> <20040402030835.GA25571@nevyn.them.org> From: Jim Blandy Date: Fri, 02 Apr 2004 03:47:00 -0000 In-Reply-To: <20040402030835.GA25571@nevyn.them.org> Message-ID: User-Agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2004-04/txt/msg00049.txt.bz2 Daniel Jacobowitz writes: > On Thu, Apr 01, 2004 at 05:40:01PM -0500, Jim Blandy wrote: > > If they sit there for very long, I will have neglected my duties. But > > you've been working steadily on this for quite a while, so short of > > some disaster for you, I'm confident it'll get done. So if it would > > produce interference with your other patches, leave it for later. > > May as well get it right the first time, since I will need to merge > this to the branch anyway for the renamed variables. This better? > I got the most blatantly overlong lines, but not all of them; I'll add > re-indenting this file thoroughly to my list of things to do when I'm > done churning it. I also intend to move a lot of functions around for > more logical grouping. Glorious. If tested, please commit. > 2004-04-01 Daniel Jacobowitz > > * dwarf2read.c (dwarf2_objfile_data_key): New. > (struct dwarf2_per_objfile, dwarf2_per_objfile): New. > (dwarf_info_size, dwarf_abbrev_size, dwarf_line_size) > (dwarf_pubnames_size, dwarf_aranges_size, dwarf_loc_size) > (dwarf_macinfo_size, dwarf_str_size, dwarf_ranges_size) > (dwarf_frame_size, dwarf_eh_frame_size, dwarf_info_buffer) > (dwarf_abbrev_buffer, dwarf_line_buffer, dwarf_str_buffer) > (dwarf_macinfo_buffer, dwarf_ranges_buffer, dwarf_loc_buffer): > Remove variables. > (struct dwarf2_pinfo): Remove per-objfile members. Update comments. > (DWARF_ABBREV_SIZE, DWARF_LINE_SIZE) > (DWARF_LOC_SIZE, DWARF_MACINFO_SIZE, DWARF_STR_SIZE) > (DWARF_RANGES_SIZE, DWARF_INFO_BUFFER) > (DWARF_ABBREV_BUFFER, DWARF_LINE_BUFFER, DWARF_STR_BUFFER) > (DWARF_MACINFO_BUFFER, DWARF_RANGES_BUFFER, DWARF_LOC_BUFFER): > Remove macros. > (dwarf2_has_info): Take an objfile argument. Allocate per-objfile > data. > (dwarf2_locate_sections, dwarf2_build_psymtabs) > (dwarf2_build_psymtabs_easy, dwarf2_build_psymtabs_hard) > (skip_one_die, dwarf2_get_pc_bounds, dwarf2_read_abbrevs) > (read_partial_die, read_full_die, read_indirect_string) > (dwarf_decode_line_header, dwarf_decode_macros) > (dwarf2_symbol_mark_computed): Remove use of removed macros. > Update uses of removed variables. > (psymtab_to_symtab_1): Restore per-objfile data pointer. Remove use > of removed macros. > (_initialize_dwarf2_read): New function. > * symfile.h (dwarf2_has_info): Update prototype. > * coffread.c (coff_symfile_read): Update call to dwarf2_has_info. > * elfread.c (elf_symfile_read): Likewise.