OSDN Git Service

PR fortran/25078
[pf3gnuchains/gcc-fork.git] / gcc / fortran / match.c
index 3b36178..fd4fe33 100644 (file)
@@ -1,5 +1,5 @@
 /* Matching subroutines in all sizes, shapes and colors.
 /* Matching subroutines in all sizes, shapes and colors.
-   Copyright (C) 2000, 2001, 2002, 2003, 2004 Free Software Foundation,
+   Copyright (C) 2000, 2001, 2002, 2003, 2004, 2005 Free Software Foundation,
    Inc.
    Contributed by Andy Vaught
 
    Inc.
    Contributed by Andy Vaught
 
@@ -17,8 +17,8 @@ for more details.
 
 You should have received a copy of the GNU General Public License
 along with GCC; see the file COPYING.  If not, write to the Free
 
 You should have received a copy of the GNU General Public License
 along with GCC; see the file COPYING.  If not, write to the Free
-Software Foundation, 59 Temple Place - Suite 330, Boston, MA
-02111-1307, USA.  */
+Software Foundation, 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301, USA.  */
 
 
 #include "config.h"
 
 
 #include "config.h"
@@ -217,7 +217,7 @@ gfc_match_small_int (int *value)
    do most of the work.  */
 
 match
    do most of the work.  */
 
 match
-gfc_match_st_label (gfc_st_label ** label, int allow_zero)
+gfc_match_st_label (gfc_st_label ** label)
 {
   locus old_loc;
   match m;
 {
   locus old_loc;
   match m;
@@ -229,13 +229,16 @@ gfc_match_st_label (gfc_st_label ** label, int allow_zero)
   if (m != MATCH_YES)
     return m;
 
   if (m != MATCH_YES)
     return m;
 
-  if (((i == 0) && allow_zero) || i <= 99999)
+  if (i > 0 && i <= 99999)
     {
       *label = gfc_get_st_label (i);
       return MATCH_YES;
     }
 
     {
       *label = gfc_get_st_label (i);
       return MATCH_YES;
     }
 
-  gfc_error ("Statement label at %C is out of range");
+  if (i == 0)
+    gfc_error ("Statement label at %C is zero");
+  else
+    gfc_error ("Statement label at %C is out of range");
   gfc_current_locus = old_loc;
   return MATCH_ERROR;
 }
   gfc_current_locus = old_loc;
   return MATCH_ERROR;
 }
@@ -250,7 +253,6 @@ match
 gfc_match_label (void)
 {
   char name[GFC_MAX_SYMBOL_LEN + 1];
 gfc_match_label (void)
 {
   char name[GFC_MAX_SYMBOL_LEN + 1];
-  gfc_state_data *p;
   match m;
 
   gfc_new_block = NULL;
   match m;
 
   gfc_new_block = NULL;
@@ -265,17 +267,15 @@ gfc_match_label (void)
       return MATCH_ERROR;
     }
 
       return MATCH_ERROR;
     }
 
-  if (gfc_new_block->attr.flavor != FL_LABEL
-      && gfc_add_flavor (&gfc_new_block->attr, FL_LABEL, NULL) == FAILURE)
-    return MATCH_ERROR;
+  if (gfc_new_block->attr.flavor == FL_LABEL)
+    {
+      gfc_error ("Duplicate construct label '%s' at %C", name);
+      return MATCH_ERROR;
+    }
 
 
-  for (p = gfc_state_stack; p; p = p->previous)
-    if (p->sym == gfc_new_block)
-      {
-       gfc_error ("Label %s at %C already in use by a parent block",
-                  gfc_new_block->name);
-       return MATCH_ERROR;
-      }
+  if (gfc_add_flavor (&gfc_new_block->attr, FL_LABEL,
+                     gfc_new_block->name, NULL) == FAILURE)
+    return MATCH_ERROR;
 
   return MATCH_YES;
 }
 
   return MATCH_YES;
 }
@@ -450,6 +450,8 @@ gfc_match_symbol (gfc_symbol ** matched_symbol, int host_assoc)
       else
         *matched_symbol = NULL;
     }
       else
         *matched_symbol = NULL;
     }
+  else
+    *matched_symbol = NULL;
   return m;
 }
 
   return m;
 }
 
