[PATCH/committed] sim: h8300: clean up various warnings

[PATCH/committed] sim: h8300: clean up various warnings

[Mike Frysinger]

A random grab bag of minor fixes to enable -Werror for this port.

Disable h8_set_macS for now as it's unused.
Initialize trace & intMask before using them.
Mark local set_h8300h function static.
---
 sim/h8300/ChangeLog    |  8 ++++++++
 sim/h8300/compile.c    |  7 +++++--
 sim/h8300/configure    | 13 ++++++++-----
 sim/h8300/configure.ac |  1 -
 4 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/sim/h8300/ChangeLog b/sim/h8300/ChangeLog
index c922aaf73ba9..578836eb2ee4 100644
--- a/sim/h8300/ChangeLog
+++ b/sim/h8300/ChangeLog
@@ -1,3 +1,11 @@
+2021-05-08  Mike Frysinger  <vapier@gentoo.org>
+
+	* compile.c (h8_set_macS): Disable with #if 0.
+	(step_once): Set trace & intMask to 0.
+	(set_h8300h): Mark static.
+	* configure.ac: Delete SIM_AC_OPTION_WARNINGS call.
+	* configure: Regenerate.
+
 2021-05-04  Mike Frysinger  <vapier@gentoo.org>
 
 	* configure: Regenerate.
diff --git a/sim/h8300/compile.c b/sim/h8300/compile.c
index fb0e51b2e58d..365f8667c6a5 100644
--- a/sim/h8300/compile.c
+++ b/sim/h8300/compile.c
@@ -307,11 +307,13 @@ h8_get_macS (SIM_DESC sd)
   return (STATE_CPU (sd, 0)) -> macS;
 }
 
+#if 0
 static void
 h8_set_macS (SIM_DESC sd, int val)
 {
   (STATE_CPU (sd, 0)) -> macS = (val != 0);
 }
+#endif
 
 /* MAC Zero Flag */
 static int
@@ -1716,7 +1718,8 @@ step_once (SIM_DESC sd, SIM_CPU *cpu)
   int bit;
   int pc;
   int c, nz, v, n, u, h, ui, intMaskBit;
-  int trace, intMask;
+  int trace = 0;
+  int intMask = 0;
   int oldmask;
   host_callback *sim_callback = STATE_CALLBACK (sd);
 
@@ -4552,7 +4555,7 @@ sim_info (SIM_DESC sd, int verbose)
 /* Indicate whether the cpu is an H8/300 or H8/300H.
    FLAG is non-zero for the H8/300H.  */
 
-void
+static void
 set_h8300h (unsigned long machine)
 {
   /* FIXME: Much of the code in sim_load can be moved to sim_open.
diff --git a/sim/h8300/configure.ac b/sim/h8300/configure.ac
index 8dc33c4cf3ab..dc24462742dc 100644
--- a/sim/h8300/configure.ac
+++ b/sim/h8300/configure.ac
@@ -6,7 +6,6 @@ SIM_AC_COMMON
 
 SIM_AC_OPTION_ENDIAN(BIG)
 SIM_AC_OPTION_ALIGNMENT(NONSTRICT_ALIGNMENT)
-SIM_AC_OPTION_WARNINGS(no)
 
 AC_CHECK_HEADERS_ONCE(sys/param.h)
 
-- 
2.31.1