------- Comment #22 from jvdelisle at gcc dot gnu dot org 2008-01-24 04:52 ------- Putting this back in fixes the new test case in comment #21
Index: primary.c =================================================================== --- primary.c (revision 131752) +++ primary.c (working copy) @@ -2538,6 +2538,10 @@ match_variable (gfc_expr **result, int e || sym->attr.pointer || sym->as != NULL) flavor = FL_VARIABLE; + if (sym->attr.access == ACCESS_PUBLIC + || sym->attr.access == ACCESS_PRIVATE) + break; + if (flavor != FL_UNKNOWN && gfc_add_flavor (&sym->attr, flavor, sym->name, NULL) == FAILURE) return MATCH_ERROR; Shall we go back to this? or just add it in as above? It does seem simpler and more elegant and it makes sense. I am regression testing now. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32760