diff options
| author | Tobias Burnus <burnus@net-b.de> | 2008-01-06 10:07:47 +0100 |
|---|---|---|
| committer | Tobias Burnus <burnus@gcc.gnu.org> | 2008-01-06 10:07:47 +0100 |
| commit | a5c655e832e56b78b5cc4cab0452f4263740e938 (patch) | |
| tree | cbdd147b3dc5efc685848f0b3084142c55373706 | |
| parent | 5c09795d82c0d78be059a3e94b5bb9cef99305ff (diff) | |
| download | gcc-a5c655e832e56b78b5cc4cab0452f4263740e938.zip gcc-a5c655e832e56b78b5cc4cab0452f4263740e938.tar.gz gcc-a5c655e832e56b78b5cc4cab0452f4263740e938.tar.bz2 | |
re PR fortran/34662 (inout argument with parameter)
2008-01-06 Tobias Burnus <burnus@net-b.de>
PR fortran/34662
* interface.c (compare_actual_formal): Reject parameter
actual to intent(out) dummy.
2008-01-06 Tobias Burnus <burnus@net-b.de>
PR fortran/34662
* gfortran.dg/intent_out_3.f90: New.
From-SVN: r131348
| -rw-r--r-- | gcc/fortran/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/fortran/interface.c | 8 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gfortran.dg/intent_out_3.f90 | 19 |
4 files changed, 35 insertions, 3 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 82f4cd7..43d7519 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,9 @@ +2008-01-06 Tobias Burnus <burnus@net-b.de> + + PR fortran/34662 + * interface.c (compare_actual_formal): Reject parameter + actual to intent(out) dummy. + 2008-01-04 Tobias Burnus <burnus@net-b.de> PR fortran/34557 diff --git a/gcc/fortran/interface.c b/gcc/fortran/interface.c index 519251e..b900b10 100644 --- a/gcc/fortran/interface.c +++ b/gcc/fortran/interface.c @@ -1908,13 +1908,15 @@ compare_actual_formal (gfc_actual_arglist **ap, gfc_formal_arglist *formal, } /* Check intent = OUT/INOUT for definable actual argument. */ - if (a->expr->expr_type != EXPR_VARIABLE + if ((a->expr->expr_type != EXPR_VARIABLE + || a->expr->symtree->n.sym->attr.flavor != FL_VARIABLE) && (f->sym->attr.intent == INTENT_OUT || f->sym->attr.intent == INTENT_INOUT)) { if (where) - gfc_error ("Actual argument at %L must be definable to " - "match dummy INTENT = OUT/INOUT", &a->expr->where); + gfc_error ("Actual argument at %L must be definable as " + "the dummy argument '%s' is INTENT = OUT/INOUT", + &a->expr->where, f->sym->name); return 0; } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8519cae..5cecf06 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2008-01-06 Tobias Burnus <burnus@net-b.de> + + PR fortran/34662 + * gfortran.dg/intent_out_3.f90: New. + 2008-01-05 Richard Sandiford <rsandifo@nildram.co.uk> * objc.dg/stabs-1.m: Allow labels to be begin with $L as well as .L. diff --git a/gcc/testsuite/gfortran.dg/intent_out_3.f90 b/gcc/testsuite/gfortran.dg/intent_out_3.f90 new file mode 100644 index 0000000..7b68a35 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/intent_out_3.f90 @@ -0,0 +1,19 @@ +! { dg-do compile } +! +! PR fortran/34662 +! The INTENT error was not detected. +! Test case contributed by Joost VandeVondele. +! +MODULE M1 + TYPE T1 + INTEGER :: I(3) + END TYPE T1 + TYPE(T1), PARAMETER :: D1=T1((/1,2,3/)) +CONTAINS + SUBROUTINE S1(J) + INTEGER, INTENT(INOUT) :: J + END SUBROUTINE S1 +END MODULE M1 +USE M1 +CALL S1(D1%I(3)) ! { dg-error "must be definable" } +END |