On 26/03/2025 18:34, David Malcolm wrote:
> Found by dg-lint.
>
> gcc/testsuite/ChangeLog:
> * gcc.target/aarch64/atomic-inst-ldlogic.c: Add missing trailing
> " }" for 2 dg-final directives.
> * gcc.target/aarch64/saturating_arithmetic_1.c: Fix dg-do compile.
> * gcc.target/aarch64/saturating_arithmetic_2.c: Likewise.
> ---
> gcc/testsuite/gcc.target/aarch64/atomic-inst-ldlogic.c | 4 ++--
> gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_1.c | 4 ++--
> gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_2.c | 4 ++--
> 3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/aarch64/atomic-inst-ldlogic.c
> b/gcc/testsuite/gcc.target/aarch64/atomic-inst-ldlogic.c
> index 4879d52b9b4f..ef79396151c6 100644
> --- a/gcc/testsuite/gcc.target/aarch64/atomic-inst-ldlogic.c
> +++ b/gcc/testsuite/gcc.target/aarch64/atomic-inst-ldlogic.c
> @@ -128,7 +128,7 @@ TEST (xor_load_notreturn, XOR_LOAD_NORETURN)
> /* { dg-final { scan-assembler-times "ldclrlh\t" 8} } */
^
Shouldn't there also be a space here (and all the other instances)?
But this is OK as far as it goes.
R.
> /* { dg-final { scan-assembler-times "ldclralh\t" 16} } */
>
> -/* { dg-final { scan-assembler-times "ldclr\t" 16} */
> +/* { dg-final { scan-assembler-times "ldclr\t" 16} } */
> /* { dg-final { scan-assembler-times "ldclra\t" 32} } */
> /* { dg-final { scan-assembler-times "ldclrl\t" 16} } */
> /* { dg-final { scan-assembler-times "ldclral\t" 32} } */
> @@ -145,7 +145,7 @@ TEST (xor_load_notreturn, XOR_LOAD_NORETURN)
> /* { dg-final { scan-assembler-times "ldeorlh\t" 8} } */
> /* { dg-final { scan-assembler-times "ldeoralh\t" 16} } */
>
> -/* { dg-final { scan-assembler-times "ldeor\t" 16} */
> +/* { dg-final { scan-assembler-times "ldeor\t" 16} } */
> /* { dg-final { scan-assembler-times "ldeora\t" 32} } */
> /* { dg-final { scan-assembler-times "ldeorl\t" 16} } */
> /* { dg-final { scan-assembler-times "ldeoral\t" 32} } */
> diff --git a/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_1.c
> b/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_1.c
> index 2ac0c376d126..acd2e11f41d3 100644
> --- a/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_1.c
> +++ b/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_1.c
> @@ -1,4 +1,4 @@
> -/* { dg-do-compile } */
> +/* { dg-do compile } */
> /* { dg-options "-O2 --save-temps -fno-schedule-insns2" } */
> /* { dg-final { check-function-bodies "**" "" "" } } */
>
> @@ -33,4 +33,4 @@
> #define UMAX UCHAR_MAX
> #define UMIN 0
>
> -#include "saturating_arithmetic.inc"
> \ No newline at end of file
> +#include "saturating_arithmetic.inc"
> diff --git a/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_2.c
> b/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_2.c
> index 2a55aa9f2218..86c88f8447c3 100644
> --- a/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_2.c
> +++ b/gcc/testsuite/gcc.target/aarch64/saturating_arithmetic_2.c
> @@ -1,4 +1,4 @@
> -/* { dg-do-compile } */
> +/* { dg-do compile } */
> /* { dg-options "-O2 --save-temps -fno-schedule-insns2" } */
> /* { dg-final { check-function-bodies "**" "" "" } } */
>
> @@ -33,4 +33,4 @@
> #define UMAX USHRT_MAX
> #define UMIN 0
>
> -#include "saturating_arithmetic.inc"
> \ No newline at end of file
> +#include "saturating_arithmetic.inc"
