From: Jakub Jelinek <jakub@redhat.com>
Date: Tue, 8 Mar 2016 08:05:26 +0000 (+0100)
Subject: re PR target/70110 (ICE at -O3 in the 32-bit mode in set_last_insn, at emit-rtl.h... 
X-Git-Tag: basepoints/gcc-7~528
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=12b81409d97f6d617f58f974c6cbedc6bf9f5552;p=thirdparty%2Fgcc.git

re PR target/70110 (ICE at -O3 in the 32-bit mode in set_last_insn, at emit-rtl.h:420)

	PR target/70110
	* config/i386/i386.c (scalar_chain::make_vector_copies,
	scalar_chain::convert_reg): Call end_sequence in between
	get_insns and emit_conversion_insns rather than after both
	calls.

	* gcc.dg/pr70110.c: New test.

From-SVN: r234057
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 38c47b696560..3bfd5f45f908 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,11 @@
+2016-03-08  Jakub Jelinek  <jakub@redhat.com>
+
+	PR target/70110
+	* config/i386/i386.c (scalar_chain::make_vector_copies,
+	scalar_chain::convert_reg): Call end_sequence in between
+	get_insns and emit_conversion_insns rather than after both
+	calls.
+
 2016-03-07  Uros Bizjak  <ubizjak@gmail.com>
 
 	PR target/70064
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 5155a00ccc93..25a6467bbd1a 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -3272,8 +3272,9 @@ scalar_chain::make_vector_copies (unsigned regno)
 			    gen_rtx_SUBREG (SImode, reg, 4));
 	    emit_move_insn (vreg, tmp);
 	  }
-	emit_conversion_insns (get_insns (), insn);
+	rtx_insn *seq = get_insns ();
 	end_sequence ();
+	emit_conversion_insns (seq, insn);
 
 	if (dump_file)
 	  fprintf (dump_file,
@@ -3348,8 +3349,9 @@ scalar_chain::convert_reg (unsigned regno)
 	      emit_move_insn (gen_rtx_SUBREG (SImode, scopy, 4),
 			      adjust_address (tmp, SImode, 4));
 	    }
-	  emit_conversion_insns (get_insns (), insn);
+	  rtx_insn *seq = get_insns ();
 	  end_sequence ();
+	  emit_conversion_insns (seq, insn);
 
 	  if (dump_file)
 	    fprintf (dump_file,
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0dfe344bc807..28229ee4cf84 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2016-03-08  Jakub Jelinek  <jakub@redhat.com>
+
+	PR target/70110
+	* gcc.dg/pr70110.c: New test.
+
 2016-03-07  Martin Jambor  <mjambor@suse.cz>
 
 	* c-c++-common/gomp/clauses-1.c: Remove dg-options.
diff --git a/gcc/testsuite/gcc.dg/pr70110.c b/gcc/testsuite/gcc.dg/pr70110.c
new file mode 100644
index 000000000000..42bbe3a956b4
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr70110.c
@@ -0,0 +1,39 @@
+/* PR target/70110 */
+/* { dg-do compile } */
+/* { dg-options "-O3" } */
+/* { dg-additional-options "-msse2" { target i?86-*-* x86_64-*-* } } */
+
+int a, c, d, f, h;
+long long b;
+
+static inline void
+foo (void)
+{
+  if (a) 
+    foo ();
+  b = c;
+}
+
+static inline void 
+bar (int p)
+{
+  if (p)
+    f = 0; 
+  b |= c; 
+} 
+
+void
+baz (int g, int i)
+{
+  for (b = d; (d = 1) != 0; )
+    {
+      if (a)
+	foo ();
+      b |= c;
+      bar (h); 
+      bar (g); 
+      bar (h); 
+      bar (i); 
+      bar (h);
+    }
+}