OSDN Git Service

2010-10-06 Mikael Morin <mikael@gcc.gnu.org>
[pf3gnuchains/gcc-fork.git] / gcc / fortran / primary.c
index 113729f..f6ceae9 100644 (file)
@@ -1,5 +1,5 @@
 /* Primary expression subroutines
-   Copyright (C) 2000, 2001, 2002, 2004, 2005, 2006, 2007, 2008
+   Copyright (C) 2000, 2001, 2002, 2004, 2005, 2006, 2007, 2008, 2009, 2010
    Free Software Foundation, Inc.
    Contributed by Andy Vaught
 
@@ -26,7 +26,7 @@ along with GCC; see the file COPYING3.  If not see
 #include "arith.h"
 #include "match.h"
 #include "parse.h"
-#include "toplev.h"
+#include "constructor.h"
 
 /* Matches a kind-parameter expression, which is either a named
    symbolic constant or a nonnegative integer constant.  If
@@ -242,7 +242,7 @@ match_hollerith_constant (gfc_expr **result)
   locus old_loc;
   gfc_expr *e = NULL;
   const char *msg;
-  int num;
+  int num, pad;
   int i;  
 
   old_loc = gfc_current_locus;
@@ -276,10 +276,13 @@ match_hollerith_constant (gfc_expr **result)
       else
        {
          gfc_free_expr (e);
-         e = gfc_constant_result (BT_HOLLERITH, gfc_default_character_kind,
-                                  &gfc_current_locus);
+         e = gfc_get_constant_expr (BT_HOLLERITH, gfc_default_character_kind,
+                                    &gfc_current_locus);
 
-         e->representation.string = XCNEWVEC (char, num + 1);
+         /* Calculate padding needed to fit default integer memory.  */
+         pad = gfc_default_integer_kind - (num % gfc_default_integer_kind);
+
+         e->representation.string = XCNEWVEC (char, num + pad + 1);
 
          for (i = 0; i < num; i++)
            {
@@ -294,8 +297,13 @@ match_hollerith_constant (gfc_expr **result)
              e->representation.string[i] = (unsigned char) c;
            }
 
-         e->representation.string[num] = '\0';
-         e->representation.length = num;
+         /* Now pad with blanks and end with a null char.  */
+         for (i = 0; i < pad; i++)
+           e->representation.string[num + i] = ' ';
+
+         e->representation.string[num + i] = '\0';
+         e->representation.length = num + pad;
+         e->ts.u.pad = pad;
 
          *result = e;
          return MATCH_YES;
@@ -711,7 +719,7 @@ match_substring (gfc_charlen *cl, int init, gfc_ref **result)
 
       ref->type = REF_SUBSTRING;
       if (start == NULL)
-       start = gfc_int_expr (1);
+       start = gfc_get_int_expr (gfc_default_integer_kind, NULL, 1);
       ref->u.ss.start = start;
       if (end == NULL && cl)
        end = gfc_copy_expr (cl->length);
@@ -867,12 +875,11 @@ match_string_constant (gfc_expr **result)
 
   gfc_gobble_whitespace ();
 
-  start_locus = gfc_current_locus;
-
   c = gfc_next_char ();
   if (c == '\'' || c == '"')
     {
       kind = gfc_default_character_kind;
+      start_locus = gfc_current_locus;
       goto got_delim;
     }
 
@@ -916,12 +923,13 @@ match_string_constant (gfc_expr **result)
     goto no_match;
 
   gfc_gobble_whitespace ();
-  start_locus = gfc_current_locus;
 
   c = gfc_next_char ();
   if (c != '\'' && c != '"')
     goto no_match;
 
