* config/m88k/{tm-delta88.h,tm-delta88v4.h}, m88k-tdep.c:
[deliverable/binutils-gdb.git] / gdb / symfile.c
index 395d7306ddffe2dfc2992ee9787eb245cfbc3075..52028238f09ae67ede9164e6c093050e7410f2e4 100644 (file)
@@ -1,34 +1,38 @@
 /* Generic symbol file reading for the GNU debugger, GDB.
-   Copyright 1990, 1991 Free Software Foundation, Inc.
+   Copyright 1990, 1991, 1992, 1993 Free Software Foundation, Inc.
    Contributed by Cygnus Support, using pieces from other GDB modules.
 
 This file is part of GDB.
 
-GDB is free software; you can redistribute it and/or modify
+This program is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 1, or (at your option)
-any later version.
+the Free Software Foundation; either version 2 of the License, or
+(at your option) any later version.
 
-GDB is distributed in the hope that it will be useful,
+This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
-along with GDB; see the file COPYING.  If not, write to
-the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
+along with this program; if not, write to the Free Software
+Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
-#include <stdio.h>
 #include "defs.h"
 #include "symtab.h"
-#include "param.h"
+#include "gdbtypes.h"
 #include "gdbcore.h"
 #include "frame.h"
 #include "target.h"
 #include "value.h"
 #include "symfile.h"
+#include "objfiles.h"
 #include "gdbcmd.h"
 #include "breakpoint.h"
+#include "language.h"
+#include "complaints.h"
+#include "demangle.h"
+#include "inferior.h" /* for write_pc */
 
 #include <obstack.h>
 #include <assert.h>
@@ -37,94 +41,144 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include <fcntl.h>
 #include <string.h>
 #include <sys/stat.h>
+#include <ctype.h>
 
-extern int info_verbose;
+#ifndef O_BINARY
+#define O_BINARY 0
+#endif
 
-extern int close ();
-extern void qsort ();
-extern char *getenv ();
+/* Global variables owned by this file */
 
-/* Functions this file defines */
-static bfd *symfile_open();
-static struct sym_fns *symfile_init();
-static void clear_symtab_users_once();
+int readnow_symbol_files;              /* Read full symbols immediately */
 
-/* List of all available sym_fns.  */
+struct complaint oldsyms_complaint = {
+  "Replacing old symbols for `%s'", 0, 0
+};
 
-struct sym_fns *symtab_fns = NULL;
+struct complaint empty_symtab_complaint = {
+  "Empty symbol table found for `%s'", 0, 0
+};
 
-/* Saves the sym_fns of the current symbol table, so we can call
-   the right sym_discard function when we free it.  */
+/* External variables and functions referenced. */
 
-static struct sym_fns *symfile_fns;
+extern int info_verbose;
 
-/* Allocate an obstack to hold objects that should be freed
-   when we load a new symbol table.
-   This includes the symbols made by dbxread
-   and the types that are not permanent.  */
+/* Functions this file defines */
 
-struct obstack obstack1;
+static void
+set_initial_language PARAMS ((void));
 
-struct obstack *symbol_obstack = &obstack1;
+static void
+load_command PARAMS ((char *, int));
 
-/* This obstack will be used for partial_symbol objects.  It can
-   probably actually be the same as the symbol_obstack above, but I'd
-   like to keep them seperate for now.  If I want to later, I'll
-   replace one with the other.  */
+static void
+add_symbol_file_command PARAMS ((char *, int));
 
-struct obstack obstack2;
+static void
+cashier_psymtab PARAMS ((struct partial_symtab *));
 
-struct obstack *psymbol_obstack = &obstack2;
+static int
+compare_psymbols PARAMS ((const void *, const void *));
+
+static int
+compare_symbols PARAMS ((const void *, const void *));
 
-/* File name symbols were loaded from.  */
+static bfd *
+symfile_bfd_open PARAMS ((char *));
 
-char *symfile = 0;
+static void
+find_sym_fns PARAMS ((struct objfile *));
 
-/* The modification date of the file when they were loaded.  */
+/* List of all available sym_fns.  On gdb startup, each object file reader
+   calls add_symtab_fns() to register information on each format it is
+   prepared to read. */
 
-int symfile_mtime = 0;
+static struct sym_fns *symtab_fns = NULL;
 
 /* Structures with which to manage partial symbol allocation.  */
 
 struct psymbol_allocation_list global_psymbols = {0}, static_psymbols = {0};
 
-/* Structure to manage complaints about symbol file contents.  */
+/* Flag for whether user will be reloading symbols multiple times.
+   Defaults to ON for VxWorks, otherwise OFF.  */
 
-struct complaint complaint_root[1] = {
-  {(char *)0, 0, complaint_root},
-};
+#ifdef SYMBOL_RELOADING_DEFAULT
+int symbol_reloading = SYMBOL_RELOADING_DEFAULT;
+#else
+int symbol_reloading = 0;
+#endif
 
-/* Some actual complaints.  */
+\f
+/* Since this function is called from within qsort, in an ANSI environment
+   it must conform to the prototype for qsort, which specifies that the
+   comparison function takes two "void *" pointers. */
 
-struct complaint oldsyms_complaint = {
-       "Replacing old symbols for `%s'", 0, 0 };
+static int
+compare_symbols (s1p, s2p)
+     const PTR s1p;
+     const PTR s2p;
+{
+  register struct symbol **s1, **s2;
 
-struct complaint empty_symtab_complaint = {
-       "Empty symbol table found for `%s'", 0, 0 };
+  s1 = (struct symbol **) s1p;
+  s2 = (struct symbol **) s2p;
 
-\f
-/* In the following sort, we always make sure that
-   register debug symbol declarations always come before regular
-   debug symbol declarations (as might happen when parameters are
-   then put into registers by the compiler).  */
+  return (STRCMP (SYMBOL_NAME (*s1), SYMBOL_NAME (*s2)));
+}
+
+/*
+
+LOCAL FUNCTION
+
+       compare_psymbols -- compare two partial symbols by name
+
+DESCRIPTION
+
+       Given pointer to two partial symbol table entries, compare
+       them by name and return -N, 0, or +N (ala strcmp).  Typically
+       used by sorting routines like qsort().
+
+NOTES
+
+       Does direct compare of first two characters before punting
+       and passing to strcmp for longer compares.  Note that the
+       original version had a bug whereby two null strings or two
+       identically named one character strings would return the
+       comparison of memory following the null byte.
+
+ */
 
 static int
-compare_symbols (s1, s2)
-     struct symbol **s1, **s2;
+compare_psymbols (s1p, s2p)
+     const PTR s1p;
+     const PTR s2p;
 {
-  register int namediff;
+  register char *st1 = SYMBOL_NAME ((struct partial_symbol *) s1p);
+  register char *st2 = SYMBOL_NAME ((struct partial_symbol *) s2p);
 
-  /* Compare the initial characters.  */
-  namediff = SYMBOL_NAME (*s1)[0] - SYMBOL_NAME (*s2)[0];
-  if (namediff != 0) return namediff;
+  if ((st1[0] - st2[0]) || !st1[0])
+    {
+      return (st1[0] - st2[0]);
+    }
+  else if ((st1[1] - st2[1]) || !st1[1])
+    {
+      return (st1[1] - st2[1]);
+    }
+  else
+    {
+      return (STRCMP (st1 + 2, st2 + 2));
+    }
+}
 
-  /* If they match, compare the rest of the names.  */
-  namediff = strcmp (SYMBOL_NAME (*s1), SYMBOL_NAME (*s2));
-  if (namediff != 0) return namediff;
+void
+sort_pst_symbols (pst)
+     struct partial_symtab *pst;
+{
+  /* Sort the global list; don't sort the static list */
 
-  /* For symbols of the same name, registers should come first.  */
-  return ((SYMBOL_CLASS (*s2) == LOC_REGISTER)
-         - (SYMBOL_CLASS (*s1) == LOC_REGISTER));
+  qsort (pst -> objfile -> global_psymbols.list + pst -> globals_offset,
+        pst -> n_global_syms, sizeof (struct partial_symbol),
+        compare_psymbols);
 }
 
 /* Call sort_block_syms to sort alphabetically the symbols of one block.  */
@@ -144,11 +198,15 @@ void
 sort_symtab_syms (s)
      register struct symtab *s;
 {
-  register struct blockvector *bv = BLOCKVECTOR (s);
-  int nbl = BLOCKVECTOR_NBLOCKS (bv);
+  register struct blockvector *bv;
+  int nbl;
   int i;
   register struct block *b;
 
+  if (s == 0)
+    return;
+  bv = BLOCKVECTOR (s);
+  nbl = BLOCKVECTOR_NBLOCKS (bv);
   for (i = 0; i < nbl; i++)
     {
       b = BLOCKVECTOR_BLOCK (bv, i);
@@ -157,28 +215,18 @@ sort_symtab_syms (s)
     }
 }
 
