On Wed, Jun 12, 2024 at 07:02:31PM -0500, Peter Bergner wrote:
> On 6/12/24 3:00 PM, Segher Boessenkool wrote:
> >> /* { dg-do compile { target { powerpc64*-*-* } } } */
> >
> > Probably should be an "lp64" instead?
>
> Actually, there is nothing inherently 64-bit about the test case.
> I removed the target test altogether and it executes just fine on
> our BE system in both 32-bit and 64-bit modes, so I'll just drop
> the target test as part of the patch.
Ha, even better!
> >> /* { dg-require-effective-target powerpc_vsx } */
> >
> > This isn't needed either.
>
> Maybe not strictly needed, but it shields us from users who force
> some options to be used via RUNTESTFLAGS env var that can cause the
> test case to FAIL. I'm going to leave this for someone else to
> clean up.
Users can make most tests fail in interesting and exciting ways like
that, heh.
In general, only realistic settings are supported: things for which
hardware exists, an OS exists for, etc. With any other settings many
things can fail, and that is Just Fine.
Thanks again,
Segher
