diff mbox series

testsuite/aarch64: Skip SLP diagnostic under ILP32 (PR target/96974)

Message ID 1617017997-32250-1-git-send-email-christophe.lyon@linaro.org
State New
Headers show
Series testsuite/aarch64: Skip SLP diagnostic under ILP32 (PR target/96974) | expand

Commit Message

Christophe Lyon March 29, 2021, 11:39 a.m. UTC 
The vectorizer has a very different effect with -mabi=ilp32, and
doesn't emit the expecte diagnostic, so this patch expects it only
under lp64.

2021-03-29  Christophe Lyon  <christophe.lyon@linaro.org>

	gcc/testsuite/
	PR target/96974
	* g++.target/aarch64/sve/pr96974.C: Expect SLP diagnostic only
	under lp64.
---
 gcc/testsuite/g++.target/aarch64/sve/pr96974.C | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Richard Biener March 31, 2021, 10:07 a.m. UTC | #1 
On Mon, Mar 29, 2021 at 1:40 PM Christophe Lyon via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> The vectorizer has a very different effect with -mabi=ilp32, and
> doesn't emit the expecte diagnostic, so this patch expects it only
> under lp64.

OK (please also backport as necessary)

> 2021-03-29  Christophe Lyon  <christophe.lyon@linaro.org>
>
>         gcc/testsuite/
>         PR target/96974
>         * g++.target/aarch64/sve/pr96974.C: Expect SLP diagnostic only
>         under lp64.
> ---
>  gcc/testsuite/g++.target/aarch64/sve/pr96974.C | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/g++.target/aarch64/sve/pr96974.C b/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
> index 363241d..54000f5 100644
> --- a/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
> +++ b/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
> @@ -15,4 +15,4 @@ struct c {
>      int coeffs[10];
>  } f;
>
> -/* { dg-final { scan-tree-dump "Not vectorized: Incompatible number of vector subparts between" "slp1" } } */
> +/* { dg-final { scan-tree-dump "Not vectorized: Incompatible number of vector subparts between" "slp1" { target lp64 } } } */
> --
> 2.7.4
>
Christophe Lyon March 31, 2021, 2 p.m. UTC | #2 
On Wed, 31 Mar 2021 at 12:08, Richard Biener <richard.guenther@gmail.com> wrote:
>
> On Mon, Mar 29, 2021 at 1:40 PM Christophe Lyon via Gcc-patches
> <gcc-patches@gcc.gnu.org> wrote:
> >
> > The vectorizer has a very different effect with -mabi=ilp32, and
> > doesn't emit the expecte diagnostic, so this patch expects it only
> > under lp64.
>
> OK (please also backport as necessary)
>
Thanks, pushed to trunk and gcc-10

Christophe

> > 2021-03-29  Christophe Lyon  <christophe.lyon@linaro.org>
> >
> >         gcc/testsuite/
> >         PR target/96974
> >         * g++.target/aarch64/sve/pr96974.C: Expect SLP diagnostic only
> >         under lp64.
> > ---
> >  gcc/testsuite/g++.target/aarch64/sve/pr96974.C | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/gcc/testsuite/g++.target/aarch64/sve/pr96974.C b/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
> > index 363241d..54000f5 100644
> > --- a/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
> > +++ b/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
> > @@ -15,4 +15,4 @@ struct c {
> >      int coeffs[10];
> >  } f;
> >
> > -/* { dg-final { scan-tree-dump "Not vectorized: Incompatible number of vector subparts between" "slp1" } } */
> > +/* { dg-final { scan-tree-dump "Not vectorized: Incompatible number of vector subparts between" "slp1" { target lp64 } } } */
> > --
> > 2.7.4
> >
diff mbox series

Patch

diff --git a/gcc/testsuite/g++.target/aarch64/sve/pr96974.C b/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
index 363241d..54000f5 100644
--- a/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
+++ b/gcc/testsuite/g++.target/aarch64/sve/pr96974.C
@@ -15,4 +15,4 @@  struct c {
     int coeffs[10];
 } f;
 
-/* { dg-final { scan-tree-dump "Not vectorized: Incompatible number of vector subparts between" "slp1" } } */
+/* { dg-final { scan-tree-dump "Not vectorized: Incompatible number of vector subparts between" "slp1" { target lp64 } } } */