MIPS: Complete `status' to `err' renaming in `mips_breakpoint_from_pc'
authorMaciej W. Rozycki <macro@imgtec.com>
Thu, 7 Jan 2016 19:12:44 +0000 (19:12 +0000)
committerMaciej W. Rozycki <macro@imgtec.com>
Thu, 7 Jan 2016 19:12:44 +0000 (19:12 +0000)
Complement commit d09f2c3f [target_read_memory&co: no longer return
target_xfer_status] and apply the same change made to the big-endian leg
of the function to the little-endian leg as well.

gdb/
* mips-tdep.c (mips_breakpoint_from_pc): Rename local `status'
to `err' in the little-endian leg.

gdb/ChangeLog
gdb/mips-tdep.c

index e11ea1a..73ba8cd 100644 (file)
@@ -1,3 +1,8 @@
+2016-01-07  Maciej W. Rozycki  <macro@imgtec.com>
+
+       * mips-tdep.c (mips_breakpoint_from_pc): Rename local `status'
+       to `err' in the little-endian leg.
+
 2016-01-06  Yao Qi  <yao.qi@linaro.org>
 
        * arch/arm-get-next-pcs.c (arm_get_next_pcs): Move it to some
index c8e12e8..ca17864 100644 (file)
@@ -7170,12 +7170,13 @@ mips_breakpoint_from_pc (struct gdbarch *gdbarch,
          static gdb_byte micromips16_little_breakpoint[] = { 0x85, 0x46 };
          static gdb_byte micromips32_little_breakpoint[] = { 0x5, 0, 0x7, 0 };
          ULONGEST insn;
-         int status;
+         int err;
          int size;
 
-         insn = mips_fetch_instruction (gdbarch, ISA_MICROMIPS, pc, &status);
-         size = status ? 2
-                       : mips_insn_size (ISA_MICROMIPS, insn) == 2 ? 2 : 4;
+         insn = mips_fetch_instruction (gdbarch, ISA_MICROMIPS, pc, &err);
+         size = (err != 0
+                 ? 2 : (mips_insn_size (ISA_MICROMIPS, insn) == 2
+                        ? 2 : 4));
          *pcptr = unmake_compact_addr (pc);
          *lenptr = size;
          return (size == 2) ? micromips16_little_breakpoint