* objfiles.c (objfile_relocate): When relocating ->sections, use

	objfile not symfile_objfile.

	* symtab.h, minsyms.c (minsyms_sort): New function.
	* objfiles.c (objfile_relocate): Call it.

	* remote-vx.c (vx_add_symbols): Call breakpoint_re_set.

	* objfiles.c, objfiles.h (objfile_to_front): New function.
	* remote-vx.c (vx_add_symbols): Call it.

	* coffread.c (coff_symtab_read): Handle common symbols the same
	way that partial-stab.h does.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e009c262d98fef001aac52d24b4ffc53dfe454ae..c961404117b138e0c3e9b75162d02c9ba94970c4 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,24 @@
+Wed Oct 19 11:32:15 1994  Jim Kingdon  <kingdon@cygnus.com>
+
+       * objfiles.c (objfile_relocate): When relocating ->sections, use
+       objfile not symfile_objfile.
+
+       * symtab.h, minsyms.c (minsyms_sort): New function.
+       * objfiles.c (objfile_relocate): Call it.
+
+       * remote-vx.c (vx_add_symbols): Call breakpoint_re_set.
+
+       * objfiles.c, objfiles.h (objfile_to_front): New function.
+       * remote-vx.c (vx_add_symbols): Call it.
+
+       * coffread.c (coff_symtab_read): Handle common symbols the same
+       way that partial-stab.h does.
+
+Wed Oct 19 21:06:12 1994  Rob Savoye  (rob@cirdan.cygnus.com)
+
+       * hppa-tdep.c: Remove include files a.out.h, ioctl.h, and
+       machine/psl.h. These are host files.
+
 Wed Oct 19 15:13:51 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
 
        * objfiles.h (struct objfile): Fix comment--minimal_symbol_count

diff --git a/gdb/coffread.c b/gdb/coffread.c
index 82b72995339179f75692b54f4aa16c7cf12f83e2..ddd05117bebcb0b507155f28d1a863353057209d 100644 (file)
--- a/gdb/coffread.c
+++ b/gdb/coffread.c
@@ -38,6 +38,7 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include "gdb-stabs.h"
 #include "stabsread.h"
 #include "complaints.h"
+#include "target.h"
 
 struct coff_symfile_info {
   file_ptr min_lineno_offset;          /* Where in file lowest line#s are */
@@ -908,10 +909,29 @@ coff_symtab_read (symtab_offset, nsyms, section_offsets, objfile)
 
              /* FIXME: should use mst_abs, and not relocate, if absolute.  */
              enum minimal_symbol_type ms_type;
-             int sec = cs_to_section (cs, objfile);
-             tmpaddr = cs->c_value;
-             if (cs->c_sclass != C_STAT)
-               tmpaddr += ANOFFSET (section_offsets, sec);
+             int sec;
+
+             if (cs->c_secnum == N_UNDEF)
+               {
+                 /* This is a common symbol.  See if the target
+                    environment knows where it has been relocated to.  */
+                 CORE_ADDR reladdr;
+                 if (target_lookup_symbol (cs->c_name, &reladdr))
+                   {
+                     /* Error in lookup; ignore symbol.  */
+                     break;
+                   }
+                 tmpaddr = reladdr;
+                 sec = SECT_OFF_BSS;
+               }
+             else
+               {
+                 sec = cs_to_section (cs, objfile);
+                 tmpaddr = cs->c_value;
+                 if (cs->c_sclass != C_STAT)
+                   tmpaddr += ANOFFSET (section_offsets, sec);
+               }
+
              switch (sec)
                {
                case SECT_OFF_TEXT:

diff --git a/gdb/minsyms.c b/gdb/minsyms.c
index fde97e9e6cfb12cbe481374f68292cf435e0641c..77d7466f792e6f8a5ea5648ff8f6da0a30214495 100644 (file)
--- a/gdb/minsyms.c
+++ b/gdb/minsyms.c
@@ -601,6 +601,16 @@ install_minimal_symbols (objfile)
     }
 }
 
+/* Sort all the minimal symbols in OBJFILE.  */
+
+void
+msymbols_sort (objfile)
+     struct objfile *objfile;
+{
+  qsort (objfile->msymbols, objfile->minimal_symbol_count,
+        sizeof (struct minimal_symbol), compare_minimal_symbols);
+}
+
 /* Check if PC is in a shared library trampoline code stub.
    Return minimal symbol for the trampoline entry or NULL if PC is not
    in a trampoline code stub.  */

