From 84a1243b15122dfe6414a4f9bdd82096b37bc625 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Mon, 7 Oct 2013 19:40:38 +0000 Subject: [PATCH] move the demangled_names_hash into the per-BFD This moves the demangled_names_hash from the objfile into the per-BFD object. This is part of the objfile splitting project. The demangled names hash is independent of the program space. And, it is needed by the symbol tables. Both of these things indicate that it must be pushed into the per-BFD object, which this patch does. Built and regtested on x86-64 Fedora 18. * objfiles.c (free_objfile_per_bfd_storage): Delete the demangled_names_hash. (free_objfile): Don't delete the demangled_names_hash. * objfiles.h (struct objfile_per_bfd_storage) : New field. (struct objfile) : Move to objfile_per_bfd_storage. * symfile.c (reread_symbols): Don't delete the demangled_names_hash. * symtab.c (create_demangled_names_hash): Update. (symbol_set_names): Update. --- gdb/ChangeLog | 14 ++++++++++++++ gdb/objfiles.c | 4 ++-- gdb/objfiles.h | 14 +++++++------- gdb/symfile.c | 5 ----- gdb/symtab.c | 21 +++++++++++---------- 5 files changed, 34 insertions(+), 24 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 5a10cbaa39..a0c0b97cbe 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,17 @@ +2013-10-07 Tom Tromey + + * objfiles.c (free_objfile_per_bfd_storage): Delete the + demangled_names_hash. + (free_objfile): Don't delete the demangled_names_hash. + * objfiles.h (struct objfile_per_bfd_storage) + : New field. + (struct objfile) : Move to + objfile_per_bfd_storage. + * symfile.c (reread_symbols): Don't delete the + demangled_names_hash. + * symtab.c (create_demangled_names_hash): Update. + (symbol_set_names): Update. + 2013-10-07 Tom Tromey * gdb_bfd.c (struct gdb_bfd_data) filename_cache); bcache_xfree (storage->macro_cache); + if (storage->demangled_names_hash) + htab_delete (storage->demangled_names_hash); obstack_free (&storage->storage_obstack, 0); } @@ -655,8 +657,6 @@ free_objfile (struct objfile *objfile) xfree (objfile->static_psymbols.list); /* Free the obstacks for non-reusable objfiles. */ psymbol_bcache_free (objfile->psymbol_cache); - if (objfile->demangled_names_hash) - htab_delete (objfile->demangled_names_hash); obstack_free (&objfile->objfile_obstack, 0); /* Rebuild section map next time we need it. */ diff --git a/gdb/objfiles.h b/gdb/objfiles.h index 72cef50229..8586e5a12b 100644 --- a/gdb/objfiles.h +++ b/gdb/objfiles.h @@ -185,6 +185,13 @@ struct objfile_per_bfd_storage differ from this e.g. with respect to register types and names. */ struct gdbarch *gdbarch; + + /* Hash table for mapping symbol names to demangled names. Each + entry in the hash table is actually two consecutive strings, + both null-terminated; the first one is a mangled or linkage + name, and the second is the demangled name or just a zero byte + if the name doesn't demangle. */ + struct htab *demangled_names_hash; }; /* Master structure for keeping track of each file from which @@ -270,13 +277,6 @@ struct objfile struct psymbol_bcache *psymbol_cache; /* Byte cache for partial syms. */ - /* Hash table for mapping symbol names to demangled names. Each - entry in the hash table is actually two consecutive strings, - both null-terminated; the first one is a mangled or linkage - name, and the second is the demangled name or just a zero byte - if the name doesn't demangle. */ - struct htab *demangled_names_hash; - /* Vectors of all partial symbols read in from file. The actual data is stored in the objfile_obstack. */ diff --git a/gdb/symfile.c b/gdb/symfile.c index ecf4e32835..d260ff9fcd 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -2497,11 +2497,6 @@ reread_symbols (void) /* Free the obstacks for non-reusable objfiles. */ psymbol_bcache_free (objfile->psymbol_cache); objfile->psymbol_cache = psymbol_bcache_init (); - if (objfile->demangled_names_hash != NULL) - { - htab_delete (objfile->demangled_names_hash); - objfile->demangled_names_hash = NULL; - } obstack_free (&objfile->objfile_obstack, 0); objfile->sections = NULL; objfile->symtabs = NULL; diff --git a/gdb/symtab.c b/gdb/symtab.c index d3622e508f..3660f1ab6c 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -592,7 +592,7 @@ create_demangled_names_hash (struct objfile *objfile) Choosing a much larger table size wastes memory, and saves only about 1% in symbol reading. */ - objfile->demangled_names_hash = htab_create_alloc + objfile->per_bfd->demangled_names_hash = htab_create_alloc (256, hash_demangled_name_entry, eq_demangled_name_entry, NULL, xcalloc, xfree); } @@ -687,7 +687,7 @@ symbol_find_demangled_name (struct general_symbol_info *gsymbol, objfile), and it will not be copied. The hash table corresponding to OBJFILE is used, and the memory - comes from that objfile's objfile_obstack. LINKAGE_NAME is copied, + comes from the per-BFD storage_obstack. LINKAGE_NAME is copied, so the pointer can be discarded after calling this function. */ /* We have to be careful when dealing with Java names: when we run @@ -723,6 +723,7 @@ symbol_set_names (struct general_symbol_info *gsymbol, /* The length of lookup_name. */ int lookup_len; struct demangled_name_entry entry; + struct objfile_per_bfd_storage *per_bfd = objfile->per_bfd; if (gsymbol->language == language_ada) { @@ -738,18 +739,18 @@ symbol_set_names (struct general_symbol_info *gsymbol, gsymbol->name = linkage_name; else { - char *name = obstack_alloc (&objfile->objfile_obstack, len + 1); + char *name = obstack_alloc (&per_bfd->storage_obstack, len + 1); memcpy (name, linkage_name, len); name[len] = '\0'; gsymbol->name = name; } - symbol_set_demangled_name (gsymbol, NULL, &objfile->objfile_obstack); + symbol_set_demangled_name (gsymbol, NULL, &per_bfd->storage_obstack); return; } - if (objfile->demangled_names_hash == NULL) + if (per_bfd->demangled_names_hash == NULL) create_demangled_names_hash (objfile); /* The stabs reader generally provides names that are not @@ -789,7 +790,7 @@ symbol_set_names (struct general_symbol_info *gsymbol, entry.mangled = lookup_name; slot = ((struct demangled_name_entry **) - htab_find_slot (objfile->demangled_names_hash, + htab_find_slot (per_bfd->demangled_names_hash, &entry, INSERT)); /* If this name is not in the hash table, add it. */ @@ -814,7 +815,7 @@ symbol_set_names (struct general_symbol_info *gsymbol, us better bcache hit rates for partial symbols. */ if (!copy_name && lookup_name == linkage_name) { - *slot = obstack_alloc (&objfile->objfile_obstack, + *slot = obstack_alloc (&per_bfd->storage_obstack, offsetof (struct demangled_name_entry, demangled) + demangled_len + 1); @@ -827,7 +828,7 @@ symbol_set_names (struct general_symbol_info *gsymbol, /* If we must copy the mangled name, put it directly after the demangled name so we can have a single allocation. */ - *slot = obstack_alloc (&objfile->objfile_obstack, + *slot = obstack_alloc (&per_bfd->storage_obstack, offsetof (struct demangled_name_entry, demangled) + lookup_len + demangled_len + 2); @@ -848,9 +849,9 @@ symbol_set_names (struct general_symbol_info *gsymbol, gsymbol->name = (*slot)->mangled + lookup_len - len; if ((*slot)->demangled[0] != '\0') symbol_set_demangled_name (gsymbol, (*slot)->demangled, - &objfile->objfile_obstack); + &per_bfd->storage_obstack); else - symbol_set_demangled_name (gsymbol, NULL, &objfile->objfile_obstack); + symbol_set_demangled_name (gsymbol, NULL, &per_bfd->storage_obstack); } /* Return the source code name of a symbol. In languages where