Patchwork PATCH: PR target/46285: ICE: in final_scan_insn, at final.c:2642 with -mavx -fsplit-stack

login
register
mail settings
Submitter H.J. Lu
Date Nov. 4, 2010, 2:45 a.m.
Message ID <20101104024541.GA661@intel.com>
Download mbox | patch
Permalink /patch/70095/
State New
Headers show

Comments

H.J. Lu - Nov. 4, 2010, 2:45 a.m.
On Tue, Nov 02, 2010 at 09:42:53PM -0700, H.J. Lu wrote:
> Hi,
> 
> We should generate avx_vzeroupper after reload is completed.  OK for
> trunk?
> 
> Thanks.
> 
> 
> H.J.
> ---
> gcc/
> 
> 2010-11-02  H.J. Lu  <hongjiu.lu@intel.com>
> 
> 	PR target/46285
> 	* config/i386/i386.c (ix86_expand_call): Conditionally decorate
> 	calls with UNSPEC_CALL_NEEDS_VZEROUPPER unspec before reload
> 	is completed.
> 
> gcc/testsuite/
> 
> 2010-11-02  H.J. Lu  <hongjiu.lu@intel.com>
> 
> 	PR target/46285
> 	* gcc.target/i386/pr46285.c.
> 

Hi,

I checked in this patch as an obvious fix.

Thanks.


H.J.
---

Patch

Index: gcc.target/i386/pr46285.c
===================================================================
--- gcc.target/i386/pr46285.c	(revision 166297)
+++ gcc.target/i386/pr46285.c	(working copy)
@@ -1,5 +1,6 @@ 
 /* { dg-do compile } */
 /* { dg-options "-mavx -fsplit-stack -mtune=generic" } */
+/* { dg-require-effective-target split_stack } */
 
 typedef char __m256 __attribute__ ((__vector_size__ (32)));
 void foo (__m256 x) {}
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 166297)
+++ ChangeLog	(working copy)
@@ -1,3 +1,7 @@ 
+2010-11-03  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* gcc.target/i386/pr46285.c: Require split_stack target.
+
 2010-11-03  Jason Merrill  <jason@redhat.com>
 
 	PR c++/46293