Hi! On Mon, Feb 17, 2025 at 02:28:50PM +0000, Alex Coplan wrote: > After the recent alignment peeling enhancements in the vectorizer we > started vectorizing the "checking" loops (that check for the right > result) in gcc.target/powerpc/vsx-vectorize-*.c, thus skewing the > expected counts of various scan-dump-times tests (causing them to FAIL). > This adds #pragma GCC novector above the relevant loops to prevent them > from being vectorized, thereby fixing the test failures.
It is fundamentally incorrect to add a "novector" attribute to testcases called "vsx-vectorize-*". Even if it is just to some part of the code; if you want to only test some part of code, put it in a different file. If the testcases are less than stellar (and pretty much anything using scan-assembler-times is!), fix *that*? Segher
