From: Steven G. Kargl <kargl@gcc.gnu.org>
Date: Fri, 22 Jul 2016 18:53:11 +0000 (+0000)
Subject: re PR fortran/71935 (ICE is_c_interoperable(): gfc_simplify_expr failed)
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d52d37673383f479aec07664bac5eec07f19d2da;p=gcc.git

re PR fortran/71935 (ICE is_c_interoperable(): gfc_simplify_expr failed)

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

	PR fortran/71935
	* check.c (is_c_interoperable): Simplify right expression.

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

	PR fortran/71935
	* gfortran.dg/c_sizeof_1.f90: Move invalid code to ...
	* gfortran.dg/c_sizeof_6.f90: here.  Test for error.
	* gfortran.dg/pr71935.f90: New test.

From-SVN: r238665
---

diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index ff12b56ec2e..b056e0e86e8 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,8 @@
+2016-07-22  Steven G. Kargl  <kargl@gcc.gnu.org>
+
+	PR fortran/71935
+	* check.c (is_c_interoperable): Simplify right expression. 
+
 2016-07-22  Thomas Koenig  <tkoenig@gcc.gnu.org>
 
 	PR fortran/71795
diff --git a/gcc/fortran/check.c b/gcc/fortran/check.c
index d26e45ec406..7fc608081cc 100644
--- a/gcc/fortran/check.c
+++ b/gcc/fortran/check.c
@@ -4278,7 +4278,7 @@ is_c_interoperable (gfc_expr *expr, const char **msg, bool c_loc, bool c_f_ptr)
       }
 
     if (expr->ts.u.cl && expr->ts.u.cl->length
-	&& !gfc_simplify_expr (expr, 0))
+	&& !gfc_simplify_expr (expr->ts.u.cl->length, 0))
       gfc_internal_error ("is_c_interoperable(): gfc_simplify_expr failed");
 
     if (!c_loc && expr->ts.u.cl
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index c38c41e492b..2c381514fb2 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,10 @@
+2016-07-22  Steven G. Kargl  <kargl@gcc.gnu.org>
+
+	PR fortran/71935
+	* gfortran.dg/c_sizeof_1.f90: Move invalid code to ...
+	* gfortran.dg/c_sizeof_6.f90: here.  Test for error.
+	* gfortran.dg/pr71935.f90: New test.
+
 2016-07-22 Martin Sebor  <msebor@redhat.com>
 
 	PR c++/71675
diff --git a/gcc/testsuite/gfortran.dg/c_sizeof_1.f90 b/gcc/testsuite/gfortran.dg/c_sizeof_1.f90
index 4a8385b8d8f..d601f1ec8ec 100644
--- a/gcc/testsuite/gfortran.dg/c_sizeof_1.f90
+++ b/gcc/testsuite/gfortran.dg/c_sizeof_1.f90
@@ -22,9 +22,6 @@ if (i /= 4) call abort()
 i = c_sizeof(str2(1))
 if (i /= 1) call abort()
 
-i = c_sizeof(str2(1:3))
-if (i /= 3) call abort()
-
 write(*,*) c_sizeof(cptr), c_sizeof(iptr), c_sizeof(C_NULL_PTR)
 
 ! Using GNU's SIZEOF
diff --git a/gcc/testsuite/gfortran.dg/c_sizeof_6.f90 b/gcc/testsuite/gfortran.dg/c_sizeof_6.f90
new file mode 100644
index 00000000000..0b57964ea5b
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/c_sizeof_6.f90
@@ -0,0 +1,16 @@
+! { dg-do compile }
+!
+program foo
+
+   use iso_c_binding, only: c_int, c_char, c_sizeof
+
+   integer(kind=c_int) :: i
+
+   character(kind=c_char,len=1),parameter :: str2(4) = ["a","b","c","d"]
+
+   i = c_sizeof(str2(1:3)) ! { dg-error "must be an interoperable data" }
+
+   if (i /= 3) call abort()
+
+end program foo
+
diff --git a/gcc/testsuite/gfortran.dg/pr71935.f90 b/gcc/testsuite/gfortran.dg/pr71935.f90
new file mode 100644
index 00000000000..495759b1f6f
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/pr71935.f90
@@ -0,0 +1,7 @@
+! { dg-do compile }
+program p
+   use iso_c_binding
+   character(len=1, kind=c_char), parameter :: z(2) = 'z'
+   print *, sizeof(z(3))      ! { dg-warning "is out of bounds" }
+   print *, c_sizeof(z(3))    ! { dg-warning "is out of bounds" }
+end