Re: [PATCH, rs6000] Restrict reload use of FLOAT_REGS

2014-03-03 Thread Bill Schmidt
Uli, thanks. Sorry I misunderstood what you said the first time. I'm currently testing this version, which I've verified fixes the bug. I'll plan to commit the new version after testing provided neither you nor David objects. Thanks! Bill On Mon, 2014-03-03 at 17:17 +0100, Ulrich Weigand wrote

Re: [PATCH, rs6000] Restrict reload use of FLOAT_REGS

2014-03-03 Thread Ulrich Weigand
David Edelsohn wrote: > On Fri, Feb 28, 2014 at 7:11 PM, Bill Schmidt > wrote: > > * config/rs6000/rs6000.c (rs6000_preferred_reload_class): Disallow > > PLUS rtx's from reloading into a superset of FLOAT_REGS; relax > > constraint on constants to only prevent them from bei

Re: [PATCH, rs6000] Restrict reload use of FLOAT_REGS

2014-02-28 Thread David Edelsohn
On Fri, Feb 28, 2014 at 7:11 PM, Bill Schmidt wrote: > Hi, > > We've encountered a rare bug that occurs when attempting to reload for > an unaligned store in DImode. For an unaligned store, using stfd gets > preference over std since stfd doesn't have an alignment restriction and > therefore the

[PATCH, rs6000] Restrict reload use of FLOAT_REGS

2014-02-28 Thread Bill Schmidt
Hi, We've encountered a rare bug that occurs when attempting to reload for an unaligned store in DImode. For an unaligned store, using stfd gets preference over std since stfd doesn't have an alignment restriction and therefore the "m" constraint matches. However, when there is not a register av