From: Tom Tromey <tromey@redhat.com>
To: gdb-patches@sourceware.org
Subject: RFA: remove unused fields
Date: Sun, 17 Aug 2008 17:48:00 -0000 [thread overview]
Message-ID: <m3bpzrzg8q.fsf@fleche.redhat.com> (raw)
This patch removes a few unused fields from objfile. I happened
across these while reading code.
This only tidies things up a bit; due to the large size of struct
objfile it doesn't save any meaningful amount of memory.
Built and regtested on x86-64 (compile farm). Also, I searched for
these symbols in the source tree to verify that they do not occur
anywhere else.
Ok?
Tom
:ADDPATCH objfiles:
2008-08-17 Tom Tromey <tromey@redhat.com>
* symfile.c (reread_symbols): Update.
* solib-sunos.c (allocate_rt_common_objfile): Update.
* objfiles.c (allocate_objfile): Update.
* objfiles.h (struct objfile) <md, mmfd, deprecated_obj_private>:
Remove.
diff --git a/gdb/objfiles.c b/gdb/objfiles.c
index 68cec1b..2969a8a 100644
--- a/gdb/objfiles.c
+++ b/gdb/objfiles.c
@@ -163,7 +163,6 @@ allocate_objfile (bfd *abfd, int flags)
{
objfile = (struct objfile *) xmalloc (sizeof (struct objfile));
memset (objfile, 0, sizeof (struct objfile));
- objfile->md = NULL;
objfile->psymbol_cache = bcache_xmalloc ();
objfile->macro_cache = bcache_xmalloc ();
/* We could use obstack_specify_allocation here instead, but
diff --git a/gdb/objfiles.h b/gdb/objfiles.h
index 7e39b07..56a00f1 100644
--- a/gdb/objfiles.h
+++ b/gdb/objfiles.h
@@ -293,18 +293,6 @@ struct objfile
struct minimal_symbol *msymbol_demangled_hash[MINIMAL_SYMBOL_HASH_SIZE];
- /* The mmalloc() malloc-descriptor for this objfile if we are using
- the memory mapped malloc() package to manage storage for this objfile's
- data. NULL if we are not. */
-
- void *md;
-
- /* The file descriptor that was used to obtain the mmalloc descriptor
- for this objfile. If we call mmalloc_detach with the malloc descriptor
- we should then close this file descriptor. */
-
- int mmfd;
-
/* Structure which keeps track of functions that manipulate objfile's
of the same type as this objfile. I.E. the function to read partial
symbols for example. Note that this structure is in statically
@@ -338,17 +326,10 @@ struct objfile
void *deprecated_sym_private;
- /* Hook for target-architecture-specific information. This must
- point to memory allocated on one of the obstacks in this objfile,
- so that it gets freed automatically when reading a new object
- file. */
-
- void *deprecated_obj_private;
-
/* Per objfile data-pointers required by other GDB modules. */
/* FIXME: kettenis/20030711: This mechanism could replace
- deprecated_sym_stab_info, deprecated_sym_private and
- deprecated_obj_private entirely. */
+ deprecated_sym_stab_info and deprecated_sym_private
+ entirely. */
void **data;
unsigned num_data;
diff --git a/gdb/solib-sunos.c b/gdb/solib-sunos.c
index 0834d0f..554ffdb 100644
--- a/gdb/solib-sunos.c
+++ b/gdb/solib-sunos.c
@@ -184,7 +184,6 @@ allocate_rt_common_objfile (void)
objfile = (struct objfile *) xmalloc (sizeof (struct objfile));
memset (objfile, 0, sizeof (struct objfile));
- objfile->md = NULL;
objfile->psymbol_cache = bcache_xmalloc ();
objfile->macro_cache = bcache_xmalloc ();
obstack_init (&objfile->objfile_obstack);
diff --git a/gdb/symfile.c b/gdb/symfile.c
index e0d94f3..b80ef2a 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -2421,8 +2421,6 @@ reread_symbols (void)
(*objfile->sf->sym_finish) (objfile);
}
- /* We never make this a mapped file. */
- objfile->md = NULL;
objfile->psymbol_cache = bcache_xmalloc ();
objfile->macro_cache = bcache_xmalloc ();
/* obstack_init also initializes the obstack so it is
next reply other threads:[~2008-08-17 17:48 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-17 17:48 Tom Tromey [this message]
2008-08-18 12:29 ` Joel Brobecker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3bpzrzg8q.fsf@fleche.redhat.com \
--to=tromey@redhat.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox