* config/tc-rx.c (rx_bytesT): Add grown/shrank counters for

relaxation.
(rx_relax_frag): Prevent infinite loops of grow/shrink/grow/etc.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index ef3ac8b8323355253592a190bf3a8ac187f42e90..47708c7c4a16bb2d0271b1b474ddc1f4c9074e10 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,9 @@
+2010-07-02  DJ Delorie  <dj@redhat.com>
+
+       * config/tc-rx.c (rx_bytesT): Add grown/shrank counters for
+       relaxation.
+       (rx_relax_frag): Prevent infinite loops of grow/shrink/grow/etc.
+
 2010-07-01  H.J. Lu  <hongjiu.lu@intel.com>
 
        AVX Programming Reference (June, 2010)

diff --git a/gas/config/tc-rx.c b/gas/config/tc-rx.c
index 3bb9a5afe7b48de2ec60c7b49f768c7836ca9ff7..00101e0318b79116e12d5e8a1d8e0a1601705eee 100644 (file)
--- a/gas/config/tc-rx.c
+++ b/gas/config/tc-rx.c
@@ -624,6 +624,8 @@ typedef struct rx_bytesT
   int n_relax;
   int link_relax;
   fixS *link_relax_fixP;
+  char times_grown;
+  char times_shrank;
 } rx_bytesT;
 
 static rx_bytesT rx_bytes;
@@ -1485,6 +1487,21 @@ rx_relax_frag (segT segment ATTRIBUTE_UNUSED, fragS * fragP, long stretch)
        break;
       }
 
+  /* This prevents infinite loops in align-heavy sources.  */
+  if (newsize < oldsize)
+    {
+      if (fragP->tc_frag_data->times_shrank > 10
+         && fragP->tc_frag_data->times_grown > 10)
+       newsize = oldsize;
+      if (fragP->tc_frag_data->times_shrank < 20)
+       fragP->tc_frag_data->times_shrank ++;
+    }
+  else if (newsize > oldsize)
+    {
+      if (fragP->tc_frag_data->times_grown < 20)
+       fragP->tc_frag_data->times_grown ++;
+    }
+
   fragP->fr_subtype = newsize;
   tprintf (" -> new %d old %d delta %d\n", newsize, oldsize, newsize-oldsize);
   return newsize - oldsize;