Remove addrmap_create_mutable
authorTom Tromey <tom@tromey.com>
Sat, 16 Apr 2022 15:48:12 +0000 (09:48 -0600)
committerTom Tromey <tom@tromey.com>
Sun, 12 Jun 2022 16:49:48 +0000 (10:49 -0600)
This removes addrmap_create_mutable in favor of using 'new' at the
spots where the addrmap is created.

gdb/addrmap.c
gdb/addrmap.h
gdb/buildsym.c
gdb/buildsym.h
gdb/dwarf2/read.c

index b9a787135b8014b09e9e00d7bd3ef549d0d0ac51..29e6b2e8a31a90efb2a8ba65c1f765345f32715c 100644 (file)
@@ -406,12 +406,6 @@ addrmap_mutable::addrmap_mutable (struct obstack *obs)
 }
 
 
-struct addrmap *
-addrmap_create_mutable (struct obstack *obstack)
-{
-  return new (obstack) struct addrmap_mutable (obstack);
-}
-
 /* See addrmap.h.  */
 
 void
@@ -483,7 +477,8 @@ test_addrmap ()
   /* Create mutable addrmap.  */
   struct obstack temp_obstack;
   obstack_init (&temp_obstack);
-  struct addrmap *map = addrmap_create_mutable (&temp_obstack);
+  struct addrmap_mutable *map
+    = new (&temp_obstack) addrmap_mutable (&temp_obstack);
   SELF_CHECK (map != nullptr);
 
   /* Check initial state.  */
index 01569d3fa0a684fe27987fd9e3421b34abf65988..8794f1fefbf1675dab8501f8ce93a50d0d1e528a 100644 (file)
@@ -202,10 +202,6 @@ private:
 };
 
 
-/* Create a mutable address map which maps every address to NULL.
-   Allocate entries in OBSTACK.  */
-struct addrmap *addrmap_create_mutable (struct obstack *obstack);
-
 /* Dump the addrmap to OUTFILE.  If PAYLOAD is non-NULL, only dump any
    components that map to PAYLOAD.  (If PAYLOAD is NULL, the entire
    map is dumped.)  */
index f057f5c4b3f49eddfa8ed6eae064ba6e790ce321..d4a90abcee436aba420d81f7a4026d0f3f694530 100644 (file)
@@ -419,7 +419,9 @@ buildsym_compunit::record_block_range (struct block *block,
     m_pending_addrmap_interesting = true;
 
   if (m_pending_addrmap == nullptr)
-    m_pending_addrmap = addrmap_create_mutable (&m_pending_addrmap_obstack);
+    m_pending_addrmap
+      = (new (&m_pending_addrmap_obstack) addrmap_mutable
+        (&m_pending_addrmap_obstack));
 
   m_pending_addrmap->set_empty (start, end_inclusive, block);
 }
index ee75e6fd95d355865bbbcb6e910d9057aa124c51..c1cd5192a79e66e0c4dec7f45baa2c975a547265 100644 (file)
@@ -385,7 +385,7 @@ private:
   /* The mutable address map for the compilation unit whose symbols
      we're currently reading.  The symtabs' shared blockvector will
      point to a fixed copy of this.  */
-  struct addrmap *m_pending_addrmap = nullptr;
+  struct addrmap_mutable *m_pending_addrmap = nullptr;
 
   /* The obstack on which we allocate pending_addrmap.
      If pending_addrmap is NULL, this is uninitialized; otherwise, it is
index 2af1a49f45df5d3b620e02e0edae37add553e897..216b211bf9df769933eac6c912e70d7922079cf2 100644 (file)
@@ -2267,12 +2267,12 @@ create_addrmap_from_index (dwarf2_per_objfile *per_objfile,
   dwarf2_per_bfd *per_bfd = per_objfile->per_bfd;
   struct gdbarch *gdbarch = objfile->arch ();
   const gdb_byte *iter, *end;
-  struct addrmap *mutable_map;
+  struct addrmap_mutable *mutable_map;
   CORE_ADDR baseaddr;
 
   auto_obstack temp_obstack;
 
-  mutable_map = addrmap_create_mutable (&temp_obstack);
+  mutable_map = new (&temp_obstack) addrmap_mutable (&temp_obstack);
 
   iter = index->address_table.data ();
   end = iter + index->address_table.size ();
@@ -2496,7 +2496,8 @@ create_addrmap_from_aranges (dwarf2_per_objfile *per_objfile,
   dwarf2_per_bfd *per_bfd = per_objfile->per_bfd;
 
   auto_obstack temp_obstack;
-  addrmap *mutable_map = addrmap_create_mutable (&temp_obstack);
+  addrmap_mutable *mutable_map
+    = new (&temp_obstack) addrmap_mutable (&temp_obstack);
 
   if (read_addrmap_from_aranges (per_objfile, section, mutable_map))
     per_bfd->index_addrmap = mutable_map->create_fixed (&per_bfd->obstack);
@@ -6552,7 +6553,7 @@ public:
                                        xcalloc, xfree)),
       m_index (new cooked_index),
       m_addrmap_storage (),
-      m_addrmap (addrmap_create_mutable (&m_addrmap_storage))
+      m_addrmap (new (&m_addrmap_storage) addrmap_mutable (&m_addrmap_storage))
   {
   }
 
@@ -6607,7 +6608,7 @@ public:
   }
 
   /* Return the mutable addrmap that is currently being created.  */
-  addrmap *get_addrmap ()
+  addrmap_mutable *get_addrmap ()
   {
     return m_addrmap;
   }
@@ -6639,7 +6640,7 @@ private:
   /* Storage for the writeable addrmap.  */
   auto_obstack m_addrmap_storage;
   /* A writeable addrmap being constructed by this scanner.  */
-  addrmap *m_addrmap;
+  addrmap_mutable *m_addrmap;
 };
 
 /* An instance of this is created to index a CU.  */
@@ -6655,7 +6656,7 @@ public:
       m_per_cu (per_cu),
       m_language (language),
       m_obstack (),
-      m_die_range_map (addrmap_create_mutable (&m_obstack))
+      m_die_range_map (new (&m_obstack) addrmap_mutable (&m_obstack))
   {
   }