On Tue, Jul 09, 2019 at 10:28:06AM +0800, Kewen.Lin wrote:
> on 2019/7/9 上午12:32, Segher Boessenkool wrote:
> > On Mon, Jul 08, 2019 at 04:07:00PM +0800, Kewen.Lin wrote:
> >> --- /dev/null
> >> +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr88497-1.c
> >> @@ -0,0 +1,60 @@
> >> +/* { dg-do run } */
> >> +/* { dg-require-effective-target vect_double } */
> >> +/* { dg-require-effective-target powerpc_vsx_ok { target { powerpc*-*-* }
> >> } } */
> >
> > For "dg-do run" tests, you need "powerpc_vsx_hw". "_ok" only tests if
> > the assembler can handle VSX instructions, not whether the test system
> > can run them. (powerpc_vsx_ok is what you need for "dg-do assemble" or
> > "dg-do link" tests. It also tests if you can use -mvsx, but that doesn't
> > do what you might hope it does: you can use -mvsx together with a -mcpu=
> > that doesn't support VSX, for example).
>
> Thanks, I will update it. But sorry that I can't find "powerpc_vsx_hw" but
> "vsx_hw_available". I guess it's the one you are referring to?
Yeah, sorry. You can also use just "vsx_hw".
> And I happened
> to find the vect_double will force powerpc to check vsx_hw_available.
Yes :-) So this whole line is unnecessary.
Segher
