* ppc-linux-nat.c (fetch_register): Replace 'gdb_assert (0)' with

a call to 'internal_error', with a more helpful error message.
* rs6000-tdep.c (e500_pseudo_register_read,
e500_pseudo_register_write): Same.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 43b87f94e340621a494bd14f275e469fff599130..cb9e7094e43581511acea8535d9cf6a1e1964da1 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2004-08-04  Jim Blandy  <jimb@redhat.com>
+
+       * ppc-linux-nat.c (fetch_register): Replace 'gdb_assert (0)' with
+       a call to 'internal_error', with a more helpful error message.
+       * rs6000-tdep.c (e500_pseudo_register_read,
+       e500_pseudo_register_write): Same.
+
 2004-08-04  Andrew Cagney  <cagney@gnu.org>
 
        * Makefile.in: Update dependencies.

diff --git a/gdb/ppc-linux-nat.c b/gdb/ppc-linux-nat.c
index 02095e992f7e48593438bd492e48fabb37118ca4..c98bb93f879f61449d0d8a9775f16a0661975d2d 100644 (file)
--- a/gdb/ppc-linux-nat.c
+++ b/gdb/ppc-linux-nat.c
@@ -399,7 +399,9 @@ fetch_register (int tid, int regno)
       regcache_raw_supply (current_regcache, regno, buf + padding);
     }
   else 
-    gdb_assert (0);
+    internal_error (__FILE__, __LINE__,
+                    "fetch_register: unexpected byte order: %d",
+                    gdbarch_byte_order (current_gdbarch));
 }
 
 static void

diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index 52812d5ad24ceaf21ab62c07f1d3757ad68a82ae..fcd14635938e79128d93c784c9bda5bfa0502a24 100644 (file)
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -1941,8 +1941,10 @@ e500_pseudo_register_read (struct gdbarch *gdbarch, struct regcache *regcache,
       && reg_nr < tdep->ppc_ev0_regnum + ppc_num_gprs)
     e500_move_ev_register (regcache_raw_read, regcache, reg_nr, buffer);
   else
-    /* We should only be called on pseudo-registers.  */
-    gdb_assert (0);
+    internal_error (__FILE__, __LINE__,
+                    "e500_pseudo_register_read: "
+                    "called on unexpected register '%s' (%d)",
+                    gdbarch_register_name (gdbarch, reg_nr), reg_nr);
 }
 
 static void
@@ -1960,8 +1962,10 @@ e500_pseudo_register_write (struct gdbarch *gdbarch, struct regcache *regcache,
                            regcache_raw_write,
                            regcache, reg_nr, (void *) buffer);
   else
-    /* We should only be called on pseudo-registers.  */
-    gdb_assert (0);
+    internal_error (__FILE__, __LINE__,
+                    "e500_pseudo_register_read: "
+                    "called on unexpected register '%s' (%d)",
+                    gdbarch_register_name (gdbarch, reg_nr), reg_nr);
 }
 
 /* The E500 needs a custom reggroup function: it has anonymous raw
@@ -2095,7 +2099,9 @@ rs6000_store_return_value (struct type *type,
           && TYPE_VECTOR (type))
         regnum = tdep->ppc_vr0_regnum + 2;
       else
-        gdb_assert (0);
+        internal_error (__FILE__, __LINE__,
+                        "rs6000_store_return_value: "
+                        "unexpected array return type");
     }
   else
     /* Everything else is returned in GPR3 and up.  */