From: Jim Wilson <wilson@tuliptree.org>
Date: Wed, 8 Jun 2005 19:47:59 +0000 (+0000)
Subject: Warning and partial fix for gcc -freorder-blocks-and-partition related problem.
X-Git-Tag: gdb-csl-arm-20051020-branchpoint~1064
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4dddc1d199cc4301671b6612fcbcb5372f9d6237;p=external%2Fbinutils.git

Warning and partial fix for gcc -freorder-blocks-and-partition related problem.
PR 994
* config/tc-ia64.c (slot_index): Revert last change.  If first_frag
is NULL, then emit a warning, and return the current index.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 5ea1397..17abc8a 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,9 @@
+2005-06-08  James E Wilson  <wilson@specifixinc.com>
+
+	PR 994
+	* config/tc-ia64.c (slot_index): Revert last change.  If first_frag
+	is NULL, then emit a warning, and return the current index.
+
 2005-06-08  Tomas Hurka  <tom@hukatronic.cz>
 
 	PR991
diff --git a/gas/config/tc-ia64.c b/gas/config/tc-ia64.c
index b6eb6bb..cab32cd 100644
--- a/gas/config/tc-ia64.c
+++ b/gas/config/tc-ia64.c
@@ -2655,7 +2655,7 @@ slot_index (slot_addr, slot_frag, first_addr, first_frag, before_relax)
   /* If the two addresses are in different frags, then we need to add in
      the remaining size of this frag, and then the entire size of intermediate
      frags.  */
-  while (first_frag != NULL && slot_frag != first_frag)
+  while (slot_frag != first_frag)
     {
       unsigned long start_addr = (unsigned long) &first_frag->fr_literal;
 
@@ -2708,6 +2708,26 @@ slot_index (slot_addr, slot_frag, first_addr, first_frag, before_relax)
       /* Move to the beginning of the next frag.  */
       first_frag = first_frag->fr_next;
       first_addr = (unsigned long) &first_frag->fr_literal;
+
+      /* This can happen if there is section switching in the middle of a
+	 function, causing the frag chain for the function to be broken.  */
+      if (first_frag == NULL)
+	{
+	  /* We get six warnings for one problem, because of the loop in
+	     fixup_unw_records, and because fixup_unw_records is called 3
+	     times: once before creating the variant frag, once to estimate
+	     its size, and once to relax it.  This is unreasonable, so we use
+	     a static var to make sure we only emit the warning once.  */
+	  static int warned = 0;
+
+	  if (!warned)
+	    {
+	      as_warn ("Corrupted unwind info due to unsupported section switching");
+	      warned = 1;
+	    }
+
+	  return index;
+	}
     }
 
   /* Add in the used part of the last frag.  */