re PR middle-end/67271 (ICE with -O1,2,3 on darwin14.5 x86_64)

2015-09-16  Richard Biener  <rguenther@suse.de>

	PR middle-end/67271
	* fold-const.c (native_encode_expr): Bail out on bogus offsets.

	* gcc.dg/pr67271.c: New testcase.

From-SVN: r227825

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 085de31e554bb76892cfdb5727ce4e330b919af4..2ec6aefd507709062e295d92cf51061c2c4f2e88 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2015-09-16  Richard Biener  <rguenther@suse.de>
+
+       PR middle-end/67271
+       * fold-const.c (native_encode_expr): Bail out on bogus offsets.
+
 2015-09-16  Eric Botcazou  <ebotcazou@adacore.com>
 
        * tree-ssa-sccvn.c (ao_ref_init_from_vn_reference): Use offset_int for

diff --git a/gcc/fold-const.c b/gcc/fold-const.c
index fd1c87ed5ad08af7b38d0cc4a3c8b2418da65f3a..15214507f083f7440392cf2513efaa89477c3bf8 100644 (file)
--- a/gcc/fold-const.c
+++ b/gcc/fold-const.c
@@ -7110,6 +7110,10 @@ native_encode_string (const_tree expr, unsigned char *ptr, int len, int off)
 int
 native_encode_expr (const_tree expr, unsigned char *ptr, int len, int off)
 {
+  /* We don't support starting at negative offset and -1 is special.  */
+  if (off < -1)
+    return 0;
+
   switch (TREE_CODE (expr))
     {
     case INTEGER_CST:

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 945db718bd04bcbd9fd73329e1927b121a947bed..87bcebfd00f973969887b7c9bfd35c58efc54077 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2015-09-16  Richard Biener  <rguenther@suse.de>
+
+       PR middle-end/67271
+       * gcc.dg/pr67271.c: New testcase.
+
 2015-09-16  Eric Botcazou  <ebotcazou@adacore.com>
 
        * gnat.dg/opt49.adb: New test.

diff --git a/gcc/testsuite/gcc.dg/pr67271.c b/gcc/testsuite/gcc.dg/pr67271.c
new file mode 100644 (file)
index 0000000..b6988a3
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr67271.c
@@ -0,0 +1,12 @@
+/* { dg-do compile } */
+/* { dg-options "-O" } */
+
+extern long int labs (long int j);
+int
+main ()
+{
+  long *a = (long *)"empty";
+  int i = 1441516387;
+  a[i] = labs (a[i]);
+  return 0;
+}