-void
-sort_all_symtab_syms ()
-{
-  register struct symtab *s;
-
-  for (s = symtab_list; s; s = s->next)
-    {
-      sort_symtab_syms (s);
-    }
-}
-
 /* Make a copy of the string at PTR with SIZE characters in the symbol obstack
    (and add a null character at the end in the copy).
    Returns the address of the copy.  */
 
 char *
-obsavestring (ptr, size)
+obsavestring (ptr, size, obstackp)
      char *ptr;
      int size;
+     struct obstack *obstackp;
 {
-  register char *p = (char *) obstack_alloc (symbol_obstack, size + 1);
-  /* Open-coded bcopy--saves function call time.
+  register char *p = (char *) obstack_alloc (obstackp, size + 1);
+  /* Open-coded memcpy--saves function call time.
      These strings are usually short.  */
   {
     register char *p1 = ptr;
@@ -195,185 +243,273 @@ obsavestring (ptr, size)
    Space is found in the symbol_obstack.  */
 
 char *
-obconcat (s1, s2, s3)
-     char *s1, *s2, *s3;
+obconcat (obstackp, s1, s2, s3)
+     struct obstack *obstackp;
+     const char *s1, *s2, *s3;
 {
   register int len = strlen (s1) + strlen (s2) + strlen (s3) + 1;
-  register char *val = (char *) obstack_alloc (symbol_obstack, len);
+  register char *val = (char *) obstack_alloc (obstackp, len);
   strcpy (val, s1);
   strcat (val, s2);
   strcat (val, s3);
   return val;
 }
-\f
-/* Accumulate the misc functions in bunches of 127.
-   At the end, copy them all into one newly allocated structure.  */
 
-#define MISC_BUNCH_SIZE 127
+/* Get the symbol table that corresponds to a partial_symtab.
+   This is fast after the first time you do it.  In fact, there
+   is an even faster macro PSYMTAB_TO_SYMTAB that does the fast
+   case inline.  */
 
-struct misc_bunch
+struct symtab *
+psymtab_to_symtab (pst)
+     register struct partial_symtab *pst;
 {
-  struct misc_bunch *next;
-  struct misc_function contents[MISC_BUNCH_SIZE];
-};
+  /* If it's been looked up before, return it. */
+  if (pst->symtab)
+    return pst->symtab;
 
-/* Bunch currently being filled up.
-   The next field points to chain of filled bunches.  */
+  /* If it has not yet been read in, read it.  */
+  if (!pst->readin)
+    { 
+      (*pst->read_symtab) (pst);
+    }
 
-static struct misc_bunch *misc_bunch;
+  return pst->symtab;
+}
 
-/* Number of slots filled in current bunch.  */
+/* Initialize entry point information for this objfile. */
 
-static int misc_bunch_index;
+void
+init_entry_point_info (objfile)
+     struct objfile *objfile;
+{
+  /* Save startup file's range of PC addresses to help blockframe.c
+     decide where the bottom of the stack is.  */
 
-/* Total number of misc functions recorded so far.  */
+  if (bfd_get_file_flags (objfile -> obfd) & EXEC_P)
+    {
+      /* Executable file -- record its entry point so we'll recognize
+        the startup file because it contains the entry point.  */
+      objfile -> ei.entry_point = bfd_get_start_address (objfile -> obfd);
+    }
+  else
+    {
+      /* Examination of non-executable.o files.  Short-circuit this stuff.  */
+      /* ~0 will not be in any file, we hope.  */
+      objfile -> ei.entry_point = ~0;
+      /* set the startup file to be an empty range.  */
+      objfile -> ei.entry_file_lowpc = 0;
+      objfile -> ei.entry_file_highpc = 0;
+    }
+}
 
-static int misc_count;
+/* Get current entry point address.  */
 
-void
-init_misc_bunches ()
+CORE_ADDR
+entry_point_address()
 {
-  misc_count = 0;
-  misc_bunch = 0;
-  misc_bunch_index = MISC_BUNCH_SIZE;
+  return symfile_objfile ? symfile_objfile->ei.entry_point : 0;
 }
 
-void
-prim_record_misc_function (name, address, misc_type)
-     char *name;
-     CORE_ADDR address;
-     enum misc_function_type misc_type;
+/* Remember the lowest-addressed loadable section we've seen.  
+   This function is called via bfd_map_over_sections.  */
+
+#if 0  /* Not used yet */
+static void
+find_lowest_section (abfd, sect, obj)
+     bfd *abfd;
+     asection *sect;
+     PTR obj;
 {
-  register struct misc_bunch *new;
+  asection **lowest = (asection **)obj;
 
-  if (misc_bunch_index == MISC_BUNCH_SIZE)
-    {
-      new = (struct misc_bunch *) xmalloc (sizeof (struct misc_bunch));
-      misc_bunch_index = 0;
-      new->next = misc_bunch;
-      misc_bunch = new;
-    }
-  misc_bunch->contents[misc_bunch_index].name = name;
-  misc_bunch->contents[misc_bunch_index].address = address;
-  misc_bunch->contents[misc_bunch_index].type = misc_type;
-  misc_bunch->contents[misc_bunch_index].misc_info = 0;
-  misc_bunch_index++;
-  misc_count++;
+  if (0 == (bfd_get_section_flags (abfd, sect) & SEC_LOAD))
+    return;
+  if (!*lowest)
+    *lowest = sect;            /* First loadable section */
+  else if (bfd_section_vma (abfd, *lowest) >= bfd_section_vma (abfd, sect))
+    *lowest = sect;            /* A lower loadable section */
 }
+#endif 
 
-static int
-compare_misc_functions (fn1, fn2)
-     struct misc_function *fn1, *fn2;
-{
-  /* Return a signed result based on unsigned comparisons
-     so that we sort into unsigned numeric order.  */
-  if (fn1->address < fn2->address)
-    return -1;
-  if (fn1->address > fn2->address)
-    return 1;
-  return 0;
-}
+/* Process a symbol file, as either the main file or as a dynamically
+   loaded file.
+
+   NAME is the file name (which will be tilde-expanded and made
+   absolute herein) (but we don't free or modify NAME itself).
+   FROM_TTY says how verbose to be.  MAINLINE specifies whether this
+   is the main symbol file, or whether it's an extra symbol file such
+   as dynamically loaded code.  If !mainline, ADDR is the address
+   where the text segment was loaded.  If VERBO, the caller has printed
+   a verbose message about the symbol reading (and complaints can be
+   more terse about it).  */
 
-/* ARGSUSED */
 void
-discard_misc_bunches (foo)
-     int foo;
+syms_from_objfile (objfile, addr, mainline, verbo)
+     struct objfile *objfile;
+     CORE_ADDR addr;
+     int mainline;
+     int verbo;
 {
-  register struct misc_bunch *next;
+  struct section_offsets *section_offsets;
+  asection *lowest_sect;
+  struct cleanup *old_chain;
+
+  init_entry_point_info (objfile);
+  find_sym_fns (objfile);
 
-  while (misc_bunch)
+  /* Make sure that partially constructed symbol tables will be cleaned up
+     if an error occurs during symbol reading.  */
+  old_chain = make_cleanup (free_objfile, objfile);
+
+  if (mainline) 
     {
-      next = misc_bunch->next;
-      free (misc_bunch);
-      misc_bunch = next;
+      /* We will modify the main symbol table, make sure that all its users
+        will be cleaned up if an error occurs during symbol reading.  */
+      make_cleanup (clear_symtab_users, 0);
+
+      /* Since no error yet, throw away the old symbol table.  */
+
+      if (symfile_objfile != NULL)
+       {
+         free_objfile (symfile_objfile);
+         symfile_objfile = NULL;
+       }
+
+      /* Currently we keep symbols from the add-symbol-file command.
+        If the user wants to get rid of them, they should do "symbol-file"
+        without arguments first.  Not sure this is the best behavior
+        (PR 2207).  */
+
+      (*objfile -> sf -> sym_new_init) (objfile);
     }
-}
 
-/* INCLINK nonzero means bunches are from an incrementally-linked file.
-   Add them to the existing bunches.
-   Otherwise INCLINK is zero, and we start from scratch. */
-void
-condense_misc_bunches (inclink)
-     int inclink;
-{
-  register int i, j;
-  register struct misc_bunch *bunch;
+  /* Convert addr into an offset rather than an absolute address.
+     We find the lowest address of a loaded segment in the objfile,
+     and assume that <addr> is where that got loaded.  Due to historical
+     precedent, we warn if that doesn't happen to be the ".text"
+     segment.  */
 
-  if (inclink)
+  if (mainline)
     {
-      misc_function_vector
-       = (struct misc_function *)
-         xrealloc (misc_function_vector, (misc_count + misc_function_count)
-                   * sizeof (struct misc_function));
-      j = misc_function_count;
+      addr = 0;                /* No offset from objfile addresses.  */
     }
   else
     {
-      misc_function_vector
-       = (struct misc_function *)
-         xmalloc (misc_count * sizeof (struct misc_function));
-      j = 0;
+      lowest_sect = bfd_get_section_by_name (objfile->obfd, ".text");
+#if 0
+      lowest_sect = 0;
+      bfd_map_over_sections (objfile->obfd, find_lowest_section,
+                            (PTR) &lowest_sect);
+#endif
+
+      if (lowest_sect == 0)
+       warning ("no loadable sections found in added symbol-file %s",
+                objfile->name);
+      else if (0 == bfd_get_section_name (objfile->obfd, lowest_sect)
+              || !STREQ (".text",
+                             bfd_get_section_name (objfile->obfd, lowest_sect)))
+       warning ("Lowest section in %s is %s at 0x%lx",
+                objfile->name,
+                bfd_section_name (objfile->obfd, lowest_sect),
+                (unsigned long) bfd_section_vma (objfile->obfd, lowest_sect));
+
+      if (lowest_sect)
+       addr -= bfd_section_vma (objfile->obfd, lowest_sect);
     }
 
-  bunch = misc_bunch;
-  while (bunch)
+  /* Initialize symbol reading routines for this objfile, allow complaints to
+     appear for this new file, and record how verbose to be, then do the
+     initial symbol reading for this file. */
+
+  (*objfile -> sf -> sym_init) (objfile);
+  clear_complaints (1, verbo);
+
+  section_offsets = (*objfile -> sf -> sym_offsets) (objfile, addr);
+  objfile->section_offsets = section_offsets;
+
+#ifndef IBM6000_TARGET
+  /* This is a SVR4/SunOS specific hack, I think.  In any event, it
+     screws RS/6000.  sym_offsets should be doing this sort of thing,
+     because it knows the mapping between bfd sections and
+     section_offsets.  */
+  /* This is a hack.  As far as I can tell, section offsets are not
+     target dependent.  They are all set to addr with a couple of
+     exceptions.  The exceptions are sysvr4 shared libraries, whose
+     offsets are kept in solib structures anyway and rs6000 xcoff
+     which handles shared libraries in a completely unique way.
+
+     Section offsets are built similarly, except that they are built
+     by adding addr in all cases because there is no clear mapping
+     from section_offsets into actual sections.  Note that solib.c
+     has a different algorythm for finding section offsets.
+
+     These should probably all be collapsed into some target
+     independent form of shared library support.  FIXME.  */
+
+  if (addr)
     {
-      for (i = 0; i < misc_bunch_index; i++, j++)
-        {
-         misc_function_vector[j] = bunch->contents[i];
-#ifdef NAMES_HAVE_UNDERSCORE
-         if (misc_function_vector[j].name[0] == '_')
-             misc_function_vector[j].name++;
-#endif
+      struct obj_section *s;
+
+      for (s = objfile->sections; s < objfile->sections_end; ++s)
+       {
+         s->addr -= s->offset;
+         s->addr += addr;
+         s->endaddr -= s->offset;
+         s->endaddr += addr;
+         s->offset += addr;
        }
-      bunch = bunch->next;
-      misc_bunch_index = MISC_BUNCH_SIZE;
     }
+#endif /* not IBM6000_TARGET */
 
-  if (misc_function_count + misc_count != j)           /* DEBUG */
-    printf_filtered ("Function counts are off!  %d + %d != %d\n",
-      misc_function_count, misc_count, j);
+  (*objfile -> sf -> sym_read) (objfile, section_offsets, mainline);
 
-  misc_function_count = j;
+  /* Don't allow char * to have a typename (else would get caddr_t).
+     Ditto void *.  FIXME: Check whether this is now done by all the
+     symbol readers themselves (many of them now do), and if so remove
+     it from here.  */
 
-  /* Sort the misc functions by address.  */
+  TYPE_NAME (lookup_pointer_type (builtin_type_char)) = 0;
+  TYPE_NAME (lookup_pointer_type (builtin_type_void)) = 0;
 
-  qsort (misc_function_vector, misc_function_count,
-        sizeof (struct misc_function),
-        compare_misc_functions);
-}
+  /* Mark the objfile has having had initial symbol read attempted.  Note
+     that this does not mean we found any symbols... */
 
+  objfile -> flags |= OBJF_SYMS;
 
-/* Get the symbol table that corresponds to a partial_symtab.
-   This is fast after the first time you do it.  In fact, there
-   is an even faster macro PSYMTAB_TO_SYMTAB that does the fast
-   case inline.  */
+  /* Discard cleanups as symbol reading was successful.  */
 
-struct symtab *
-psymtab_to_symtab (pst)
-     register struct partial_symtab *pst;
+  discard_cleanups (old_chain);
+}
+
+/* Perform required actions after either reading in the initial
+   symbols for a new objfile, or mapping in the symbols from a reusable
+   objfile. */
+   
+void
+new_symfile_objfile (objfile, mainline, verbo)
+     struct objfile *objfile;
+     int mainline;
+     int verbo;
 {
-  register struct symtab *result;
 
-  /* If it's been looked up before, return it. */
-  if (pst->symtab)
-    return pst->symtab;
+  /* If this is the main symbol file we have to clean up all users of the
+     old main symbol file. Otherwise it is sufficient to fixup all the
+     breakpoints that may have been redefined by this symbol file.  */
+  if (mainline)
+    {
+      /* OK, make it the "real" symbol file.  */
+      symfile_objfile = objfile;
 
-  /* If it has not yet been read in, read it.  */
-  if (!pst->readin)
-    { 
-      (*pst->read_symtab) (pst);
+      clear_symtab_users ();
+    }
+  else
+    {
+      breakpoint_re_set ();
     }
 
-  /* Search through list for correct name. */
-  for (result = symtab_list; result; result = result->next)
-    if (!strcmp (result->filename, pst->filename))
-      {
-       pst->symtab = result;           /* Remember where it was.  */
-       return result;
-      }
-
-  return 0;
+  /* We're done reading the symbol file; finish off complaints.  */
+  clear_complaints (0, verbo);
 }
 
 /* Process a symbol file, as either the main file or as a dynamically
@@ -384,179 +520,262 @@ psymtab_to_symtab (pst)
    FROM_TTY says how verbose to be.  MAINLINE specifies whether this
    is the main symbol file, or whether it's an extra symbol file such
    as dynamically loaded code.  If !mainline, ADDR is the address
-   where the text segment was loaded.  */
+   where the text segment was loaded.
 
-void
-symbol_file_add (name, from_tty, addr, mainline)
+   Upon success, returns a pointer to the objfile that was added.
+   Upon failure, jumps back to command level (never returns). */
+
+struct objfile *
+symbol_file_add (name, from_tty, addr, mainline, mapped, readnow)
      char *name;
      int from_tty;
      CORE_ADDR addr;
      int mainline;
+     int mapped;
+     int readnow;
 {
-  bfd *sym_bfd;
-  asection *text_sect;
-  struct sym_fns *sf;
-  char *realname;
-
-  sym_bfd = symfile_open (name);
-
-  entry_point = bfd_get_start_address (sym_bfd);
-
-  if (mainline)
-    symfile_mtime = bfd_get_mtime (sym_bfd);
+  struct objfile *objfile;
+  struct partial_symtab *psymtab;
+  bfd *abfd;
 
-  /* There is a distinction between having no symbol table
-     (we refuse to read the file, leaving the old set of symbols around)
-     and having no debugging symbols in your symbol table (we read
-     the file and end up with a mostly empty symbol table).  */
+  /* Open a bfd for the file, and give user a chance to burp if we'd be
+     interactively wiping out any existing symbols.  */
 
-  if (!(bfd_get_file_flags (sym_bfd) & HAS_SYMS))
-    {
-      error ("%s has no symbol-table", name);
-    }
+  abfd = symfile_bfd_open (name);
 
-  if ((symtab_list || partial_symtab_list)
+  if ((have_full_symbols () || have_partial_symbols ())
       && mainline
       && from_tty
       && !query ("Load new symbol table from \"%s\"? ", name))
-    error ("Not confirmed.");
+      error ("Not confirmed.");
 
-  if (from_tty)
-    {
-      printf_filtered ("Reading symbol data from %s...", name);
-      wrap_here ("");
-      fflush (stdout);
-    }
+  objfile = allocate_objfile (abfd, mapped);
 
-  sf = symfile_init (sym_bfd);
-  realname = bfd_get_filename (sym_bfd);
-  realname = savestring (realname, strlen (realname));
-  /* FIXME, this probably creates a storage leak... */
+  /* If the objfile uses a mapped symbol file, and we have a psymtab for
+     it, then skip reading any symbols at this time. */
 
-  if (mainline) 
+  if ((objfile -> flags & OBJF_MAPPED) && (objfile -> flags & OBJF_SYMS))
     {
-      /* Since no error yet, throw away the old symbol table.  */
-
-      if (symfile)
-       free (symfile);
-      symfile = 0;
-      free_all_symtabs ();
-      free_all_psymtabs ();
-
-      (*sf->sym_new_init) ();
-
-      /* For mainline, caller didn't know the specified address of the
-         text section.  We fix that here.  */
-      text_sect = bfd_get_section_by_name (sym_bfd, ".text");
-      addr = bfd_section_vma (sym_bfd, text_sect);
+      /* We mapped in an existing symbol table file that already has had
+        initial symbol reading performed, so we can skip that part.  Notify
+        the user that instead of reading the symbols, they have been mapped.
+        */
+      if (from_tty || info_verbose)
+       {
+         printf_filtered ("Mapped symbols for %s...", name);
+         wrap_here ("");
+         gdb_flush (gdb_stdout);
+       }
+      init_entry_point_info (objfile);
+      find_sym_fns (objfile);
     }
+  else
+    {
+      /* We either created a new mapped symbol table, mapped an existing
+        symbol table file which has not had initial symbol reading
+        performed, or need to read an unmapped symbol table. */
+      if (from_tty || info_verbose)
+       {
+         printf_filtered ("Reading symbols from %s...", name);
+         wrap_here ("");
+         gdb_flush (gdb_stdout);
+       }
+      syms_from_objfile (objfile, addr, mainline, from_tty);
+    }      
 
-  clear_complaints();  /* Allow complaints to appear for this new file. */
-
-  (*sf->sym_read) (sf, addr, mainline);
-
-  /* Don't allow char * to have a typename (else would get caddr_t.)  */
-  /* Ditto void *.  FIXME should do this for all the builtin types.  */
-
-  TYPE_NAME (lookup_pointer_type (builtin_type_char)) = 0;
-  TYPE_NAME (lookup_pointer_type (builtin_type_void)) = 0;
+  /* We now have at least a partial symbol table.  Check to see if the
+     user requested that all symbols be read on initial access via either
+     the gdb startup command line or on a per symbol file basis.  Expand
+     all partial symbol tables for this objfile if so. */
 
-  if (mainline)
+  if (readnow || readnow_symbol_files)
     {
-      /* OK, make it the "real" symbol file.  */
-      symfile = realname;
-      symfile_fns = sf;
-    }
+      if (from_tty || info_verbose)
+       {
+         printf_filtered ("expanding to full symbols...");
+         wrap_here ("");
+         gdb_flush (gdb_stdout);
+       }
 
-  /* If we have wiped out any old symbol tables, clean up.  */
-  clear_symtab_users_once ();
+      for (psymtab = objfile -> psymtabs;
+          psymtab != NULL;
+          psymtab = psymtab -> next)
+       {
+         psymtab_to_symtab (psymtab);
+       }
+    }
 
-  if (from_tty)
+  if (from_tty || info_verbose)
     {
       printf_filtered ("done.\n");
-      fflush (stdout);
+      gdb_flush (gdb_stdout);
     }
+
+  new_symfile_objfile (objfile, mainline, from_tty);
+      
+  /* Getting new symbols may change our opinion about what is
+     frameless.  */
+
+  reinit_frame_cache ();
+
+  return (objfile);
 }
 
 /* This is the symbol-file command.  Read the file, analyze its symbols,
-   and add a struct symtab to symtab_list.  */
+   and add a struct symtab to a symtab list.  */
 
 void
-symbol_file_command (name, from_tty)
-     char *name;
+symbol_file_command (args, from_tty)
+     char *args;
      int from_tty;
 {
+  char **argv;
+  char *name = NULL;
+  struct cleanup *cleanups;
+  int mapped = 0;
+  int readnow = 0;
 
   dont_repeat ();
 
-  if (name == 0)
+  if (args == NULL)
     {
-      if ((symtab_list || partial_symtab_list)
+      if ((have_full_symbols () || have_partial_symbols ())
          && from_tty
-         && !query ("Discard symbol table from `%s'? ", symfile))
+         && !query ("Discard symbol table from `%s'? ",
+                    symfile_objfile -> name))
        error ("Not confirmed.");
-      if (symfile)
-       free (symfile);
-      symfile = 0;
-      free_all_symtabs ();
-      free_all_psymtabs ();
-      /* FIXME, this does not account for the main file and subsequent
-         files (shared libs, dynloads, etc) having different formats. 
-         It only calls the cleanup routine for the main file's format.  */
-      if (symfile_fns) {
-        (*symfile_fns->sym_new_init) ();
-        free (symfile_fns);
-        symfile_fns = 0;
-      }
-      return;
+      free_all_objfiles ();
+      symfile_objfile = NULL;
+      if (from_tty)
+       {
+         printf_unfiltered ("No symbol file now.\n");
+       }
     }
+  else
+    {
+      if ((argv = buildargv (args)) == NULL)
+       {
+         nomem (0);
+       }
+      cleanups = make_cleanup (freeargv, (char *) argv);
+      while (*argv != NULL)
+       {
+         if (STREQ (*argv, "-mapped"))
+           {
+             mapped = 1;
+           }
+         else if (STREQ (*argv, "-readnow"))
+           {
+             readnow = 1;
+           }
+         else if (**argv == '-')
+           {
+             error ("unknown option `%s'", *argv);
+           }
+         else
+           {
+             name = *argv;
+           }
+         argv++;
+       }
 
-  symbol_file_add (name, from_tty, (CORE_ADDR)0, 1);
+      if (name == NULL)
+       {
+         error ("no symbol file name was specified");
+       }
+      else
+       {
+         symbol_file_add (name, from_tty, (CORE_ADDR)0, 1, mapped, readnow);
+         set_initial_language ();
+       }
+      do_cleanups (cleanups);
+    }
 }
 
-/* Open NAME and hand it off to BFD for preliminary analysis.  Result
-   is a BFD *, which includes a new copy of NAME dynamically allocated
-   (which will be freed by the cleanup chain).  In case of trouble,
-   error() is called.  */
+/* Set the initial language.
+
+   A better solution would be to record the language in the psymtab when reading
+   partial symbols, and then use it (if known) to set the language.  This would
+   be a win for formats that encode the language in an easily discoverable place,
+   such as DWARF.  For stabs, we can jump through hoops looking for specially
+   named symbols or try to intuit the language from the specific type of stabs
+   we find, but we can't do that until later when we read in full symbols.
+   FIXME.  */
+
+static void
+set_initial_language ()
+{
+  struct partial_symtab *pst;
+  enum language lang = language_unknown;       
+
+  pst = find_main_psymtab ();
+  if (pst != NULL)
+    {
+      if (pst -> filename != NULL)
+       {
+         lang = deduce_language_from_filename (pst -> filename);
+        }
+      if (lang == language_unknown)
+       {
+           /* Make C the default language */
+           lang = language_c;
+       }
+      set_language (lang);
+      expected_language = current_language;    /* Don't warn the user */
+    }
+}
+
+/* Open file specified by NAME and hand it off to BFD for preliminary
+   analysis.  Result is a newly initialized bfd *, which includes a newly
+   malloc'd` copy of NAME (tilde-expanded and made absolute).
+   In case of trouble, error() is called.  */
 
 static bfd *
-symfile_open (name)
+symfile_bfd_open (name)
      char *name;
 {
   bfd *sym_bfd;
   int desc;
   char *absolute_name;
 
-  name = tilde_expand (name);
-  make_cleanup (free, name);
+  name = tilde_expand (name);  /* Returns 1st new malloc'd copy */
 
-  desc = openp (getenv ("PATH"), 1, name, O_RDONLY, 0, &absolute_name);
+  /* Look down path for it, allocate 2nd new malloc'd copy.  */
+  desc = openp (getenv ("PATH"), 1, name, O_RDONLY | O_BINARY, 0, &absolute_name);
   if (desc < 0)
-    perror_with_name (name);
-  else
     {
-      make_cleanup (free, absolute_name);
-      name = absolute_name;
+      make_cleanup (free, name);
+      perror_with_name (name);
     }
+  free (name);                 /* Free 1st new malloc'd copy */
+  name = absolute_name;                /* Keep 2nd malloc'd copy in bfd */
+                               /* It'll be freed in free_objfile(). */
 
-  sym_bfd = bfd_fdopenr (name, NULL, desc);
+  sym_bfd = bfd_fdopenr (name, gnutarget, desc);
   if (!sym_bfd)
     {
       close (desc);
-      error ("Could not open `%s' to read symbols: %s",
-            name, bfd_errmsg (bfd_error));
+      make_cleanup (free, name);
+      error ("\"%s\": can't open to read symbols: %s.", name,
+            bfd_errmsg (bfd_error));
     }
-  make_cleanup (bfd_close, sym_bfd);
+  sym_bfd->cacheable = true;
 
   if (!bfd_check_format (sym_bfd, bfd_object))
-    error ("\"%s\": can't read symbols: %s.",
-          name, bfd_errmsg (bfd_error));
+    {
+      bfd_close (sym_bfd);     /* This also closes desc */
+      make_cleanup (free, name);
+      error ("\"%s\": can't read symbols: %s.", name,
+            bfd_errmsg (bfd_error));
+    }
 
-  return sym_bfd;
+  return (sym_bfd);
 }
 
-/* Link a new symtab_fns into the global symtab_fns list.
-   Called by various _initialize routines.  */
+/* Link a new symtab_fns into the global symtab_fns list.  Called on gdb
+   startup by the _initialize routine in each object file format reader,
+   to register information about each format the the reader is prepared
+   to handle. */
 
 void
 add_symtab_fns (sf)
@@ -568,34 +787,36 @@ add_symtab_fns (sf)
 
 
 /* Initialize to read symbols from the symbol file sym_bfd.  It either
-   returns or calls error().  The result is a malloc'd struct sym_fns
-   that contains cached information about the symbol file.  */
+   returns or calls error().  The result is an initialized struct sym_fns
+   in the objfile structure, that contains cached information about the
+   symbol file.  */
 
-static struct sym_fns *
-symfile_init (sym_bfd)
-     bfd *sym_bfd;
+static void
+find_sym_fns (objfile)
+     struct objfile *objfile;
 {
-  struct sym_fns *sf, *sf2;
+  struct sym_fns *sf;
+  enum bfd_flavour our_flavour = bfd_get_flavour (objfile -> obfd);
 
-  for (sf = symtab_fns; sf != NULL; sf = sf->next)
+  /* Special kludge for RS/6000.  See xcoffread.c.  */
+  if (STREQ (bfd_get_target (objfile -> obfd), "aixcoff-rs6000"))
+    our_flavour = (enum bfd_flavour)-1;
+
+  for (sf = symtab_fns; sf != NULL; sf = sf -> next)
     {
-      if (!strncmp (bfd_get_target (sym_bfd), sf->sym_name, sf->sym_namelen))
+      if (our_flavour == sf -> sym_flavour)
        {
-         sf2 = (struct sym_fns *)xmalloc (sizeof (*sf2));      
-         /* FIXME, who frees this? */
-         *sf2 = *sf;
-         sf2->sym_bfd = sym_bfd;
-         sf2->sym_private = 0;                 /* Not alloc'd yet */
-         (*sf2->sym_init) (sf2);
-         return sf2;
+         objfile -> sf = sf;
+         return;
        }
     }
-  error ("I'm sorry, Dave, I can't do that.  Symbol format unknown.");
+  error ("I'm sorry, Dave, I can't do that.  Symbol format `%s' unknown.",
+        bfd_get_target (objfile -> obfd));
 }
 \f
 /* This function runs the load command of our current target.  */
 
-void
+static void
 load_command (arg, from_tty)
      char *arg;
      int from_tty;
@@ -603,136 +824,445 @@ load_command (arg, from_tty)
   target_load (arg, from_tty);
 }
 
-/* This function runs the add_syms command of our current target.  */
+/* This version of "load" should be usable for any target.  Currently
+   it is just used for remote targets, not inftarg.c or core files,
+   on the theory that only in that case is it useful.
 
+   Avoiding xmodem and the like seems like a win (a) because we don't have
+   to worry about finding it, and (b) On VMS, fork() is very slow and so
+   we don't want to run a subprocess.  On the other hand, I'm not sure how
+   performance compares.  */
 void
-add_symbol_file_command (args, from_tty)
-     char *args;
-     int from_tty;
+generic_load (filename, from_tty)
+    char *filename;
+    int from_tty;
 {
-  target_add_syms (args, from_tty);
+  struct cleanup *old_cleanups;
+  asection *s;
+  bfd *loadfile_bfd = bfd_openr (filename, gnutarget);
+  if (loadfile_bfd == NULL)
+    {
+      perror_with_name (filename);
+      return;
+    }
+  old_cleanups = make_cleanup (bfd_close, loadfile_bfd);
+
+  if (!bfd_check_format (loadfile_bfd, bfd_object)) 
+    {
+      error ("\"%s\" is not an object file: %s", filename,
+            bfd_errmsg (bfd_error));
+    }
+  
+  for (s = loadfile_bfd->sections; s; s = s->next) 
+    {
+      if (s->flags & SEC_LOAD) 
+       {
+         bfd_size_type size;
+
+         size = bfd_get_section_size_before_reloc (s);
+         if (size > 0)
+           {
+             char *buffer;
+             struct cleanup *old_chain;
+             bfd_vma vma;
+
+             buffer = xmalloc (size);
+             old_chain = make_cleanup (free, buffer);
+
+             vma = bfd_get_section_vma (loadfile_bfd, s);
+
+             /* Is this really necessary?  I guess it gives the user something
+                to look at during a long download.  */
+             printf_filtered ("Loading section %s, size 0x%lx vma 0x%lx\n",
+                              bfd_get_section_name (loadfile_bfd, s),
+                              (unsigned long) size, (unsigned long) vma);
+
+             bfd_get_section_contents (loadfile_bfd, s, buffer, 0, size);
+
+             target_write_memory (vma, buffer, size);
+
+             do_cleanups (old_chain);
+           }
+       }
+    }
+
+  /* We were doing this in remote-mips.c, I suspect it is right
+     for other targets too.  */
+  write_pc (loadfile_bfd->start_address);
+
+  /* FIXME: are we supposed to call symbol_file_add or not?  According to
+     a comment from remote-mips.c (where a call to symbol_file_add was
+     commented out), making the call confuses GDB if more than one file is
+     loaded in.  remote-nindy.c had no call to symbol_file_add, but remote-vx.c
+     does.  */
+
+  do_cleanups (old_cleanups);
 }
 
-/* This function allows the addition of incrementally linked object files.  */
+/* This function allows the addition of incrementally linked object files.
+   It does not modify any state in the target, only in the debugger.  */
 
-void
-add_syms_addr_command (arg_string, from_tty)
-     char* arg_string;
+/* ARGSUSED */
+static void
+add_symbol_file_command (args, from_tty)
+     char *args;
      int from_tty;
 {
-  char *name;
+  char *name = NULL;
   CORE_ADDR text_addr;
+  char *arg;
+  int readnow = 0;
+  int mapped = 0;
   
-  if (arg_string == 0)
-    error ("add-symbol-file takes a file name and an address");
+  dont_repeat ();
 
-  arg_string = tilde_expand (arg_string);
-  make_cleanup (free, arg_string);
+  if (args == NULL)
+    {
+      error ("add-symbol-file takes a file name and an address");
+    }
 
-  for( ; *arg_string == ' '; arg_string++ );
-  name = arg_string;
-  for( ; *arg_string && *arg_string != ' ' ; arg_string++ );
-  *arg_string++ = (char) 0;
+  /* Make a copy of the string that we can safely write into. */
 
-  if (name[0] == 0)
-    error ("add-symbol-file takes a file name and an address");
+  args = strdup (args);
+  make_cleanup (free, args);
 
-  text_addr = parse_and_eval_address (arg_string);
+  /* Pick off any -option args and the file name. */
 
-  dont_repeat ();
+  while ((*args != '\000') && (name == NULL))
+    {
+      while (isspace (*args)) {args++;}
+      arg = args;
+      while ((*args != '\000') && !isspace (*args)) {args++;}
+      if (*args != '\000')
+       {
+         *args++ = '\000';
+       }
+      if (*arg != '-')
+       {
+         name = arg;
+       }
+      else if (STREQ (arg, "-mapped"))
+       {
+         mapped = 1;
+       }
+      else if (STREQ (arg, "-readnow"))
+       {
+         readnow = 1;
+       }
+      else
+       {
+         error ("unknown option `%s'", arg);
+       }
+    }
+
+  /* After picking off any options and the file name, args should be
+     left pointing at the remainder of the command line, which should
+     be the address expression to evaluate. */
+
+  if ((name == NULL) || (*args == '\000') )
+    {
+      error ("add-symbol-file takes a file name and an address");
+    }
+  name = tilde_expand (name);
+  make_cleanup (free, name);
 
-  if (!query ("add symbol table from file \"%s\" at text_addr = 0x%x\n",
-             name, text_addr))
+  text_addr = parse_and_eval_address (args);
+
+  if (!query ("add symbol table from file \"%s\" at text_addr = %s?\n",
+             name, local_hex_string ((unsigned long)text_addr)))
     error ("Not confirmed.");
 
-  symbol_file_add (name, 0, text_addr, 0);
+  symbol_file_add (name, 0, text_addr, 0, mapped, readnow);
 }
 \f
-/* Re-read symbols if the symbol-file has changed.  */
+/* Re-read symbols if a symbol-file has changed.  */
 void
 reread_symbols ()
 {
-  struct stat symstat;
+  struct objfile *objfile;
+  long new_modtime;
+  int reread_one = 0;
+  struct stat new_statbuf;
+  int res;
 
   /* With the addition of shared libraries, this should be modified,
      the load time should be saved in the partial symbol tables, since
      different tables may come from different source files.  FIXME.
      This routine should then walk down each partial symbol table
-     and see if the symbol table that it originates from has been changed
-  */
+     and see if the symbol table that it originates from has been changed */
 
-  if (stat (symfile, &symstat) < 0)
-    /* Can't read symbol-file.  Assume it is up to date.  */
-    return;
+  for (objfile = object_files; objfile; objfile = objfile->next) {
+    if (objfile->obfd) {
+#ifdef IBM6000_TARGET
+     /* If this object is from a shared library, then you should
+        stat on the library name, not member name. */
 
-  if (symstat.st_mtime > symfile_mtime)
-    {
-      printf_filtered ("Symbol file has changed; re-reading symbols.\n");
-      symbol_file_command (symfile, 0);
-      breakpoint_re_set ();
+     if (objfile->obfd->my_archive)
+       res = stat (objfile->obfd->my_archive->filename, &new_statbuf);
+     else
+#endif
+      res = stat (objfile->name, &new_statbuf);
+      if (res != 0) {
+       /* FIXME, should use print_sys_errmsg but it's not filtered. */
+       printf_filtered ("`%s' has disappeared; keeping its symbols.\n",
+                        objfile->name);
+       continue;
+      }
+      new_modtime = new_statbuf.st_mtime;
+      if (new_modtime != objfile->mtime)
+       {
+         struct cleanup *old_cleanups;
+         struct section_offsets *offsets;
+         int num_offsets;
+         int section_offsets_size;
+
+         printf_filtered ("`%s' has changed; re-reading symbols.\n",
+                          objfile->name);
+
+         /* There are various functions like symbol_file_add,
+            symfile_bfd_open, syms_from_objfile, etc., which might
+            appear to do what we want.  But they have various other
+            effects which we *don't* want.  So we just do stuff
+            ourselves.  We don't worry about mapped files (for one thing,
+            any mapped file will be out of date).  */
+
+         /* If we get an error, blow away this objfile (not sure if
+            that is the correct response for things like shared
+            libraries).  */
+         old_cleanups = make_cleanup (free_objfile, objfile);
+         /* We need to do this whenever any symbols go away.  */
+         make_cleanup (clear_symtab_users, 0);
+
+         /* Clean up any state BFD has sitting around.  We don't need
+            to close the descriptor but BFD lacks a way of closing the
+            BFD without closing the descriptor.  */
+         if (!bfd_close (objfile->obfd))
+           error ("Can't close BFD for %s.", objfile->name);
+         objfile->obfd = bfd_openr (objfile->name, gnutarget);
+         if (objfile->obfd == NULL)
+           error ("Can't open %s to read symbols.", objfile->name);
+         /* bfd_openr sets cacheable to true, which is what we want.  */
+         if (!bfd_check_format (objfile->obfd, bfd_object))
+           error ("Can't read symbols from %s: %s.", objfile->name,
+                  bfd_errmsg (bfd_error));
+
+         /* Save the offsets, we will nuke them with the rest of the
+            psymbol_obstack.  */
+         num_offsets = objfile->num_sections;
+         section_offsets_size =
+           sizeof (struct section_offsets)
+             + sizeof (objfile->section_offsets->offsets) * num_offsets;
+         offsets = (struct section_offsets *) alloca (section_offsets_size);
+         memcpy (offsets, objfile->section_offsets, section_offsets_size);
+
+         /* Nuke all the state that we will re-read.  Much of the following
+            code which sets things to NULL really is necessary to tell
+            other parts of GDB that there is nothing currently there.  */
+
+         /* FIXME: Do we have to free a whole linked list, or is this
+            enough?  */
+         if (objfile->global_psymbols.list)
+           mfree (objfile->md, objfile->global_psymbols.list);
+         objfile->global_psymbols.list = NULL;
+         objfile->global_psymbols.size = 0;
+         if (objfile->static_psymbols.list)
+           mfree (objfile->md, objfile->static_psymbols.list);
+         objfile->static_psymbols.list = NULL;
+         objfile->static_psymbols.size = 0;
+
+         /* Free the obstacks for non-reusable objfiles */
+         obstack_free (&objfile -> psymbol_obstack, 0);
+         obstack_free (&objfile -> symbol_obstack, 0);
+         obstack_free (&objfile -> type_obstack, 0);
+         objfile->sections = NULL;
+         objfile->symtabs = NULL;
+         objfile->psymtabs = NULL;
+         objfile->free_psymtabs = NULL;
+         objfile->msymbols = NULL;
+         objfile->minimal_symbol_count= 0;
+         objfile->fundamental_types = NULL;
+         if (objfile -> sf != NULL)
+           {
+             (*objfile -> sf -> sym_finish) (objfile);
+           }
+
+         /* We never make this a mapped file.  */
+         objfile -> md = NULL;
+         /* obstack_specify_allocation also initializes the obstack so
+            it is empty.  */
+         obstack_specify_allocation (&objfile -> psymbol_obstack, 0, 0,
+                                     xmalloc, free);
+         obstack_specify_allocation (&objfile -> symbol_obstack, 0, 0,
+                                     xmalloc, free);
+         obstack_specify_allocation (&objfile -> type_obstack, 0, 0,
+                                     xmalloc, free);
+         if (build_objfile_section_table (objfile))
+           {
+             error ("Can't find the file sections in `%s': %s", 
+                    objfile -> name, bfd_errmsg (bfd_error));
+           }
+
+         /* We use the same section offsets as from last time.  I'm not
+            sure whether that is always correct for shared libraries.  */
+         objfile->section_offsets = (struct section_offsets *)
+           obstack_alloc (&objfile -> psymbol_obstack, section_offsets_size);
+         memcpy (objfile->section_offsets, offsets, section_offsets_size);
+         objfile->num_sections = num_offsets;
+
+         /* What the hell is sym_new_init for, anyway?  The concept of
+            distinguishing between the main file and additional files
+            in this way seems rather dubious.  */
+         if (objfile == symfile_objfile)
+           (*objfile->sf->sym_new_init) (objfile);
+
+         (*objfile->sf->sym_init) (objfile);
+         clear_complaints (1, 1);
+         /* The "mainline" parameter is a hideous hack; I think leaving it
+            zero is OK since dbxread.c also does what it needs to do if
+            objfile->global_psymbols.size is 0.  */
+         (*objfile->sf->sym_read) (objfile, objfile->section_offsets, 0);
+         objfile -> flags |= OBJF_SYMS;
+
+         /* We're done reading the symbol file; finish off complaints.  */
+         clear_complaints (0, 1);
+
+         /* Getting new symbols may change our opinion about what is
+            frameless.  */
+
+         reinit_frame_cache ();
+
+         /* Discard cleanups as symbol reading was successful.  */
+         discard_cleanups (old_cleanups);
+
+         /* If the mtime has changed between the time we set new_modtime
+            and now, we *want* this to be out of date, so don't call stat
+            again now.  */
+         objfile->mtime = new_modtime;
+         reread_one = 1;
+       }
     }
-}
+  }
 
+  if (reread_one)
+    clear_symtab_users ();
+}
 
-/* This function is really horrible, but to avoid it, there would need
-   to be more filling in of forward references.  */
-int
-fill_in_vptr_fieldno (type)
-     struct type *type;
+\f
+enum language
+deduce_language_from_filename (filename)
+     char *filename;
 {
-  check_stub_type (type);
-  if (TYPE_VPTR_FIELDNO (type) < 0)
-    TYPE_VPTR_FIELDNO (type) =
-      fill_in_vptr_fieldno (TYPE_BASECLASS (type, 1));
-  return TYPE_VPTR_FIELDNO (type);
+  char *c;
+  
+  if (0 == filename) 
+    ; /* Get default */
+  else if (0 == (c = strrchr (filename, '.')))
+    ; /* Get default. */
+  else if (STREQ(c,".mod"))
+    return language_m2;
+  else if (STREQ(c,".c"))
+    return language_c;
+  else if (STREQ (c,".cc") || STREQ (c,".C") || STREQ (c, ".cxx"))
+    return language_cplus;
+  else if (STREQ (c,".ch") || STREQ (c,".c186") || STREQ (c,".c286"))
+    return language_chill;
+
+  return language_unknown;             /* default */
 }
 \f
-/* Functions to handle complaints during symbol reading.  */
+/* allocate_symtab:
 
-/* How many complaints about a particular thing should be printed before
-   we stop whining about it?  */
+   Allocate and partly initialize a new symbol table.  Return a pointer
+   to it.  error() if no space.
 
-static unsigned stop_whining = 1;
+   Caller must set these fields:
+       LINETABLE(symtab)
+       symtab->blockvector
+       symtab->dirname
+       symtab->free_code
+       symtab->free_ptr
+       initialize any EXTRA_SYMTAB_INFO
+       possibly free_named_symtabs (symtab->filename);
+ */
 
-/* Print a complaint about the input symbols, and link the complaint block
-   into a chain for later handling.  Result is 1 if the complaint was
-   printed, 0 if it was suppressed.  */
+struct symtab *
+allocate_symtab (filename, objfile)
+     char *filename;
+     struct objfile *objfile;
+{
+  register struct symtab *symtab;
 
-int
-complain (complaint, val)
-     struct complaint *complaint;
-     char *val;
-{
-  complaint->counter++;
-  if (complaint->next == 0) {
-    complaint->next = complaint_root->next;
-    complaint_root->next = complaint;
-  }
-  if (complaint->counter > stop_whining)
-    return 0;
-  wrap_here ("");
-  if (!info_verbose) {
-    puts_filtered ("During symbol reading...");
-  }
-  printf_filtered (complaint->message, val);
-  puts_filtered ("...");
-  wrap_here("");
-  if (!info_verbose)
-    puts_filtered ("\n");
-  return 1;
-}
+  symtab = (struct symtab *)
+    obstack_alloc (&objfile -> symbol_obstack, sizeof (struct symtab));
+  memset (symtab, 0, sizeof (*symtab));
+  symtab -> filename = obsavestring (filename, strlen (filename),
+                                    &objfile -> symbol_obstack);
+  symtab -> fullname = NULL;
+  symtab -> language = deduce_language_from_filename (filename);
 
-/* Clear out all complaint counters that have ever been incremented.  */
+  /* Hook it to the objfile it comes from */
 
-void
-clear_complaints ()
+  symtab -> objfile = objfile;
+  symtab -> next = objfile -> symtabs;
+  objfile -> symtabs = symtab;
+
+#ifdef INIT_EXTRA_SYMTAB_INFO
+  INIT_EXTRA_SYMTAB_INFO (symtab);
+#endif
+
+  return (symtab);
+}
+
+struct partial_symtab *
+allocate_psymtab (filename, objfile)
+     char *filename;
+     struct objfile *objfile;
 {
-  struct complaint *p;
+  struct partial_symtab *psymtab;
 
-  for (p = complaint_root->next; p != complaint_root; p = p->next)
-    p->counter = 0;
+  if (objfile -> free_psymtabs)
+    {
+      psymtab = objfile -> free_psymtabs;
+      objfile -> free_psymtabs = psymtab -> next;
+    }
+  else
+    psymtab = (struct partial_symtab *)
+      obstack_alloc (&objfile -> psymbol_obstack,
+                    sizeof (struct partial_symtab));
+
+  memset (psymtab, 0, sizeof (struct partial_symtab));
+  psymtab -> filename = obsavestring (filename, strlen (filename),
+                                     &objfile -> psymbol_obstack);
+  psymtab -> symtab = NULL;
+
+  /* Hook it to the objfile it comes from */
+
+  psymtab -> objfile = objfile;
+  psymtab -> next = objfile -> psymtabs;
+  objfile -> psymtabs = psymtab;
+  
+  return (psymtab);
 }
+
 \f
+/* Reset all data structures in gdb which may contain references to symbol
+   table date.  */
+
+void
+clear_symtab_users ()
+{
+  /* Someday, we should do better than this, by only blowing away
+     the things that really need to be blown.  */
+  clear_value_history ();
+  clear_displays ();
+  clear_internalvars ();
+  breakpoint_re_set ();
+  set_default_breakpoint (0, 0, 0, 0);
+  current_source_symtab = 0;
+  current_source_line = 0;
+  clear_pc_function_cache ();
+}
+
 /* clear_symtab_users_once:
 
    This function is run after symbol reading, or from a cleanup.
@@ -758,6 +1288,12 @@ clear_complaints ()
    reading, is because the cleanup protects us in case of errors, but is
    discarded if symbol reading is successful.  */
 
+#if 0
+/* FIXME:  As free_named_symtabs is currently a big noop this function
+   is no longer needed.  */
+static void
+clear_symtab_users_once PARAMS ((void));
+
 static int clear_symtab_users_queued;
 static int clear_symtab_users_done;
 
@@ -769,28 +1305,21 @@ clear_symtab_users_once ()
     return;
   clear_symtab_users_done = clear_symtab_users_queued;
 
-  printf ("Resetting debugger state after updating old symbol tables\n");
-
-  /* Someday, we should do better than this, by only blowing away
-     the things that really need to be blown.  */
-  clear_value_history ();
-  clear_displays ();
-  clear_internalvars ();
-  breakpoint_re_set ();
-  set_default_breakpoint (0, 0, 0, 0);
-  current_source_symtab = 0;
+  clear_symtab_users ();
 }