@@ -691,7 +693,7 @@ loop:
 
        case 'l':
          label = va_arg (argp, gfc_st_label **);
 
        case 'l':
          label = va_arg (argp, gfc_st_label **);
-         n = gfc_match_st_label (label, 0);
+         n = gfc_match_st_label (label);
          if (n != MATCH_YES)
            {
              m = n;
          if (n != MATCH_YES)
            {
              m = n;
@@ -806,7 +808,7 @@ gfc_match_program (void)
   if (m == MATCH_ERROR)
     return m;
 
   if (m == MATCH_ERROR)
     return m;
 
-  if (gfc_add_flavor (&sym->attr, FL_PROGRAM, NULL) == FAILURE)
+  if (gfc_add_flavor (&sym->attr, FL_PROGRAM, sym->name, NULL) == FAILURE)
     return MATCH_ERROR;
 
   gfc_new_block = sym;
     return MATCH_ERROR;
 
   gfc_new_block = sym;
@@ -898,6 +900,43 @@ cleanup:
 }
 
 
 }
 
 
+/* We try to match an easy arithmetic IF statement. This only happens
+   when just after having encountered a simple IF statement. This code
+   is really duplicate with parts of the gfc_match_if code, but this is
+   *much* easier.  */
+static match
+match_arithmetic_if (void)
+{
+  gfc_st_label *l1, *l2, *l3;
+  gfc_expr *expr;
+  match m;
+
+  m = gfc_match (" ( %e ) %l , %l , %l%t", &expr, &l1, &l2, &l3);
+  if (m != MATCH_YES)
+    return m;
+
+  if (gfc_reference_st_label (l1, ST_LABEL_TARGET) == FAILURE
+      || gfc_reference_st_label (l2, ST_LABEL_TARGET) == FAILURE
+      || gfc_reference_st_label (l3, ST_LABEL_TARGET) == FAILURE)
+    {
+      gfc_free_expr (expr);
+      return MATCH_ERROR;
+    }
+
+  if (gfc_notify_std (GFC_STD_F95_DEL,
+                     "Obsolete: arithmetic IF statement at %C") == FAILURE)
+    return MATCH_ERROR;
+
+  new_st.op = EXEC_ARITHMETIC_IF;
+  new_st.expr = expr;
+  new_st.label = l1;
+  new_st.label2 = l2;
+  new_st.label3 = l3;
+
+  return MATCH_YES;
+}
+
+
 /* The IF statement is a bit of a pain.  First of all, there are three
    forms of it, the simple IF, the IF that starts a block and the
    arithmetic IF.
 /* The IF statement is a bit of a pain.  First of all, there are three
    forms of it, the simple IF, the IF that starts a block and the
    arithmetic IF.
@@ -959,6 +998,11 @@ gfc_match_if (gfc_statement * if_type)
          gfc_free_expr (expr);
          return MATCH_ERROR;
        }
          gfc_free_expr (expr);
          return MATCH_ERROR;
        }
+      
+      if (gfc_notify_std (GFC_STD_F95_DEL,
+                         "Obsolete: arithmetic IF statement at %C")
+         == FAILURE)
+        return MATCH_ERROR;
 
       new_st.op = EXEC_ARITHMETIC_IF;
       new_st.expr = expr;
 
       new_st.op = EXEC_ARITHMETIC_IF;
       new_st.expr = expr;
@@ -1033,8 +1077,10 @@ gfc_match_if (gfc_statement * if_type)
     match ("deallocate", gfc_match_deallocate, ST_DEALLOCATE)
     match ("end file", gfc_match_endfile, ST_END_FILE)
     match ("exit", gfc_match_exit, ST_EXIT)
     match ("deallocate", gfc_match_deallocate, ST_DEALLOCATE)
     match ("end file", gfc_match_endfile, ST_END_FILE)
     match ("exit", gfc_match_exit, ST_EXIT)
+    match ("flush", gfc_match_flush, ST_FLUSH)
     match ("forall", match_simple_forall, ST_FORALL)
     match ("go to", gfc_match_goto, ST_GOTO)
     match ("forall", match_simple_forall, ST_FORALL)
     match ("go to", gfc_match_goto, ST_GOTO)
+    match ("if", match_arithmetic_if, ST_ARITHMETIC_IF)
     match ("inquire", gfc_match_inquire, ST_INQUIRE)
     match ("nullify", gfc_match_nullify, ST_NULLIFY)
     match ("open", gfc_match_open, ST_OPEN)
     match ("inquire", gfc_match_inquire, ST_INQUIRE)
     match ("nullify", gfc_match_nullify, ST_NULLIFY)
     match ("open", gfc_match_open, ST_OPEN)
@@ -1199,7 +1245,7 @@ gfc_match_do (void)
   if (gfc_match (" do") != MATCH_YES)
     return MATCH_NO;
 
   if (gfc_match (" do") != MATCH_YES)
     return MATCH_NO;
 
-  m = gfc_match_st_label (&label, 0);
+  m = gfc_match_st_label (&label);
   if (m == MATCH_ERROR)
     goto cleanup;
 
   if (m == MATCH_ERROR)
     goto cleanup;
 
@@ -1232,7 +1278,7 @@ gfc_match_do (void)
   gfc_match_label ();          /* This won't error */
   gfc_match (" do ");          /* This will work */
 
   gfc_match_label ();          /* This won't error */
   gfc_match (" do ");          /* This will work */
 
