From: Jakub Jelinek <jakub@redhat.com>
Date: Tue, 26 Apr 2016 20:32:05 +0000 (+0200)
Subject: reorg.c (try_merge_delay_insns): Declare i and j inside the for loops rather than... 
X-Git-Tag: upstream/12.2.0~47597
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5e851c020abe8d78f73ace4ff8a3a2e047662865;p=platform%2Fupstream%2Fgcc.git

reorg.c (try_merge_delay_insns): Declare i and j inside the for loops rather than one for the whole function.

	* reorg.c (try_merge_delay_insns): Declare i and j inside the
	for loops rather than one for the whole function.

From-SVN: r235451
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 325eb87..2385291 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2016-04-26  Jakub Jelinek  <jakub@redhat.com>
+
+	* reorg.c (try_merge_delay_insns): Declare i and j inside the
+	for loops rather than one for the whole function.
+
 2016-04-26  Marc Glisse  <marc.glisse@inria.fr>
 
 	* match.pd (X + CST CMP X): New transformation.
diff --git a/gcc/reorg.c b/gcc/reorg.c
index 8fb912a..c58d608 100644
--- a/gcc/reorg.c
+++ b/gcc/reorg.c
@@ -1260,7 +1260,6 @@ try_merge_delay_insns (rtx_insn *insn, rtx_insn *thread)
   rtx next_to_match = XVECEXP (PATTERN (insn), 0, slot_number);
   struct resources set, needed, modified;
   auto_vec<std::pair<rtx_insn *, bool>, 10> merged_insns;
-  int i, j;
   int flags;
 
   flags = get_jump_flags (delay_insn, JUMP_LABEL (delay_insn));
@@ -1275,7 +1274,7 @@ try_merge_delay_insns (rtx_insn *insn, rtx_insn *thread)
      will essentially disable this optimization.  This method is somewhat of
      a kludge, but I don't see a better way.)  */
   if (! annul_p)
-    for (i = 1 ; i < num_slots; i++)
+    for (int i = 1; i < num_slots; i++)
       if (XVECEXP (PATTERN (insn), 0, i))
 	mark_referenced_resources (XVECEXP (PATTERN (insn), 0, i), &needed,
 				   true);
@@ -1346,19 +1345,19 @@ try_merge_delay_insns (rtx_insn *insn, rtx_insn *thread)
       mark_set_resources (filled_insn, &set, 0, MARK_SRC_DEST_CALL);
       mark_referenced_resources (filled_insn, &needed, true);
 
-      for (i = 1; i < pat->len (); i++)
+      for (int i = 1; i < pat->len (); i++)
 	{
 	  rtx_insn *dtrial = pat->insn (i);
 
 	  CLEAR_RESOURCE (&modified);
 	  /* Account for resources set by the insn following NEXT_TO_MATCH
 	     inside INSN's delay list. */
-	  for (j = 1; slot_number + j < num_slots; j++)
+	  for (int j = 1; slot_number + j < num_slots; j++)
 	    mark_set_resources (XVECEXP (PATTERN (insn), 0, slot_number + j),
 				&modified, 0, MARK_SRC_DEST_CALL);
 	  /* Account for resources set by the insn before DTRIAL and inside
 	     TRIAL's delay list. */
-	  for (j = 1; j < i; j++)
+	  for (int j = 1; j < i; j++)
 	    mark_set_resources (XVECEXP (pat, 0, j),
 				&modified, 0, MARK_SRC_DEST_CALL); 
 	  if (! insn_references_resource_p (dtrial, &set, true)
@@ -1411,24 +1410,22 @@ try_merge_delay_insns (rtx_insn *insn, rtx_insn *thread)
     {
       unsigned int len = merged_insns.length ();
       for (unsigned int i = len - 1; i < len; i--)
-	{
-	  if (merged_insns[i].second)
-	    {
-	      update_block (merged_insns[i].first, thread);
-	      rtx_insn *new_rtx = delete_from_delay_slot (merged_insns[i].first);
-	      if (thread->deleted ())
-		thread = new_rtx;
-	    }
-	  else
-	    {
-	      update_block (merged_insns[i].first, thread);
-	      delete_related_insns (merged_insns[i].first);
-	    }
-	}
+	if (merged_insns[i].second)
+	  {
+	    update_block (merged_insns[i].first, thread);
+	    rtx_insn *new_rtx = delete_from_delay_slot (merged_insns[i].first);
+	    if (thread->deleted ())
+	      thread = new_rtx;
+	  }
+	else
+	  {
+	    update_block (merged_insns[i].first, thread);
+	    delete_related_insns (merged_insns[i].first);
+	  }
 
       INSN_ANNULLED_BRANCH_P (delay_insn) = 0;
 
-      for (i = 0; i < XVECLEN (PATTERN (insn), 0); i++)
+      for (int i = 0; i < XVECLEN (PATTERN (insn), 0); i++)
 	INSN_FROM_TARGET_P (XVECEXP (PATTERN (insn), 0, i)) = 0;
     }
 }