https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116500
--- Comment #9 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot
Uni-Bielefeld.DE> ---
> --- Comment #8 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
> (In reply to andi from comment #7)
>> Thanks. Updated patch. This one seems obvious so I'll commit soon.
>>
>> diff --git a/gcc/testsuite/gcc.dg/vect/vect-switch-ifcvt-1.c
>> b/gcc/testsuite/gcc.dg/vect/vect-switch-ifcvt-1.c
>> index f5352ef8ed7a..2e3a9ae3c249 100644
>> --- a/gcc/testsuite/gcc.dg/vect/vect-switch-ifcvt-1.c
>> +++ b/gcc/testsuite/gcc.dg/vect/vect-switch-ifcvt-1.c
>> @@ -1,4 +1,4 @@
>> -/* { dg-require-effective-target vect_int } */
>> +/* { dg-require-effective-target vect_condition } */
>> #include "tree-vect.h"
>>
>> extern void abort (void);
>
> Most likely should be `{ vect_int && vect_condition }` too.
Not with that syntax: dg-require-effective-target only accepts a single
effective-target keyword. You can easily keep the vect_int requirement
and add vect_condition like Andi has done. I cannot tell whether or not
you need both or just vect_condition is fine, though (so maybe not quite
so obvious after all ;-)
I've tested a patch requiring both on sparc-sun-solaris2.11
(UNSUPPORTED) and i386-pc-solaris2.11 (PASS) for good measure.
