re PR fortran/83633 (gfortran internal compiler error for explicit-shape array with...
authorSteven G. Kargl <kargl@gcc.gnu.org>
Sun, 25 Feb 2018 16:50:50 +0000 (16:50 +0000)
committerSteven G. Kargl <kargl@gcc.gnu.org>
Sun, 25 Feb 2018 16:50:50 +0000 (16:50 +0000)
2018-02-25  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/83633
* decl.c (variable_decl): Check that an explicit-shape-array with
nonconstant bounds is allowed.

2018-02-25  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/83633
* gfortran.dg/explicit_shape_1.f90: New test.
* gfortran.dg/automatic_module_variable.f90: Update regex.
* gfortran.dg/bad_automatic_objects_1.f90: Ditto.
* gfortran.dg/constant_shape.f90: Ditto.
* gfortran.dg/dec_structure_23.f90: Ditto.
* gfortran.dg/pr78240.f90: Ditto.

From-SVN: r257971

gcc/fortran/ChangeLog
gcc/fortran/decl.c
gcc/testsuite/ChangeLog
gcc/testsuite/gfortran.dg/automatic_module_variable.f90
gcc/testsuite/gfortran.dg/bad_automatic_objects_1.f90
gcc/testsuite/gfortran.dg/constant_shape.f90
gcc/testsuite/gfortran.dg/dec_structure_23.f90
gcc/testsuite/gfortran.dg/explicit_shape_1.f90 [new file with mode: 0644]
gcc/testsuite/gfortran.dg/pr78240.f90

index 2c2aa2b8d9031dc5a74dc508384a7cff9f708075..76ac274674ad5f7511177522923fcf4817cb074b 100644 (file)
@@ -1,3 +1,9 @@
+2018-02-25  Steven G. Kargl  <kargl@gcc.gnu.org>
+
+       PR fortran/83633
+       * decl.c (variable_decl): Check that an explicit-shape-array with
+       nonconstant bounds is allowed.
+
 2018-02-25  Paul Thomas  <pault@gcc.gnu.org>
 
        PR fortran/84523
index e377a213db49ff1217dc3d4b61ec37c9260945e7..dcfda27a3c640862a868b84cae39a458084b34cc 100644 (file)
@@ -2304,7 +2304,10 @@ variable_decl (int elem)
   /* At this point, we know for sure if the symbol is PARAMETER and can thus
      determine (and check) whether it can be implied-shape.  If it
      was parsed as assumed-size, change it because PARAMETERs can not
-     be assumed-size.  */
+     be assumed-size.
+
+     An explicit-shape-array cannot appear under several conditions.
+     That check is done here as well.  */
   if (as)
     {
       if (as->type == AS_IMPLIED_SHAPE && current_attr.flavor != FL_PARAMETER)
@@ -2326,6 +2329,50 @@ variable_decl (int elem)
          m = MATCH_ERROR;
          goto cleanup;
        }
+
+      /* F2018:C830 (R816) An explicit-shape-spec whose bounds are not
+        constant expressions shall appear only in a subprogram, derived
+        type definition, BLOCK construct, or interface body.  */
+      if (as->type == AS_EXPLICIT
+         && gfc_current_state () != COMP_BLOCK
+         && gfc_current_state () != COMP_DERIVED
+         && gfc_current_state () != COMP_FUNCTION
+         && gfc_current_state () != COMP_INTERFACE
+         && gfc_current_state () != COMP_SUBROUTINE)
+       {
+         gfc_expr *e;
+         bool not_constant = false;
+
+         for (int i = 0; i < as->rank; i++)
+           {
+             e = gfc_copy_expr (as->lower[i]);
+             gfc_resolve_expr (e);
+             gfc_simplify_expr (e, 0);
+             if (e && (e->expr_type != EXPR_CONSTANT))
+               {
+                 not_constant = true;
+                 break;
+               }
+             gfc_free_expr (e);
+
+             e = gfc_copy_expr (as->upper[i]);
+             gfc_resolve_expr (e);
+             gfc_simplify_expr (e, 0);
+             if (e && (e->expr_type != EXPR_CONSTANT))
+               {
+                 not_constant = true;
+                 break;
+               }
+             gfc_free_expr (e);
+           }
+
+         if (not_constant)
+           { 
+             gfc_error ("Explicit shaped array with nonconstant bounds at %C");
+             m = MATCH_ERROR;
+             goto cleanup;
+           }
+       }
     }
 
   char_len = NULL;
index 0a015c8ca593847631d663b54175522ec3b4253a..86fce26c0835f972befd75a530b061effd69ecac 100644 (file)
@@ -1,3 +1,13 @@
+2018-02-25  Steven G. Kargl  <kargl@gcc.gnu.org>
+
+       PR fortran/83633
+       * gfortran.dg/explicit_shape_1.f90: New test.
+       * gfortran.dg/automatic_module_variable.f90: Update regex.
+       * gfortran.dg/bad_automatic_objects_1.f90: Ditto.
+       * gfortran.dg/constant_shape.f90: Ditto.
+       * gfortran.dg/dec_structure_23.f90: Ditto.
+       * gfortran.dg/pr78240.f90: Ditto.
+
 2018-02-25  Paul Thomas  <pault@gcc.gnu.org>
 
        PR fortran/84523
