X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=gdb%2Fbuildsym.c;h=3d92a85450fcc6cf84f68684efe2d4ac2b4199b3;hb=7bab9b58fd9c05c20cbd56d778ffa05d9fe3af3e;hp=cee0cc54c880a96eebf07198322c720e8c628da5;hpb=f56ce88372c4b5848638bab8b10b71042725a100;p=deliverable%2Fbinutils-gdb.git diff --git a/gdb/buildsym.c b/gdb/buildsym.c index cee0cc54c8..3d92a85450 100644 --- a/gdb/buildsym.c +++ b/gdb/buildsym.c @@ -1,5 +1,5 @@ /* Support routines for building symbol tables in GDB's internal format. - Copyright (C) 1986-2013 Free Software Foundation, Inc. + Copyright (C) 1986-2014 Free Software Foundation, Inc. This file is part of GDB. @@ -30,9 +30,7 @@ #include "symfile.h" #include "objfiles.h" #include "gdbtypes.h" -#include "gdb_assert.h" #include "complaints.h" -#include "gdb_string.h" #include "expression.h" /* For "enum exp_opcode" used by... */ #include "bcache.h" #include "filenames.h" /* For DOSish file names. */ @@ -58,6 +56,12 @@ static struct subfile *subfiles; +/* The "main" subfile. + In C this is the ".c" file (and similarly for other languages). + This becomes the "primary" symtab of the compilation unit. */ + +static struct subfile *main_subfile; + /* List of free `struct pending' structures for reuse. */ static struct pending *free_pendings; @@ -672,6 +676,26 @@ start_subfile (const char *name, const char *dirname) } } +/* Delete the subfiles list. */ + +static void +free_subfiles_list (void) +{ + struct subfile *subfile, *nextsub; + + for (subfile = subfiles; subfile != NULL; subfile = nextsub) + { + nextsub = subfile->next; + xfree (subfile->name); + xfree (subfile->dirname); + xfree (subfile->line_vector); + xfree (subfile); + } + subfiles = NULL; + current_subfile = NULL; + main_subfile = NULL; +} + /* For stabs readers, the first N_SO symbol is assumed to be the source file name, and the subfile struct is initialized using that assumption. If another N_SO symbol is later seen, immediately @@ -864,6 +888,9 @@ start_symtab (const char *name, const char *dirname, CORE_ADDR start_addr) restart_symtab (start_addr); set_last_source_file (name); start_subfile (name, dirname); + /* Save this so that we don't have to go looking for it at the end + of the subfiles list. */ + main_subfile = current_subfile; } /* Restart compilation for a symtab. @@ -894,10 +921,10 @@ restart_symtab (CORE_ADDR start_addr) /* We shouldn't have any address map at this point. */ gdb_assert (! pending_addrmap); - /* Initialize the list of sub source files with one entry for this - file (the top-level source file). */ - subfiles = NULL; - current_subfile = NULL; + /* Reset the sub source files list. The list should already be empty, + but free it anyway in case some code didn't finish cleaning up after + an error. */ + free_subfiles_list (); } /* Subroutine of end_symtab to simplify it. Look for a subfile that @@ -912,30 +939,20 @@ restart_symtab (CORE_ADDR start_addr) static void watch_main_source_file_lossage (void) { - struct subfile *mainsub, *subfile; + struct subfile *subfile; - /* Find the main source file. - This loop could be eliminated if start_symtab saved it for us. */ - mainsub = NULL; - for (subfile = subfiles; subfile; subfile = subfile->next) - { - /* The main subfile is guaranteed to be the last one. */ - if (subfile->next == NULL) - mainsub = subfile; - } + /* We have to watch for mainsub == NULL here. It's a quirk of + end_symtab, it can return NULL so there may not be a main subfile. */ + if (main_subfile == NULL) + return; /* If the main source file doesn't have any line number or symbol - info, look for an alias in another subfile. - - We have to watch for mainsub == NULL here. It's a quirk of - end_symtab, it can return NULL so there may not be a main - subfile. */ + info, look for an alias in another subfile. */ - if (mainsub - && mainsub->line_vector == NULL - && mainsub->symtab == NULL) + if (main_subfile->line_vector == NULL + && main_subfile->symtab == NULL) { - const char *mainbase = lbasename (mainsub->name); + const char *mainbase = lbasename (main_subfile->name); int nr_matches = 0; struct subfile *prevsub; struct subfile *mainsub_alias = NULL; @@ -958,15 +975,15 @@ watch_main_source_file_lossage (void) if (nr_matches == 1) { - gdb_assert (mainsub_alias != NULL && mainsub_alias != mainsub); + gdb_assert (mainsub_alias != NULL && mainsub_alias != main_subfile); /* Found a match for the main source file. Copy its line_vector and symtab to the main subfile and then discard it. */ - mainsub->line_vector = mainsub_alias->line_vector; - mainsub->line_vector_length = mainsub_alias->line_vector_length; - mainsub->symtab = mainsub_alias->symtab; + main_subfile->line_vector = mainsub_alias->line_vector; + main_subfile->line_vector_length = mainsub_alias->line_vector_length; + main_subfile->symtab = mainsub_alias->symtab; if (prev_mainsub_alias == NULL) subfiles = mainsub_alias->next; @@ -996,7 +1013,7 @@ static void reset_symtab_globals (void) { set_last_source_file (NULL); - current_subfile = NULL; + free_subfiles_list (); pending_macros = NULL; if (pending_addrmap) { @@ -1108,41 +1125,69 @@ end_symtab_get_static_block (CORE_ADDR end_addr, struct objfile *objfile, } } -/* Implementation of the second part of end_symtab. Pass STATIC_BLOCK - as value returned by end_symtab_get_static_block. +/* Subroutine of end_symtab_from_static_block to simplify it. + Handle the "no blockvector" case. + When this happens there is nothing to record, so just free up + any memory we allocated while reading debug info. */ - SECTION is the same as for end_symtab: the section number - (in objfile->section_offsets) of the blockvector and linetable. +static void +end_symtab_without_blockvector (struct objfile *objfile) +{ + struct subfile *subfile; - If EXPANDABLE is non-zero the GLOBAL_BLOCK dictionary is made - expandable. */ + /* Since we are ignoring these subfiles, we also need + to unlink the associated empty symtab that we created. + Otherwise, we can run into trouble because various parts + such as the block-vector are uninitialized whereas + the rest of the code assumes that they are. -struct symtab * -end_symtab_from_static_block (struct block *static_block, - struct objfile *objfile, int section, - int expandable) + We can only unlink the symtab. We can't free it because + it was allocated on the objfile obstack. */ + + for (subfile = subfiles; subfile != NULL; subfile = subfile->next) + { + if (subfile->symtab) + { + struct symtab *s; + + if (objfile->symtabs == subfile->symtab) + objfile->symtabs = objfile->symtabs->next; + else + ALL_OBJFILE_SYMTABS (objfile, s) + if (s->next == subfile->symtab) + { + s->next = s->next->next; + break; + } + subfile->symtab = NULL; + } + } +} + +/* Subroutine of end_symtab_from_static_block to simplify it. + Handle the "have blockvector" case. + See end_symtab_from_static_block for a description of the arguments. */ + +static struct symtab * +end_symtab_with_blockvector (struct block *static_block, + struct objfile *objfile, int section, + int expandable) { - struct symtab *symtab = NULL; + struct symtab *symtab; struct blockvector *blockvector; struct subfile *subfile; - struct subfile *nextsub; + CORE_ADDR end_addr; - if (static_block == NULL) - { - /* Ignore symtabs that have no functions with real debugging info. */ - blockvector = NULL; - } - else - { - CORE_ADDR end_addr = BLOCK_END (static_block); - - /* Define after STATIC_BLOCK also GLOBAL_BLOCK, and build the - blockvector. */ - finish_block_internal (NULL, &global_symbols, NULL, - last_source_start_addr, end_addr, objfile, - 1, expandable); - blockvector = make_blockvector (objfile); - } + gdb_assert (static_block != NULL); + gdb_assert (subfiles != NULL); + + end_addr = BLOCK_END (static_block); + + /* Create the GLOBAL_BLOCK and build the blockvector. */ + finish_block_internal (NULL, &global_symbols, NULL, + last_source_start_addr, end_addr, objfile, + 1, expandable); + blockvector = make_blockvector (objfile); /* Read the line table if it has to be read separately. This is only used by xcoffread.c. */ @@ -1155,172 +1200,146 @@ end_symtab_from_static_block (struct block *static_block, watch_main_source_file_lossage (); /* Now create the symtab objects proper, one for each subfile. */ - /* (The main file is the last one on the chain.) */ - for (subfile = subfiles; subfile; subfile = nextsub) + for (subfile = subfiles; subfile != NULL; subfile = subfile->next) { int linetablesize = 0; - symtab = NULL; - /* If we have blocks of symbols, make a symtab. Otherwise, just - ignore this file and any line number info in it. */ - if (blockvector) + if (subfile->line_vector) { - if (subfile->line_vector) - { - linetablesize = sizeof (struct linetable) + - subfile->line_vector->nitems * sizeof (struct linetable_entry); - - /* Like the pending blocks, the line table may be - scrambled in reordered executables. Sort it if - OBJF_REORDERED is true. */ - if (objfile->flags & OBJF_REORDERED) - qsort (subfile->line_vector->item, - subfile->line_vector->nitems, - sizeof (struct linetable_entry), compare_line_numbers); - } - - /* Now, allocate a symbol table. */ - if (subfile->symtab == NULL) - symtab = allocate_symtab (subfile->name, objfile); - else - symtab = subfile->symtab; - - /* Fill in its components. */ - symtab->blockvector = blockvector; - symtab->macro_table = pending_macros; - if (subfile->line_vector) - { - /* Reallocate the line table on the symbol obstack. */ - symtab->linetable = (struct linetable *) - obstack_alloc (&objfile->objfile_obstack, linetablesize); - memcpy (symtab->linetable, subfile->line_vector, linetablesize); - } - else - { - symtab->linetable = NULL; - } - symtab->block_line_section = section; - if (subfile->dirname) - { - /* Reallocate the dirname on the symbol obstack. */ - symtab->dirname = (char *) - obstack_alloc (&objfile->objfile_obstack, - strlen (subfile->dirname) + 1); - strcpy (symtab->dirname, subfile->dirname); - } - else - { - symtab->dirname = NULL; - } - - /* Use whatever language we have been using for this - subfile, not the one that was deduced in allocate_symtab - from the filename. We already did our own deducing when - we created the subfile, and we may have altered our - opinion of what language it is from things we found in - the symbols. */ - symtab->language = subfile->language; - - /* Save the debug format string (if any) in the symtab. */ - symtab->debugformat = subfile->debugformat; - - /* Similarly for the producer. */ - symtab->producer = subfile->producer; + linetablesize = sizeof (struct linetable) + + subfile->line_vector->nitems * sizeof (struct linetable_entry); + + /* Like the pending blocks, the line table may be + scrambled in reordered executables. Sort it if + OBJF_REORDERED is true. */ + if (objfile->flags & OBJF_REORDERED) + qsort (subfile->line_vector->item, + subfile->line_vector->nitems, + sizeof (struct linetable_entry), compare_line_numbers); + } - /* All symtabs for the main file and the subfiles share a - blockvector, so we need to clear primary for everything - but the main file. */ + /* Allocate a symbol table if necessary. */ + if (subfile->symtab == NULL) + subfile->symtab = allocate_symtab (subfile->name, objfile); + symtab = subfile->symtab; - symtab->primary = 0; + /* Fill in its components. */ + symtab->blockvector = blockvector; + symtab->macro_table = pending_macros; + if (subfile->line_vector) + { + /* Reallocate the line table on the symbol obstack. */ + symtab->linetable = (struct linetable *) + obstack_alloc (&objfile->objfile_obstack, linetablesize); + memcpy (symtab->linetable, subfile->line_vector, linetablesize); } else - { - if (subfile->symtab) - { - /* Since we are ignoring that subfile, we also need - to unlink the associated empty symtab that we created. - Otherwise, we can run into trouble because various parts - such as the block-vector are uninitialized whereas - the rest of the code assumes that they are. - - We can only unlink the symtab because it was allocated - on the objfile obstack. */ - struct symtab *s; - - if (objfile->symtabs == subfile->symtab) - objfile->symtabs = objfile->symtabs->next; - else - ALL_OBJFILE_SYMTABS (objfile, s) - if (s->next == subfile->symtab) - { - s->next = s->next->next; - break; - } - subfile->symtab = NULL; - } - } - if (subfile->name != NULL) { - xfree ((void *) subfile->name); + symtab->linetable = NULL; } - if (subfile->dirname != NULL) + symtab->block_line_section = section; + if (subfile->dirname) { - xfree ((void *) subfile->dirname); + /* Reallocate the dirname on the symbol obstack. */ + symtab->dirname = + obstack_copy0 (&objfile->objfile_obstack, + subfile->dirname, + strlen (subfile->dirname)); } - if (subfile->line_vector != NULL) + else { - xfree ((void *) subfile->line_vector); + symtab->dirname = NULL; } - nextsub = subfile->next; - xfree ((void *) subfile); - } + /* Use whatever language we have been using for this + subfile, not the one that was deduced in allocate_symtab + from the filename. We already did our own deducing when + we created the subfile, and we may have altered our + opinion of what language it is from things we found in + the symbols. */ + symtab->language = subfile->language; - /* Set this for the main source file. */ - if (symtab) - { - symtab->primary = 1; + /* Save the debug format string (if any) in the symtab. */ + symtab->debugformat = subfile->debugformat; - if (symtab->blockvector) - { - struct block *b = BLOCKVECTOR_BLOCK (symtab->blockvector, - GLOBAL_BLOCK); + /* Similarly for the producer. */ + symtab->producer = subfile->producer; - set_block_symtab (b, symtab); - } + /* All symtabs for the main file and the subfiles share a + blockvector, so we need to clear primary for everything + but the main file. */ + set_symtab_primary (symtab, 0); } - /* Default any symbols without a specified symtab to the primary - symtab. */ - if (blockvector) - { - int block_i; + /* The main source file is the primary symtab. */ + gdb_assert (main_subfile->symtab != NULL); + symtab = main_subfile->symtab; + set_symtab_primary (symtab, 1); + { + struct block *b = BLOCKVECTOR_BLOCK (symtab->blockvector, GLOBAL_BLOCK); - for (block_i = 0; block_i < BLOCKVECTOR_NBLOCKS (blockvector); block_i++) - { - struct block *block = BLOCKVECTOR_BLOCK (blockvector, block_i); - struct symbol *sym; - struct dict_iterator iter; + set_block_symtab (b, symtab); + } - /* Inlined functions may have symbols not in the global or - static symbol lists. */ - if (BLOCK_FUNCTION (block) != NULL) - if (SYMBOL_SYMTAB (BLOCK_FUNCTION (block)) == NULL) - SYMBOL_SYMTAB (BLOCK_FUNCTION (block)) = symtab; + /* Default any symbols without a specified symtab to the primary symtab. */ + { + int block_i; + + for (block_i = 0; block_i < BLOCKVECTOR_NBLOCKS (blockvector); block_i++) + { + struct block *block = BLOCKVECTOR_BLOCK (blockvector, block_i); + struct symbol *sym; + struct dict_iterator iter; + + /* Inlined functions may have symbols not in the global or + static symbol lists. */ + if (BLOCK_FUNCTION (block) != NULL) + if (SYMBOL_SYMTAB (BLOCK_FUNCTION (block)) == NULL) + SYMBOL_SYMTAB (BLOCK_FUNCTION (block)) = symtab; + + /* Note that we only want to fix up symbols from the local + blocks, not blocks coming from included symtabs. That is why + we use ALL_DICT_SYMBOLS here and not ALL_BLOCK_SYMBOLS. */ + ALL_DICT_SYMBOLS (BLOCK_DICT (block), iter, sym) + if (SYMBOL_SYMTAB (sym) == NULL) + SYMBOL_SYMTAB (sym) = symtab; + } + } - /* Note that we only want to fix up symbols from the local - blocks, not blocks coming from included symtabs. That is why - we use ALL_DICT_SYMBOLS here and not ALL_BLOCK_SYMBOLS. */ - ALL_DICT_SYMBOLS (BLOCK_DICT (block), iter, sym) - if (SYMBOL_SYMTAB (sym) == NULL) - SYMBOL_SYMTAB (sym) = symtab; - } + return symtab; +} + +/* Implementation of the second part of end_symtab. Pass STATIC_BLOCK + as value returned by end_symtab_get_static_block. + + SECTION is the same as for end_symtab: the section number + (in objfile->section_offsets) of the blockvector and linetable. + + If EXPANDABLE is non-zero the GLOBAL_BLOCK dictionary is made + expandable. */ + +struct symtab * +end_symtab_from_static_block (struct block *static_block, + struct objfile *objfile, int section, + int expandable) +{ + struct symtab *s; + + if (static_block == NULL) + { + end_symtab_without_blockvector (objfile); + s = NULL; + } + else + { + s = end_symtab_with_blockvector (static_block, objfile, section, + expandable); } reset_symtab_globals (); - return symtab; + return s; } /* Finish the symbol definitions for one main source file, close off @@ -1391,7 +1410,7 @@ set_missing_symtab (struct pending *pending_list, struct symtab *symtab) void augment_type_symtab (struct objfile *objfile, struct symtab *primary_symtab) { - struct blockvector *blockvector = primary_symtab->blockvector; + const struct blockvector *blockvector = primary_symtab->blockvector; if (context_stack_depth > 0) {