Jonathan Wright <jonathan.wri...@arm.com> writes:
> Hi,
>
> I've corrected the quoting and moved everything on to one line.
>
> Ok for master?
>
> Thanks,
> Jonathan
>
> ---
>
> gcc/testsuite/ChangeLog:
>
> 2021-08-04  Jonathan Wright  <jonathan.wri...@arm.com>
>
>         * gcc.target/aarch64/vector_structure_intrinsics.c: Restrict
>         tests to little-endian targets.

OK, thanks.

Richard

> From: Richard Sandiford <richard.sandif...@arm.com>
> Sent: 06 August 2021 13:24
> To: Jonathan Wright <jonathan.wri...@arm.com>
> Cc: gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>; Christophe Lyon 
> <christophe.lyon....@gmail.com>
> Subject: Re: [PATCH] testsuite: aarch64: Fix failing vector structure tests 
> on big-endian
>
> Jonathan Wright <jonathan.wri...@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} }  */
>
> diff --git a/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c 
> b/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c
> index 
> 3e7e572bf39659ecf2f17751d92a4a99a4f2bf8b..89e9de18a92dbc00e58261e4558b3cff38c7ca75
>  100644
> --- a/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c
> +++ b/gcc/testsuite/gcc.target/aarch64/vector_structure_intrinsics.c
> @@ -263,7 +263,7 @@ 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 } 
> } } */
>  
>  /* { dg-final { scan-assembler-times "tbl\\t" 18} }  */
>  /* { dg-final { scan-assembler-times "tbx\\t" 18} }  */

Reply via email to