* mips-tdep.c (mips_next_pc): Use is_mips16_addr() instead of

a direct test.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 475f38003687fafa2d5e81291c6519234a3492c4..aaac11dbf89f756589e0178e295a515679c724dc 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2007-07-20  Maciej W. Rozycki  <macro@mips.com>
+
+       * mips-tdep.c (mips_next_pc): Use is_mips16_addr() instead of
+       a direct test.
+
 2007-07-20  Chris Dearman  <chris@mips.com>
             Maciej W. Rozycki  <macro@mips.com>
 

diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index 751945e54e887a1d98d36044793b3576da0c1682..b4153fd0fdf3f7898e1877dc2ff6bcf1f0f3af1d 100644 (file)
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -1435,11 +1435,11 @@ mips16_next_pc (struct frame_info *frame, CORE_ADDR pc)
    target monitor or stub is not developed enough to do a single_step.
    It works by decoding the current instruction and predicting where a
    branch will go. This isnt hard because all the data is available.
-   The MIPS32 and MIPS16 variants are quite different */
+   The MIPS32 and MIPS16 variants are quite different.  */
 static CORE_ADDR
 mips_next_pc (struct frame_info *frame, CORE_ADDR pc)
 {
-  if (pc & 0x01)
+  if (is_mips16_addr (pc))
     return mips16_next_pc (frame, pc);
   else
     return mips32_next_pc (frame, pc);