On Mon, 6 Nov 2023, Tamar Christina wrote:
> Hi All,
>
> This adds pragma GCC novector to testcases that have showed up
> since last regression run and due to this series detecting more.
>
> Is it ok that when it comes time to commit I can just update any
> new cases before committing? since this seems a cat and mouse game..
Yeah, just update.
> Bootstrapped Regtested on aarch64-none-linux-gnu and no issues.
>
> Ok for master?
OK.
> Thanks,
> Tamar
>
> gcc/testsuite/ChangeLog:
>
> * gcc.dg/vect/no-scevccp-slp-30.c: Add pragma novector.
> * gcc.dg/vect/no-scevccp-slp-31.c: Likewise.
> * gcc.dg/vect/no-section-anchors-vect-69.c: Likewise.
> * gcc.target/aarch64/vect-xorsign_exec.c: Likewise.
>
> --- inline copy of patch --
> diff --git a/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-30.c
> b/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-30.c
> index
> 00d0eca56eeca6aee6f11567629dc955c0924c74..534bee4a1669a7cbd95cf6007f28dafd23bab8da
> 100644
> --- a/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-30.c
> +++ b/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-30.c
> @@ -24,9 +24,9 @@ main1 ()
> }
>
> /* check results: */
> -#pragma GCC novector
> for (j = 0; j < N; j++)
> {
> +#pragma GCC novector
> for (i = 0; i < N; i++)
> {
> if (out[i*4] != 8
> diff --git a/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-31.c
> b/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-31.c
> index
> 48b6a9b0681cf1fe410755c3e639b825b27895b0..22817a57ef81398cc018a78597755397d20e0eb9
> 100644
> --- a/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-31.c
> +++ b/gcc/testsuite/gcc.dg/vect/no-scevccp-slp-31.c
> @@ -27,6 +27,7 @@ main1 ()
> #pragma GCC novector
> for (i = 0; i < N; i++)
> {
> +#pragma GCC novector
> for (j = 0; j < N; j++)
> {
> if (a[i][j] != 8)
> diff --git a/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-69.c
> b/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-69.c
> index
> a0e53d5fef91868dfdbd542dd0a98dff92bd265b..0861d488e134d3f01a2fa83c56eff7174f36ddfb
> 100644
> --- a/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-69.c
> +++ b/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-69.c
> @@ -83,9 +83,9 @@ int main1 ()
> }
>
> /* check results: */
> -#pragma GCC novector
> for (i = 0; i < N; i++)
> {
> +#pragma GCC novector
> for (j = 0; j < N; j++)
> {
> if (tmp1[2].e.n[1][i][j] != 8)
> @@ -103,9 +103,9 @@ int main1 ()
> }
>
> /* check results: */
> -#pragma GCC novector
> for (i = 0; i < N - NINTS; i++)
> {
> +#pragma GCC novector
> for (j = 0; j < N - NINTS; j++)
> {
> if (tmp2[2].e.n[1][i][j] != 8)
> diff --git a/gcc/testsuite/gcc.target/aarch64/vect-xorsign_exec.c
> b/gcc/testsuite/gcc.target/aarch64/vect-xorsign_exec.c
> index
> cfa22115831272cb1d4e1a38512f10c3a1c6ad77..84f33d3f6cce9b0017fd12ab961019041245ffae
> 100644
> --- a/gcc/testsuite/gcc.target/aarch64/vect-xorsign_exec.c
> +++ b/gcc/testsuite/gcc.target/aarch64/vect-xorsign_exec.c
> @@ -33,6 +33,7 @@ main (void)
> r[i] = a[i] * __builtin_copysignf (1.0f, b[i]);
>
> /* check results: */
> +#pragma GCC novector
> for (i = 0; i < N; i++)
> if (r[i] != a[i] * __builtin_copysignf (1.0f, b[i]))
> abort ();
> @@ -41,6 +42,7 @@ main (void)
> rd[i] = ad[i] * __builtin_copysign (1.0d, bd[i]);
>
> /* check results: */
> +#pragma GCC novector
> for (i = 0; i < N; i++)
> if (rd[i] != ad[i] * __builtin_copysign (1.0d, bd[i]))
> abort ();
>
>
>
>
>
--
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH,
Frankenstrasse 146, 90461 Nuernberg, Germany;
GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)
