sim/h8300: avoid self assignment

There are two places in the h8300 simulator where we assign a variable
to itself.  Clang gives a warning for this, which is converted into an
error by -Werror.

Silence the warning by removing the self assignments.  As these
assignments were in a complex if/then/else tree, rather than try to
adjust all the conditions, I've just replaced the self assignments
with a comment and an empty statement.

diff --git a/sim/h8300/compile.c b/sim/h8300/compile.c
index 9be7dd565a922466cede1fae3fea94f5f5bab809..5f64b4752f05e2776f8ba903f97a16ec3d3be7a5 100644 (file)
--- a/sim/h8300/compile.c
+++ b/sim/h8300/compile.c
@@ -4141,7 +4141,7 @@ step_once (SIM_DESC sd, SIM_CPU *cpu)
          res = GET_B_REG (code->src.reg);      /* FIXME fetch? */
          if (!c && (0 <= (res >>  4) && (res >>  4) <= 9) && 
              !h && (0 <= (res & 0xf) && (res & 0xf) <= 9))
-           res = res;          /* Value added == 0.  */
+           /* Nothing.  */;            /* Value added == 0.  */
          else if (!c && (0  <= (res >>  4) && (res >>  4) <=  8) && 
                   !h && (10 <= (res & 0xf) && (res & 0xf) <= 15))
            res = res + 0x6;            /* Value added == 6.  */
@@ -4174,7 +4174,7 @@ step_once (SIM_DESC sd, SIM_CPU *cpu)
          res = GET_B_REG (code->src.reg); /* FIXME fetch, fetch2... */
          if (!c && (0 <= (res >>  4) && (res >>  4) <= 9) && 
              !h && (0 <= (res & 0xf) && (res & 0xf) <= 9))
-           res = res;          /* Value added == 0.  */
+           /* Nothing.  */;            /* Value added == 0.  */
          else if (!c && (0 <= (res >>  4) && (res >>  4) <=  8) && 
                    h && (6 <= (res & 0xf) && (res & 0xf) <= 15))
            res = res + 0xfa;           /* Value added == 0xfa.  */