aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJanus Weil <janus@gcc.gnu.org>2011-10-01 13:41:41 +0200
committerJanus Weil <janus@gcc.gnu.org>2011-10-01 13:41:41 +0200
commite323640fb2d6fc3f9ed467b7dad246ce953441f8 (patch)
tree4513af351f32c0335af08877e33a022f8c519019
parent48843306cdc9307594a3cd04800ec92f44df8217 (diff)
downloadgcc-e323640fb2d6fc3f9ed467b7dad246ce953441f8.zip
gcc-e323640fb2d6fc3f9ed467b7dad246ce953441f8.tar.gz
gcc-e323640fb2d6fc3f9ed467b7dad246ce953441f8.tar.bz2
re PR fortran/50585 (ICE with assumed length character array argument)
2011-10-01 Janus Weil <janus@gcc.gnu.org> PR fortran/50585 * interface.c (get_expr_storage_size): Check if 'length' component is associated. 2011-10-01 Janus Weil <janus@gcc.gnu.org> PR fortran/50585 * gfortran.dg/assumed_charlen_arg_2.f90: New. From-SVN: r179413
-rw-r--r--gcc/fortran/ChangeLog6
-rw-r--r--gcc/fortran/interface.c2
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gfortran.dg/assumed_charlen_arg_2.f9016
4 files changed, 28 insertions, 1 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index ecd9fa3..c6bea2a 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,9 @@
+2011-10-01 Janus Weil <janus@gcc.gnu.org>
+
+ PR fortran/50585
+ * interface.c (get_expr_storage_size): Check if 'length' component is
+ associated.
+
2011-09-29 Janus Weil <janus@gcc.gnu.org>
PR fortran/50547
diff --git a/gcc/fortran/interface.c b/gcc/fortran/interface.c
index f65087b..aa075a9 100644
--- a/gcc/fortran/interface.c
+++ b/gcc/fortran/interface.c
@@ -1959,7 +1959,7 @@ get_expr_storage_size (gfc_expr *e)
{
/* The string length is the substring length.
Set now to full string length. */
- if (ref->u.ss.length == NULL
+ if (!ref->u.ss.length || !ref->u.ss.length->length
|| ref->u.ss.length->length->expr_type != EXPR_CONSTANT)
return 0;
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index d7b163a..075edfc 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2011-10-01 Janus Weil <janus@gcc.gnu.org>
+
+ PR fortran/50585
+ * gfortran.dg/assumed_charlen_arg_2.f90: New.
+
2011-09-30 Janis Johnson <janisjo@codesourcery.com>
PR c++/44473
diff --git a/gcc/testsuite/gfortran.dg/assumed_charlen_arg_2.f90 b/gcc/testsuite/gfortran.dg/assumed_charlen_arg_2.f90
new file mode 100644
index 0000000..e9481d8
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/assumed_charlen_arg_2.f90
@@ -0,0 +1,16 @@
+! { dg-do compile }
+!
+! PR 50585: [4.6/4.7 Regression] ICE with assumed length character array argument
+!
+! Contributed by Stuart Mentzer <sgm@objexx.com>
+
+SUBROUTINE SUB1( str )
+ IMPLICIT NONE
+ CHARACTER(len=*) :: str(2)
+ CALL SUB2( str(1)(:3) )
+END SUBROUTINE
+
+SUBROUTINE SUB2( str )
+ IMPLICIT NONE
+ CHARACTER(*) :: str
+END SUBROUTINE