Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-26 Thread H.J. Lu
On Thu, Dec 25, 2014 at 8:56 PM, Gerald Pfeifer wrote: > On Wednesday 2014-12-17 06:49, H.J. Lu wrote: >> Index: gcc-5/changes.html >> === >> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v >> retrieving revision 1.52 >> diff -

Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-25 Thread Gerald Pfeifer
On Wednesday 2014-12-17 06:49, H.J. Lu wrote: > Index: gcc-5/changes.html > === > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v > retrieving revision 1.52 > diff -u -p -r1.52 changes.html > --- gcc-5/changes.html15 Dec

Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-17 Thread H.J. Lu
On Wed, Dec 17, 2014 at 03:31:50PM +0100, Uros Bizjak wrote: > On Wed, Dec 17, 2014 at 2:10 PM, H.J. Lu wrote: > > > Add -malign-data={abi|compat,cachineline} to control how GCC aligns > > variables. "compat" uses increased alignment value compatible with > > GCC 4.8 and earlier, "abi" uses alig

Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-17 Thread Uros Bizjak
On Wed, Dec 17, 2014 at 2:10 PM, H.J. Lu wrote: > Add -malign-data={abi|compat,cachineline} to control how GCC aligns > variables. "compat" uses increased alignment value compatible with > GCC 4.8 and earlier, "abi" uses alignment value as specified by the > psABI, and "cacheline" with increased

Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-17 Thread Uros Bizjak
On Wed, Dec 17, 2014 at 2:14 PM, Jakub Jelinek wrote: > On Wed, Dec 17, 2014 at 05:10:36AM -0800, H.J. Lu wrote: >> PR target/61296 >> * config/i386/i386-opts.h (ix86_align_data): New enum. >> * config/i386/i386.c (ix86_data_alignment): Return the ABI >> alignment value for

Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-17 Thread H.J. Lu
On Wed, Dec 17, 2014 at 02:14:34PM +0100, Jakub Jelinek wrote: > On Wed, Dec 17, 2014 at 05:10:36AM -0800, H.J. Lu wrote: > > PR target/61296 > > * config/i386/i386-opts.h (ix86_align_data): New enum. > > * config/i386/i386.c (ix86_data_alignment): Return the ABI > > alignment value

Re: PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-17 Thread Jakub Jelinek
On Wed, Dec 17, 2014 at 05:10:36AM -0800, H.J. Lu wrote: > PR target/61296 > * config/i386/i386-opts.h (ix86_align_data): New enum. > * config/i386/i386.c (ix86_data_alignment): Return the ABI > alignment value for -malign-data=abi, the cachine line size > for -malign-

PATCH: PR target/61296: Excessive alignment in ix86_data_alignment

2014-12-17 Thread H.J. Lu
> From: Jakub Jelinek > > On Wed, Dec 17, 2014 at 09:33:23AM +0100, Uros Bizjak wrote: > > Oops, I was under impression that HJ sent the patch on public list. > > The ix86_data_alignment change looks too ugly and too complicated to me. > > I think it would be much cleaner to add after: > > /