-  gfc_match_st_label (&label, 0);      /* Can't error out */
+  gfc_match_st_label (&label); /* Can't error out */
   gfc_match_char (',');                /* Optional comma */
 
   m = gfc_match_iterator (&iter, 0);
   gfc_match_char (',');                /* Optional comma */
 
   m = gfc_match_iterator (&iter, 0);
@@ -1362,7 +1408,7 @@ gfc_match_stopcode (gfc_statement st)
   gfc_expr *e;
   match m;
 
   gfc_expr *e;
   match m;
 
-  stop_code = 0;
+  stop_code = -1;
   e = NULL;
 
   if (gfc_match_eos () != MATCH_YES)
   e = NULL;
 
   if (gfc_match_eos () != MATCH_YES)
@@ -1525,7 +1571,6 @@ gfc_match_goto (void)
          == FAILURE)
        return MATCH_ERROR;
 
          == FAILURE)
        return MATCH_ERROR;
 
-      expr->symtree->n.sym->attr.assign = 1;
       new_st.op = EXEC_GOTO;
       new_st.expr = expr;
 
       new_st.op = EXEC_GOTO;
       new_st.expr = expr;
 
@@ -1543,7 +1588,7 @@ gfc_match_goto (void)
 
       do
        {
 
       do
        {
-         m = gfc_match_st_label (&label, 0);
+         m = gfc_match_st_label (&label);
          if (m != MATCH_YES)
            goto syntax;
 
          if (m != MATCH_YES)
            goto syntax;
 
@@ -1589,7 +1634,7 @@ gfc_match_goto (void)
 
   do
     {
 
   do
     {
-      m = gfc_match_st_label (&label, 0);
+      m = gfc_match_st_label (&label);
       if (m != MATCH_YES)
        goto syntax;
 
       if (m != MATCH_YES)
        goto syntax;
 
@@ -1940,12 +1985,7 @@ gfc_match_return (void)
   gfc_expr *e;
   match m;
   gfc_compile_state s;
   gfc_expr *e;
   match m;
   gfc_compile_state s;
-
-  gfc_enclosing_unit (&s);
-  if (s == COMP_PROGRAM
-      && gfc_notify_std (GFC_STD_GNU, "Extension: RETURN statement in "
-                        "main program at %C") == FAILURE)
-      return MATCH_ERROR;
+  int c;
 
   e = NULL;
   if (gfc_match_eos () == MATCH_YES)
 
   e = NULL;
   if (gfc_match_eos () == MATCH_YES)
@@ -1958,7 +1998,18 @@ gfc_match_return (void)
       goto cleanup;
     }
 
       goto cleanup;
     }
 
-  m = gfc_match ("% %e%t", &e);
+  if (gfc_current_form == FORM_FREE)
+    {
+      /* The following are valid, so we can't require a blank after the
+        RETURN keyword:
+          return+1
+          return(1)  */
+      c = gfc_peek_char ();
+      if (ISALPHA (c) || ISDIGIT (c))
+       return MATCH_NO;
+    }
+
+  m = gfc_match (" %e%t", &e);
   if (m == MATCH_YES)
     goto done;
   if (m == MATCH_ERROR)
   if (m == MATCH_YES)
     goto done;
   if (m == MATCH_ERROR)
@@ -1971,6 +2022,12 @@ cleanup:
   return MATCH_ERROR;
 
 done:
   return MATCH_ERROR;
 
 done:
+  gfc_enclosing_unit (&s);
+  if (s == COMP_PROGRAM
+      && gfc_notify_std (GFC_STD_GNU, "Extension: RETURN statement in "
+                        "main program at %C") == FAILURE)
+      return MATCH_ERROR;
+
   new_st.op = EXEC_RETURN;
   new_st.expr = e;
 
   new_st.op = EXEC_RETURN;
   new_st.expr = e;
 
@@ -2013,7 +2070,7 @@ gfc_match_call (void)
 
   if (!sym->attr.generic
       && !sym->attr.subroutine
 
   if (!sym->attr.generic
       && !sym->attr.subroutine
-      && gfc_add_subroutine (&sym->attr, NULL) == FAILURE)
+      && gfc_add_subroutine (&sym->attr, sym->name, NULL) == FAILURE)
     return MATCH_ERROR;
 
   if (gfc_match_eos () != MATCH_YES)
     return MATCH_ERROR;
 
   if (gfc_match_eos () != MATCH_YES)
@@ -2172,10 +2229,11 @@ match_common_name (char *name)
 match
 gfc_match_common (void)
 {
 match
 gfc_match_common (void)
 {
-  gfc_symbol *sym, **head, *tail, *old_blank_common;
+  gfc_symbol *sym, **head, *tail, *other, *old_blank_common;
   char name[GFC_MAX_SYMBOL_LEN+1];
   gfc_common_head *t;
   gfc_array_spec *as;
   char name[GFC_MAX_SYMBOL_LEN+1];
   gfc_common_head *t;
   gfc_array_spec *as;
+  gfc_equiv * e1, * e2;
   match m;
 
   old_blank_common = gfc_current_ns->blank_common.head;
   match m;
 
   old_blank_common = gfc_current_ns->blank_common.head;
@@ -2187,9 +2245,6 @@ gfc_match_common (void)
 
   as = NULL;
 
 
   as = NULL;
 
-  if (gfc_match_eos () == MATCH_YES)
-    goto syntax;
-
   for (;;)
     {
       m = match_common_name (name);
   for (;;)
     {
       m = match_common_name (name);
@@ -2219,9 +2274,6 @@ gfc_match_common (void)
        }
 
       /* Grab the list of symbols.  */
        }
 
       /* Grab the list of symbols.  */
-      if (gfc_match_eos () == MATCH_YES)
-       goto done;
-  
       for (;;)
        {
          m = gfc_match_symbol (&sym, 0);
       for (;;)
        {
          m = gfc_match_symbol (&sym, 0);
@@ -2237,7 +2289,7 @@ gfc_match_common (void)
              goto cleanup;
            }
 
              goto cleanup;
            }
 
-         if (gfc_add_in_common (&sym->attr, NULL) == FAILURE) 
+         if (gfc_add_in_common (&sym->attr, sym->name, NULL) == FAILURE) 
            goto cleanup;
 
          if (sym->value != NULL
            goto cleanup;
 
          if (sym->value != NULL
@@ -2252,7 +2304,7 @@ gfc_match_common (void)
              goto cleanup;
            }
 
              goto cleanup;
            }
 
-         if (gfc_add_in_common (&sym->attr, NULL) == FAILURE)
+         if (gfc_add_in_common (&sym->attr, sym->name, NULL) == FAILURE)
            goto cleanup;
 
          /* Derived type names must have the SEQUENCE attribute.  */
            goto cleanup;
 
          /* Derived type names must have the SEQUENCE attribute.  */
@@ -2287,7 +2339,7 @@ gfc_match_common (void)
                  goto cleanup;
                }
 
                  goto cleanup;
                }
 
-             if (gfc_add_dimension (&sym->attr, NULL) == FAILURE)
+             if (gfc_add_dimension (&sym->attr, sym->name, NULL) == FAILURE)
                goto cleanup;
 
              if (sym->attr.pointer)
                goto cleanup;
 
              if (sym->attr.pointer)
@@ -2300,8 +2352,46 @@ gfc_match_common (void)
 
              sym->as = as;
              as = NULL;
 
              sym->as = as;
              as = NULL;
+
+           }
+
+         sym->common_head = t;
+
+         /* Check to see if the symbol is already in an equivalence group.
+            If it is, set the other members as being in common.  */
+         if (sym->attr.in_equivalence)
+           {
+             for (e1 = gfc_current_ns->equiv; e1; e1 = e1->next)
+               {
+                 for (e2 = e1; e2; e2 = e2->eq)
+                   if (e2->expr->symtree->n.sym == sym)
+                     goto equiv_found;
+
+                 continue;
+
+         equiv_found:
+
+                 for (e2 = e1; e2; e2 = e2->eq)
+                   {
+                     other = e2->expr->symtree->n.sym;
+                     if (other->common_head
+                           && other->common_head != sym->common_head)
+                       {
+                         gfc_error ("Symbol '%s', in COMMON block '%s' at "
+                                    "%C is being indirectly equivalenced to "
+                                    "another COMMON block '%s'",
+                                    sym->name,
+                                    sym->common_head->name,
+                                    other->common_head->name);
+                           goto cleanup;
+                       }
+                     other->attr.in_common = 1;
+                     other->common_head = t;
+                   }
+               }
            }
 
            }
 
