Split up end_symtab_from_static_block into two.
[deliverable/binutils-gdb.git] / gdb / buildsym.c
index 2c4a946ce5ee9f8e4f00b73941aa53d44db2016b..3d92a85450fcc6cf84f68684efe2d4ac2b4199b3 100644 (file)
@@ -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.  */
 
 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;
@@ -102,13 +106,24 @@ struct pending_block
    associated symtab.  */
 
 static struct pending_block *pending_blocks;
-\f
+
+struct subfile_stack
+  {
+    struct subfile_stack *next;
+    char *name;
+  };
+
+static struct subfile_stack *subfile_stack;
+
+/* The macro table for the compilation unit whose symbols we're
+   currently reading.  All the symtabs for the CU will point to this.  */
+static struct macro_table *pending_macros;
+
 static int compare_line_numbers (const void *ln1p, const void *ln2p);
 
 static void record_pending_block (struct objfile *objfile,
                                  struct block *block,
                                  struct pending_block *opblock);
-\f
 
 /* Initial sizes of data structures.  These are realloc'd larger if
    needed, and realloc'd down to the size actually used, when
@@ -661,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
@@ -681,7 +716,7 @@ patch_subfile_names (struct subfile *subfile, char *name)
     {
       subfile->dirname = subfile->name;
       subfile->name = xstrdup (name);
-      last_source_file = name;
+      set_last_source_file (name);
 
       /* Default the source language to whatever can be deduced from
          the filename.  If nothing can be deduced (such as for a C/C++
@@ -825,6 +860,19 @@ compare_line_numbers (const void *ln1p, const void *ln2p)
   return ln1->line - ln2->line;
 }
 \f
+/* Return the macro table.
+   Initialize it if this is the first use.  */
+
+struct macro_table *
+get_macro_table (struct objfile *objfile, const char *comp_dir)
+{
+  if (! pending_macros)
+    pending_macros = new_macro_table (&objfile->per_bfd->storage_obstack,
+                                     objfile->per_bfd->macro_cache,
+                                     comp_dir);
+  return pending_macros;
+}
+\f
 /* Start a new symtab for a new source file.  Called, for example,
    when a stabs symbol of type N_SO is seen, or when a DWARF
    TAG_compile_unit DIE is seen.  It indicates the start of data for
@@ -835,11 +883,14 @@ compare_line_numbers (const void *ln1p, const void *ln2p)
    lowest address of objects in the file (or 0 if not known).  */
 
 void
-start_symtab (char *name, char *dirname, CORE_ADDR start_addr)
+start_symtab (const char *name, const char *dirname, CORE_ADDR start_addr)
 {
   restart_symtab (start_addr);
-  last_source_file = name;
+  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.
@@ -850,7 +901,7 @@ start_symtab (char *name, char *dirname, CORE_ADDR start_addr)
 void
 restart_symtab (CORE_ADDR start_addr)
 {
-  last_source_file = NULL;
+  set_last_source_file (NULL);
   last_source_start_addr = start_addr;
   file_symbols = NULL;
   global_symbols = NULL;
@@ -870,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
@@ -888,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;
@@ -934,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;
@@ -971,8 +1012,8 @@ block_compar (const void *ap, const void *bp)
 static void
 reset_symtab_globals (void)
 {
-  last_source_file = NULL;
-  current_subfile = NULL;
+  set_last_source_file (NULL);
+  free_subfiles_list ();
   pending_macros = NULL;
   if (pending_addrmap)
     {
@@ -1084,45 +1125,74 @@ 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);
 
-  /* Read the line table if it has to be read separately.  */
+  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.  */
   if (objfile->sf->sym_read_linetable != NULL)
-    objfile->sf->sym_read_linetable ();
+    objfile->sf->sym_read_linetable (objfile);
 
   /* Handle the case where the debug info specifies a different path
      for the main source file.  It can cause us to lose track of its
@@ -1130,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
@@ -1366,8 +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;
-  int i;
+  const struct blockvector *blockvector = primary_symtab->blockvector;
 
   if (context_stack_depth > 0)
     {
@@ -1500,6 +1543,32 @@ merge_symbol_lists (struct pending **srclist, struct pending **targetlist)
   free_pendings = (*srclist);
 }
 \f
+
+/* Name of source file whose symbol data we are now processing.  This
+   comes from a symbol of type N_SO for stabs.  For Dwarf it comes
+   from the DW_AT_name attribute of a DW_TAG_compile_unit DIE.  */
+
+static char *last_source_file;
+
+/* See buildsym.h.  */
+
+void
+set_last_source_file (const char *name)
+{
+  xfree (last_source_file);
+  last_source_file = name == NULL ? NULL : xstrdup (name);
+}
+
+/* See buildsym.h.  */
+
+const char *
+get_last_source_file (void)
+{
+  return last_source_file;
+}
+
+\f
+
 /* Initialize anything that needs initializing when starting to read a
    fresh piece of a symbol file, e.g. reading in the stuff
    corresponding to a psymtab.  */
@@ -1513,6 +1582,7 @@ buildsym_init (void)
   pending_blocks = NULL;
   pending_macros = NULL;
   using_directives = NULL;
+  subfile_stack = NULL;
 
   /* We shouldn't have any address map at this point.  */
   gdb_assert (! pending_addrmap);
This page took 0.033709 seconds and 4 git commands to generate.