+  start_locus = gfc_current_locus;
+
   if (kind == -1)
     {
       q = gfc_extract_int (sym->value, &kind);
@@ -969,28 +977,19 @@ got_delim:
   if (peek == 'b' || peek == 'o' || peek =='z' || peek == 'x')
     goto no_match;
 
-
-  e = gfc_get_expr ();
-
-  e->expr_type = EXPR_CONSTANT;
+  e = gfc_get_character_expr (kind, &start_locus, NULL, length);
   e->ref = NULL;
-  e->ts.type = BT_CHARACTER;
-  e->ts.kind = kind;
   e->ts.is_c_interop = 0;
   e->ts.is_iso_c = 0;
-  e->where = start_locus;
-
-  e->value.character.string = p = gfc_get_wide_string (length + 1);
-  e->value.character.length = length;
 
   gfc_current_locus = start_locus;
-  gfc_next_char ();            /* Skip delimiter */
 
   /* We disable the warning for the following loop as the warning has already
      been printed in the loop above.  */
   warn_ampersand = gfc_option.warn_ampersand;
   gfc_option.warn_ampersand = 0;
 
+  p = e->value.character.string;
   for (i = 0; i < length; i++)
     {
       c = next_string_char (delimiter, &ret);
@@ -1084,15 +1083,9 @@ match_logical_constant (gfc_expr **result)
       return MATCH_ERROR;
     }
 
-  e = gfc_get_expr ();
-
-  e->expr_type = EXPR_CONSTANT;
-  e->value.logical = i;
-  e->ts.type = BT_LOGICAL;
-  e->ts.kind = kind;
+  e = gfc_get_logical_expr (kind, &gfc_current_locus, i);
   e->ts.is_c_interop = 0;
   e->ts.is_iso_c = 0;
-  e->where = gfc_current_locus;
 
   *result = e;
   return MATCH_YES;
@@ -1746,13 +1739,37 @@ gfc_match_varspec (gfc_expr *primary, int equiv_flag, bool sub_flag,
   tail = NULL;
 
   gfc_gobble_whitespace ();
+
+  if (gfc_peek_ascii_char () == '[')
+    {
+      if (sym->attr.dimension)
+       {
+         gfc_error ("Array section designator, e.g. '(:)', is required "
+                    "besides the coarray designator '[...]' at %C");
+         return MATCH_ERROR;
+       }
+      if (!sym->attr.codimension)
+       {
+         gfc_error ("Coarray designator at %C but '%s' is not a coarray",
+                    sym->name);
+         return MATCH_ERROR;
+       }
+    }
+
+  /* For associate names, we may not yet know whether they are arrays or not.
+     Thus if we have one and parentheses follow, we have to assume that it
+     actually is one for now.  The final decision will be made at
+     resolution time, of course.  */
+  if (sym->assoc && gfc_peek_ascii_char () == '(')
+    sym->attr.dimension = 1;
+
   if ((equiv_flag && gfc_peek_ascii_char () == '(')
+      || gfc_peek_ascii_char () == '[' || sym->attr.codimension
       || (sym->attr.dimension && !sym->attr.proc_pointer
          && !gfc_is_proc_ptr_comp (primary, NULL)
          && !(gfc_matching_procptr_assignment
               && sym->attr.flavor == FL_PROCEDURE))
-      || (sym->ts.type == BT_CLASS
-         && sym->ts.u.derived->components->attr.dimension))
+      || (sym->ts.type == BT_CLASS && CLASS_DATA (sym)->attr.dimension))
     {
       /* In EQUIVALENCE, we don't know yet whether we are seeing
         an array, character variable or array of character
@@ -1761,7 +1778,7 @@ gfc_match_varspec (gfc_expr *primary, int equiv_flag, bool sub_flag,
       tail->type = REF_ARRAY;
 
       m = gfc_match_array_ref (&tail->u.ar, equiv_flag ? NULL : sym->as,
-                              equiv_flag);
+                              equiv_flag, sym->as ? sym->as->corank : 0);
       if (m != MATCH_YES)
        return m;
 
@@ -1771,7 +1788,7 @@ gfc_match_varspec (gfc_expr *primary, int equiv_flag, bool sub_flag,
          tail = extend_ref (primary, tail);
          tail->type = REF_ARRAY;
 
-         m = gfc_match_array_ref (&tail->u.ar, NULL, equiv_flag);
+         m = gfc_match_array_ref (&tail->u.ar, NULL, equiv_flag, 0);
          if (m != MATCH_YES)
            return m;
        }
@@ -1881,20 +1898,21 @@ gfc_match_varspec (gfc_expr *primary, int equiv_flag, bool sub_flag,
          tail = extend_ref (primary, tail);
          tail->type = REF_ARRAY;
 
-         m = gfc_match_array_ref (&tail->u.ar, component->as, equiv_flag);
+         m = gfc_match_array_ref (&tail->u.ar, component->as, equiv_flag,
+                         component->as->corank);
          if (m != MATCH_YES)
            return m;
        }
       else if (component->ts.type == BT_CLASS
-              && component->ts.u.derived->components->as != NULL
+              && CLASS_DATA (component)->as != NULL
               && !component->attr.proc_pointer)
        {
          tail = extend_ref (primary, tail);
          tail->type = REF_ARRAY;
 
-         m = gfc_match_array_ref (&tail->u.ar,
-                                  component->ts.u.derived->components->as,
-                                  equiv_flag);
+         m = gfc_match_array_ref (&tail->u.ar, CLASS_DATA (component)->as,
+                                  equiv_flag,
+                                  CLASS_DATA (component)->as->corank);
          if (m != MATCH_YES)
            return m;
        }
@@ -1949,6 +1967,13 @@ check_substring:
        }
     }
 
+  /* F2008, C727.  */
+  if (primary->expr_type == EXPR_PPC && gfc_is_coindexed (primary))
+    {
+      gfc_error ("Coindexed procedure-pointer component at %C");
+      return MATCH_ERROR;
+    }
+
   return MATCH_YES;
 }
 
@@ -1982,15 +2007,14 @@ gfc_variable_attr (gfc_expr *expr, gfc_typespec *ts)
   if (expr->expr_type != EXPR_VARIABLE && expr->expr_type != EXPR_FUNCTION)
     gfc_internal_error ("gfc_variable_attr(): Expression isn't a variable");
 
-  ref = expr->ref;
   sym = expr->symtree->n.sym;
   attr = sym->attr;
 
   if (sym->ts.type == BT_CLASS)
     {
-      dimension = sym->ts.u.derived->components->attr.dimension;
-      pointer = sym->ts.u.derived->components->attr.pointer;
-      allocatable = sym->ts.u.derived->components->attr.allocatable;
+      dimension = CLASS_DATA (sym)->attr.dimension;
+      pointer = CLASS_DATA (sym)->attr.class_pointer;
+      allocatable = CLASS_DATA (sym)->attr.allocatable;
     }
   else
     {
@@ -2006,7 +2030,7 @@ gfc_variable_attr (gfc_expr *expr, gfc_typespec *ts)
   if (ts != NULL && expr->ts.type == BT_UNKNOWN)
     *ts = sym->ts;
 
-  for (; ref; ref = ref->next)
+  for (ref = expr->ref; ref; ref = ref->next)
     switch (ref->type)
       {
       case REF_ARRAY:
@@ -2023,7 +2047,9 @@ gfc_variable_attr (gfc_expr *expr, gfc_typespec *ts)
            break;
 
          case AR_ELEMENT:
-           allocatable = pointer = 0;
+           /* Handle coarrays.  */
+           if (ref->u.ar.dimen > 0)
+             allocatable = pointer = 0;
            break;
 
          case AR_UNKNOWN:
@@ -2047,8 +2073,8 @@ gfc_variable_attr (gfc_expr *expr, gfc_typespec *ts)
 
        if (comp->ts.type == BT_CLASS)
          {
-           pointer = comp->ts.u.derived->components->attr.pointer;
-           allocatable = comp->ts.u.derived->components->attr.allocatable;
+           pointer = CLASS_DATA (comp)->attr.class_pointer;
+           allocatable = CLASS_DATA (comp)->attr.allocatable;
          }
        else
          {
@@ -2069,6 +2095,7 @@ gfc_variable_attr (gfc_expr *expr, gfc_typespec *ts)
   attr.pointer = pointer;
   attr.allocatable = allocatable;
   attr.target = target;
+  attr.save = sym->attr.save;
 
   return attr;
 }
@@ -2096,9 +2123,9 @@ gfc_expr_attr (gfc_expr *e)
          attr = sym->attr;
          if (sym->ts.type == BT_CLASS)
            {
-             attr.dimension = sym->ts.u.derived->components->attr.dimension;
-             attr.pointer = sym->ts.u.derived->components->attr.pointer;
-             attr.allocatable = sym->ts.u.derived->components->attr.allocatable;
+             attr.dimension = CLASS_DATA (sym)->attr.dimension;
+             attr.pointer = CLASS_DATA (sym)->attr.class_pointer;
+             attr.allocatable = CLASS_DATA (sym)->attr.allocatable;
            }
        }
       else
@@ -2146,10 +2173,9 @@ gfc_free_structure_ctor_component (gfc_structure_ctor_component *comp)
    for components without explicit value given.  */
 static gfc_try
 build_actual_constructor (gfc_structure_ctor_component **comp_head,
-                         gfc_constructor **ctor_head, gfc_symbol *sym)
+                         gfc_constructor_base *ctor_head, gfc_symbol *sym)
 {
   gfc_structure_ctor_component *comp_iter;
-  gfc_constructor *ctor_tail = NULL;
   gfc_component *comp;
 
   for (comp = sym->components; comp; comp = comp->next)
@@ -2170,11 +2196,10 @@ build_actual_constructor (gfc_structure_ctor_component **comp_head,
         a value expression for the parent derived type and calling self.  */
       if (!comp_iter && comp == sym->components && sym->attr.extension)
        {
-         value = gfc_get_expr ();
-         value->expr_type = EXPR_STRUCTURE;
-         value->value.constructor = NULL;
+         value = gfc_get_structure_constructor_expr (comp->ts.type,
+                                                     comp->ts.kind,
+                                                     &gfc_current_locus);
          value->ts = comp->ts;
-         value->where = gfc_current_locus;
 
          if (build_actual_constructor (comp_head, &value->value.constructor,
                                        comp->ts.u.derived) == FAILURE)
@@ -2182,8 +2207,8 @@ build_actual_constructor (gfc_structure_ctor_component **comp_head,
              gfc_free_expr (value);
              return FAILURE;
            }
-         *ctor_head = ctor_tail = gfc_get_constructor ();
-         ctor_tail->expr = value;
+
+         gfc_constructor_append_expr (ctor_head, value, NULL);
          continue;
        }
 
@@ -2210,15 +2235,7 @@ build_actual_constructor (gfc_structure_ctor_component **comp_head,
        value = comp_iter->val;
 
       /* Add the value to the constructor chain built.  */
-      if (ctor_tail)
-       {
-         ctor_tail->next = gfc_get_constructor ();
-         ctor_tail = ctor_tail->next;
-       }
-      else
-       *ctor_head = ctor_tail = gfc_get_constructor ();
-      gcc_assert (value);
-      ctor_tail->expr = value;
+      gfc_constructor_append_expr (ctor_head, value, NULL);
 
       /* Remove the entry from the component list.  We don't want the expression
         value to be free'd, so set it to NULL.  */
@@ -2237,7 +2254,7 @@ gfc_match_structure_constructor (gfc_symbol *sym, gfc_expr **result,
                                 bool parent)
 {
   gfc_structure_ctor_component *comp_tail, *comp_head, *comp_iter;
-  gfc_constructor *ctor_head, *ctor_tail;
+  gfc_constructor_base ctor_head = NULL;
   gfc_component *comp; /* Is set NULL when named component is first seen */
   gfc_expr *e;
   locus where;
@@ -2245,7 +2262,6 @@ gfc_match_structure_constructor (gfc_symbol *sym, gfc_expr **result,
   const char* last_name = NULL;
 
   comp_tail = comp_head = NULL;
-  ctor_head = ctor_tail = NULL;
 
   if (!parent && gfc_match_char ('(') != MATCH_YES)
     goto syntax;
@@ -2349,6 +2365,15 @@ gfc_match_structure_constructor (gfc_symbol *sym, gfc_expr **result,
          if (m == MATCH_ERROR)
            goto cleanup;
 
+         /* F2008, R457/C725, for PURE C1283.  */
+          if (this_comp->attr.pointer && gfc_is_coindexed (comp_tail->val))
+           {
+             gfc_error ("Coindexed expression to pointer component '%s' in "
+                        "structure constructor at %C!", comp_tail->name);
+             goto cleanup;
+           }
+
+
          /* If not explicitly a parent constructor, gather up the components
             and build one.  */
          if (comp && comp == sym->components
@@ -2401,14 +2426,8 @@ gfc_match_structure_constructor (gfc_symbol *sym, gfc_expr **result,
   else
     gcc_assert (!comp_head);
 
-  e = gfc_get_expr ();
-
-  e->expr_type = EXPR_STRUCTURE;
-
-  e->ts.type = BT_DERIVED;
+  e = gfc_get_structure_constructor_expr (BT_DERIVED, 0, &where);
   e->ts.u.derived = sym;
-  e->where = where;
-
   e->value.constructor = ctor_head;
 
   *result = e;
@@ -2424,7 +2443,7 @@ cleanup:
       gfc_free_structure_ctor_component (comp_iter);
       comp_iter = next;
     }
-  gfc_free_constructor (ctor_head);
+  gfc_constructor_free (ctor_head);
   return MATCH_ERROR;
 }
 
@@ -2966,11 +2985,7 @@ match_variable (gfc_expr **result, int equiv_flag, int host_flag)
   switch (sym->attr.flavor)
     {
     case FL_VARIABLE:
-      if (sym->attr.is_protected && sym->attr.use_assoc)
-       {
-         gfc_error ("Assigning to PROTECTED variable at %C");
-         return MATCH_ERROR;
-       }
+      /* Everything is alright.  */
       break;
 
     case FL_UNKNOWN:
@@ -3002,22 +3017,24 @@ match_variable (gfc_expr **result, int equiv_flag, int host_flag)
 
     case FL_PARAMETER:
       if (equiv_flag)
-       gfc_error ("Named constant at %C in an EQUIVALENCE");
-      else
-       gfc_error ("Cannot assign to a named constant at %C");
-      return MATCH_ERROR;
+       {
+         gfc_error ("Named constant at %C in an EQUIVALENCE");
+         return MATCH_ERROR;
+       }
+      /* Otherwise this is checked for and an error given in the
+        variable definition context checks.  */
       break;
 
     case FL_PROCEDURE:
       /* Check for a nonrecursive function result variable.  */
       if (sym->attr.function
-          && !sym->attr.external
-          && sym->result == sym
-          && (gfc_is_function_return_value (sym, gfc_current_ns)
-              || (sym->attr.entry
-                  && sym->ns == gfc_current_ns)
-              || (sym->attr.entry
-                  && sym->ns == gfc_current_ns->parent)))
+         && !sym->attr.external
+         && sym->result == sym
+         && (gfc_is_function_return_value (sym, gfc_current_ns)
+             || (sym->attr.entry
+                 && sym->ns == gfc_current_ns)
+             || (sym->attr.entry
+                 && sym->ns == gfc_current_ns->parent)))
        {
          /* If a function result is a derived type, then the derived
             type may still have to be resolved.  */