aboutsummaryrefslogtreecommitdiff
path: root/gdb
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2020-09-29 18:49:08 -0600
committerTom Tromey <tom@tromey.com>2020-09-29 20:29:06 -0600
commit630ed6b975d2d2f950523dd2ad9ac77614f5222c (patch)
tree1d3434e503cc0f281807ffe8a379e2163409225d /gdb
parent9d2246fce014a0133882051f8527e2dbd869a31b (diff)
downloadfsf-binutils-gdb-630ed6b975d2d2f950523dd2ad9ac77614f5222c.zip
fsf-binutils-gdb-630ed6b975d2d2f950523dd2ad9ac77614f5222c.tar.gz
fsf-binutils-gdb-630ed6b975d2d2f950523dd2ad9ac77614f5222c.tar.bz2
Remove DW_SIGNATURE
This removes DW_SIGNATURE in favor of methods on struct attribute. As usual, the methods check the form, which DW_SIGNATURE did not do. gdb/ChangeLog 2020-09-29 Tom Tromey <tom@tromey.com> * dwarf2/read.c (read_attribute_value, lookup_die_type) (dump_die_shallow, follow_die_sig, get_DW_AT_signature_type): Update. * dwarf2/attribute.h (struct attribute) <as_signature, set_signature>: New methods. (DW_SIGNATURE): Remove.
Diffstat (limited to 'gdb')
-rw-r--r--gdb/ChangeLog9
-rw-r--r--gdb/dwarf2/attribute.h16
-rw-r--r--gdb/dwarf2/read.c10
3 files changed, 29 insertions, 6 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index ed4fa18..cab1d8a 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,14 @@
2020-09-29 Tom Tromey <tom@tromey.com>
+ * dwarf2/read.c (read_attribute_value, lookup_die_type)
+ (dump_die_shallow, follow_die_sig, get_DW_AT_signature_type):
+ Update.
+ * dwarf2/attribute.h (struct attribute) <as_signature,
+ set_signature>: New methods.
+ (DW_SIGNATURE): Remove.
+
+2020-09-29 Tom Tromey <tom@tromey.com>
+
* dwarf2/read.c (read_call_site_scope)
(handle_data_member_location, dwarf2_add_member_fn)
(mark_common_block_symbol_computed, attr_to_dynamic_prop)
diff --git a/gdb/dwarf2/attribute.h b/gdb/dwarf2/attribute.h
index 0e2e86d..800ea8d 100644
--- a/gdb/dwarf2/attribute.h
+++ b/gdb/dwarf2/attribute.h
@@ -57,6 +57,14 @@ struct attribute
return u.blk;
}
+ /* Return the signature. The attribute must have signature
+ form. */
+ ULONGEST as_signature () const
+ {
+ gdb_assert (form == DW_FORM_ref_sig8);
+ return u.signature;
+ }
+
/* 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.
@@ -160,6 +168,13 @@ struct attribute
u.blk = blk;
}
+ /* Set the signature value for this attribute. */
+ void set_signature (ULONGEST signature)
+ {
+ gdb_assert (form == DW_FORM_ref_sig8);
+ u.signature = signature;
+ }
+
ENUM_BITFIELD(dwarf_attribute) name : 16;
ENUM_BITFIELD(dwarf_form) form : 15;
@@ -192,6 +207,5 @@ private:
#define DW_UNSND(attr) ((attr)->u.unsnd)
#define DW_SND(attr) ((attr)->u.snd)
#define DW_ADDR(attr) ((attr)->u.addr)
-#define DW_SIGNATURE(attr) ((attr)->u.signature)
#endif /* GDB_DWARF2_ATTRIBUTE_H */
diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c
index cd7f5c4..e8170eb 100644
--- a/gdb/dwarf2/read.c
+++ b/gdb/dwarf2/read.c
@@ -19850,7 +19850,7 @@ read_attribute_value (const struct die_reader_specs *reader,
info_ptr += 8;
break;
case DW_FORM_ref_sig8:
- DW_SIGNATURE (attr) = read_8_bytes (abfd, info_ptr);
+ attr->set_signature (read_8_bytes (abfd, info_ptr));
info_ptr += 8;
break;
case DW_FORM_ref_udata:
@@ -22093,7 +22093,7 @@ lookup_die_type (struct die_info *die, const struct attribute *attr,
}
else if (attr->form == DW_FORM_ref_sig8)
{
- ULONGEST signature = DW_SIGNATURE (attr);
+ ULONGEST signature = attr->as_signature ();
return get_signatured_type (die, signature, cu);
}
@@ -22782,7 +22782,7 @@ dump_die_shallow (struct ui_file *f, int indent, struct die_info *die)
break;
case DW_FORM_ref_sig8:
fprintf_unfiltered (f, "signature: %s",
- hex_string (DW_SIGNATURE (&die->attrs[i])));
+ hex_string (die->attrs[i].as_signature ()));
break;
case DW_FORM_string:
case DW_FORM_strp:
@@ -23360,7 +23360,7 @@ static struct die_info *
follow_die_sig (struct die_info *src_die, const struct attribute *attr,
struct dwarf2_cu **ref_cu)
{
- ULONGEST signature = DW_SIGNATURE (attr);
+ ULONGEST signature = attr->as_signature ();
struct signatured_type *sig_type;
struct die_info *die;
@@ -23468,7 +23468,7 @@ get_DW_AT_signature_type (struct die_info *die, const struct attribute *attr,
}
else if (attr->form == DW_FORM_ref_sig8)
{
- return get_signatured_type (die, DW_SIGNATURE (attr), cu);
+ return get_signatured_type (die, attr->as_signature (), cu);
}
else
{