Hi Jeff,
> On 1/24/24 07:40, Robin Dapp wrote:
>> Hi,
>> on Solaris/SPARC several vector tests appeared to be regressing. They
>> were never vectorized but the checks before r14-3612-ge40edf64995769
>> would match regardless if a loop was actually vectorized or not.
>> The refined checks only match a successful vectorization attempt
>> but are run unconditionally. This patch adds target checks to them.
>> Bootstrapped (unnecessarily) and regtested on x86, aarch64 and
>> power10. Regtested on riscv and (the previous version that
>> missed vect-reduc-pattern-2a.c) on Solaris/SPARC by Rainer Orth.
>> Is this OK if Rainer's second run is successful?
>> Regards
>> Robin
>> gcc/testsuite/ChangeLog:
>> PR testsuite/113558
>> * gcc.dg/vect/no-scevccp-outer-7.c: Add target check.
>> * gcc.dg/vect/vect-outer-4c-big-array.c: Ditto.
>> * gcc.dg/vect/vect-reduc-dot-s16a.c: Ditto.
>> * gcc.dg/vect/vect-reduc-dot-s8a.c: Ditto.
>> * gcc.dg/vect/vect-reduc-dot-s8b.c: Ditto.
>> * gcc.dg/vect/vect-reduc-dot-u16b.c: Ditto.
>> * gcc.dg/vect/vect-reduc-dot-u8a.c: Ditto.
>> * gcc.dg/vect/vect-reduc-dot-u8b.c: Ditto.
>> * gcc.dg/vect/vect-reduc-pattern-1a.c: Ditto.
>> * gcc.dg/vect/vect-reduc-pattern-1b-big-array.c: Ditto.
>> * gcc.dg/vect/vect-reduc-pattern-1c-big-array.c: Ditto.
>> * gcc.dg/vect/vect-reduc-pattern-2a.c: Ditto.
>> * gcc.dg/vect/vect-reduc-pattern-2b-big-array.c: Ditto.
>> * gcc.dg/vect/wrapv-vect-reduc-dot-s8b.c: Ditto.
> Yes, OK if Rainer's run is successful. If you need to make similiar
> changes to other tests, consider them pre-approved.
sparc-sun-solaris2.11 testing just completed successfully.
Thanks.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
