Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-11-03 Thread Peter Bergner
On Wed, 2012-10-31 at 21:54 -0500, Peter Bergner wrote: > On Wed, 2012-10-31 at 09:02 -0500, Peter Bergner wrote: > > On Wed, 2012-10-31 at 14:55 +0100, Jakub Jelinek wrote: > > > On Wed, Oct 31, 2012 at 08:53:31AM -0500, Peter Bergner wrote: > > > > Great. Jakub, were you going to commit your cha

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-31 Thread Peter Bergner
On Wed, 2012-10-31 at 09:02 -0500, Peter Bergner wrote: > On Wed, 2012-10-31 at 14:55 +0100, Jakub Jelinek wrote: > > On Wed, Oct 31, 2012 at 08:53:31AM -0500, Peter Bergner wrote: > > > Great. Jakub, were you going to commit your change or did you want me > > > to do that? > > > > I haven't test

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-31 Thread Peter Bergner
On Wed, 2012-10-31 at 14:55 +0100, Jakub Jelinek wrote: > On Wed, Oct 31, 2012 at 08:53:31AM -0500, Peter Bergner wrote: > > Great. Jakub, were you going to commit your change or did you want me > > to do that? > > I haven't tested it, you did, so please do that yourself. Thanks. I tested it on

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-31 Thread Jakub Jelinek
On Wed, Oct 31, 2012 at 08:53:31AM -0500, Peter Bergner wrote: > Great. Jakub, were you going to commit your change or did you want me > to do that? I haven't tested it, you did, so please do that yourself. Thanks. Jakub

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-31 Thread Peter Bergner
On Wed, 2012-10-31 at 10:12 +0100, Richard Biener wrote: > On Tue, Oct 30, 2012 at 9:23 PM, Peter Bergner wrote: > > On Tue, 2012-10-30 at 20:37 +0100, Jakub Jelinek wrote: > >> On Tue, Oct 30, 2012 at 02:03:44PM -0500, Peter Bergner wrote: > >> > Ok, then I'll bootstrap and regtest your suggested

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-31 Thread Richard Biener
On Tue, Oct 30, 2012 at 9:23 PM, Peter Bergner wrote: > On Tue, 2012-10-30 at 20:37 +0100, Jakub Jelinek wrote: >> On Tue, Oct 30, 2012 at 02:03:44PM -0500, Peter Bergner wrote: >> > Ok, then I'll bootstrap and regtest your suggested change while we >> > wait for richi to comment. I'm fine with w

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-30 Thread Peter Bergner
On Tue, 2012-10-30 at 20:37 +0100, Jakub Jelinek wrote: > On Tue, Oct 30, 2012 at 02:03:44PM -0500, Peter Bergner wrote: > > Ok, then I'll bootstrap and regtest your suggested change while we > > wait for richi to comment. I'm fine with whatever you and richi > > decide is best. The ObjC guys sho

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-30 Thread Jakub Jelinek
On Tue, Oct 30, 2012 at 02:03:44PM -0500, Peter Bergner wrote: > Ok, then I'll bootstrap and regtest your suggested change while we > wait for richi to comment. I'm fine with whatever you and richi > decide is best. The ObjC guys should probably test it though too. > > I assume you think we shou

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-30 Thread Peter Bergner
On Tue, 2012-10-30 at 19:55 +0100, Jakub Jelinek wrote: > On Tue, Oct 30, 2012 at 01:43:33PM -0500, Peter Bergner wrote: > > Commenting on Richard's question from the bugzilla: > > > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53708#c10 > > > > I suppose if attribute((__aligned__)) truly does

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-30 Thread Jakub Jelinek
On Tue, Oct 30, 2012 at 01:43:33PM -0500, Peter Bergner wrote: > Commenting on Richard's question from the bugzilla: > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53708#c10 > > I suppose if attribute((__aligned__)) truly does just set a minimum alignment > value (and the documentation seems t

Re: [PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-30 Thread Peter Bergner
On Tue, 2012-10-30 at 11:58 -0500, Peter Bergner wrote: > I'm hitting the same bug as in PR53708 when compiling GLIBC's dlfcn.c when > vectorization is enabled on powerpc64-linux. A reduced test case is: > > bergner@bns:~/gcc/BUGS> cat foo.i > static void (*const init_array []) (void) > __attr

[PATCH, GCC 4.7] Backport fix for PR tree-optimization/53708

2012-10-30 Thread Peter Bergner
I'm hitting the same bug as in PR53708 when compiling GLIBC's dlfcn.c when vectorization is enabled on powerpc64-linux. A reduced test case is: bergner@bns:~/gcc/BUGS> cat foo.i static void (*const init_array []) (void) __attribute__ ((section (".init_array"), aligned (sizeof (void *)), used))