[binutils-gdb] MIPS: Remove remains of legacy remote target support

[Maciej W.Rozycki <macro at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=7e3d947dd6ae45d71aa175b96fbd42a51e93b3f8

commit 7e3d947dd6ae45d71aa175b96fbd42a51e93b3f8
Author: Maciej W. Rozycki <macro@imgtec.com>
Date:   Mon Oct 31 02:08:06 2016 +0000

    MIPS: Remove remains of legacy remote target support
    
    Complement commit f7c382926d78 ("Remove support for "target m32rsdi" and
    "target mips/pmon/ddb/rockhopper/lsi"") and remove dead MIPS target code
    which used to support these legacy remote targets.
    
    	gdb/
    	* mips-tdep.c (mips_r3041_reg_names): Remove.
    	(mips_breakpoint_from_pc): Remove IDT and PMON breakpoint
    	encodings.

Diff:
---
 gdb/ChangeLog   |  6 ++++++
 gdb/mips-tdep.c | 40 ++--------------------------------------
 2 files changed, 8 insertions(+), 38 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index ba4c58a..f917305 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,11 @@
 2016-10-31  Maciej W. Rozycki  <macro@imgtec.com>
 
+	* mips-tdep.c (mips_r3041_reg_names): Remove.
+	(mips_breakpoint_from_pc): Remove IDT and PMON breakpoint
+	encodings.
+
+2016-10-31  Maciej W. Rozycki  <macro@imgtec.com>
+
 	* defs.h (gdb_osabi): Remove GDB_OSABI_IRIX enum value.
 	* osabi.c (gdb_osabi_names): Remove "Irix" entry.
 	* mips-tdep.c (mips_irix_reg_names): Remove.
diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index eaed79f..962f9ca 100644
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -560,19 +560,6 @@ static const char *mips_generic_reg_names[NUM_MIPS_PROCESSOR_REGS] = {
   "fsr", "fir",
 };
 
-/* Names of IDT R3041 registers.  */
-
-static const char *mips_r3041_reg_names[] = {
-  "sr", "lo", "hi", "bad", "cause", "pc",
-  "f0", "f1", "f2", "f3", "f4", "f5", "f6", "f7",
-  "f8", "f9", "f10", "f11", "f12", "f13", "f14", "f15",
-  "f16", "f17", "f18", "f19", "f20", "f21", "f22", "f23",
-  "f24", "f25", "f26", "f27", "f28", "f29", "f30", "f31",
-  "fsr", "fir", "", /*"fp" */ "",
-  "", "", "bus", "ccfg", "", "", "", "",
-  "", "", "port", "cmp", "", "", "epc", "prid",
-};
-
 /* Names of tx39 registers.  */
 
 static const char *mips_tx39_reg_names[NUM_MIPS_PROCESSOR_REGS] = {
@@ -7070,23 +7057,10 @@ mips_breakpoint_from_pc (struct gdbarch *gdbarch,
 	}
       else
 	{
-	  /* The IDT board uses an unusual breakpoint value, and
-	     sometimes gets confused when it sees the usual MIPS
-	     breakpoint instruction.  */
 	  static gdb_byte big_breakpoint[] = { 0, 0x5, 0, 0xd };
-	  static gdb_byte pmon_big_breakpoint[] = { 0, 0, 0, 0xd };
-	  static gdb_byte idt_big_breakpoint[] = { 0, 0, 0x0a, 0xd };
 
 	  *lenptr = sizeof (big_breakpoint);
-
-	  if (strcmp (target_shortname, "mips") == 0)
-	    return idt_big_breakpoint;
-	  else if (strcmp (target_shortname, "ddb") == 0
-		   || strcmp (target_shortname, "pmon") == 0
-		   || strcmp (target_shortname, "lsi") == 0)
-	    return pmon_big_breakpoint;
-	  else
-	    return big_breakpoint;
+	  return big_breakpoint;
 	}
     }
   else
@@ -7116,19 +7090,9 @@ mips_breakpoint_from_pc (struct gdbarch *gdbarch,
       else
 	{
 	  static gdb_byte little_breakpoint[] = { 0xd, 0, 0x5, 0 };
-	  static gdb_byte pmon_little_breakpoint[] = { 0xd, 0, 0, 0 };
-	  static gdb_byte idt_little_breakpoint[] = { 0xd, 0x0a, 0, 0 };
 
 	  *lenptr = sizeof (little_breakpoint);
-
-	  if (strcmp (target_shortname, "mips") == 0)
-	    return idt_little_breakpoint;
-	  else if (strcmp (target_shortname, "ddb") == 0
-		   || strcmp (target_shortname, "pmon") == 0
-		   || strcmp (target_shortname, "lsi") == 0)
-	    return pmon_little_breakpoint;
-	  else
-	    return little_breakpoint;
+	  return little_breakpoint;
 	}
     }
 }