Fix inifnite loop problem with M32R port

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 9cb2f089a13695a977701129f31410d02f717991..e50a297a5c2d6baef2dd00efb1c704d6208b83bf 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2004-04-13  Kazuhiro Inaoka  <inaoka.kazuhiro@renesas.com>
+
+       * config/tc-m32r.c (md_assemble): Fixed infinite loop bug
+       in parallel.
+
 2004-04-11  Thiemo Seufer  <seufer@csv.ica.uni-stuttgart.de>
 
        * Makefile.am: Remove mips from aout targets.

diff --git a/gas/config/tc-m32r.c b/gas/config/tc-m32r.c
index 7b01516c71f455955781b58c8770e296607df090..e990d9d941c691bb8a52af5a715b0b71c9eac8c0 100644 (file)
--- a/gas/config/tc-m32r.c
+++ b/gas/config/tc-m32r.c
@@ -1382,6 +1382,14 @@ md_assemble (str)
         prev_insn.insn is NULL when we're on a 32 bit boundary.  */
       on_32bit_boundary_p = prev_insn.insn == NULL;
 
+      /* Change a frag to, if each insn to swap is in a different frag.
+         It must keep only one instruction in a frag.  */
+      if (parallel() && on_32bit_boundary_p)
+        {
+          frag_wane (frag_now);
+          frag_new (0);
+        }
+
       /* Look to see if this instruction can be combined with the
         previous instruction to make one, parallel, 32 bit instruction.
         If the previous instruction (potentially) changed the flow of
@@ -1442,13 +1450,25 @@ md_assemble (str)
          else if (insn.frag->fr_opcode == insn.addr)
            insn.frag->fr_opcode = prev_insn.addr;
 
-         /* Update the addresses in any fixups.
-            Note that we don't have to handle the case where each insn is in
-            a different frag as we ensure they're in the same frag above.  */
-         for (i = 0; i < prev_insn.num_fixups; ++i)
-           prev_insn.fixups[i]->fx_where += 2;
-         for (i = 0; i < insn.num_fixups; ++i)
-           insn.fixups[i]->fx_where -= 2;
+          /* Change a frag to, if each insn is in a different frag.
+            It must keep only one instruction in a frag.  */
+          if (prev_insn.frag != insn.frag)
+            {
+              for (i = 0; i < prev_insn.num_fixups; ++i)
+                prev_insn.fixups[i]->fx_frag = insn.frag;
+              for (i = 0; i < insn.num_fixups; ++i)
+                insn.fixups[i]->fx_frag = prev_insn.frag;
+            }
+          else
+           {
+             /* Update the addresses in any fixups.
+                Note that we don't have to handle the case where each insn is in
+                a different frag as we ensure they're in the same frag above.  */
+             for (i = 0; i < prev_insn.num_fixups; ++i)
+               prev_insn.fixups[i]->fx_where += 2;
+             for (i = 0; i < insn.num_fixups; ++i)
+               insn.fixups[i]->fx_where -= 2;
+           }
        }
 
       /* Keep track of whether we've seen a pair of 16 bit insns.

diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog
index febbce89481be7ab54ee8641fbf5be2a26d5750d..9ca420c3c02023a8d6bd80c0bc330bbe3fcda398 100644 (file)
--- a/gas/testsuite/ChangeLog
+++ b/gas/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2004-04-13  Kazuhiro Inaoka  <inaoka.kazuhiro@renesas.com>
+
+       * gas/m32r/parallel-2.s, gas/m32r/parallel-2.d: Add a new test
+       case for parallel code.
+
 2004-04-01  Asgari Jinia  <asgarij@kpitcummins.com>
 
        * gas/sh/renesas-1.s, gas/sh/renesas-1.d: New test for -renesas

diff --git a/gas/testsuite/gas/m32r/m32r2.exp b/gas/testsuite/gas/m32r/m32r2.exp
index 03a160aa589e23a865d426bbcf61aac77524e3bc..f5f7415d9ed233cc3c997b7373a725810db270a5 100644 (file)
--- a/gas/testsuite/gas/m32r/m32r2.exp
+++ b/gas/testsuite/gas/m32r/m32r2.exp
@@ -2,4 +2,5 @@
 
 if [istarget m32r*-*-*] {
     run_dump_test "m32r2"
+    run_dump_test "parallel-2"
 }

diff --git a/gas/testsuite/gas/m32r/parallel-2.d b/gas/testsuite/gas/m32r/parallel-2.d
new file mode 100644 (file)
index 0000000..5638c33
--- /dev/null
+++ b/gas/testsuite/gas/m32r/parallel-2.d
@@ -0,0 +1,10 @@
+#as: -m32r2 -O
+#objdump: -dr
+
+.*: +file format .*
+
+Disassembly of section .text:
+
+0+0000 <test>:
+   0:  04 a5 24 46     add r4,r5 -> st r4,@r6
+   4:  7c ff c6 04     bc 0 <test> \|\| addi r6,[#]4

diff --git a/gas/testsuite/gas/m32r/parallel-2.s b/gas/testsuite/gas/m32r/parallel-2.s
new file mode 100644 (file)
index 0000000..7ea40c2
--- /dev/null
+++ b/gas/testsuite/gas/m32r/parallel-2.s
@@ -0,0 +1,7 @@
+       .text
+test:
+        add r4,r5
+        st r4,@(r6)
+        addi r6,#4
+        .debugsym .LM568
+        bc.s test