+
          gfc_gobble_whitespace ();
          if (gfc_match_eos () == MATCH_YES)
            goto done;
          gfc_gobble_whitespace ();
          if (gfc_match_eos () == MATCH_YES)
            goto done;
@@ -2353,7 +2443,7 @@ gfc_match_block_data (void)
   if (gfc_get_symbol (name, NULL, &sym))
     return MATCH_ERROR;
 
   if (gfc_get_symbol (name, NULL, &sym))
     return MATCH_ERROR;
 
-  if (gfc_add_flavor (&sym->attr, FL_BLOCK_DATA, NULL) == FAILURE)
+  if (gfc_add_flavor (&sym->attr, FL_BLOCK_DATA, sym->name, NULL) == FAILURE)
     return MATCH_ERROR;
 
   gfc_new_block = sym;
     return MATCH_ERROR;
 
   gfc_new_block = sym;
@@ -2403,7 +2493,8 @@ gfc_match_namelist (void)
        }
 
       if (group_name->attr.flavor != FL_NAMELIST
        }
 
       if (group_name->attr.flavor != FL_NAMELIST
-         && gfc_add_flavor (&group_name->attr, FL_NAMELIST, NULL) == FAILURE)
+         && gfc_add_flavor (&group_name->attr, FL_NAMELIST,
+                            group_name->name, NULL) == FAILURE)
        return MATCH_ERROR;
 
       for (;;)
        return MATCH_ERROR;
 
       for (;;)
