[PATCH] testsuite: aarch64: Fix failing vector structure tests on big-endian

Richard Sandiford richard.sandiford@arm.com
Fri Aug 6 12:24:29 GMT 2021


Jonathan Wright <Jonathan.Wright@arm.com> writes:
> diff --git a/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c b/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c
> index 60c53bc27f8378c78b119576ed19fde0e5743894..a8e31ab85d6fd2a045c8efaf2cbc42b5f40d2411 100644
> --- a/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c
> +++ b/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c
> @@ -197,7 +197,8 @@ TEST_ST1x3 (vst1q, uint64x2x3_t, uint64_t*, u64, x3);
>  TEST_ST1x3 (vst1q, poly64x2x3_t, poly64_t*, p64, x3);
>  TEST_ST1x3 (vst1q, float64x2x3_t, float64_t*, f64, x3);
>  
> -/* { dg-final { scan-assembler-not "mov\\t" } } */
> +/* { dg-final { scan-assembler-not {"mov\\t"} {
> +		target { aarch64_little_endian } } ) }  */

I think this needs to stay on line.  We should also either keep the
original quoting on the regexp or use {mov\t}.  Having both forms
of quote would turn it into a test for the characters:

   "mov\t"

(including quotes and backslash).

Thanks,
Richard


>  
>  /* { dg-final { scan-assembler-times "tbl\\t" 18} }  */
>  /* { dg-final { scan-assembler-times "tbx\\t" 18} }  */



More information about the Gcc-patches mailing list