Hi!

On Mon, Feb 21, 2022 at 03:17:47PM -0600, Paul A. Clarke wrote:
> gcc/testsuite
>       * g++.dg/debug/dwarf2/const2.C: Move to g++.target/powerpc.
>       * g++.dg/other/darwin-minversion-1.C: Likewise.
>       * g++.dg/eh/ppc64-sighandle-cr.C: Likewise.

This one uses
// { dg-do run { target { powerpc64*-*-linux* } } }
which is never correct: the target defaulting to 64 bit does in no way
mean that this is built as 64 bit.  There is nothing in the test that
needs 64 bit afaics?

>       * g++.dg/eh/simd-5.C: Likewise.
// { dg-do run { target { powerpc_spe && { ! *-*-vxworks* } } } }

This testcase can be deleted, there is no SPE anymore.

But that is perhaps best done deleting these tests and everything that
uses it as well at the same time :-)

>       * g++.dg/eh/simd-4.C: Move to g++.target/powerpc, adjust dg directives.
>       * g++.dg/eh/uncaught3.C: Likewise.
>       * g++.dg/other/spu2vmx-1.C: Likewise.

Those things can be done later, so: okay for trunk.  Thanks!


Segher

Reply via email to