public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [patch] work-arround for misscompilation of crtstuff
@ 2007-11-13 11:12 Victor Kaplansky
  2007-11-13 13:41 ` Paolo Bonzini
  2007-11-13 18:36 ` Mark Mitchell
  0 siblings, 2 replies; 3+ messages in thread
From: Victor Kaplansky @ 2007-11-13 11:12 UTC (permalink / raw)
  To: gcc-patches; +Cc: mark, rakdver, richard.guenther


Hi,

This patch turns off vectorization for crtstuff in order
to work-arround the problem caused by increased
alignment with enabled vectorization. See a disscussion in
http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00832.html.

Bootstrapped on PPC64.
OK, for mainline testing?

-- Victor

2007-11-13  Victor Kaplansky  <victork@il.ibm.com>

        * Makefile.in (CRTSTUFF_CFLAGS): Add -fno-tree-vectorize

Index: Makefile.in
===================================================================
--- Makefile.in (revision 130086)
+++ Makefile.in (working copy)
@@ -571,7 +571,7 @@ TARGET_LIBGCC2_CFLAGS =
 # Options to use when compiling crtbegin/end.
 CRTSTUFF_CFLAGS = -O2 $(GCC_CFLAGS) $(INCLUDES) $(MULTILIB_CFLAGS) -g0 \
   -finhibit-size-directive -fno-inline-functions -fno-exceptions \
-  -fno-zero-initialized-in-bss -fno-toplevel-reorder \
+  -fno-zero-initialized-in-bss -fno-toplevel-reorder -fno-tree-vectorize \
   $(INHIBIT_LIBC_CFLAGS)

 # Additional sources to handle exceptions; overridden by targets as
needed.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] work-arround for misscompilation of crtstuff
  2007-11-13 11:12 [patch] work-arround for misscompilation of crtstuff Victor Kaplansky
@ 2007-11-13 13:41 ` Paolo Bonzini
  2007-11-13 18:36 ` Mark Mitchell
  1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2007-11-13 13:41 UTC (permalink / raw)
  To: gcc-patches

Victor Kaplansky wrote:
> Hi,
> 
> This patch turns off vectorization for crtstuff in order
> to work-arround the problem caused by increased
> alignment with enabled vectorization. See a disscussion in
> http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00832.html.
> 
> Bootstrapped on PPC64.
> OK, for mainline testing?

I don't like the approach, but it seemed that it was agreed upon.  So okay.

Paolo

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] work-arround for misscompilation of crtstuff
  2007-11-13 11:12 [patch] work-arround for misscompilation of crtstuff Victor Kaplansky
  2007-11-13 13:41 ` Paolo Bonzini
@ 2007-11-13 18:36 ` Mark Mitchell
  1 sibling, 0 replies; 3+ messages in thread
From: Mark Mitchell @ 2007-11-13 18:36 UTC (permalink / raw)
  To: Victor Kaplansky; +Cc: gcc-patches, rakdver, richard.guenther

Victor Kaplansky wrote:
> Hi,
> 
> This patch turns off vectorization for crtstuff in order
> to work-arround the problem caused by increased
> alignment with enabled vectorization. See a disscussion in
> http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00832.html.
> 
> Bootstrapped on PPC64.
> OK, for mainline testing?
> 
> -- Victor
> 
> 2007-11-13  Victor Kaplansky  <victork@il.ibm.com>
> 
>         * Makefile.in (CRTSTUFF_CFLAGS): Add -fno-tree-vectorize

OK.

If someone has a better solution, we can replace this with that, at that
point.

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-11-13 17:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-11-13 11:12 [patch] work-arround for misscompilation of crtstuff Victor Kaplansky
2007-11-13 13:41 ` Paolo Bonzini
2007-11-13 18:36 ` Mark Mitchell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).