On Oct 11, 2011, at 11:08 AM, Janis Johnson wrote: > On 10/10/2011 01:19 PM, Janis Johnson wrote: >> Tests gcc.target/powerpc/warn-[12].c fail for soft-float multilibs with >> the unexpected warning "-mvsx requires hardware floating point [enabled >> by default]". This patch skips those tests for soft-float multilibs and >> modifies the powerpc check for a soft-float effective target to return >> true for either __NO_FPRS__ or _SOFT_FLOAT being defined. >> >> Is this OK for trunk? I must admit that I'm not sure what all those >> Power float variants are for. > > On second thought these tests should use > > /* { dg-require-effective-target powerpc_vsx_ok } */ > > instead of requiring hard_float, and forget about the proposed > change to effective target hard_float. Is that OK?
I was hoping a ppc person would chime in... Ok.