Re: [PATCH] Fix PR53708

2012-06-20 Thread Richard Guenther
On Tue, 19 Jun 2012, Iain Sandoe wrote: On 19 Jun 2012, at 22:41, Mike Stump wrote: On Jun 19, 2012, at 12:22 PM, Iain Sandoe i...@codesourcery.com wrote: On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther

Re: [PATCH] Fix PR53708

2012-06-20 Thread Richard Guenther
On Tue, 19 Jun 2012, Dominique Dhumieres wrote: On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or

Re: [PATCH] Fix PR53708

2012-06-20 Thread Iain Sandoe
Hi, On 20 Jun 2012, at 09:23, Richard Guenther wrote: On Tue, 19 Jun 2012, Iain Sandoe wrote: On 19 Jun 2012, at 22:41, Mike Stump wrote: On Jun 19, 2012, at 12:22 PM, Iain Sandoe i...@codesourcery.com wrote: On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: On Tue, 19 Jun 2012,

Re: [PATCH] Fix PR53708

2012-06-20 Thread Richard Guenther
On Wed, 20 Jun 2012, Richard Guenther wrote: On Wed, 20 Jun 2012, Iain Sandoe wrote: Hi, On 20 Jun 2012, at 09:23, Richard Guenther wrote: On Tue, 19 Jun 2012, Iain Sandoe wrote: On 19 Jun 2012, at 22:41, Mike Stump wrote: On Jun 19, 2012, at 12:22 PM, Iain

[PATCH] Fix PR53708

2012-06-19 Thread Richard Guenther
We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that are used in an unknown way. Bootstrapped and tested on i686-darwin9 and x86_64-apple-darwin10 and powerpc-apple-darwin9 by darwin folks, applied.

Re: [PATCH] Fix PR53708

2012-06-19 Thread Richard Sandiford
Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that are used in an unknown way. I thought attribute((__aligned__)) only set a minimum alignment for

Re: [PATCH] Fix PR53708

2012-06-19 Thread Richard Guenther
On Tue, 19 Jun 2012, Richard Sandiford wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that are used in an unknown way. I thought

Re: [PATCH] Fix PR53708

2012-06-19 Thread Dominique Dhumieres
On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that are used in an unknown way. I thought

Re: [PATCH] Fix PR53708

2012-06-19 Thread Iain Sandoe
On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that

Re: [PATCH] Fix PR53708

2012-06-19 Thread Mike Stump
On Jun 19, 2012, at 12:22 PM, Iain Sandoe i...@codesourcery.com wrote: On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is

Re: [PATCH] Fix PR53708

2012-06-19 Thread Mike Stump
On Jun 19, 2012, at 5:53 AM, domi...@lps.ens.fr (Dominique Dhumieres) wrote: On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user

Re: [PATCH] Fix PR53708

2012-06-19 Thread Iain Sandoe
On 19 Jun 2012, at 22:41, Mike Stump wrote: On Jun 19, 2012, at 12:22 PM, Iain Sandoe i...@codesourcery.com wrote: On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: On Tue, 19 Jun 2012, Richard Guenther wrote: Richard Guenther rguent...@suse.de writes: We are too eager to bump