summaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
authorburnus <burnus@138bc75d-0d04-0410-961f-82ee72b054a4>2008-01-06 19:36:44 +0000
committerburnus <burnus@138bc75d-0d04-0410-961f-82ee72b054a4>2008-01-06 19:36:44 +0000
commit02c8c8d3c8688a4b10ec5541268e436fa13a0753 (patch)
tree898255a182219879150de12200496747201e247e /gcc
parent9edaf48c13105023cc4fe3f5ebe7b884c626c366 (diff)
downloadgcc-02c8c8d3c8688a4b10ec5541268e436fa13a0753.tar.gz
2008-01-06 Tobias Burnus <burnus@net-b.de>
PR fortran/34689 * interface.c (compare_actual_formal): Fix intent(out) check for function result variables. 2008-01-06 Tobias Burnus <burnus@net-b.de> PR fortran/34689 * gfortran.dg/intent_out_4.f90: New. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@131359 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'gcc')
-rw-r--r--gcc/fortran/ChangeLog6
-rw-r--r--gcc/fortran/interface.c3
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gfortran.dg/intent_out_4.f9018
4 files changed, 31 insertions, 1 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index 932527eadf8..09155e3f26a 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,9 @@
+2008-01-06 Tobias Burnus <burnus@net-b.de>
+
+ PR fortran/34689
+ * interface.c (compare_actual_formal): Fix intent(out) check for
+ function result variables.
+
2008-01-06 Jerry DeLisle <jvdelisle@gcc.gnu.org>
PR fortran/34387
diff --git a/gcc/fortran/interface.c b/gcc/fortran/interface.c
index b900b102fe5..8088fc65432 100644
--- a/gcc/fortran/interface.c
+++ b/gcc/fortran/interface.c
@@ -1909,7 +1909,8 @@ 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
- || a->expr->symtree->n.sym->attr.flavor != FL_VARIABLE)
+ || (a->expr->symtree->n.sym->attr.flavor != FL_VARIABLE
+ && a->expr->symtree->n.sym->attr.flavor != FL_PROCEDURE))
&& (f->sym->attr.intent == INTENT_OUT
|| f->sym->attr.intent == INTENT_INOUT))
{
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index e0c426684d1..1631fe90021 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2008-01-06 Tobias Burnus <burnus@net-b.de>
+
+ PR fortran/34689
+ * gfortran.dg/intent_out_4.f90: New.
+
2008-01-06 Jerry DeLisle <jvdelisle@gcc.gnu.org>
PR libfortran/34540
diff --git a/gcc/testsuite/gfortran.dg/intent_out_4.f90 b/gcc/testsuite/gfortran.dg/intent_out_4.f90
new file mode 100644
index 00000000000..e34c452e8e6
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/intent_out_4.f90
@@ -0,0 +1,18 @@
+! { dg-do compile }
+!
+! PR fortran/34689
+!
+! The following (cf. libgomp.fortran/appendix-a/a.33.3.f90)
+! was rejected because the intent check missed a FL_FUNCTION
+! for the result variable.
+!
+function test()
+ implicit none
+ integer :: test
+ interface
+ subroutine foo(a)
+ integer(inout) :: a
+ end subroutine foo
+ end interface
+ call foo(a)
+end function test