From patchwork Thu Jan 23 00:56:38 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 37483 Received: (qmail 79953 invoked by alias); 23 Jan 2020 00:57:29 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 79662 invoked by uid 89); 23 Jan 2020 00:57:27 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-20.2 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_STOCKGEN, RCVD_IN_DNSWL_NONE, SPF_HELO_PASS autolearn=ham version=3.3.1 spammy=dw_at_alignment, DW_AT_alignment, sk:DW_AT_s, sk:dw_at_s X-HELO: gateway32.websitewelcome.com Received: from gateway32.websitewelcome.com (HELO gateway32.websitewelcome.com) (192.185.145.119) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 23 Jan 2020 00:57:21 +0000 Received: from cm17.websitewelcome.com (cm17.websitewelcome.com [100.42.49.20]) by gateway32.websitewelcome.com (Postfix) with ESMTP id 95D35363F55D for ; Wed, 22 Jan 2020 18:57:19 -0600 (CST) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with SMTP id uQoBiNCUx9jb7uQoBiTpPq; Wed, 22 Jan 2020 18:57:19 -0600 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=tromey.com; s=default; h=References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From: Sender:Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=/KIAbfyGdii8HhkH4pgTiJZwhhQS5gaX2MuvDJm22uQ=; b=ing2aSJxY6dPXNeOG3wd+xV47F a8VXACA5qY2ci5t5UloquUg8lnyEaM8/3Nm3VeTV+GlBDdU4q6Og7KX2wYN2jktIt7Tq7uiXUzcjI ZBwG/CTdLwVIV7gc7KY1oxxn1; Received: from 75-166-123-50.hlrn.qwest.net ([75.166.123.50]:40800 helo=bapiya.Home) by box5379.bluehost.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.92) (envelope-from ) id 1iuQoB-004Kfy-D7; Wed, 22 Jan 2020 17:57:19 -0700 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH 06/38] Change some attribute functions to be methods Date: Wed, 22 Jan 2020 17:56:38 -0700 Message-Id: <20200123005710.7978-7-tom@tromey.com> In-Reply-To: <20200123005710.7978-1-tom@tromey.com> References: <20200123005710.7978-1-tom@tromey.com> This changes most of the attribute-related functions to be methods. (attr_form_is_block changed in a subsequent patch.) gdb/ChangeLog 2020-01-22 Tom Tromey * dwarf2read.c (dwarf2_find_base_address, ) (read_call_site_scope, rust_containing_type) (dwarf2_get_pc_bounds, dwarf2_record_block_ranges) (handle_data_member_location, dwarf2_add_member_fn) (get_alignment, read_structure_type, process_structure_scope) (mark_common_block_symbol_computed, read_common_block) (read_tag_string_type, attr_to_dynamic_prop, read_subrange_type) (partial_die_info::read, read_attribute_value, new_symbol) (lookup_die_type, dwarf2_get_ref_die_offset) (dwarf2_get_attr_constant_value, follow_die_ref_or_sig) (dwarf2_fetch_die_loc_sect_off, get_DW_AT_signature_type) (dwarf2_symbol_mark_computed): Update. * dwarf2/attribute.h (struct attribute) : Declare methods. (value_as_address, attr_form_is_section_offset) (attr_form_is_constant, attr_form_is_ref): Don't declare. * dwarf2/attribute.c (attribute::value_as_address) (attribute::form_is_section_offset, attribute::form_is_constant) (attribute::form_is_ref): Now methods. Change-Id: I320dad13002c59b848dc86c39d5d7111c8a15bdc --- gdb/ChangeLog | 23 +++++++++++ gdb/dwarf2/attribute.c | 40 +++++++++---------- gdb/dwarf2/attribute.h | 78 ++++++++++++++++++------------------- gdb/dwarf2read.c | 88 +++++++++++++++++++++--------------------- 4 files changed, 126 insertions(+), 103 deletions(-) diff --git a/gdb/dwarf2/attribute.c b/gdb/dwarf2/attribute.c index 75a2fa3774b..6e51fff5362 100644 --- a/gdb/dwarf2/attribute.c +++ b/gdb/dwarf2/attribute.c @@ -30,12 +30,12 @@ /* See attribute.h. */ CORE_ADDR -attr_value_as_address (struct attribute *attr) +attribute::value_as_address () const { CORE_ADDR addr; - if (attr->form != DW_FORM_addr && attr->form != DW_FORM_addrx - && attr->form != DW_FORM_GNU_addr_index) + if (form != DW_FORM_addr && form != DW_FORM_addrx + && form != DW_FORM_GNU_addr_index) { /* Aside from a few clearly defined exceptions, attributes that contain an address must always be in DW_FORM_addr form. @@ -49,10 +49,10 @@ attr_value_as_address (struct attribute *attr) as well as update callers to pass in at least the CU's DWARF version. This is more overhead than what we're willing to expand for a pretty rare case. */ - addr = DW_UNSND (attr); + addr = DW_UNSND (this); } else - addr = DW_ADDR (attr); + addr = DW_ADDR (this); return addr; } @@ -72,20 +72,20 @@ attr_form_is_block (const struct attribute *attr) /* See attribute.h. */ -int -attr_form_is_section_offset (const struct attribute *attr) +bool +attribute::form_is_section_offset () const { - return (attr->form == DW_FORM_data4 - || attr->form == DW_FORM_data8 - || attr->form == DW_FORM_sec_offset); + return (form == DW_FORM_data4 + || form == DW_FORM_data8 + || form == DW_FORM_sec_offset); } /* See attribute.h. */ -int -attr_form_is_constant (const struct attribute *attr) +bool +attribute::form_is_constant () const { - switch (attr->form) + switch (form) { case DW_FORM_sdata: case DW_FORM_udata: @@ -94,19 +94,19 @@ attr_form_is_constant (const struct attribute *attr) case DW_FORM_data4: case DW_FORM_data8: case DW_FORM_implicit_const: - return 1; + return true; default: - return 0; + return false; } } /* DW_ADDR is always stored already as sect_offset; despite for the forms besides DW_FORM_ref_addr it is stored as cu_offset in the DWARF file. */ -int -attr_form_is_ref (const struct attribute *attr) +bool +attribute::form_is_ref () const { - switch (attr->form) + switch (form) { case DW_FORM_ref_addr: case DW_FORM_ref1: @@ -115,8 +115,8 @@ attr_form_is_ref (const struct attribute *attr) case DW_FORM_ref8: case DW_FORM_ref_udata: case DW_FORM_GNU_ref_alt: - return 1; + return true; default: - return 0; + return false; } } diff --git a/gdb/dwarf2/attribute.h b/gdb/dwarf2/attribute.h index 11c6cb929d9..2e663f1560c 100644 --- a/gdb/dwarf2/attribute.h +++ b/gdb/dwarf2/attribute.h @@ -41,6 +41,45 @@ struct dwarf_block /* Attributes have a name and a value. */ struct attribute { + /* Read the given attribute value as an address, taking the + attribute's form into account. */ + CORE_ADDR value_as_address () const; + + /* Return non-zero if ATTR's value is a section offset --- classes + lineptr, loclistptr, macptr or rangelistptr --- or zero, otherwise. + You may use DW_UNSND (attr) to retrieve such offsets. + + Section 7.5.4, "Attribute Encodings", explains that no attribute + may have a value that belongs to more than one of these classes; it + would be ambiguous if we did, because we use the same forms for all + of them. */ + + bool form_is_section_offset () const; + + /* Return non-zero if ATTR's value falls in the 'constant' class, or + zero otherwise. When this function returns true, you can apply + dwarf2_get_attr_constant_value to it. + + However, note that for some attributes you must check + attr_form_is_section_offset before using this test. DW_FORM_data4 + and DW_FORM_data8 are members of both the constant class, and of + the classes that contain offsets into other debug sections + (lineptr, loclistptr, macptr or rangelistptr). The DWARF spec says + that, if an attribute's can be either a constant or one of the + section offset classes, DW_FORM_data4 and DW_FORM_data8 should be + taken as section offsets, not constants. + + DW_FORM_data16 is not considered as dwarf2_get_attr_constant_value + cannot handle that. */ + + bool form_is_constant () const; + + /* DW_ADDR is always stored already as sect_offset; despite for the forms + besides DW_FORM_ref_addr it is stored as cu_offset in the DWARF file. */ + + bool form_is_ref () const; + + ENUM_BITFIELD(dwarf_attribute) name : 16; ENUM_BITFIELD(dwarf_form) form : 15; @@ -71,48 +110,9 @@ struct attribute #define DW_ADDR(attr) ((attr)->u.addr) #define DW_SIGNATURE(attr) ((attr)->u.signature) -/* Read the given attribute value as an address, taking the attribute's - form into account. */ - -extern CORE_ADDR attr_value_as_address (struct attribute *attr); - /* Check if the attribute's form is a DW_FORM_block* if so return true else false. */ extern int attr_form_is_block (const struct attribute *attr); -/* Return non-zero if ATTR's value is a section offset --- classes - lineptr, loclistptr, macptr or rangelistptr --- or zero, otherwise. - You may use DW_UNSND (attr) to retrieve such offsets. - - Section 7.5.4, "Attribute Encodings", explains that no attribute - may have a value that belongs to more than one of these classes; it - would be ambiguous if we did, because we use the same forms for all - of them. */ - -extern int attr_form_is_section_offset (const struct attribute *attr); - -/* Return non-zero if ATTR's value falls in the 'constant' class, or - zero otherwise. When this function returns true, you can apply - dwarf2_get_attr_constant_value to it. - - However, note that for some attributes you must check - attr_form_is_section_offset before using this test. DW_FORM_data4 - and DW_FORM_data8 are members of both the constant class, and of - the classes that contain offsets into other debug sections - (lineptr, loclistptr, macptr or rangelistptr). The DWARF spec says - that, if an attribute's can be either a constant or one of the - section offset classes, DW_FORM_data4 and DW_FORM_data8 should be - taken as section offsets, not constants. - - DW_FORM_data16 is not considered as dwarf2_get_attr_constant_value - cannot handle that. */ - -extern int attr_form_is_constant (const struct attribute *attr); - -/* DW_ADDR is always stored already as sect_offset; despite for the forms - besides DW_FORM_ref_addr it is stored as cu_offset in the DWARF file. */ - -extern int attr_form_is_ref (const struct attribute *attr); - #endif /* GDB_DWARF2_ATTRIBUTE_H */ diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index 5b77e397899..87b8aaaf679 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -6117,7 +6117,7 @@ dwarf2_find_base_address (struct die_info *die, struct dwarf2_cu *cu) attr = dwarf2_attr (die, DW_AT_entry_pc, cu); if (attr != nullptr) { - cu->base_address = attr_value_as_address (attr); + cu->base_address = attr->value_as_address (); cu->base_known = 1; } else @@ -6125,7 +6125,7 @@ dwarf2_find_base_address (struct die_info *die, struct dwarf2_cu *cu) attr = dwarf2_attr (die, DW_AT_low_pc, cu); if (attr != nullptr) { - cu->base_address = attr_value_as_address (attr); + cu->base_address = attr->value_as_address (); cu->base_known = 1; } } @@ -13632,7 +13632,7 @@ read_call_site_scope (struct die_info *die, struct dwarf2_cu *cu) sect_offset_str (die->sect_off), objfile_name (objfile)); return; } - pc = attr_value_as_address (attr) + baseaddr; + pc = attr->value_as_address () + baseaddr; pc = gdbarch_adjust_dwarf2_addr (gdbarch, pc); if (cu->call_site_htab == NULL) @@ -13751,7 +13751,7 @@ read_call_site_scope (struct die_info *die, struct dwarf2_cu *cu) SET_FIELD_DWARF_BLOCK (call_site->target, dlbaton); } - else if (attr_form_is_ref (attr)) + else if (attr->form_is_ref ()) { struct dwarf2_cu *target_cu = cu; struct die_info *target_die; @@ -13826,7 +13826,7 @@ read_call_site_scope (struct die_info *die, struct dwarf2_cu *cu) for DW_AT_call_parameter. */ origin = dwarf2_attr (child_die, DW_AT_abstract_origin, cu); } - if (loc == NULL && origin != NULL && attr_form_is_ref (origin)) + if (loc == NULL && origin != NULL && origin->form_is_ref ()) { parameter->kind = CALL_SITE_PARAMETER_PARAM_OFFSET; @@ -13928,7 +13928,7 @@ rust_containing_type (struct die_info *die, struct dwarf2_cu *cu) struct die_info *type_die = NULL; struct dwarf2_cu *type_cu = cu; - if (attr_form_is_ref (attr)) + if (attr->form_is_ref ()) type_die = follow_die_ref (die, attr, &type_cu); if (type_die == NULL) return NULL; @@ -14339,9 +14339,9 @@ dwarf2_get_pc_bounds (struct die_info *die, CORE_ADDR *lowpc, attr = dwarf2_attr (die, DW_AT_low_pc, cu); if (attr != nullptr) { - low = attr_value_as_address (attr); - high = attr_value_as_address (attr_high); - if (cu->header.version >= 4 && attr_form_is_constant (attr_high)) + low = attr->value_as_address (); + high = attr_high->value_as_address (); + if (cu->header.version >= 4 && attr_high->form_is_constant ()) high += low; } else @@ -14512,10 +14512,10 @@ dwarf2_record_block_ranges (struct die_info *die, struct block *block, attr = dwarf2_attr (die, DW_AT_low_pc, cu); if (attr != nullptr) { - CORE_ADDR low = attr_value_as_address (attr); - CORE_ADDR high = attr_value_as_address (attr_high); + CORE_ADDR low = attr->value_as_address (); + CORE_ADDR high = attr_high->value_as_address (); - if (cu->header.version >= 4 && attr_form_is_constant (attr_high)) + if (cu->header.version >= 4 && attr_high->form_is_constant ()) high += low; low = gdbarch_adjust_dwarf2_addr (gdbarch, low + baseaddr); @@ -14668,9 +14668,9 @@ handle_data_member_location (struct die_info *die, struct dwarf2_cu *cu, This is because DW_AT_data_member_location is new in DWARF 4, so if we see it, we can assume that a constant form is really a constant and not a section offset. */ - if (attr_form_is_constant (attr)) + if (attr->form_is_constant ()) *offset = dwarf2_get_attr_constant_value (attr, 0); - else if (attr_form_is_section_offset (attr)) + else if (attr->form_is_section_offset ()) dwarf2_complex_location_expr_complaint (); else if (attr_form_is_block (attr)) *offset = decode_locdesc (DW_BLOCK (attr), cu); @@ -15291,7 +15291,7 @@ dwarf2_add_member_fn (struct field_info *fip, struct die_info *die, } } } - else if (attr_form_is_section_offset (attr)) + else if (attr->form_is_section_offset ()) { dwarf2_complex_location_expr_complaint (); } @@ -15418,7 +15418,7 @@ get_alignment (struct dwarf2_cu *cu, struct die_info *die) if (attr == nullptr) return 0; - if (!attr_form_is_constant (attr)) + if (!attr->form_is_constant ()) { complaint (_("DW_AT_alignment must have constant form" " - DIE at %s [in module %s]"), @@ -15620,7 +15620,7 @@ read_structure_type (struct die_info *die, struct dwarf2_cu *cu) attr = dwarf2_attr (die, DW_AT_byte_size, cu); if (attr != nullptr) { - if (attr_form_is_constant (attr)) + if (attr->form_is_constant ()) TYPE_LENGTH (type) = DW_UNSND (attr); else { @@ -15785,7 +15785,7 @@ process_structure_scope (struct die_info *die, struct dwarf2_cu *cu) In this case arrange not to check the offset. */ is_variant_part = false; } - else if (attr_form_is_ref (discr)) + else if (discr->form_is_ref ()) { struct dwarf2_cu *target_cu = cu; struct die_info *target_die = follow_die_ref (die, discr, &target_cu); @@ -16463,7 +16463,7 @@ mark_common_block_symbol_computed (struct symbol *sym, gdb_assert (common_loc && member_loc); gdb_assert (attr_form_is_block (common_loc)); gdb_assert (attr_form_is_block (member_loc) - || attr_form_is_constant (member_loc)); + || member_loc->form_is_constant ()); baton = XOBNEW (&objfile->objfile_obstack, struct dwarf2_locexpr_baton); baton->per_cu = cu->per_cu; @@ -16471,7 +16471,7 @@ mark_common_block_symbol_computed (struct symbol *sym, baton->size = 5 /* DW_OP_call4 */ + 1 /* DW_OP_plus */; - if (attr_form_is_constant (member_loc)) + if (member_loc->form_is_constant ()) { offset = dwarf2_get_attr_constant_value (member_loc, 0); baton->size += 1 /* DW_OP_addr */ + cu->header.addr_size; @@ -16487,7 +16487,7 @@ mark_common_block_symbol_computed (struct symbol *sym, store_unsigned_integer (ptr, 4, byte_order, cu_off); ptr += 4; - if (attr_form_is_constant (member_loc)) + if (member_loc->form_is_constant ()) { *ptr++ = DW_OP_addr; store_unsigned_integer (ptr, cu->header.addr_size, byte_order, offset); @@ -16527,7 +16527,7 @@ read_common_block (struct die_info *die, struct dwarf2_cu *cu) { /* Ok. */ } - else if (attr_form_is_section_offset (attr)) + else if (attr->form_is_section_offset ()) { dwarf2_complex_location_expr_complaint (); attr = NULL; @@ -16588,9 +16588,9 @@ read_common_block (struct die_info *die, struct dwarf2_cu *cu) sect_offset_str (child_die->sect_off), objfile_name (objfile)); - if (attr_form_is_section_offset (member_loc)) + if (member_loc->form_is_section_offset ()) dwarf2_complex_location_expr_complaint (); - else if (attr_form_is_constant (member_loc) + else if (member_loc->form_is_constant () || attr_form_is_block (member_loc)) { if (attr != nullptr) @@ -17051,7 +17051,7 @@ read_tag_string_type (struct die_info *die, struct dwarf2_cu *cu) } attr = dwarf2_attr (die, DW_AT_string_length, cu); - if (attr != nullptr && !attr_form_is_constant (attr)) + if (attr != nullptr && !attr->form_is_constant ()) { /* The string length describes the location at which the length of the string can be found. The size of the length field can be @@ -17061,7 +17061,7 @@ read_tag_string_type (struct die_info *die, struct dwarf2_cu *cu) = dwarf2_attr (die, DW_AT_string_length_byte_size, cu); if (len == nullptr) len = dwarf2_attr (die, DW_AT_byte_size, cu); - if (len != nullptr && attr_form_is_constant (len)) + if (len != nullptr && len->form_is_constant ()) { /* Pass 0 as the default as we know this attribute is constant and the default value will not be returned. */ @@ -17595,7 +17595,7 @@ attr_to_dynamic_prop (const struct attribute *attr, struct die_info *die, prop->kind = PROP_LOCEXPR; gdb_assert (prop->data.baton != NULL); } - else if (attr_form_is_ref (attr)) + else if (attr->form_is_ref ()) { struct dwarf2_cu *target_cu = cu; struct die_info *target_die; @@ -17612,7 +17612,7 @@ attr_to_dynamic_prop (const struct attribute *attr, struct die_info *die, switch (target_attr->name) { case DW_AT_location: - if (attr_form_is_section_offset (target_attr)) + if (target_attr->form_is_section_offset ()) { baton = XOBNEW (obstack, struct dwarf2_property_baton); baton->property_type = die_type (target_die, target_cu); @@ -17659,7 +17659,7 @@ attr_to_dynamic_prop (const struct attribute *attr, struct die_info *die, } } } - else if (attr_form_is_constant (attr)) + else if (attr->form_is_constant ()) { prop->data.const_val = dwarf2_get_attr_constant_value (attr, 0); prop->kind = PROP_CONST; @@ -17843,7 +17843,7 @@ read_subrange_type (struct die_info *die, struct dwarf2_cu *cu) LONGEST bias = 0; struct attribute *bias_attr = dwarf2_attr (die, DW_AT_GNU_bias, cu); - if (bias_attr != nullptr && attr_form_is_constant (bias_attr)) + if (bias_attr != nullptr && bias_attr->form_is_constant ()) bias = dwarf2_get_attr_constant_value (bias_attr, 0); /* Normally, the DWARF producers are expected to use a signed @@ -18521,12 +18521,12 @@ partial_die_info::read (const struct die_reader_specs *reader, break; case DW_AT_low_pc: has_low_pc_attr = 1; - lowpc = attr_value_as_address (&attr); + lowpc = attr.value_as_address (); break; case DW_AT_high_pc: has_high_pc_attr = 1; - highpc = attr_value_as_address (&attr); - if (cu->header.version >= 4 && attr_form_is_constant (&attr)) + highpc = attr.value_as_address (); + if (cu->header.version >= 4 && attr.form_is_constant ()) high_pc_relative = 1; break; case DW_AT_location: @@ -18535,7 +18535,7 @@ partial_die_info::read (const struct die_reader_specs *reader, { d.locdesc = DW_BLOCK (&attr); } - else if (attr_form_is_section_offset (&attr)) + else if (attr.form_is_section_offset ()) { dwarf2_complex_location_expr_complaint (); } @@ -19207,7 +19207,7 @@ read_attribute_value (const struct die_reader_specs *reader, } /* Super hack. */ - if (cu->per_cu->is_dwz && attr_form_is_ref (attr)) + if (cu->per_cu->is_dwz && attr->form_is_ref ()) attr->form = DW_FORM_GNU_ref_alt; /* We have seen instances where the compiler tried to emit a byte @@ -21321,7 +21321,7 @@ new_symbol (struct die_info *die, struct type *type, struct dwarf2_cu *cu, { CORE_ADDR addr; - addr = attr_value_as_address (attr); + addr = attr->value_as_address (); addr = gdbarch_adjust_dwarf2_addr (gdbarch, addr + baseaddr); SET_SYMBOL_VALUE_ADDRESS (sym, addr); } @@ -21939,7 +21939,7 @@ lookup_die_type (struct die_info *die, const struct attribute *attr, dwarf2_per_objfile); this_type = get_die_type_at_offset (sect_off, per_cu); } - else if (attr_form_is_ref (attr)) + else if (attr->form_is_ref ()) { sect_offset sect_off = dwarf2_get_ref_die_offset (attr); @@ -21967,7 +21967,7 @@ lookup_die_type (struct die_info *die, const struct attribute *attr, struct die_info *type_die = NULL; struct dwarf2_cu *type_cu = cu; - if (attr_form_is_ref (attr)) + if (attr->form_is_ref ()) type_die = follow_die_ref (die, attr, &type_cu); if (type_die == NULL) return build_error_marker_type (cu, die); @@ -22871,7 +22871,7 @@ store_in_ref_table (struct die_info *die, struct dwarf2_cu *cu) static sect_offset dwarf2_get_ref_die_offset (const struct attribute *attr) { - if (attr_form_is_ref (attr)) + if (attr->form_is_ref ()) return (sect_offset) DW_UNSND (attr); complaint (_("unsupported die ref attribute form: '%s'"), @@ -22895,7 +22895,7 @@ dwarf2_get_attr_constant_value (const struct attribute *attr, int default_value) return DW_UNSND (attr); else { - /* For DW_FORM_data16 see attr_form_is_constant. */ + /* For DW_FORM_data16 see attribute::form_is_constant. */ complaint (_("Attribute value is not a constant (%s)"), dwarf_form_name (attr->form)); return default_value; @@ -22912,7 +22912,7 @@ follow_die_ref_or_sig (struct die_info *src_die, const struct attribute *attr, { struct die_info *die; - if (attr_form_is_ref (attr)) + if (attr->form_is_ref ()) die = follow_die_ref (src_die, attr, ref_cu); else if (attr->form == DW_FORM_ref_sig8) die = follow_die_sig (src_die, attr, ref_cu); @@ -23086,7 +23086,7 @@ dwarf2_fetch_die_loc_sect_off (sect_offset sect_off, retval.data = NULL; retval.size = 0; } - else if (attr_form_is_section_offset (attr)) + else if (attr->form_is_section_offset ()) { struct dwarf2_loclist_baton loclist_baton; CORE_ADDR pc = (*get_frame_pc) (baton); @@ -23474,7 +23474,7 @@ get_DW_AT_signature_type (struct die_info *die, const struct attribute *attr, struct dwarf2_cu *cu) /* ARI: editCase function */ { /* Yes, DW_AT_signature can use a non-ref_sig8 reference. */ - if (attr_form_is_ref (attr)) + if (attr->form_is_ref ()) { struct dwarf2_cu *type_cu = cu; struct die_info *type_die = follow_die_ref (die, attr, &type_cu); @@ -24828,7 +24828,7 @@ dwarf2_symbol_mark_computed (const struct attribute *attr, struct symbol *sym, struct objfile *objfile = dwarf2_per_objfile->objfile; struct dwarf2_section_info *section = cu_debug_loc_section (cu); - if (attr_form_is_section_offset (attr) + if (attr->form_is_section_offset () /* .debug_loc{,.dwo} may not exist at all, or the offset may be outside the section. If so, fall through to the complaint in the other branch. */