Avoid using DW_* macros in dwarf2/attribute.c
authorTom Tromey <tom@tromey.com>
Wed, 30 Sep 2020 00:49:08 +0000 (18:49 -0600)
committerTom Tromey <tom@tromey.com>
Wed, 30 Sep 2020 02:29:06 +0000 (20:29 -0600)
There's no need to use the DW_* accessor macros in dwarf2/attribute.c,
and this is a necessary step toward our goal of removing them
entirely.

gdb/ChangeLog
2020-09-29  Tom Tromey  <tom@tromey.com>

* dwarf2/attribute.c (attribute::address): Don't use DW_UNSND or
DW_ADDR.
(attribute::string): Don't use DW_STRING.
(attribute::get_ref_die_offset): Don't use DW_UNSND.
(attribute::constant_value): Don't use DW_UNSND or DW_SND.

gdb/ChangeLog
gdb/dwarf2/attribute.c

index b561f8a8302aa8fb225af2510292ac758793479b..b78cffe5978aac07b054bad9f5f2e1113bd10253 100644 (file)
@@ -1,3 +1,11 @@
+2020-09-29  Tom Tromey  <tom@tromey.com>
+
+       * dwarf2/attribute.c (attribute::address): Don't use DW_UNSND or
+       DW_ADDR.
+       (attribute::string): Don't use DW_STRING.
+       (attribute::get_ref_die_offset): Don't use DW_UNSND.
+       (attribute::constant_value): Don't use DW_UNSND or DW_SND.
+
 2020-09-29  Tom Tromey  <tom@tromey.com>
 
        * dwarf2/read.c (dwarf2_find_base_address, read_call_site_scope)
index 1345e0a3f867e1dd6b1d2c05b13d3112a1cecef6..af24d660918c8e6766fa50c0d77ae8b66a7d0953 100644 (file)
@@ -51,10 +51,10 @@ attribute::as_address () const
         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 (this);
+      addr = u.unsnd;
     }
   else
-    addr = DW_ADDR (this);
+    addr = u.addr;
 
   return addr;
 }
@@ -73,7 +73,7 @@ attribute::as_string () const
       || form == DW_FORM_strx4
       || form == DW_FORM_GNU_str_index
       || form == DW_FORM_GNU_strp_alt)
-    return DW_STRING (this);
+    return u.str;
   return nullptr;
 }
 
@@ -135,13 +135,13 @@ LONGEST
 attribute::constant_value (int default_value) const
 {
   if (form == DW_FORM_sdata || form == DW_FORM_implicit_const)
-    return DW_SND (this);
+    return u.snd;
   else if (form == DW_FORM_udata
           || form == DW_FORM_data1
           || form == DW_FORM_data2
           || form == DW_FORM_data4
           || form == DW_FORM_data8)
-    return DW_UNSND (this);
+    return u.unsnd;
   else
     {
       /* For DW_FORM_data16 see attribute::form_is_constant.  */