diff options
Diffstat (limited to 'gcc/fortran')
| -rw-r--r-- | gcc/fortran/ChangeLog | 8 | ||||
| -rw-r--r-- | gcc/fortran/parse.c | 15 | ||||
| -rw-r--r-- | gcc/fortran/resolve.c | 11 |
3 files changed, 19 insertions, 15 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 7be47e2..d5f0cfe 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,11 @@ +2006-08-06 Paul Thomas <pault@gcc.gnu.org> + + PR fortran/28590 + * parse.c (parse_derived): Remove the test for sequence type + components of a sequence type. + * resolve.c (resolve_fl_derived): Put the test here so that + pointer components are tested. + 2006-08-05 Steven G. Kargl <kargls@comcast.nt> PR fortran/28548 diff --git a/gcc/fortran/parse.c b/gcc/fortran/parse.c index 972805e..0416d28 100644 --- a/gcc/fortran/parse.c +++ b/gcc/fortran/parse.c @@ -1498,7 +1498,6 @@ parse_derived (void) { int compiling_type, seen_private, seen_sequence, seen_component, error_flag; gfc_statement st; - gfc_component *c; gfc_state_data s; error_flag = 0; @@ -1596,20 +1595,6 @@ parse_derived (void) } } - /* Sanity checks on the structure. If the structure has the - SEQUENCE attribute, then all component structures must also have - SEQUENCE. */ - if (error_flag == 0 && gfc_current_block ()->attr.sequence) - for (c = gfc_current_block ()->components; c; c = c->next) - { - if (c->ts.type == BT_DERIVED && c->ts.derived->attr.sequence == 0) - { - gfc_error - ("Component %s of SEQUENCE type declared at %C does not " - "have the SEQUENCE attribute", c->ts.derived->name); - } - } - pop_state (); } diff --git a/gcc/fortran/resolve.c b/gcc/fortran/resolve.c index c327a82..cb45a2b 100644 --- a/gcc/fortran/resolve.c +++ b/gcc/fortran/resolve.c @@ -5392,6 +5392,17 @@ resolve_fl_derived (gfc_symbol *sym) return FAILURE; } + if (sym->attr.sequence) + { + if (c->ts.type == BT_DERIVED && c->ts.derived->attr.sequence == 0) + { + gfc_error ("Component %s of SEQUENCE type declared at %L does " + "not have the SEQUENCE attribute", + c->ts.derived->name, &sym->declared_at); + return FAILURE; + } + } + if (c->pointer || c->as == NULL) continue; |