aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
authorTobias Burnus <burnus@net-b.de>2011-01-28 20:49:25 +0100
committerTobias Burnus <burnus@gcc.gnu.org>2011-01-28 20:49:25 +0100
commit36ea267b8a5ed4ce63b864ce447e59ebc4e17bb6 (patch)
tree0b7402a91c513d3f4534f6e36ee16a517b28e7dd /gcc
parent5582f599f61e783867a944f192f34e6ad1b764ca (diff)
downloadgcc-36ea267b8a5ed4ce63b864ce447e59ebc4e17bb6.zip
gcc-36ea267b8a5ed4ce63b864ce447e59ebc4e17bb6.tar.gz
gcc-36ea267b8a5ed4ce63b864ce447e59ebc4e17bb6.tar.bz2
re PR fortran/47507 (PURE functions with VALUE arguments invalidly rejectd)
2011-01-28 Tobias Burnus <burnus@net-b.de> PR fortran/47507 * resolve.c (resolve_formal_arglist): Allow arguments with VALUE attribute also without INTENT. 2011-01-28 Tobias Burnus <burnus@net-b.de> PR fortran/47507 * gfortran.dg/pure_formal_1.f90: New. From-SVN: r169372
Diffstat (limited to 'gcc')
-rw-r--r--gcc/fortran/ChangeLog6
-rw-r--r--gcc/fortran/resolve.c12
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gfortran.dg/pure_formal_1.f9016
4 files changed, 34 insertions, 5 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index 336f28e..b706492 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,5 +1,11 @@
2011-01-28 Tobias Burnus <burnus@net-b.de>
+ PR fortran/47507
+ * resolve.c (resolve_formal_arglist): Allow arguments with VALUE
+ attribute also without INTENT.
+
+2011-01-28 Tobias Burnus <burnus@net-b.de>
+
* gfortran.texi (Fortran 2003 status): Mention support for
nonconstant namelist variables.
diff --git a/gcc/fortran/resolve.c b/gcc/fortran/resolve.c
index 2436283..55b5183 100644
--- a/gcc/fortran/resolve.c
+++ b/gcc/fortran/resolve.c
@@ -338,15 +338,17 @@ resolve_formal_arglist (gfc_symbol *proc)
if (gfc_pure (proc) && !sym->attr.pointer
&& sym->attr.flavor != FL_PROCEDURE)
{
- if (proc->attr.function && sym->attr.intent != INTENT_IN)
+ if (proc->attr.function && sym->attr.intent != INTENT_IN
+ && !sym->attr.value)
gfc_error ("Argument '%s' of pure function '%s' at %L must be "
- "INTENT(IN)", sym->name, proc->name,
+ "INTENT(IN) or VALUE", sym->name, proc->name,
&sym->declared_at);
- if (proc->attr.subroutine && sym->attr.intent == INTENT_UNKNOWN)
+ if (proc->attr.subroutine && sym->attr.intent == INTENT_UNKNOWN
+ && !sym->attr.value)
gfc_error ("Argument '%s' of pure subroutine '%s' at %L must "
- "have its INTENT specified", sym->name, proc->name,
- &sym->declared_at);
+ "have its INTENT specified or have the VALUE "
+ "attribute", sym->name, proc->name, &sym->declared_at);
}
if (proc->attr.implicit_pure && !sym->attr.pointer
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 4d6acf0..892a543 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2011-01-28 Tobias Burnus <burnus@net-b.de>
+
+ PR fortran/47507
+ * gfortran.dg/pure_formal_1.f90: New.
+
2011-01-28 Jakub Jelinek <jakub@redhat.com>
PR target/42894
diff --git a/gcc/testsuite/gfortran.dg/pure_formal_1.f90 b/gcc/testsuite/gfortran.dg/pure_formal_1.f90
new file mode 100644
index 0000000..4e62cf9
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/pure_formal_1.f90
@@ -0,0 +1,16 @@
+! { dg-do compile }
+!
+! PR fortran/47507
+!
+! PURE procedures: Allow arguments w/o INTENT if they are VALUE
+!
+
+pure function f(x)
+ real, VALUE :: x
+ real :: f
+ f = sin(x)
+end function f
+
+pure subroutine sub(x)
+ real, VALUE :: x
+end subroutine sub