This is the mail archive of the
gdb@sourceware.org
mailing list for the GDB project.
RFC: free pending blocks
- From: Tom Tromey <tromey at redhat dot com>
- To: GDB Development <gdb at sourceware dot org>
- Date: Wed, 28 Nov 2012 14:26:39 -0700
- Subject: RFC: free pending blocks
Another patch I've had kicking around for a while.
Currently gdb allocates pending blocks on the objfile obstack.
These aren't used once the blocks are all created, so this is just
waste.
This patch arranges to free them instead.
It also makes 'struct pending_block' private to buildsym.
Built and regtested on x86-64 Fedora 16.
Tom
2012-11-28 Tom Tromey <tromey@redhat.com>
* buildsym.c (struct pending_block): Move from buildsym.h.
(pending_blocks): Likewise; now static.
(free_pending_blocks): Free blocks.
(record_pending_block): Use XNEW, not obstack_alloc.
* buildsym.h (struct pending_block): Move definition to
buildsym.c.
(pending_blocks): Don't declare.
diff --git a/gdb/buildsym.c b/gdb/buildsym.c
index 8f2e732..3acdd06 100644
--- a/gdb/buildsym.c
+++ b/gdb/buildsym.c
@@ -83,6 +83,21 @@ static struct obstack pending_addrmap_obstack;
the end, then we just toss the addrmap. */
static int pending_addrmap_interesting;
+/* List of blocks already made (lexical contexts already closed).
+ This is used at the end to make the blockvector. */
+
+struct pending_block
+ {
+ struct pending_block *next;
+ struct block *block;
+ };
+
+/* Pointer to the head of a linked list of symbol blocks which have
+ already been finalized (lexical contexts already closed) and which
+ are just waiting to be built into a blockvector when finalizing the
+ associated symtab. */
+
+static struct pending_block *pending_blocks;
static int compare_line_numbers (const void *ln1p, const void *ln2p);
@@ -205,9 +220,13 @@ really_free_pendings (void *dummy)
void
free_pending_blocks (void)
{
- /* The links are made in the objfile_obstack, so we only need to
- reset PENDING_BLOCKS. */
- pending_blocks = NULL;
+ while (pending_blocks)
+ {
+ struct pending_block *next = pending_blocks->next;
+
+ xfree (pending_blocks);
+ pending_blocks = next;
+ }
}
/* Take one of the lists of symbols and make a block from it. Keep
@@ -418,8 +437,7 @@ record_pending_block (struct objfile *objfile, struct block *block,
{
struct pending_block *pblock;
- pblock = (struct pending_block *)
- obstack_alloc (&objfile->objfile_obstack, sizeof (struct pending_block));
+ pblock = XNEW (struct pending_block);
pblock->block = block;
if (opblock)
{
diff --git a/gdb/buildsym.h b/gdb/buildsym.h
index 0db96bf..b7d4a76 100644
--- a/gdb/buildsym.h
+++ b/gdb/buildsym.h
@@ -37,6 +37,7 @@ struct addrmap;
this technique. */
struct block;
+struct pending_block;
#ifndef EXTERN
#define EXTERN extern
@@ -177,21 +178,6 @@ EXTERN int context_stack_size;
EXTERN int within_function;
-/* List of blocks already made (lexical contexts already closed).
- This is used at the end to make the blockvector. */
-
-struct pending_block
- {
- struct pending_block *next;
- struct block *block;
- };
-
-/* Pointer to the head of a linked list of symbol blocks which have
- already been finalized (lexical contexts already closed) and which
- are just waiting to be built into a blockvector when finalizing the
- associated symtab. */
-
-EXTERN struct pending_block *pending_blocks;
struct subfile_stack