Re: PowerPC64 reload failure with misaligned fp load

2011-03-25 Thread Alan Modra
On Fri, Mar 25, 2011 at 09:36:04PM +0100, Ulrich Weigand wrote: > Looks good to me, except ... > > + mem = change_address (mem, VOIDmode, scratch_or_premodify); > > Maybe replace_equiv_address instead, to avoid losing the memory > attribute information (alignment, alias set, ...)? Yes, in fact

Re: PowerPC64 reload failure with misaligned fp load

2011-03-25 Thread Ulrich Weigand
David Edelsohn wrote: > On Fri, Mar 25, 2011 at 11:09 AM, Alan Modra wrote: > > > Much of the following patch is based on Michael Meissner's support for > > vector reloads. =A0The predicates.md change teaches the predicate used > > by the "Y" constraint to check cmodel medium addresses in case su

Re: PowerPC64 reload failure with misaligned fp load

2011-03-25 Thread David Edelsohn
On Fri, Mar 25, 2011 at 11:09 AM, Alan Modra wrote: > Much of the following patch is based on Michael Meissner's support for > vector reloads.  The predicates.md change teaches the predicate used > by the "Y" constraint to check cmodel medium addresses in case such > addresses should ever be gene

PowerPC64 reload failure with misaligned fp load

2011-03-25 Thread Alan Modra
Compiler errors like the following have plagued powerpc64 gcc for a long time. error: insn does not satisfy its constraints: (insn 43020 13791 13792 1245 _thread.c:8306 (set (reg:DF 12 12) (mem:DF (plus:DI (reg:DI 3 3 [orig:4092 D.11951 ] [4092]) (const_int 15 [0xf])) [0 S8