2003-07-09 Michael Snyder <msnyder@redhat.com>

        * gencode.c (ppi_gensim): For a conditional ppi insn, if the
        condition is false, we want to return (not break).  A break
        will take us to the end of the function where registers will
	be updated, whereas the desired outcome is for nothing to change.

diff --git a/sim/sh/ChangeLog b/sim/sh/ChangeLog
index 8a5e9e3f0c8723362b0f952cab1bc29fd8f3d654..50a0e506e138a154b9600b3306f5380c1cd1089e 100644 (file)
--- a/sim/sh/ChangeLog
+++ b/sim/sh/ChangeLog
@@ -1,3 +1,10 @@
+2003-07-09  Michael Snyder  <msnyder@redhat.com>
+
+       * gencode.c (ppi_gensim): For a conditional ppi insn, if the
+       condition is false, we want to return (not break).  A break
+       will take us to the end of the function where registers will
+       be updated, whereas the desired outcome is for nothing to change.
+
 2003-07-03  Michael Snyder  <msnyder@redhat.com>
 
        * gencode.c (movs): Fix a couple of text transpositions.

diff --git a/sim/sh/gencode.c b/sim/sh/gencode.c
index e7b6b4dc548f063b489d6a07c59ed4f4b9600834..f0af175c4eec89afdc99084e88d76d35154c012d 100644 (file)
--- a/sim/sh/gencode.c
+++ b/sim/sh/gencode.c
@@ -2469,7 +2469,7 @@ ppi_gensim ()
              break;
            case 'c':
              printf ("      if ((((iword >> 8) ^ DSR) & 1) == 0)\n");
-             printf ("\tbreak;\n");
+             printf ("\treturn;\n");
              printf ("    }\n");
              printf ("  case %d:      \n", p->index + 1);
              printf ("    {\n");