This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Do not ICE in resolve.c (PR fortran/71799)


Hi.

Suggested patch was just grabbed from the PR.

Patch can bootstrap on x86_64-linux-gnu and survives regression tests.

Ready to be installed?
Martin
>From e6c55da822bc5a7464edc9cedb9b08f56e310885 Mon Sep 17 00:00:00 2001
From: marxin <mliska@suse.cz>
Date: Mon, 18 Jul 2016 16:47:27 +0200
Subject: [PATCH] Do not ICE in resolve.c (PR fortran/71799)

gcc/testsuite/ChangeLog:

2016-07-18  Martin Liska  <mliska@suse.cz>

	* gfortran.dg/pr71799.f90: New test.

gcc/fortran/ChangeLog:

2016-07-18  Steven G. Kargl  <kargl@gcc.gnu.org>

	* resolve.c (gfc_resolve_iterator): Generate error instead
	of ICE.
---
 gcc/fortran/resolve.c                 |  6 +++---
 gcc/testsuite/gfortran.dg/pr71799.f90 | 12 ++++++++++++
 2 files changed, 15 insertions(+), 3 deletions(-)
 create mode 100644 gcc/testsuite/gfortran.dg/pr71799.f90

diff --git a/gcc/fortran/resolve.c b/gcc/fortran/resolve.c
index 1fc540a..23da9ac 100644
--- a/gcc/fortran/resolve.c
+++ b/gcc/fortran/resolve.c
@@ -6515,15 +6515,15 @@ gfc_resolve_iterator (gfc_iterator *iter, bool real_ok, bool own_scope)
   /* Convert start, end, and step to the same type as var.  */
   if (iter->start->ts.kind != iter->var->ts.kind
       || iter->start->ts.type != iter->var->ts.type)
-    gfc_convert_type (iter->start, &iter->var->ts, 2);
+    gfc_convert_type (iter->start, &iter->var->ts, 1);
 
   if (iter->end->ts.kind != iter->var->ts.kind
       || iter->end->ts.type != iter->var->ts.type)
-    gfc_convert_type (iter->end, &iter->var->ts, 2);
+    gfc_convert_type (iter->end, &iter->var->ts, 1);
 
   if (iter->step->ts.kind != iter->var->ts.kind
       || iter->step->ts.type != iter->var->ts.type)
-    gfc_convert_type (iter->step, &iter->var->ts, 2);
+    gfc_convert_type (iter->step, &iter->var->ts, 1);
 
   if (iter->start->expr_type == EXPR_CONSTANT
       && iter->end->expr_type == EXPR_CONSTANT
diff --git a/gcc/testsuite/gfortran.dg/pr71799.f90 b/gcc/testsuite/gfortran.dg/pr71799.f90
new file mode 100644
index 0000000..4760cc3
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/pr71799.f90
@@ -0,0 +1,12 @@
+! PR fortran/71799
+! { dg-do compile }
+
+subroutine test2(array, s, block)
+integer(1) :: i, block(9), array(2)
+integer (8) :: s
+
+do i = 10, HUGE(i) - 10, 222 ! { dg-error "Arithmetic overflow converting INTEGER\\(4\\) to INTEGER\\(1\\)" }
+  s = s + 1
+end do
+
+end subroutine test2
-- 
2.8.4


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]