[binutils-gdb] gdb/dwarf: set is_debug_types in signatured_type constructor

Simon Marchi simark@sourceware.org
Wed Feb 19 16:17:23 GMT 2025


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=8af6d60eb9714201c663117169ef4b6ab5b0582b

commit 8af6d60eb9714201c663117169ef4b6ab5b0582b
Author: Simon Marchi <simon.marchi@polymtl.ca>
Date:   Mon Feb 17 14:59:35 2025 -0500

    gdb/dwarf: set is_debug_types in signatured_type constructor
    
    This makes it more obvious that all created signatured_type objects have
    this flag set.
    
    Also, remove an unnecessary assignment in create_cus_hash_table: when
    constructing the dwarf2_per_cu_data object, is_debug_types is already
    initialized to 0/false.
    
    Change-Id: I6d28b17ac77edc040172254f6970d05ebc4a47f4
    Approved-By: Tom Tromey <tom@tromey.com>

Diff:
---
 gdb/dwarf2/read.c | 2 --
 gdb/dwarf2/read.h | 4 +++-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c
index de932a82fe0..e5bb4e26050 100644
--- a/gdb/dwarf2/read.c
+++ b/gdb/dwarf2/read.c
@@ -1835,7 +1835,6 @@ dwarf2_per_bfd::allocate_signatured_type (dwarf2_section_info *section,
   auto result
     = std::make_unique<signatured_type> (this, section, sect_off, signature);
   result->index = all_units.size ();
-  result->is_debug_types = true;
   tu_stats.nr_tus++;
   return result;
 }
@@ -7149,7 +7148,6 @@ create_cus_hash_table (dwarf2_per_objfile *per_objfile,
       sect_offset sect_off = (sect_offset) (info_ptr - section.buffer);
 
       dwarf2_per_cu_data per_cu (per_bfd, &section, sect_off);
-      per_cu.is_debug_types = 0;
 
       cutu_reader reader (&per_cu, per_objfile, cu, &dwo_file);
       if (!reader.dummy_p)
diff --git a/gdb/dwarf2/read.h b/gdb/dwarf2/read.h
index 2aa6e58ddaf..56705c5a193 100644
--- a/gdb/dwarf2/read.h
+++ b/gdb/dwarf2/read.h
@@ -375,7 +375,9 @@ struct signatured_type : public dwarf2_per_cu_data
 		   sect_offset sect_off, ULONGEST signature)
     : dwarf2_per_cu_data (per_bfd, section, sect_off),
       signature (signature)
-  {}
+  {
+    this->is_debug_types = true;
+  }
 
   /* The type's signature.  */
   ULONGEST signature;


More information about the Gdb-cvs mailing list