From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from barracuda.ebox.ca (barracuda.ebox.ca [96.127.255.19]) by sourceware.org (Postfix) with ESMTPS id 74BCE3959C8A for ; Tue, 12 May 2020 21:15:17 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 74BCE3959C8A X-ASG-Debug-ID: 1589318116-0c856e314b893790001-fS2M51 Received: from smtp.ebox.ca (smtp.ebox.ca [96.127.255.82]) by barracuda.ebox.ca with ESMTP id FPBrr3cQprtJMduR (version=TLSv1 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Tue, 12 May 2020 17:15:16 -0400 (EDT) X-Barracuda-Envelope-From: simon.marchi@efficios.com X-Barracuda-RBL-Trusted-Forwarder: 96.127.255.82 Received: from epycamd.internal.efficios.com (192-222-181-218.qc.cable.ebox.net [192.222.181.218]) by smtp.ebox.ca (Postfix) with ESMTP id 94D32441B21; Tue, 12 May 2020 17:15:16 -0400 (EDT) From: Simon Marchi X-Barracuda-Effective-Source-IP: 192-222-181-218.qc.cable.ebox.net[192.222.181.218] X-Barracuda-Apparent-Source-IP: 192.222.181.218 X-Barracuda-RBL-IP: 192.222.181.218 To: gdb-patches@sourceware.org Cc: Simon Marchi Subject: [PATCH v2 22/42] Move int type methods out of dwarf2_per_cu_data Date: Tue, 12 May 2020 17:12:30 -0400 X-ASG-Orig-Subj: [PATCH v2 22/42] Move int type methods out of dwarf2_per_cu_data Message-Id: <20200512211250.6230-23-simon.marchi@efficios.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200512210913.5593-1-simon.marchi@efficios.com> References: <20200512210913.5593-1-simon.marchi@efficios.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Barracuda-Connect: smtp.ebox.ca[96.127.255.82] X-Barracuda-Start-Time: 1589318116 X-Barracuda-Encrypted: DHE-RSA-AES256-SHA X-Barracuda-URL: https://96.127.255.19:443/cgi-mod/mark.cgi X-Barracuda-Scan-Msg-Size: 9811 X-Virus-Scanned: by bsmtpd at ebox.ca X-Barracuda-BRTS-Status: 1 X-Barracuda-Spam-Score: 0.00 X-Barracuda-Spam-Status: No, SCORE=0.00 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=8.0 tests= X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.81805 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- X-Spam-Status: No, score=-27.3 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_NONE, KAM_DMARC_STATUS, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_SOFTFAIL, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 12 May 2020 21:15:19 -0000 These methods rely on the current objfile to create types based on it. Since dwarf2_per_cu_data is to become objfile-independent, these methods need to mvoe. int_type can be in dwarf2_per_objfile, as it only requires knowing about the objfile. addr_sized_int_type and addr_type also need to know about the DWARF address type size, which is CU-specific. The dwarf2_cu objects seems like a good place for it, as it knows both about the current objfile and the current CU. gdb/ChangeLog: * dwarf2/read.h (struct dwarf2_per_cu_data) : Move to dwarf2_cu. : Move to dwarf2_per_objfile. (struct dwarf2_per_objfile) : Move here. * dwarf2/read.c (struct dwarf2_cu) : Move here. (read_func_scope): Update. (read_array_type): Update. (read_tag_string_type): Update. (attr_to_dynamic_prop): Update. (dwarf2_per_cu_data::int_type): Rename to... (dwarf2_per_objfile::int_type): ... this. (dwarf2_per_cu_data::addr_sized_int_type): Rename to... (dwarf2_cu::addr_sized_int_type): ... this. (read_subrange_type): Update. (dwarf2_per_cu_data::addr_type): Rename to... (dwarf2_cu::addr_type): ... this. (set_die_type): Update. --- gdb/dwarf2/read.c | 49 +++++++++++++++++++++++++++-------------------- gdb/dwarf2/read.h | 18 ++++------------- 2 files changed, 32 insertions(+), 35 deletions(-) diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index 7470acef879..c9bced5d967 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -421,6 +421,16 @@ struct dwarf2_cu /* Reset the builder. */ void reset_builder () { m_builder.reset (); } + /* Return a type that is a generic pointer type, the size of which + matches the address size given in the compilation unit header for + this CU. */ + struct type *addr_type () const; + + /* Find an integer type the same size as the address size given in + the compilation unit header for this CU. UNSIGNED_P controls if + the integer is unsigned or not. */ + struct type *addr_sized_int_type (bool unsigned_p) const; + /* The header of the compilation unit. */ struct comp_unit_head header {}; @@ -13069,7 +13079,7 @@ read_func_scope (struct die_info *die, struct dwarf2_cu *cu) newobj->static_link = XOBNEW (&objfile->objfile_obstack, struct dynamic_prop); attr_to_dynamic_prop (attr, die, cu, newobj->static_link, - cu->per_cu->addr_type ()); + cu->addr_type ()); } cu->list_in_scope = cu->get_builder ()->get_local_symbols (); @@ -15461,8 +15471,7 @@ read_structure_type (struct die_info *die, struct dwarf2_cu *cu) else { struct dynamic_prop prop; - if (attr_to_dynamic_prop (attr, die, cu, &prop, - cu->per_cu->addr_type ())) + if (attr_to_dynamic_prop (attr, die, cu, &prop, cu->addr_type ())) type->add_dyn_prop (DYN_PROP_BYTE_SIZE, prop); TYPE_LENGTH (type) = 0; } @@ -16147,7 +16156,7 @@ read_array_type (struct die_info *die, struct dwarf2_cu *cu) if (attr != NULL) { int stride_ok; - struct type *prop_type = cu->per_cu->addr_sized_int_type (false); + struct type *prop_type = cu->addr_sized_int_type (false); byte_stride_prop = (struct dynamic_prop *) alloca (sizeof (struct dynamic_prop)); @@ -16947,13 +16956,13 @@ read_tag_string_type (struct die_info *die, struct dwarf2_cu *cu) /* Pass 0 as the default as we know this attribute is constant and the default value will not be returned. */ LONGEST sz = len->constant_value (0); - prop_type = cu->per_cu->int_type (sz, true); + prop_type = cu->per_objfile->int_type (sz, true); } else { /* If the size is not specified then we assume it is the size of an address on this target. */ - prop_type = cu->per_cu->addr_sized_int_type (true); + prop_type = cu->addr_sized_int_type (true); } /* Convert the attribute into a dynamic property. */ @@ -17583,9 +17592,8 @@ attr_to_dynamic_prop (const struct attribute *attr, struct die_info *die, /* See read.h. */ struct type * -dwarf2_per_cu_data::int_type (int size_in_bytes, bool unsigned_p) const +dwarf2_per_objfile::int_type (int size_in_bytes, bool unsigned_p) const { - struct objfile *objfile = dwarf2_per_objfile->objfile; struct type *int_type; /* Helper macro to examine the various builtin types. */ @@ -17610,10 +17618,10 @@ dwarf2_per_cu_data::int_type (int size_in_bytes, bool unsigned_p) const /* See read.h. */ struct type * -dwarf2_per_cu_data::addr_sized_int_type (bool unsigned_p) const +dwarf2_cu::addr_sized_int_type (bool unsigned_p) const { - int addr_size = this->addr_size (); - return int_type (addr_size, unsigned_p); + int addr_size = this->per_cu->addr_size (); + return this->per_objfile->int_type (addr_size, unsigned_p); } /* Read the DW_AT_type attribute for a sub-range. If this attribute is not @@ -17638,7 +17646,7 @@ read_subrange_index_type (struct die_info *die, struct dwarf2_cu *cu) FIXME: muller/2010-05-28: Possible references to object for low bound, high bound or count are not yet handled by this code. */ if (TYPE_CODE (index_type) == TYPE_CODE_VOID) - index_type = cu->per_cu->addr_sized_int_type (false); + index_type = cu->addr_sized_int_type (false); return index_type; } @@ -17768,7 +17776,7 @@ read_subrange_type (struct die_info *die, struct dwarf2_cu *cu) attribute *attr_byte_stride = dwarf2_attr (die, DW_AT_byte_stride, cu); if (attr_byte_stride != nullptr) { - struct type *prop_type = cu->per_cu->addr_sized_int_type (false); + struct type *prop_type = cu->addr_sized_int_type (false); attr_to_dynamic_prop (attr_byte_stride, die, cu, &byte_stride_prop, prop_type); } @@ -17788,7 +17796,7 @@ read_subrange_type (struct die_info *die, struct dwarf2_cu *cu) } else { - struct type *prop_type = cu->per_cu->addr_sized_int_type (false); + struct type *prop_type = cu->addr_sized_int_type (false); attr_to_dynamic_prop (attr_bit_stride, die, cu, &bit_stride_prop, prop_type); } @@ -23356,12 +23364,12 @@ dwarf2_per_cu_data::text_offset () const /* See read.h. */ struct type * -dwarf2_per_cu_data::addr_type () const +dwarf2_cu::addr_type () const { - struct objfile *objfile = dwarf2_per_objfile->objfile; + struct objfile *objfile = this->per_objfile->objfile; struct type *void_type = objfile_type (objfile)->builtin_void; struct type *addr_type = lookup_pointer_type (void_type); - int addr_size = this->addr_size (); + int addr_size = this->per_cu->addr_size (); if (TYPE_LENGTH (addr_type) == addr_size) return addr_type; @@ -23699,7 +23707,7 @@ set_die_type (struct die_info *die, struct type *type, struct dwarf2_cu *cu) attr = dwarf2_attr (die, DW_AT_allocated, cu); if (attr != NULL && attr->form_is_block ()) { - struct type *prop_type = cu->per_cu->addr_sized_int_type (false); + struct type *prop_type = cu->addr_sized_int_type (false); if (attr_to_dynamic_prop (attr, die, cu, &prop, prop_type)) type->add_dyn_prop (DYN_PROP_ALLOCATED, prop); } @@ -23714,7 +23722,7 @@ set_die_type (struct die_info *die, struct type *type, struct dwarf2_cu *cu) attr = dwarf2_attr (die, DW_AT_associated, cu); if (attr != NULL && attr->form_is_block ()) { - struct type *prop_type = cu->per_cu->addr_sized_int_type (false); + struct type *prop_type = cu->addr_sized_int_type (false); if (attr_to_dynamic_prop (attr, die, cu, &prop, prop_type)) type->add_dyn_prop (DYN_PROP_ASSOCIATED, prop); } @@ -23727,8 +23735,7 @@ set_die_type (struct die_info *die, struct type *type, struct dwarf2_cu *cu) /* Read DW_AT_data_location and set in type. */ attr = dwarf2_attr (die, DW_AT_data_location, cu); - if (attr_to_dynamic_prop (attr, die, cu, &prop, - cu->per_cu->addr_type ())) + if (attr_to_dynamic_prop (attr, die, cu, &prop, cu->addr_type ())) type->add_dyn_prop (DYN_PROP_DATA_LOCATION, prop); if (dwarf2_per_objfile->die_type_hash == NULL) diff --git a/gdb/dwarf2/read.h b/gdb/dwarf2/read.h index e9c74a40a87..77c1f246db7 100644 --- a/gdb/dwarf2/read.h +++ b/gdb/dwarf2/read.h @@ -301,6 +301,10 @@ struct dwarf2_per_objfile /* Set the compunit_symtab associated to PER_CU. */ void set_symtab (const dwarf2_per_cu_data *per_cu, compunit_symtab *symtab); + /* Find an integer type SIZE_IN_BYTES bytes in size and return it. + UNSIGNED_P controls if the integer is unsigned or not. */ + struct type *int_type (int size_in_bytes, bool unsigned_p) const; + /* Back link. */ struct objfile *objfile; @@ -510,20 +514,6 @@ struct dwarf2_per_cu_data corresponding offset in the parent objfile. */ CORE_ADDR text_offset () const; - /* Return a type that is a generic pointer type, the size of which - matches the address size given in the compilation unit header for - this CU. */ - struct type *addr_type () const; - - /* Find an integer type SIZE_IN_BYTES bytes in size and return it. - UNSIGNED_P controls if the integer is unsigned or not. */ - struct type *int_type (int size_in_bytes, bool unsigned_p) const; - - /* Find an integer type the same size as the address size given in - the compilation unit header for this CU. UNSIGNED_P controls if - the integer is unsigned or not. */ - struct type *addr_sized_int_type (bool unsigned_p) const; - /* Return DWARF version number of this CU. */ short version () const { -- 2.26.2