Re: [PATCH 2/6] rs6000: Do not allow lo_sum accesses >= 4 bytes if unaligned

2015-01-17 Thread Segher Boessenkool
On Sat, Jan 17, 2015 at 09:30:07AM -0500, David Edelsohn wrote: > On Sat, Jan 17, 2015 at 3:48 AM, Segher Boessenkool > wrote: > > This fixes 29 FAILs. > > > > The ld, lwa etc. insns do not encode the low two bits of the offset in > > the opcode; those have to be zero. For -m64 this seemed to nev

Re: [PATCH 2/6] rs6000: Do not allow lo_sum accesses >= 4 bytes if unaligned

2015-01-17 Thread David Edelsohn
On Sat, Jan 17, 2015 at 3:48 AM, Segher Boessenkool wrote: > This fixes 29 FAILs. > > The ld, lwa etc. insns do not encode the low two bits of the offset in > the opcode; those have to be zero. For -m64 this seemed to never matter, > datums are always aligned; but for -m32 -mpowerpc64 you can get

[PATCH 2/6] rs6000: Do not allow lo_sum accesses >= 4 bytes if unaligned

2015-01-17 Thread Segher Boessenkool
This fixes 29 FAILs. The ld, lwa etc. insns do not encode the low two bits of the offset in the opcode; those have to be zero. For -m64 this seemed to never matter, datums are always aligned; but for -m32 -mpowerpc64 you can get symbols that are not sufficiently aligned. So check for that. [ Hr