Re: [PATCH, testsuite] Fix PR34168 - fortran vectorized tests on x86 without support of SSE2

[Victor Kaplansky <VICTORK at il dot ibm dot com>]

> You don't need to set -msse2 to use cpuid.

OK, thanks, I'll remove it.

>
> I think you have the logic reversed ;)
>
> You can perhaps code:
>
> if (__get_cpuid (...))
>   return edx & bit_SSE2;
>
> return 0;

No, in this case return is used to pass exit status of the main()
program, so the zero value means "true".


> Please also put this check in gcc.dg/vect/vect.exp and
> g++.dg/vect/vect.exp. In fact, it looks that all these three files
> should be synchronized w.r.t to unsupported target handling. Perhaps
> we need a global function in testsuite/lib/target-supports.exp (lets
> name it something like check_vector_hw_available) to be shared between
> these three vect.exp files, so we don't have to change this part in
> three separate places.

I agree. Do we want to make this reorganization as part of this PR or as
a separate patch? I prefer to fix PR and reorganize vect.exp later.
-- Victor