testsuite: add missing dg-require ifunc in pr105554.c

Message ID 20231207162651.685329-1-poulhies@adacore.com
State Accepted
Headers
Series testsuite: add missing dg-require ifunc in pr105554.c |

Checks

Context Check Description
snail/gcc-patch-check success Github commit url

Commit Message

Marc Poulhiès Dec. 7, 2023, 4:25 p.m. UTC
  The 'target_clones' attribute depends on the ifunc support.

gcc/testsuite/ChangeLog:
	* gcc.target/i386/pr105554.c: Add dg-require ifunc.
---
Tested on x86_64-linux and x86_64-elf.

Ok for master?

 gcc/testsuite/gcc.target/i386/pr105554.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Jakub Jelinek Dec. 7, 2023, 4:40 p.m. UTC | #1
On Thu, Dec 07, 2023 at 05:25:39PM +0100, Marc Poulhiès wrote:
> The 'target_clones' attribute depends on the ifunc support.
> 
> gcc/testsuite/ChangeLog:
> 	* gcc.target/i386/pr105554.c: Add dg-require ifunc.
> ---
> Tested on x86_64-linux and x86_64-elf.
> 
> Ok for master?
> 
>  gcc/testsuite/gcc.target/i386/pr105554.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/gcc/testsuite/gcc.target/i386/pr105554.c b/gcc/testsuite/gcc.target/i386/pr105554.c
> index e9ef494270a..420987e5df8 100644
> --- a/gcc/testsuite/gcc.target/i386/pr105554.c
> +++ b/gcc/testsuite/gcc.target/i386/pr105554.c
> @@ -2,6 +2,7 @@
>  /* { dg-do compile } */
>  /* { dg-require-ifunc "" } */
>  /* { dg-options "-O2 -Wno-psabi -mno-sse3" } */
> +/* { dg-require-ifunc "" } */

That is 2 lines above this already...

	Jakub
  
Marc Poulhiès Dec. 8, 2023, 8:29 a.m. UTC | #2
Jakub Jelinek <jakub@redhat.com> writes:

> On Thu, Dec 07, 2023 at 05:25:39PM +0100, Marc Poulhiès wrote:
>> The 'target_clones' attribute depends on the ifunc support.
>>
>> gcc/testsuite/ChangeLog:
>> 	* gcc.target/i386/pr105554.c: Add dg-require ifunc.
>> ---
>> Tested on x86_64-linux and x86_64-elf.
>>
>> Ok for master?
>>
>>  gcc/testsuite/gcc.target/i386/pr105554.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/gcc/testsuite/gcc.target/i386/pr105554.c b/gcc/testsuite/gcc.target/i386/pr105554.c
>> index e9ef494270a..420987e5df8 100644
>> --- a/gcc/testsuite/gcc.target/i386/pr105554.c
>> +++ b/gcc/testsuite/gcc.target/i386/pr105554.c
>> @@ -2,6 +2,7 @@
>>  /* { dg-do compile } */
>>  /* { dg-require-ifunc "" } */
>>  /* { dg-options "-O2 -Wno-psabi -mno-sse3" } */
>> +/* { dg-require-ifunc "" } */
>
> That is 2 lines above this already...

Oh right, sorry about that. I didn't catch this when rebasing.

Marc
  

Patch

diff --git a/gcc/testsuite/gcc.target/i386/pr105554.c b/gcc/testsuite/gcc.target/i386/pr105554.c
index e9ef494270a..420987e5df8 100644
--- a/gcc/testsuite/gcc.target/i386/pr105554.c
+++ b/gcc/testsuite/gcc.target/i386/pr105554.c
@@ -2,6 +2,7 @@ 
 /* { dg-do compile } */
 /* { dg-require-ifunc "" } */
 /* { dg-options "-O2 -Wno-psabi -mno-sse3" } */
+/* { dg-require-ifunc "" } */
 
 typedef long long v4di __attribute__((__vector_size__(32)));