aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTobias Burnus <burnus@net-b.de>2012-06-13 13:56:08 +0200
committerTobias Burnus <burnus@gcc.gnu.org>2012-06-13 13:56:08 +0200
commitd1f6dfe6fb26ad5bc8d7b7cd34199bd5a9cfe59a (patch)
tree00fc2b0830ef2a6f8cb308542c9977fe04a44884
parent9c78662a1bab40bad07d732879e31df3679ee945 (diff)
downloadgcc-d1f6dfe6fb26ad5bc8d7b7cd34199bd5a9cfe59a.zip
gcc-d1f6dfe6fb26ad5bc8d7b7cd34199bd5a9cfe59a.tar.gz
gcc-d1f6dfe6fb26ad5bc8d7b7cd34199bd5a9cfe59a.tar.bz2
re PR fortran/53597 (F95/F2003 constraint no longer triggers: un-SAVED default-initialized module variable)
2012-06-13 Tobias Burnus <burnus@net-b.de> PR fortran/53597 * decl.c (match_attr_spec): Only mark module variables as SAVE_IMPLICIT for Fortran 2008 and later. 2012-06-13 Tobias Burnus <burnus@net-b.de> PR fortran/53597 * gfortran.dg/save_4.f90: New. From-SVN: r188506
-rw-r--r--gcc/fortran/ChangeLog6
-rw-r--r--gcc/fortran/decl.c5
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gfortran.dg/save_4.f9013
4 files changed, 27 insertions, 2 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index 02b6445..ffe1254 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,9 @@
+2012-06-13 Tobias Burnus <burnus@net-b.de>
+
+ PR fortran/53597
+ * decl.c (match_attr_spec): Only mark module variables
+ as SAVE_IMPLICIT for Fortran 2008 and later.
+
2012-06-08 Janus Weil <janus@gcc.gnu.org>
PR fortran/52552
diff --git a/gcc/fortran/decl.c b/gcc/fortran/decl.c
index a760331..26b5059 100644
--- a/gcc/fortran/decl.c
+++ b/gcc/fortran/decl.c
@@ -3810,8 +3810,9 @@ match_attr_spec (void)
}
}
- /* Module variables implicitly have the SAVE attribute. */
- if (gfc_current_state () == COMP_MODULE && !current_attr.save)
+ /* Since Fortran 2008 module variables implicitly have the SAVE attribute. */
+ if (gfc_current_state () == COMP_MODULE && !current_attr.save
+ && (gfc_option.allow_std & GFC_STD_F2008) != 0)
current_attr.save = SAVE_IMPLICIT;
colon_seen = 1;
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index cdeb722..3649dcf 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2012-06-13 Tobias Burnus <burnus@net-b.de>
+
+ PR fortran/53597
+ * gfortran.dg/save_4.f90: New.
+
2012-06-12 Jakub Jelinek <jakub@redhat.com>
PR c/53532
diff --git a/gcc/testsuite/gfortran.dg/save_4.f90 b/gcc/testsuite/gfortran.dg/save_4.f90
new file mode 100644
index 0000000..74ea6e8
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/save_4.f90
@@ -0,0 +1,13 @@
+! { dg-do compile }
+! { dg-options "-std=f2003" }
+!
+! PR fortran/53597
+!
+MODULE somemodule
+ IMPLICIT NONE
+ TYPE sometype
+ INTEGER :: i
+ DOUBLE PRECISION, POINTER, DIMENSION(:,:) :: coef => NULL()
+ END TYPE sometype
+ TYPE(sometype) :: somevariable ! { dg-error "Fortran 2008: Implied SAVE for module variable 'somevariable' at .1., needed due to the default initialization" }
+END MODULE somemodule