Re: [RS6000] strict alignment for little-endian

2013-11-19 Thread David Edelsohn
On Tue, Nov 19, 2013 at 1:59 AM, Alan Modra wrote: > This patch disables the default -mstrict-align on power8. Rather than > testing values from enum processor_type, which is prone to error when > adding a new processor, I chose one of the mask bits that happens to > be set for power8, and is li

Re: [RS6000] strict alignment for little-endian

2013-11-19 Thread Eric Botcazou
> It's not desirable since gcc easily loses track of alignment, for instance > with -mstrict-align > > void foo (char *p, char *q) > { > __builtin_memcpy (p, q, 4); > } > > generates > > lbz 7,0(4) > lbz 8,1(4) > lbz 10,2(4) > lbz 9,3(4) > stb 7,0(3) > stb 8

Re: [RS6000] strict alignment for little-endian

2013-11-18 Thread Alan Modra
On Mon, Jun 10, 2013 at 03:14:04PM -0400, Michael Meissner wrote: > On Fri, Jun 07, 2013 at 10:54:39AM +0930, Alan Modra wrote: > > I'd like to remove -mstrict-align for little-endian powerpc, because > > the assumption that mis-aligned accesses are massively slow isn't true > > for current powerpc