+#endif
 
 /* Delete the specified psymtab, and any others that reference it.  */
 
+static void
 cashier_psymtab (pst)
      struct partial_symtab *pst;
 {
-  struct partial_symtab *ps, *pprev;
+  struct partial_symtab *ps, *pprev = NULL;
   int i;
 
   /* Find its previous psymtab in the chain */
-  for (ps = partial_symtab_list; ps; ps = ps->next) {
+  for (ps = pst->objfile->psymtabs; ps; ps = ps->next) {
     if (ps == pst)
       break;
     pprev = ps;
@@ -798,8 +1327,8 @@ cashier_psymtab (pst)
 
   if (ps) {
     /* Unhook it from the chain.  */
-    if (ps == partial_symtab_list)
-      partial_symtab_list = ps->next;
+    if (ps == pst->objfile->psymtabs)
+      pst->objfile->psymtabs = ps->next;
     else
       pprev->next = ps->next;
 
@@ -811,7 +1340,7 @@ cashier_psymtab (pst)
 
     /* We need to cashier any psymtab that has this one as a dependency... */
 again:
-    for (ps = partial_symtab_list; ps; ps = ps->next) {
+    for (ps = pst->objfile->psymtabs; ps; ps = ps->next) {
       for (i = 0; i < ps->number_of_dependencies; i++) {
        if (ps->dependencies[i] == pst) {
          cashier_psymtab (ps);
@@ -829,6 +1358,7 @@ again:
    it is not called for subsidiary files such as .h files.
 
    Return value is 1 if we blew away the environment, 0 if not.
+   FIXME.  The return valu appears to never be used.
 
    FIXME.  I think this is not the best way to do this.  We should
    work on being gentler to the environment while still cleaning up
@@ -838,18 +1368,35 @@ int
 free_named_symtabs (name)
      char *name;
 {
+#if 0
+  /* FIXME:  With the new method of each objfile having it's own
+     psymtab list, this function needs serious rethinking.  In particular,
+     why was it ever necessary to toss psymtabs with specific compilation
+     unit filenames, as opposed to all psymtabs from a particular symbol
+     file?  -- fnf
+     Well, the answer is that some systems permit reloading of particular
+     compilation units.  We want to blow away any old info about these
+     compilation units, regardless of which objfiles they arrived in. --gnu.  */
+
   register struct symtab *s;
   register struct symtab *prev;
   register struct partial_symtab *ps;
-  register struct partial_symtab *pprev;
   struct blockvector *bv;
   int blewit = 0;
 
+  /* We only wack things if the symbol-reload switch is set.  */
+  if (!symbol_reloading)
+    return 0;
+
+  /* Some symbol formats have trouble providing file names... */
+  if (name == 0 || *name == '\0')
+    return 0;
+
   /* Look for a psymtab with the specified name.  */
 
 again2:
   for (ps = partial_symtab_list; ps; ps = ps->next) {
-    if (!strcmp (name, ps->filename)) {
+    if (STREQ (name, ps->filename)) {
       cashier_psymtab (ps);    /* Blow it away...and its little dog, too.  */
       goto again2;             /* Must restart, chain has been munged */
     }
@@ -859,7 +1406,7 @@ again2:
 
   for (s = symtab_list; s; s = s->next)
     {
-      if (!strcmp (name, s->filename))
+      if (STREQ (name, s->filename))
        break;
       prev = s;
     }
@@ -881,8 +1428,8 @@ again2:
         contain the pathname of the object file.  (This problem
         has been fixed in GDB 3.9x).  */
 
-      bv = BLOCKLIST (s);
-      if (BLOCKLIST_NBLOCKS (bv) > 2
+      bv = BLOCKVECTOR (s);
+      if (BLOCKVECTOR_NBLOCKS (bv) > 2
          || BLOCK_NSYMS (BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK))
          || BLOCK_NSYMS (BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK)))
        {
@@ -898,42 +1445,156 @@ again2:
       free_symtab (s);
     }
   else
-    /* It is still possible that some breakpoints will be affected
-       even though no symtab was found, since the file might have
-       been compiled without debugging, and hence not be associated
-       with a symtab.  In order to handle this correctly, we would need
-       to keep a list of text address ranges for undebuggable files.
-       For now, we do nothing, since this is a fairly obscure case.  */
-    ;
-
-  /* FIXME, what about the misc function vector? */
+    {
+      /* It is still possible that some breakpoints will be affected
+        even though no symtab was found, since the file might have
+        been compiled without debugging, and hence not be associated
+        with a symtab.  In order to handle this correctly, we would need
+        to keep a list of text address ranges for undebuggable files.
+        For now, we do nothing, since this is a fairly obscure case.  */
+      ;
+    }
+
+  /* FIXME, what about the minimal symbol table? */
   return blewit;
+#else
+  return (0);
+#endif
+}
+\f
+/* Allocate and partially fill a partial symtab.  It will be
+   completely filled at the end of the symbol list.
+
+   SYMFILE_NAME is the name of the symbol-file we are reading from, and ADDR
+   is the address relative to which its symbols are (incremental) or 0
+   (normal). */
+
+
+struct partial_symtab *
+start_psymtab_common (objfile, section_offsets,
+                     filename, textlow, global_syms, static_syms)
+     struct objfile *objfile;
+     struct section_offsets *section_offsets;
+     char *filename;
+     CORE_ADDR textlow;
+     struct partial_symbol *global_syms;
+     struct partial_symbol *static_syms;
+{
+  struct partial_symtab *psymtab;
+
+  psymtab = allocate_psymtab (filename, objfile);
+  psymtab -> section_offsets = section_offsets;
+  psymtab -> textlow = textlow;
+  psymtab -> texthigh = psymtab -> textlow;  /* default */
+  psymtab -> globals_offset = global_syms - objfile -> global_psymbols.list;
+  psymtab -> statics_offset = static_syms - objfile -> static_psymbols.list;
+  return (psymtab);
+}
+\f
+/* Debugging versions of functions that are usually inline macros
+   (see symfile.h).  */
+
+#if !INLINE_ADD_PSYMBOL
+
+/* Add a symbol with a long value to a psymtab.
+   Since one arg is a struct, we pass in a ptr and deref it (sigh).  */
+
+void
+add_psymbol_to_list (name, namelength, namespace, class, list, val, language,
+                    objfile)
+     char *name;
+     int namelength;
+     enum namespace namespace;
+     enum address_class class;
+     struct psymbol_allocation_list *list;
+     long val;
+     enum language language;
+     struct objfile *objfile;
+{
+  register struct partial_symbol *psym;
+  register char *demangled_name;
+
+  if (list->next >= list->list + list->size)
+    {
+      extend_psymbol_list (list,objfile);
+    }
+  psym = list->next++;
+  
+  SYMBOL_NAME (psym) =
+    (char *) obstack_alloc (&objfile->psymbol_obstack, namelength + 1);
+  memcpy (SYMBOL_NAME (psym), name, namelength);
+  SYMBOL_NAME (psym)[namelength] = '\0';
+  SYMBOL_VALUE (psym) = val;
+  SYMBOL_LANGUAGE (psym) = language;
+  PSYMBOL_NAMESPACE (psym) = namespace;
+  PSYMBOL_CLASS (psym) = class;
+  SYMBOL_INIT_DEMANGLED_NAME (psym, &objfile->psymbol_obstack);
+}
+
+/* Add a symbol with a CORE_ADDR value to a psymtab. */
+
+void
+add_psymbol_addr_to_list (name, namelength, namespace, class, list, val,
+                         language, objfile)
+     char *name;
+     int namelength;
+     enum namespace namespace;
+     enum address_class class;
+     struct psymbol_allocation_list *list;
+     CORE_ADDR val;
+     enum language language;
+     struct objfile *objfile;
+{
+  register struct partial_symbol *psym;
+  register char *demangled_name;
+
+  if (list->next >= list->list + list->size)
+    {
+      extend_psymbol_list (list,objfile);
+    }
+  psym = list->next++;
+  
+  SYMBOL_NAME (psym) =
+    (char *) obstack_alloc (&objfile->psymbol_obstack, namelength + 1);
+  memcpy (SYMBOL_NAME (psym), name, namelength);
+  SYMBOL_NAME (psym)[namelength] = '\0';
+  SYMBOL_VALUE_ADDRESS (psym) = val;
+  SYMBOL_LANGUAGE (psym) = language;
+  PSYMBOL_NAMESPACE (psym) = namespace;
+  PSYMBOL_CLASS (psym) = class;
+  SYMBOL_INIT_DEMANGLED_NAME (psym, &objfile->psymbol_obstack);
 }
+
+#endif /* !INLINE_ADD_PSYMBOL */
+
 \f
 void
 _initialize_symfile ()
 {
-
-  add_com ("symbol-file", class_files, symbol_file_command,
-          "Load symbol table from executable file FILE.\n\
+  struct cmd_list_element *c;
+  
+  c = add_cmd ("symbol-file", class_files, symbol_file_command,
+   "Load symbol table from executable file FILE.\n\
 The `file' command can also load symbol tables, as well as setting the file\n\
-to execute.");
+to execute.", &cmdlist);
+  c->completer = filename_completer;
 
-  add_com ("add-symbol-file", class_files, add_symbol_file_command,
+  c = add_cmd ("add-symbol-file", class_files, add_symbol_file_command,
    "Load the symbols from FILE, assuming FILE has been dynamically loaded.\n\
-The second argument provides the starting address of the file's text.");
+The second argument provides the starting address of the file's text.",
+              &cmdlist);
+  c->completer = filename_completer;
 
-  add_com ("load", class_files, load_command,
+  c = add_cmd ("load", class_files, load_command,
    "Dynamically load FILE into the running program, and record its symbols\n\
-for access from GDB.");
+for access from GDB.", &cmdlist);
+  c->completer = filename_completer;
 
   add_show_from_set
-    (add_set_cmd ("complaints", class_support, var_uinteger,
-                 (char *)&stop_whining,
-         "Set max number of complaints about incorrect symbols.",
+    (add_set_cmd ("symbol-reloading", class_support, var_boolean,
+                 (char *)&symbol_reloading,
+         "Set dynamic symbol table reloading multiple times in one run.",
                  &setlist),
      &showlist);
 
-  obstack_init (symbol_obstack);
-  obstack_init (psymbol_obstack);
 }
This page took 0.043638 seconds and 4 git commands to generate.