]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
Make prepare_one_comp_unit a method of cutu_reader
authorTom Tromey <tom@tromey.com>
Fri, 31 Jan 2025 19:38:15 +0000 (12:38 -0700)
committerTom Tromey <tom@tromey.com>
Thu, 20 Feb 2025 00:58:04 +0000 (17:58 -0700)
This changes prepare_one_comp_unit to be a private method of
cutu_reader.  This should make it somewhat simpler to reason about.

gdb/dwarf2/read.c

index 3d04b8e6ca8e0e2f9c593c8d87cded9153cc3f1f..f31ddb5ce1e6ce25e938d47b056f4efda8843c5c 100644 (file)
@@ -631,6 +631,9 @@ private:
                                  dwarf2_cu *existing_cu,
                                  enum language pretend_language);
 
+  void prepare_one_comp_unit (struct dwarf2_cu *cu,
+                             enum language pretend_language);
+
   struct dwarf2_per_cu_data *m_this_cu;
   dwarf2_cu_up m_new_cu;
 
@@ -1061,10 +1064,6 @@ static struct dwarf2_per_cu_data *dwarf2_find_containing_comp_unit
   (sect_offset sect_off, unsigned int offset_in_dwz,
    dwarf2_per_bfd *per_bfd);
 
-static void prepare_one_comp_unit (struct dwarf2_cu *cu,
-                                  struct die_info *comp_unit_die,
-                                  enum language pretend_language);
-
 static struct type *set_die_type (struct die_info *, struct type *,
                                  struct dwarf2_cu *, bool = false);
 
@@ -3223,7 +3222,7 @@ cutu_reader::init_tu_and_read_dwo_dies (dwarf2_per_cu_data *this_cu,
       dummy_p = true;
     }
 
-  prepare_one_comp_unit (cu, comp_unit_die, pretend_language);
+  prepare_one_comp_unit (cu, pretend_language);
 }
 
 /* Initialize a CU (or TU) and read its DIEs.
@@ -3440,7 +3439,7 @@ cutu_reader::cutu_reader (dwarf2_per_cu_data *this_cu,
 
   /* Only a dummy unit can be missing the compunit DIE.  */
   gdb_assert (dummy_p || comp_unit_die != nullptr);
-  prepare_one_comp_unit (cu, comp_unit_die, pretend_language);
+  prepare_one_comp_unit (cu, pretend_language);
 }
 
 void
@@ -3533,7 +3532,7 @@ cutu_reader::cutu_reader (dwarf2_per_cu_data *this_cu,
       info_ptr = read_toplevel_die (this, &comp_unit_die, info_ptr);
     }
 
-  prepare_one_comp_unit (m_new_cu.get (), comp_unit_die, pretend_language);
+  prepare_one_comp_unit (m_new_cu.get (), pretend_language);
 }
 
 \f
@@ -21224,9 +21223,9 @@ run_test ()
    CU one with no contents; in this case default values are used for
    the fields.  */
 
-static void
-prepare_one_comp_unit (struct dwarf2_cu *cu, struct die_info *comp_unit_die,
-                      enum language pretend_language)
+void
+cutu_reader::prepare_one_comp_unit (struct dwarf2_cu *cu,
+                                   enum language pretend_language)
 {
   struct attribute *attr;