[PATCH v2, rs6000] Fix PR81833

[Segher Boessenkool]

Hi!

On Tue, Aug 29, 2017 at 04:47:18PM -0500, Bill Schmidt wrote:
> Thanks for approving the previous patch with changes.  I've made those and also
> modified the test case to require VSX hardware for execution.  I duplicated the
> test so we get coverage on P7 BE 32/64 and P8 BE/LE.  I'd appreciate it if you
> could look over the dejagnu instructions once more on these.  Thanks!

> --- gcc/testsuite/gcc.target/powerpc/pr81833-1.c	(nonexistent)
> +++ gcc/testsuite/gcc.target/powerpc/pr81833-1.c	(working copy)
> @@ -0,0 +1,59 @@
> +/* PR81833: This used to fail due to improper implementation of vec_msum.  */
> +/* Test case relies on -mcpu=power7 or later.  Currently we don't have
> +   machinery to express that, so we have two separate tests for -mcpu=power7
> +   and -mcpu=power8 to catch 32-bit BE on P7 and 64-bit BE/LE on P8.  */
> +
> +/* { dg-do run } */
> +/* { dg-require-effective-target vsx_hw } */
> +/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { "-mcpu=power8" } } */
> +/* { dg-options "-mcpu=power8 -O2" } */

As I explained off-list, but here's for the record:

Since this is a run test and you use -mcpu=power8, you probably should
use p8vector_hw instead of vsx_hw.

Okay with that.  Thanks!


Segher