diff --git a/gdb/objfiles.c b/gdb/objfiles.c
index 5cd01e5210e2223897d95c71882f0a079c5be7ac..891531d3d89c0990ee7c9cbffe6384049c6980a7 100644 (file)
--- a/gdb/objfiles.c
+++ b/gdb/objfiles.c
@@ -273,6 +273,27 @@ allocate_objfile (abfd, mapped)
   return (objfile);
 }
 
+/* Put OBJFILE at the front of the list.  */
+
+void
+objfile_to_front (objfile)
+     struct objfile *objfile;
+{
+  struct objfile **objp;
+  for (objp = &object_files; *objp != NULL; objp = &((*objp)->next))
+    {
+      if (*objp == objfile)
+       {
+         /* Unhook it from where it is.  */
+         *objp = objfile->next;
+         /* Put it in the front.  */
+         objfile->next = object_files;
+         object_files = objfile;
+         break;
+       }
+    }
+}
+
 /* Unlink OBJFILE from the list of known objfiles, if it is found in the
    list.
 
@@ -551,6 +572,9 @@ objfile_relocate (objfile, new_offsets)
       if (SYMBOL_SECTION (msym) >= 0)
        SYMBOL_VALUE_ADDRESS (msym) += ANOFFSET (delta, SYMBOL_SECTION (msym));
   }
+  /* Relocating different sections by different amounts may cause the symbols
+     to be out of order.  */
+  msymbols_sort (objfile);
 
   {
     int i;
@@ -562,10 +586,10 @@ objfile_relocate (objfile, new_offsets)
     struct obj_section *s;
     bfd *abfd;
 
-    abfd = symfile_objfile->obfd;
+    abfd = objfile->obfd;
 
-    for (s = symfile_objfile->sections;
-        s < symfile_objfile->sections_end; ++s)
+    for (s = objfile->sections;
+        s < objfile->sections_end; ++s)
       {
        flagword flags;
 

diff --git a/gdb/objfiles.h b/gdb/objfiles.h
index 2add5209d32a528181636b29f3c753b0114cda2f..0a69f20cad835f67c61fb012b64dce5e3dcd20df 100644 (file)
--- a/gdb/objfiles.h
+++ b/gdb/objfiles.h
@@ -384,6 +384,8 @@ allocate_objfile PARAMS ((bfd *, int));
 extern int
 build_objfile_section_table PARAMS ((struct objfile *));
 
+extern void objfile_to_front PARAMS ((struct objfile *));
+
 extern void
 unlink_objfile PARAMS ((struct objfile *));
 

diff --git a/gdb/remote-vx.c b/gdb/remote-vx.c
index 8f0c2d49e7ec1254c382abbb6206f6e575602f98..239ad24d0ecedd9430f7f79a9092477e3fbc55cd 100644 (file)
--- a/gdb/remote-vx.c
+++ b/gdb/remote-vx.c
@@ -669,7 +669,15 @@ vx_add_symbols (name, from_tty, text_addr, data_addr, bss_addr)
   struct objfile *objfile;
   struct find_sect_args ss;
 
+  /* It might be nice to suppress the breakpoint_re_set which happens here
+     because we are going to do one again after the objfile_relocate.  */
   objfile = symbol_file_add (name, from_tty, 0, 0, 0, 0);
+
+  /* This is a (slightly cheesy) way of superceding the old symbols.  A less
+     cheesy way would be to find the objfile with the same name and
+     free_objfile it.  */
+  objfile_to_front (objfile);
+
   offs = (struct section_offsets *)
     alloca (sizeof (struct section_offsets)
            + objfile->num_sections * sizeof (offs->offsets));
@@ -687,6 +695,9 @@ vx_add_symbols (name, from_tty, text_addr, data_addr, bss_addr)
   ANOFFSET (offs, SECT_OFF_DATA) = data_addr - ss.data_start;
   ANOFFSET (offs, SECT_OFF_BSS) = bss_addr - ss.bss_start;
   objfile_relocate (objfile, offs);
+
+  /* Need to do this *after* things are relocated.  */
+  breakpoint_re_set ();
 }
 
 /* This function allows the addition of incrementally linked object files.  */