index 201dcf4e1d3cb360d5864a205bcd3df7932283a8..ab041fcf4f5ac40cce2e8545be134b20f761a520 100644 (file)
@@ -1,10 +1,12 @@
 ! { dg-do compile }
 ! Tests fix for PR15976
 !
+! Error message update with patch for PR fortran/83633
+!
 module sd
   integer, parameter :: n = 20
   integer :: i(n)
-  integer :: j(m) ! { dg-error "must have constant shape" }
+  integer :: j(m) ! { dg-error "array with nonconstant bounds" }
   integer, pointer :: p(:)
   integer, allocatable :: q(:)
 contains
index 2734418619478f0eb0e0b88a9dac4da5c6a3c956..61db2174e5b9cca84428efde8d8d3cccd1983ddd 100644 (file)
@@ -5,16 +5,18 @@
 !
 ! Contributed by Joost VandeVondele  <jv244@cam.ac.uk>
 !
+! Error message update with patch for PR fortran/83633
+!
 module foo
   integer    ::  i
 end module foo
 module bar
   use foo
-  integer, dimension (i) :: j ! { dg-error "must have constant shape" }
+  integer, dimension (i) :: j ! { dg-error "array with nonconstant bounds" }
   character (len = i) :: c1   ! { dg-error "must have constant character length" }
 end module bar
 program foobar
   use foo
-  integer, dimension (i) :: k ! { dg-error "must have constant shape" }
+  integer, dimension (i) :: k ! { dg-error "array with nonconstant bounds" }
   character (len = i) :: c2   ! { dg-error "must have constant character length" }
 end program foobar
index c2eaf82f5ec7766a014b74fb25a9ddbea38fa279..5ee927b20e2f8058d05b1671ee163911231519fe 100644 (file)
@@ -3,7 +3,8 @@
 ! PR 78392: ICE in gfc_trans_auto_array_allocation, at fortran/trans-array.c:5979
 !
 ! Contributed by Janus Weil <janus@gcc.gnu.org>
-
+! Error message update with patch for PR fortran/83633
+!
 module mytypes
    implicit none
  contains
@@ -15,6 +16,6 @@ end module
 program test
   use mytypes
   implicit none
-  integer, dimension(get_i()) :: x  ! { dg-error "must have constant shape" }
-  print *, size (x)
+  integer, dimension(get_i()) :: x  ! { dg-error "array with nonconstant bounds" }
+  print *, size (x)                 ! { dg-error "has no IMPLICIT type" }
 end
index 3c68489c4bdb5b7272720c2910a20882d7033f3c..78db344e0fc859adfdbcf0d518830eeb03c8d8be 100644 (file)
@@ -6,14 +6,15 @@
 ! Test a regression where an ICE occurred attempting to create array variables
 ! with non-constant array-specs in legacy clist initializers.
 !
-
+! Error message update with patch for PR fortran/83633
+!
 program p
   implicit none
   integer :: nn
   real :: rr
   structure /s/
-    integer x(n)    /1/   ! { dg-error "xpected constant" }
-    integer xx(nn)  /1/   ! { dg-error "xpected constant" }
-    integer xxx(rr) /1.0/ ! { dg-error "xpected constant" }
+    integer x(n)    /1/   ! { dg-error "array with nonconstant bounds" }
+    integer xx(nn)  /1/   ! { dg-error "array with nonconstant bounds" }
+    integer xxx(rr) /1.0/ ! { dg-error "array with nonconstant bounds" }
   end structure
 end
diff --git a/gcc/testsuite/gfortran.dg/explicit_shape_1.f90 b/gcc/testsuite/gfortran.dg/explicit_shape_1.f90
new file mode 100644 (file)
index 0000000..ca3cd00
--- /dev/null
@@ -0,0 +1,7 @@
+! { dg-do compile }
+! PR fortran/83633
+! Original testcase by Nathan T. Weeks  <weeks at iastate dot edu>
+!
+integer :: A(command_argument_count()) = 1 ! { dg-error "nonconstant bounds" }
+write (*,*) A
+end
index 5373b555a5059a8f7a6e92addd68b06d26ed588c..45d3f86360b58cd0c48d5a1dcca1b3a8d9b4f752 100644 (file)
@@ -7,9 +7,10 @@
 ! to the error handling routine for non-constant array-specs in DATA list
 ! initializers.
 !
-
+! Error message update with patch for PR fortran/83633
+!
 program p
-  integer x(n)    /1/   ! { dg-error "cannot appear in the expression" }
+  integer x(n)    /1/   ! { dg-error "array with nonconstant bounds" }
 end
 ! { dg-prune-output "module or main program" }
 ! { dg-prune-output "Nonconstant array" }