On Thu, Sep 26, 2013 at 06:56:37PM -0400, David Edelsohn wrote: > On Thu, Sep 26, 2013 at 4:51 PM, Michael Meissner > <meiss...@linux.vnet.ibm.com> wrote: > > I discovered that I was setting the wv/wu constraints incorrectly to > > ALTIVEC_REGS, which leads to reload failures in some cases. > > > > Is this patch ok to apply along with the previous patch assuming it > > bootstraps > > and has no regressions with make check? It builds the programs that had > > failures with the previous patch. > > > > 2013-09-26 Michael Meissner <meiss...@linux.vnet.ibm.com> > > > > * config/rs6000/rs6000.c (rs6000_init_hard_regno_mode_ok): Don't > > allow wv/wu constraints to be ALTIVEC_REGISTERS unless DF/SF can > > occupy the Altivec registers. > > Okay. > > Can you add a testcase to catch this in the future?
I looked into this, and of the 5 spec benchmarks that did not compile without the fix, 4 are in fortran, and the 5th in C++ did not show up the error if I deleted any code, so there is no simple test case I know of. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797
