This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [testsuite] modify powerpc test for hard_float target, skip powerpc/warn-[12].c for soft-float
- From: Janis Johnson <janis_johnson at mentor dot com>
- To: <janisjo at codesourcery dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 11 Oct 2011 11:08:18 -0700
- Subject: Re: [testsuite] modify powerpc test for hard_float target, skip powerpc/warn-[12].c for soft-float
- References: <4E935366.3060700@mentor.com>
- Reply-to: <janisjo at codesourcery dot com>
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?
Janis