Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

sim: iq2000/m32r/lm32/sh64: delete dead option code

Browse Source 
The iq2000/m32r/sh64 option parsing logic appears to have always been
dead.  At least iq2000/sh64 are simply copy & paste rot from m32r.

The lm32 option parsing hack here hasn't been needed for a while -- this
was fixed back in commit 11409fac6b in the
common code.
This commit is contained in:
Mike Frysinger
2016-01-02 10:29:51 -05:00
parent d47f5b30d8
commit 027e73b217
8 changed files with 18 additions and 55 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
sim/iq2000/ChangeLog
View File

@@ -1,3 +1,7 @@
2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (sim_open): Delete #if 0 sim_add_option_table call.
2016-01-02 Mike Frysinger <vapier@gentoo.org> 2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (current_state): Delete. * sim-if.c (current_state): Delete.

8
sim/iq2000/sim-if.c
View File

@@ -82,14 +82,6 @@ sim_open (kind, callback, abfd, argv)
return 0; return 0;
} }
#if 0 /* FIXME: 'twould be nice if we could do this */
/* These options override any module options.
Obviously ambiguity should be avoided, however the caller may wish to
augment the meaning of an option. */
if (extra_options != NULL)
sim_add_option_table (sd, extra_options);
#endif
/* getopt will print the error message so we just have to exit if this fails. /* getopt will print the error message so we just have to exit if this fails.
FIXME: Hmmm... in the case of gdb we need getopt to call FIXME: Hmmm... in the case of gdb we need getopt to call
print_filtered. */ print_filtered. */

6
sim/lm32/ChangeLog
View File

@@ -1,3 +1,9 @@
2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (free_state, print_lm32_misc_cpu): Delete prototypes.
(lm32_option_handler, OPTION_ENDIAN, lm32_options): Delete.
(sim_open): Delete sim_add_option_table call.
2016-01-02 Mike Frysinger <vapier@gentoo.org> 2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (current_state): Delete. * sim-if.c (current_state): Delete.

31
sim/lm32/sim-if.c
View File

@@ -26,23 +26,6 @@
#ifdef HAVE_STDLIB_H #ifdef HAVE_STDLIB_H
#include <stdlib.h> #include <stdlib.h>
#endif #endif
static void free_state (SIM_DESC);
static void print_lm32_misc_cpu (SIM_CPU * cpu, int verbose);
static DECLARE_OPTION_HANDLER (lm32_option_handler);
enum
{
OPTION_ENDIAN = OPTION_START,
};
/* GDB passes -E, even though it's fixed, so we have to handle it here. common code only handles it if SIM_HAVE_BIENDIAN is defined, which it isn't for lm32. */
static const OPTION lm32_options[] = {
{{"endian", required_argument, NULL, OPTION_ENDIAN},
'E', "big", "Set endianness",
lm32_option_handler},
{{NULL, no_argument, NULL, 0}, '\0', NULL, NULL, NULL}
};
/* Cover function of sim_state_free to free the cpu buffers as well. */ /* Cover function of sim_state_free to free the cpu buffers as well. */
@@ -111,19 +94,6 @@ find_limit (bfd *prog_bfd)
return 0; return 0;
} }
/* Handle lm32 specific options. */
static SIM_RC
lm32_option_handler (sd, cpu, opt, arg, is_command)
SIM_DESC sd;
sim_cpu *cpu;
int opt;
char *arg;
int is_command;
{
return SIM_RC_OK;
}
/* Create an instance of the simulator. */ /* Create an instance of the simulator. */
SIM_DESC SIM_DESC
@@ -150,7 +120,6 @@ sim_open (kind, callback, abfd, argv)
free_state (sd); free_state (sd);
return 0; return 0;
} }
sim_add_option_table (sd, NULL, lm32_options);
/* getopt will print the error message so we just have to exit if this fails. /* getopt will print the error message so we just have to exit if this fails.
FIXME: Hmmm... in the case of gdb we need getopt to call FIXME: Hmmm... in the case of gdb we need getopt to call

4
sim/m32r/ChangeLog
View File

@@ -1,3 +1,7 @@
2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (sim_open): Delete #if 0 sim_add_option_table call.
2016-01-02 Mike Frysinger <vapier@gentoo.org> 2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (current_state): Delete. * sim-if.c (current_state): Delete.

8
sim/m32r/sim-if.c
View File

@@ -84,14 +84,6 @@ sim_open (kind, callback, abfd, argv)
return 0; return 0;
} }
#if 0 /* FIXME: 'twould be nice if we could do this */
/* These options override any module options.
Obviously ambiguity should be avoided, however the caller may wish to
augment the meaning of an option. */
if (extra_options != NULL)
sim_add_option_table (sd, extra_options);
#endif
/* getopt will print the error message so we just have to exit if this fails. /* getopt will print the error message so we just have to exit if this fails.
FIXME: Hmmm... in the case of gdb we need getopt to call FIXME: Hmmm... in the case of gdb we need getopt to call
print_filtered. */ print_filtered. */

4
sim/sh64/ChangeLog
View File

@@ -1,3 +1,7 @@
2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (sim_open): Delete #if 0 sim_add_option_table call.
2016-01-02 Mike Frysinger <vapier@gentoo.org> 2016-01-02 Mike Frysinger <vapier@gentoo.org>
* sim-if.c (current_state): Delete. * sim-if.c (current_state): Delete.

8
sim/sh64/sim-if.c
View File

@@ -79,14 +79,6 @@ sim_open (kind, callback, abfd, argv)
return 0; return 0;
} }
#if 0 /* FIXME: 'twould be nice if we could do this */
/* These options override any module options.
Obviously ambiguity should be avoided, however the caller may wish to
augment the meaning of an option. */
if (extra_options != NULL)
sim_add_option_table (sd, extra_options);
#endif
/* getopt will print the error message so we just have to exit if this fails. /* getopt will print the error message so we just have to exit if this fails.
FIXME: Hmmm... in the case of gdb we need getopt to call FIXME: Hmmm... in the case of gdb we need getopt to call
print_filtered. */ print_filtered. */