This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH,rs6000] Improve tests for Power9 vector shift variable instructions
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Kelvin Nilsen <kdnilsen at linux dot vnet dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 28 Jun 2016 14:14:23 -0500
- Subject: Re: [PATCH,rs6000] Improve tests for Power9 vector shift variable instructions
- Authentication-results: sourceware.org; auth=none
- References: <5772CAC6 dot 5030403 at linux dot vnet dot ibm dot com>
On Tue, Jun 28, 2016 at 01:06:46PM -0600, Kelvin Nilsen wrote:
> This patch improves upon a recently committed patch to change the
> DejaGnu conditionals that control when the tests run. The patch
> corrects failures that occur on aix platforms.
>
> This patch has bootstrapped on and regression tested on powerpc64le-
> unknown-linux-gnu with no regressions. Is this ok for trunk? Is it ok
> for gcc-6 after burn-in on the trunk?
Okay (for both). With the typoes fixed:
> /* { dg-do compile { target { powerpc*-*-* } } } */
> /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { "-mcpu=power9" } } */
> +/* { dg-require-effective-target powerpc_p9vector_ok } */
> +/* {dg-skip-if "" { powerpc*=*-aix* } } */
Space here (in all four testcases). And s/=/-/.
Thanks,
Segher