2001-08-14 Daniel Jacobowitz <drow@mvista.com>

	    H.J. Lu  (hjl@gnu.org)

	* partial-stab.h: valu should be a CORE_ADDR.

2001-08-14  H.J. Lu  (hjl@gnu.org)

	* dbxread.c (SWAP_SYMBOL): Removed.
	(INTERNALIZE_SYMBOL): Check sign extended vma.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c1567aa804019cb5c1998bfa6348208282c920cb..51b5a87144035b7a37976fbea35532af0a57eec9 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,13 @@
+2001-08-14  Daniel Jacobowitz  <drow@mvista.com>
+           H.J. Lu  (hjl@gnu.org)
+
+       * partial-stab.h: valu should be a CORE_ADDR.
+
+2001-08-14  H.J. Lu  (hjl@gnu.org)
+
+       * dbxread.c (SWAP_SYMBOL): Removed.
+       (INTERNALIZE_SYMBOL): Check sign extended vma.
+
 2001-08-13  Christopher Faylor  <cgf@cygnus.com>
 
        * top.c (SIGSETJMP): Protect env argument with parentheses.

diff --git a/gdb/dbxread.c b/gdb/dbxread.c
index 5d7ed74e8af4cddbb135ffbe1a09e10c8f035ebc..f1382e7ef045e4ffe015fd33671102b6f46f4e3c 100644 (file)
--- a/gdb/dbxread.c
+++ b/gdb/dbxread.c
@@ -946,22 +946,15 @@ fill_symbuf (bfd *sym_bfd)
   symbuf_read += nbytes;
 }
 
-#define SWAP_SYMBOL(symp, abfd) \
-  { \
-    (symp)->n_strx = bfd_h_get_32(abfd,                        \
-                               (unsigned char *)&(symp)->n_strx);      \
-    (symp)->n_desc = bfd_h_get_16 (abfd,                       \
-                               (unsigned char *)&(symp)->n_desc);      \
-    (symp)->n_value = bfd_h_get_32 (abfd,                      \
-                               (unsigned char *)&(symp)->n_value);     \
-  }
-
 #define INTERNALIZE_SYMBOL(intern, extern, abfd)                       \
   {                                                                    \
     (intern).n_type = bfd_h_get_8 (abfd, (extern)->e_type);            \
     (intern).n_strx = bfd_h_get_32 (abfd, (extern)->e_strx);           \
     (intern).n_desc = bfd_h_get_16 (abfd, (extern)->e_desc);           \
-    (intern).n_value = bfd_h_get_32 (abfd, (extern)->e_value);         \
+    if (bfd_get_sign_extend_vma (abfd))                                        \
+      (intern).n_value = bfd_h_get_signed_32 (abfd, (extern)->e_value);        \
+    else                                                               \
+      (intern).n_value = bfd_h_get_32 (abfd, (extern)->e_value);       \
   }
 
 /* Invariant: The symbol pointed to by symbuf_idx is the first one

diff --git a/gdb/partial-stab.h b/gdb/partial-stab.h
index ebedef72c53ca60a2a7e5decba2a87bd78b047b9..fe772ba588aa7649eb24bc55748f8092b5f73118 100644 (file)
--- a/gdb/partial-stab.h
+++ b/gdb/partial-stab.h
@@ -201,7 +201,7 @@ switch (CUR_SYMBOL_TYPE)
 
   case N_SO:
     {
-      unsigned long valu;
+      CORE_ADDR valu;
       static int prev_so_symnum = -10;
       static int first_so_symnum;
       char *p;
@@ -399,7 +399,7 @@ switch (CUR_SYMBOL_TYPE)
     /* See if this is an end of function stab.  */
     if (pst && CUR_SYMBOL_TYPE == N_FUN && *namestring == '\000')
       {
-       unsigned long valu;
+       CORE_ADDR valu;
 
        /* It's value is the size (in bytes) of the function for
           function relative stabs, or the address of the function's