@@ -2415,12 +2506,9 @@ gfc_match_namelist (void)
            goto error;
 
          if (sym->attr.in_namelist == 0
            goto error;
 
          if (sym->attr.in_namelist == 0
-             && gfc_add_in_namelist (&sym->attr, NULL) == FAILURE)
+             && gfc_add_in_namelist (&sym->attr, sym->name, NULL) == FAILURE)
            goto error;
 
            goto error;
 
-         /* TODO: worry about PRIVATE members of a PUBLIC namelist
-             group.  */
-
          nl = gfc_get_namelist ();
          nl->sym = sym;
 
          nl = gfc_get_namelist ();
          nl->sym = sym;
 
@@ -2474,7 +2562,8 @@ gfc_match_module (void)
   if (m != MATCH_YES)
     return m;
 
   if (m != MATCH_YES)
     return m;
 
-  if (gfc_add_flavor (&gfc_new_block->attr, FL_MODULE, NULL) == FAILURE)
+  if (gfc_add_flavor (&gfc_new_block->attr, FL_MODULE,
+                     gfc_new_block->name, NULL) == FAILURE)
     return MATCH_ERROR;
 
   return MATCH_YES;
     return MATCH_ERROR;
 
   return MATCH_YES;
@@ -2506,7 +2595,11 @@ gfc_match_equivalence (void)
 {
   gfc_equiv *eq, *set, *tail;
   gfc_ref *ref;
 {
   gfc_equiv *eq, *set, *tail;
   gfc_ref *ref;
+  gfc_symbol *sym;
   match m;
   match m;
+  gfc_common_head *common_head = NULL;
+  bool common_flag;
+  int cnt;
 
   tail = NULL;
 
 
   tail = NULL;
 
@@ -2523,15 +2616,27 @@ gfc_match_equivalence (void)
        goto syntax;
 
       set = eq;
        goto syntax;
 
       set = eq;
+      common_flag = FALSE;
+      cnt = 0;
 
       for (;;)
        {
 
       for (;;)
        {
-         m = gfc_match_variable (&set->expr, 1);
+         m = gfc_match_equiv_variable (&set->expr);
          if (m == MATCH_ERROR)
            goto cleanup;
          if (m == MATCH_NO)
            goto syntax;
 
          if (m == MATCH_ERROR)
            goto cleanup;
          if (m == MATCH_NO)
            goto syntax;
 
+         /*  count the number of objects.  */
+         cnt++;
+
+         if (gfc_match_char ('%') == MATCH_YES)
+           {
+             gfc_error ("Derived type component %C is not a "
+                        "permitted EQUIVALENCE member");
+             goto cleanup;
+           }
+
          for (ref = set->expr->ref; ref; ref = ref->next)
            if (ref->type == REF_ARRAY && ref->u.ar.type == AR_SECTION)
              {
          for (ref = set->expr->ref; ref; ref = ref->next)
            if (ref->type == REF_ARRAY && ref->u.ar.type == AR_SECTION)
              {
@@ -2541,8 +2646,21 @@ gfc_match_equivalence (void)
                goto cleanup;
              }
 
                goto cleanup;
              }
 
+         sym = set->expr->symtree->n.sym;
+
+         if (gfc_add_in_equivalence (&sym->attr, sym->name, NULL)
+               == FAILURE)
+           goto cleanup;
+
+         if (sym->attr.in_common)
+           {
+             common_flag = TRUE;
+             common_head = sym->common_head;
+           }
+
          if (gfc_match_char (')') == MATCH_YES)
            break;
          if (gfc_match_char (')') == MATCH_YES)
            break;
+
          if (gfc_match_char (',') != MATCH_YES)
            goto syntax;
 
          if (gfc_match_char (',') != MATCH_YES)
            goto syntax;
 
@@ -2550,6 +2668,32 @@ gfc_match_equivalence (void)
          set = set->eq;
        }
 
          set = set->eq;
        }
 
+      if (cnt < 2)
+       {
+         gfc_error ("EQUIVALENCE at %C requires two or more objects");
+         goto cleanup;
+       }
+
+      /* If one of the members of an equivalence is in common, then
+        mark them all as being in common.  Before doing this, check
+        that members of the equivalence group are not in different
+        common blocks. */
+      if (common_flag)
+       for (set = eq; set; set = set->eq)
+         {
+           sym = set->expr->symtree->n.sym;
+           if (sym->common_head && sym->common_head != common_head)
+             {
+               gfc_error ("Attempt to indirectly overlap COMMON "
+                          "blocks %s and %s by EQUIVALENCE at %C",
+                          sym->common_head->name,
+                          common_head->name);
+               goto cleanup;
+             }
+           sym->attr.in_common = 1;
+           sym->common_head = common_head;
+         }
+
       if (gfc_match_eos () == MATCH_YES)
        break;
       if (gfc_match_char (',') != MATCH_YES)
       if (gfc_match_eos () == MATCH_YES)
        break;
       if (gfc_match_char (',') != MATCH_YES)
@@ -2571,6 +2715,91 @@ cleanup:
   return MATCH_ERROR;
 }
 
   return MATCH_ERROR;
 }
 
