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]

[Committed] Fix PR target/29909, -Os -m64 -mno-sse ICEs


The problem here is that Jason removed the check of TARGET_64BIT from
the default alignment which meant that -Os -m64 -mno-sse would give the
wrong alignment.  This patch adds that check back.

Committed after a build and a quick test on x86_64-linux-gnu.

Thanks,
Andrew Pinski

2006-09-10  Andrew Pinski  <pinskia@physics.uc.edu>

        PR target/29009
        * config/i386/i386.c (override_options): Use 128-bit
        stack boundary always for 64bits.


Index: config/i386/i386.c
===================================================================
--- config/i386/i386.c	(revision 116830)
+++ config/i386/i386.c	(working copy)
@@ -1903,7 +1903,8 @@ override_options (void)
      don't want additional code to keep the stack aligned when
      optimizing for code size.  */
   ix86_preferred_stack_boundary
-    = ((TARGET_MACHO || TARGET_SSE || !optimize_size) ? 128 : 32);
+    = (TARGET_64BIT || TARGET_MACHO || TARGET_SSE || !optimize_size) 
+       ? 128 : 32;
   if (ix86_preferred_stack_boundary_string)
     {
       i = atoi (ix86_preferred_stack_boundary_string);
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 116830)
+++ ChangeLog	(working copy)
@@ -1,3 +1,9 @@
+2006-09-10  Andrew Pinski  <pinskia@physics.uc.edu>
+
+	PR target/29009
+	* config/i386/i386.c (override_options): Use 128-bit
+	stack boundary always for 64bits.
+
 2006-09-10  Eric Botcazou  <ebotcazou@libertysurf.fr>
 
 	PR rtl-optimization/28636

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