aarch64: Fix dejaGNU directive in clastb_8.c testcase

Message ID gkr1rj0iihd.fsf@arm.com
State New
Headers show
Series
  • aarch64: Fix dejaGNU directive in clastb_8.c testcase
Related show

Commit Message

Andrea Corallo Sept. 17, 2020, 2:30 p.m.
Hi all,

this is to fix a typo in a dejaGNU directive introduced with
with 052204fac58 "vec: don't select partial vectors when 
unnecessary".

Okay for trunk?

gcc/testsuite/ChangeLog

2020-09-17  Andre Corallo  <andrea.corallo@arm.com>

	* gcc.target/aarch64/sve/clastb_8.c: Fix a dg-final scan
	directive
From 6628a4bbfa909ac2fda789e549c7e7067ecaddad Mon Sep 17 00:00:00 2001
From: Andrea Corallo <andrea.corallo@arm.com>

Date: Thu, 17 Sep 2020 15:36:22 +0200
Subject: [PATCH] aarch64: Fix dejaGNU directive in clastb_8.c testcase

gcc/testsuite/ChangeLog

2020-09-17  Andre Corallo  <andrea.corallo@arm.com>

	* gcc.target/aarch64/sve/clastb_8.c: Fix a dg-final scan
	directive.
---
 gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.1

Comments

Richard Sandiford Sept. 17, 2020, 4:50 p.m. | #1
Andrea Corallo <andrea.corallo@arm.com> writes:
> Hi all,

>

> this is to fix a typo in a dejaGNU directive introduced with

> with 052204fac58 "vec: don't select partial vectors when 

> unnecessary".

>

> Okay for trunk?


OK, thanks.  FWIW, this would also have been OK under the
“obviously correct” rule (but asking is obviously OK too :-))

Richard

>

> gcc/testsuite/ChangeLog

>

> 2020-09-17  Andre Corallo  <andrea.corallo@arm.com>

>

> 	* gcc.target/aarch64/sve/clastb_8.c: Fix a dg-final scan

> 	directive

>         

>

> From 6628a4bbfa909ac2fda789e549c7e7067ecaddad Mon Sep 17 00:00:00 2001

> From: Andrea Corallo <andrea.corallo@arm.com>

> Date: Thu, 17 Sep 2020 15:36:22 +0200

> Subject: [PATCH] aarch64: Fix dejaGNU directive in clastb_8.c testcase

>

> gcc/testsuite/ChangeLog

>

> 2020-09-17  Andre Corallo  <andrea.corallo@arm.com>

>

> 	* gcc.target/aarch64/sve/clastb_8.c: Fix a dg-final scan

> 	directive.

> ---

>  gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c b/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c

> index e61ff4ac92d..49a91616e84 100644

> --- a/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c

> +++ b/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c

> @@ -23,4 +23,4 @@ TEST_TYPE (uint64_t);

>  /* { dg-final { scan-assembler {\tclastb\t(h[0-9]+), p[0-7], \1, z[0-9]+\.h\n} } } */

>  /* { dg-final { scan-assembler {\tclastb\t(s[0-9]+), p[0-7], \1, z[0-9]+\.s\n} } } */

>  /* { dg-final { scan-assembler {\tclastb\t(d[0-9]+), p[0-7], \1, z[0-9]+\.d\n} } } */

> -/* { dg-final { scan-assembler {\tptrue\tp[0-9]+\.b,} 4 } } */

> +/* { dg-final { scan-assembler-times {\tptrue\tp[0-9]+\.b,} 4 } } */
Andrea Corallo Sept. 17, 2020, 5:35 p.m. | #2
Richard Sandiford <richard.sandiford@arm.com> writes:

> OK, thanks.  FWIW, this would also have been OK under the

> “obviously correct” rule (but asking is obviously OK too :-))

>

> Richard


Installed as ee7aa7e7b9d

Thanks for having a look and for the info!

  Andrea

Patch

diff --git a/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c b/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c
index e61ff4ac92d..49a91616e84 100644
--- a/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c
+++ b/gcc/testsuite/gcc.target/aarch64/sve/clastb_8.c
@@ -23,4 +23,4 @@  TEST_TYPE (uint64_t);
 /* { dg-final { scan-assembler {\tclastb\t(h[0-9]+), p[0-7], \1, z[0-9]+\.h\n} } } */
 /* { dg-final { scan-assembler {\tclastb\t(s[0-9]+), p[0-7], \1, z[0-9]+\.s\n} } } */
 /* { dg-final { scan-assembler {\tclastb\t(d[0-9]+), p[0-7], \1, z[0-9]+\.d\n} } } */
-/* { dg-final { scan-assembler {\tptrue\tp[0-9]+\.b,} 4 } } */
+/* { dg-final { scan-assembler-times {\tptrue\tp[0-9]+\.b,} 4 } } */