+/* Check that a statement function is not recursive. This is done by looking
+   for the statement function symbol(sym) by looking recursively through its
+   expression(e).  If a reference to sym is found, true is returned.  */
+static bool
+recursive_stmt_fcn (gfc_expr *e, gfc_symbol *sym)
+{
+  gfc_actual_arglist *arg;
+  gfc_ref *ref;
+  int i;
+
+  if (e == NULL)
+    return false;
+
+  switch (e->expr_type)
+    {
+    case EXPR_FUNCTION:
+      for (arg = e->value.function.actual; arg; arg = arg->next)
+       {
+         if (sym->name == arg->name
+               || recursive_stmt_fcn (arg->expr, sym))
+           return true;
+       }
+
+      if (e->symtree == NULL)
+       return false;
+
+      /* Check the name before testing for nested recursion!  */
+      if (sym->name == e->symtree->n.sym->name)
+       return true;
+
+      /* Catch recursion via other statement functions.  */
+      if (e->symtree->n.sym->attr.proc == PROC_ST_FUNCTION
+           && e->symtree->n.sym->value
+           && recursive_stmt_fcn (e->symtree->n.sym->value, sym))
+       return true;
+
+      break;
+
+    case EXPR_VARIABLE:
+      if (e->symtree && sym->name == e->symtree->n.sym->name)
+       return true;
+      break;
+
+    case EXPR_OP:
+      if (recursive_stmt_fcn (e->value.op.op1, sym)
+           || recursive_stmt_fcn (e->value.op.op2, sym))
+       return true;
+      break;
+
+    default:
+      break;
+    }
+
+  /* Component references do not need to be checked.  */
+  if (e->ref)
+    {
+      for (ref = e->ref; ref; ref = ref->next)
+       {
+         switch (ref->type)
+           {
+           case REF_ARRAY:
+             for (i = 0; i < ref->u.ar.dimen; i++)
+               {
+                 if (recursive_stmt_fcn (ref->u.ar.start[i], sym)
+                       || recursive_stmt_fcn (ref->u.ar.end[i], sym)
+                       || recursive_stmt_fcn (ref->u.ar.stride[i], sym))
+                   return true;
+               }
+             break;
+
+           case REF_SUBSTRING:
+             if (recursive_stmt_fcn (ref->u.ss.start, sym)
+                   || recursive_stmt_fcn (ref->u.ss.end, sym))
+               return true;
+
+             break;
+
+           default:
+             break;
+           }
+       }
+    }
+  return false;
+}
+
 
 /* Match a statement function declaration.  It is so easy to match
    non-statement function statements with a MATCH_ERROR as opposed to
 
 /* Match a statement function declaration.  It is so easy to match
    non-statement function statements with a MATCH_ERROR as opposed to
@@ -2590,7 +2819,8 @@ gfc_match_st_function (void)
 
   gfc_push_error (&old_error);
 
 
   gfc_push_error (&old_error);
 
-  if (gfc_add_procedure (&sym->attr, PROC_ST_FUNCTION, NULL) == FAILURE)
+  if (gfc_add_procedure (&sym->attr, PROC_ST_FUNCTION,
+                        sym->name, NULL) == FAILURE)
     goto undo_error;
 
   if (gfc_match_formal_arglist (sym, 1, 0) != MATCH_YES)
     goto undo_error;
 
   if (gfc_match_formal_arglist (sym, 1, 0) != MATCH_YES)
@@ -2599,9 +2829,18 @@ gfc_match_st_function (void)
   m = gfc_match (" = %e%t", &expr);
   if (m == MATCH_NO)
     goto undo_error;
   m = gfc_match (" = %e%t", &expr);
   if (m == MATCH_NO)
     goto undo_error;
+
+  gfc_free_error (&old_error);
   if (m == MATCH_ERROR)
     return m;
 
   if (m == MATCH_ERROR)
     return m;
 
+  if (recursive_stmt_fcn (expr, sym))
+    {
+      gfc_error ("Statement function at %L is recursive",
+                &expr->where);
+      return MATCH_ERROR;
+    }
+
   sym->value = expr;
 
   return MATCH_YES;
   sym->value = expr;
 
   return MATCH_YES;
@@ -3029,9 +3268,7 @@ match_forall_iterator (gfc_forall_iterator ** result)
     }
 
   m = gfc_match_expr (&iter->start);
     }
 
   m = gfc_match_expr (&iter->start);
-  if (m == MATCH_NO)
-    goto syntax;
-  if (m == MATCH_ERROR)
+  if (m != MATCH_YES)
     goto cleanup;
 
   if (gfc_match_char (':') != MATCH_YES)
     goto cleanup;
 
   if (gfc_match_char (':') != MATCH_YES)