testsuite: Align testcase with implementation [PR105090]

Message ID 20240710162637.2213550-1-torbjorn.svensson@foss.st.com
State New
Headers
Series testsuite: Align testcase with implementation [PR105090] |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gcc_build--master-arm success Build passed
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 success Build passed
linaro-tcwg-bot/tcwg_gcc_check--master-aarch64 warning Patch is already merged
linaro-tcwg-bot/tcwg_gcc_check--master-arm warning Patch is already merged

Commit Message

Torbjorn SVENSSON July 10, 2024, 4:26 p.m. UTC
  Is this ok for the following branches?
- trunk
- releases/gcc-14
- releases/gcc-13

--

Since r13-1006-g2005b9b888eeac, the test case copysign_softfloat_1.c
no longer contains any lsr istruction, so drop the check as per
comment 9 in PR105090.

gcc/testsuite/ChangeLog:

	PR target/105090
	* gcc.target/arm/copysign_softfloat_1.c: Drop check for lsr

Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
---
 gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c | 1 -
 1 file changed, 1 deletion(-)
  

Comments

Richard Earnshaw (lists) July 10, 2024, 4:41 p.m. UTC | #1
On 10/07/2024 17:26, Torbjörn SVENSSON wrote:
> Is this ok for the following branches?
> - trunk
> - releases/gcc-14
> - releases/gcc-13
> 
> --
> 
> Since r13-1006-g2005b9b888eeac, the test case copysign_softfloat_1.c
> no longer contains any lsr istruction, so drop the check as per
> comment 9 in PR105090.
> 
> gcc/testsuite/ChangeLog:
> 
> 	PR target/105090
> 	* gcc.target/arm/copysign_softfloat_1.c: Drop check for lsr
> 
> Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
> ---
>  gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c b/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
> index a14922f1c12..50317b7abe5 100644
> --- a/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
> +++ b/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
> @@ -42,7 +42,6 @@ main (int argc, char **argv)
>    int index = 0;
>  
>  /* { dg-final { scan-assembler-times "bfi" 2 { target arm_softfloat } } } */
> -/* { dg-final { scan-assembler-times "lsr" 1 { target arm_softfloat } } } */
>    for (index; index < N; index++)
>      {
>        if (__builtin_copysignf (a_f[index], b_f[index]) != c_f[index])

OK.
R.
  
Torbjorn SVENSSON July 10, 2024, 4:58 p.m. UTC | #2
On 2024-07-10 18:41, Richard Earnshaw (lists) wrote:
> On 10/07/2024 17:26, Torbjörn SVENSSON wrote:
>> Is this ok for the following branches?
>> - trunk
>> - releases/gcc-14
>> - releases/gcc-13
>>
>> --
>>
>> Since r13-1006-g2005b9b888eeac, the test case copysign_softfloat_1.c
>> no longer contains any lsr istruction, so drop the check as per
>> comment 9 in PR105090.
>>
>> gcc/testsuite/ChangeLog:
>>
>> 	PR target/105090
>> 	* gcc.target/arm/copysign_softfloat_1.c: Drop check for lsr
>>
>> Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
>> ---
>>   gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c | 1 -
>>   1 file changed, 1 deletion(-)
>>
>> diff --git a/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c b/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
>> index a14922f1c12..50317b7abe5 100644
>> --- a/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
>> +++ b/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
>> @@ -42,7 +42,6 @@ main (int argc, char **argv)
>>     int index = 0;
>>   
>>   /* { dg-final { scan-assembler-times "bfi" 2 { target arm_softfloat } } } */
>> -/* { dg-final { scan-assembler-times "lsr" 1 { target arm_softfloat } } } */
>>     for (index; index < N; index++)
>>       {
>>         if (__builtin_copysignf (a_f[index], b_f[index]) != c_f[index])
> 
> OK.
> R.

Pushed as:

basepoints/gcc-15-1950-g4865a92b350
releases/gcc-14.1.0-232-ge7d81cf551b
releases/gcc-13.3.0-121-g4f6f63f2cfc

Kind regards,
Torbjörn
  

Patch

diff --git a/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c b/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
index a14922f1c12..50317b7abe5 100644
--- a/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
+++ b/gcc/testsuite/gcc.target/arm/copysign_softfloat_1.c
@@ -42,7 +42,6 @@  main (int argc, char **argv)
   int index = 0;
 
 /* { dg-final { scan-assembler-times "bfi" 2 { target arm_softfloat } } } */
-/* { dg-final { scan-assembler-times "lsr" 1 { target arm_softfloat } } } */
   for (index; index < N; index++)
     {
       if (__builtin_copysignf (a_f[index], b_f[index]) != c_f[index])