diff --git a/gcc/fortran/array.c b/gcc/fortran/array.c index 2fc9dfaf..1430e802 100644 --- a/gcc/fortran/array.c +++ b/gcc/fortran/array.c @@ -421,10 +421,15 @@ match_array_element_spec (gfc_array_spec *as) if (!gfc_expr_check_typed (*upper, gfc_current_ns, false)) return AS_UNKNOWN; - if ((*upper)->expr_type == EXPR_FUNCTION && (*upper)->ts.type == BT_UNKNOWN - && (*upper)->symtree && strcmp ((*upper)->symtree->name, "null") == 0) - { - gfc_error ("Expecting a scalar INTEGER expression at %C"); + if (((*upper)->expr_type == EXPR_CONSTANT + && (*upper)->ts.type != BT_INTEGER) || + ((*upper)->expr_type == EXPR_FUNCTION + && (*upper)->ts.type == BT_UNKNOWN + && (*upper)->symtree + && strcmp ((*upper)->symtree->name, "null") == 0)) + { + gfc_error ("Expecting a scalar INTEGER expression at %C, found %s", + gfc_basic_typename ((*upper)->ts.type)); return AS_UNKNOWN; } @@ -448,10 +453,15 @@ match_array_element_spec (gfc_array_spec *as) if (!gfc_expr_check_typed (*upper, gfc_current_ns, false)) return AS_UNKNOWN; - if ((*upper)->expr_type == EXPR_FUNCTION && (*upper)->ts.type == BT_UNKNOWN - && (*upper)->symtree && strcmp ((*upper)->symtree->name, "null") == 0) + if (((*upper)->expr_type == EXPR_CONSTANT + && (*upper)->ts.type != BT_INTEGER) || + ((*upper)->expr_type == EXPR_FUNCTION + && (*upper)->ts.type == BT_UNKNOWN + && (*upper)->symtree + && strcmp ((*upper)->symtree->name, "null") == 0)) { - gfc_error ("Expecting a scalar INTEGER expression at %C"); + gfc_error ("Expecting a scalar INTEGER expression at %C, found %s", + gfc_basic_typename ((*upper)->ts.type)); return AS_UNKNOWN; } diff --git a/gcc/fortran/simplify.c b/gcc/fortran/simplify.c index 12a8f32e..a6310107 100644 --- a/gcc/fortran/simplify.c +++ b/gcc/fortran/simplify.c @@ -5163,6 +5163,9 @@ gfc_simplify_reshape (gfc_expr *source, gfc_expr *shape_exp, || !is_constant_array_expr (order_exp)) return NULL; + if (source->shape == NULL) + return NULL; + /* Proceed with simplification, unpacking the array. */ mpz_init (index); diff --git a/gcc/testsuite/gfortran.dg/pr36192.f90 b/gcc/testsuite/gfortran.dg/pr36192.f90 index df3bfd75..ebf95e35 100644 --- a/gcc/testsuite/gfortran.dg/pr36192.f90 +++ b/gcc/testsuite/gfortran.dg/pr36192.f90 @@ -3,7 +3,6 @@ ! program three_body real, parameter :: n = 2, d = 2 - real, dimension(n,d) :: x ! { dg-error "of INTEGER type|of INTEGER type" } - x(1,:) = (/ 1.0, 0.0 /) + real, dimension(n,d) :: x ! { dg-error "Expecting a scalar INTEGER" } + x(1,:) = (/ 1.0, 0.0 /) ! { dg-error "Unclassifiable" } end program three_body -! { dg-prune-output "have constant shape" } diff --git a/gcc/testsuite/gfortran.dg/pr36192_1.f90 b/gcc/testsuite/gfortran.dg/pr36192_1.f90 index 77df3176..687a465f 100644 --- a/gcc/testsuite/gfortran.dg/pr36192_1.f90 +++ b/gcc/testsuite/gfortran.dg/pr36192_1.f90 @@ -2,11 +2,11 @@ ! PR fortran/36192 program three_body real, parameter :: n = 2, d = 2 - real, dimension(n,d) :: x_hq ! { dg-error "of INTEGER type|of INTEGER type" } + real, dimension(n,d) :: x_hq ! { dg-error "Expecting a scalar INTEGER" } call step(x_hq) contains subroutine step(x) real, dimension(:,:), intent(in) :: x end subroutine step end program three_body -! { dg-prune-output "must have constant shape" } +! { dg-prune-output "Rank mismatch in argument" } diff --git a/gcc/testsuite/gfortran.dg/real_dimension_1.f b/gcc/testsuite/gfortran.dg/real_dimension_1.f index 73e9131a..3dd1a5af 100644 --- a/gcc/testsuite/gfortran.dg/real_dimension_1.f +++ b/gcc/testsuite/gfortran.dg/real_dimension_1.f @@ -1,7 +1,7 @@ ! { dg-do compile } -! PR 34305 - make sure there's an error message for specifying a +! PR 34305 - Test for specifying a real as dimension program test - parameter (datasize = 1000) - dimension idata (datasize) ! { dg-error "must be of INTEGER type|must have constant shape" } - idata (1) = -1 + real , parameter :: dsize = 1000 + dimension idata (dsize) ! { dg-error "scalar INTEGER expression" } + idata (1) = -1 ! { dg-error "must have the pointer attribute" } end