[Patch, PR 57802] Wasted work in set_loop_bounds()

[pchang9@cs.wisc.edu]

Hi,

The problem appears in revision 201034 in version 4.9. I attached a
one-line patch that fixes it.  I also reported this problem
at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57802

Bootstrap and regression-tested on x86_64-linux.

In method "set_loop_bounds()" in gcc/fortran/trans-array.c, the loop
on line 4456 should break immediately after "nonoptional_arr" is set
to "true".

2013-07-23  Chang  <pchang9@cs.wisc.edu>

        * trans-array.c (set_loop_bounds): Exit loop after
        setting nonoptional_arr.


Index: gcc/fortran/trans-array.c
===================================================================
--- gcc/fortran/trans-array.c	(revision 201034)
+++ gcc/fortran/trans-array.c	(working copy)
@@ -4456,7 +4456,10 @@
       for (ss = loop->ss; ss != gfc_ss_terminator; ss = ss->loop_chain)
 	if (ss->info->type != GFC_SS_SCALAR && ss->info->type != GFC_SS_TEMP
 	    && ss->info->type != GFC_SS_REFERENCE && !ss->info->can_be_null_ref)
-	  nonoptional_arr = true;
+	  {
+	    nonoptional_arr = true;
+	    break;
+	  }

       /* We use one SS term, and use that to determine the bounds of the
 	 loop for this dimension.  We try to pick the simplest term.  */



-Chang
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pr57802_patch.diff
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130723/24f53f47/attachment.ksh>