public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r9-9878] Fortran: catch failed simplification of bad stride expression
@ 2021-12-19 20:51 Harald Anlauf
  0 siblings, 0 replies; only message in thread
From: Harald Anlauf @ 2021-12-19 20:51 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:c38611aabf4022323b7c44d0dcb3f6b076836de0

commit r9-9878-gc38611aabf4022323b7c44d0dcb3f6b076836de0
Author: Harald Anlauf <anlauf@gmx.de>
Date:   Tue Dec 7 18:46:52 2021 +0100

    Fortran: catch failed simplification of bad stride expression
    
    gcc/fortran/ChangeLog:
    
            PR fortran/103588
            * array.c (gfc_ref_dimen_size): Do not generate internal error on
            failed simplification of stride expression; just return failure.
    
    gcc/testsuite/ChangeLog:
    
            PR fortran/103588
            * gfortran.dg/pr103588.f90: New test.
    
    (cherry picked from commit 652c28736209f10bac1aa7ecb31f9056b518dacf)

Diff:
---
 gcc/fortran/array.c                    | 9 ++++-----
 gcc/testsuite/gfortran.dg/pr103588.f90 | 8 ++++++++
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/gcc/fortran/array.c b/gcc/fortran/array.c
index 2fb4e792ad4..c5d813a2dc3 100644
--- a/gcc/fortran/array.c
+++ b/gcc/fortran/array.c
@@ -2329,12 +2329,11 @@ gfc_ref_dimen_size (gfc_array_ref *ar, int dimen, mpz_t *result, mpz_t *end)
 	{
 	  stride_expr = gfc_copy_expr(ar->stride[dimen]); 
 
-	  if(!gfc_simplify_expr(stride_expr, 1))
-	    gfc_internal_error("Simplification error");
-
-	  if (stride_expr->expr_type != EXPR_CONSTANT
-	      || mpz_cmp_ui (stride_expr->value.integer, 0) == 0)
+	  if (!gfc_simplify_expr (stride_expr, 1)
+	     || stride_expr->expr_type != EXPR_CONSTANT
+	     || mpz_cmp_ui (stride_expr->value.integer, 0) == 0)
 	    {
+	      gfc_free_expr (stride_expr);
 	      mpz_clear (stride);
 	      return false;
 	    }
diff --git a/gcc/testsuite/gfortran.dg/pr103588.f90 b/gcc/testsuite/gfortran.dg/pr103588.f90
new file mode 100644
index 00000000000..198e1766cd2
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/pr103588.f90
@@ -0,0 +1,8 @@
+! { dg-do compile }
+! PR fortran/103588 - ICE: Simplification error in gfc_ref_dimen_size
+! Contributed by G.Steinmetz
+
+program p
+  integer, parameter :: a(:) = [1,2] ! { dg-error "cannot be automatic or of deferred shape" }
+  integer :: b(2) = a(::a(1))        ! { dg-error "Invalid" }
+end


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-12-19 20:51 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-19 20:51 [gcc r9-9878] Fortran: catch failed simplification of bad stride expression Harald Anlauf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).