* tst-cpu-features-supports.c build failures
@ 2021-04-21 20:52 Joseph Myers
2021-04-22 1:43 ` [PATCH] x86: tst-cpu-features-supports.c: Update AMX check H.J. Lu
0 siblings, 1 reply; 4+ messages in thread
From: Joseph Myers @ 2021-04-21 20:52 UTC (permalink / raw)
To: libc-alpha
With GCC mainline, I'm seeing glibc testsuite build failures for x86_64
and x86:
../sysdeps/x86/tst-cpu-features-supports.c: In function 'do_test':
../sysdeps/x86/tst-cpu-features-supports.c:62:3: error: parameter to builtin not valid: amx_bf16
62 | fails += CHECK_SUPPORTS (amx_bf16, AMX_BF16);
| ^~~~~
../sysdeps/x86/tst-cpu-features-supports.c:63:3: error: parameter to builtin not valid: amx_int8
63 | fails += CHECK_SUPPORTS (amx_int8, AMX_INT8);
| ^~~~~
../sysdeps/x86/tst-cpu-features-supports.c:64:3: error: parameter to builtin not valid: amx_tile
64 | fails += CHECK_SUPPORTS (amx_tile, AMX_TILE);
| ^~~~~
https://sourceware.org/pipermail/libc-testresults/2021q2/007827.html
The code in question is inside a conditional
#if __GNUC_PREREQ (11, 1)
so wasn't compiled until the GCC version number was increased to 12 (and
would probably start failing with GCC 11 branch as soon as 11.1 is
released).
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] x86: tst-cpu-features-supports.c: Update AMX check
2021-04-21 20:52 tst-cpu-features-supports.c build failures Joseph Myers
@ 2021-04-22 1:43 ` H.J. Lu
2021-04-22 17:07 ` Joseph Myers
0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2021-04-22 1:43 UTC (permalink / raw)
To: Joseph Myers; +Cc: libc-alpha
[-- Attachment #1: Type: text/plain, Size: 1095 bytes --]
On Wed, Apr 21, 2021 at 3:45 PM Joseph Myers <joseph@codesourcery.com> wrote:
>
> With GCC mainline, I'm seeing glibc testsuite build failures for x86_64
> and x86:
>
> ../sysdeps/x86/tst-cpu-features-supports.c: In function 'do_test':
> ../sysdeps/x86/tst-cpu-features-supports.c:62:3: error: parameter to builtin not valid: amx_bf16
> 62 | fails += CHECK_SUPPORTS (amx_bf16, AMX_BF16);
> | ^~~~~
> ../sysdeps/x86/tst-cpu-features-supports.c:63:3: error: parameter to builtin not valid: amx_int8
> 63 | fails += CHECK_SUPPORTS (amx_int8, AMX_INT8);
> | ^~~~~
> ../sysdeps/x86/tst-cpu-features-supports.c:64:3: error: parameter to builtin not valid: amx_tile
> 64 | fails += CHECK_SUPPORTS (amx_tile, AMX_TILE);
> | ^~~~~
>
> https://sourceware.org/pipermail/libc-testresults/2021q2/007827.html
>
> The code in question is inside a conditional
>
> #if __GNUC_PREREQ (11, 1)
>
> so wasn't compiled until the GCC version number was increased to 12 (and
> would probably start failing with GCC 11 branch as soon as 11.1 is
> released).
>
Try this.
--
H.J.
[-- Attachment #2: 0001-x86-tst-cpu-features-supports.c-Update-AMX-check.patch --]
[-- Type: text/x-patch, Size: 1195 bytes --]
From 023f78e474f844068a9fb615976ea59a1437877f Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Wed, 21 Apr 2021 18:40:08 -0700
Subject: [PATCH] x86: tst-cpu-features-supports.c: Update AMX check
Pass "amx-bf16", "amx-int8" and "amx-tile", instead of "amx_bf16",
"amx_int8" and "amx_tile", to __builtin_cpu_supports for GCC 11.
---
sysdeps/x86/tst-cpu-features-supports.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/sysdeps/x86/tst-cpu-features-supports.c b/sysdeps/x86/tst-cpu-features-supports.c
index ce78a7d8bc..a2cabc90be 100644
--- a/sysdeps/x86/tst-cpu-features-supports.c
+++ b/sysdeps/x86/tst-cpu-features-supports.c
@@ -59,9 +59,9 @@ do_test (int argc, char **argv)
fails += CHECK_SUPPORTS (aes, AES);
#endif
#if __GNUC_PREREQ (11, 1)
- fails += CHECK_SUPPORTS (amx_bf16, AMX_BF16);
- fails += CHECK_SUPPORTS (amx_int8, AMX_INT8);
- fails += CHECK_SUPPORTS (amx_tile, AMX_TILE);
+ fails += CHECK_SUPPORTS (amx-bf16, AMX_BF16);
+ fails += CHECK_SUPPORTS (amx-int8, AMX_INT8);
+ fails += CHECK_SUPPORTS (amx-tile, AMX_TILE);
#endif
fails += CHECK_SUPPORTS (avx, AVX);
fails += CHECK_SUPPORTS (avx2, AVX2);
--
2.30.2
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] x86: tst-cpu-features-supports.c: Update AMX check
2021-04-22 1:43 ` [PATCH] x86: tst-cpu-features-supports.c: Update AMX check H.J. Lu
@ 2021-04-22 17:07 ` Joseph Myers
2021-04-22 17:08 ` H.J. Lu
0 siblings, 1 reply; 4+ messages in thread
From: Joseph Myers @ 2021-04-22 17:07 UTC (permalink / raw)
To: H.J. Lu; +Cc: libc-alpha
On Wed, 21 Apr 2021, H.J. Lu via Libc-alpha wrote:
> On Wed, Apr 21, 2021 at 3:45 PM Joseph Myers <joseph@codesourcery.com> wrote:
> >
> > With GCC mainline, I'm seeing glibc testsuite build failures for x86_64
> > and x86:
> >
> > ../sysdeps/x86/tst-cpu-features-supports.c: In function 'do_test':
> > ../sysdeps/x86/tst-cpu-features-supports.c:62:3: error: parameter to builtin not valid: amx_bf16
> > 62 | fails += CHECK_SUPPORTS (amx_bf16, AMX_BF16);
> > | ^~~~~
> > ../sysdeps/x86/tst-cpu-features-supports.c:63:3: error: parameter to builtin not valid: amx_int8
> > 63 | fails += CHECK_SUPPORTS (amx_int8, AMX_INT8);
> > | ^~~~~
> > ../sysdeps/x86/tst-cpu-features-supports.c:64:3: error: parameter to builtin not valid: amx_tile
> > 64 | fails += CHECK_SUPPORTS (amx_tile, AMX_TILE);
> > | ^~~~~
> >
> > https://sourceware.org/pipermail/libc-testresults/2021q2/007827.html
> >
> > The code in question is inside a conditional
> >
> > #if __GNUC_PREREQ (11, 1)
> >
> > so wasn't compiled until the GCC version number was increased to 12 (and
> > would probably start failing with GCC 11 branch as soon as 11.1 is
> > released).
> >
>
> Try this.
That patch fixes the build failures for me.
(Note: the fix ought to go on 2.33 branch as well, otherwise that branch
will fail to build the testsuite with GCC 11.1 release.)
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] x86: tst-cpu-features-supports.c: Update AMX check
2021-04-22 17:07 ` Joseph Myers
@ 2021-04-22 17:08 ` H.J. Lu
0 siblings, 0 replies; 4+ messages in thread
From: H.J. Lu @ 2021-04-22 17:08 UTC (permalink / raw)
To: Joseph Myers, Libc-stable Mailing List; +Cc: GNU C Library
On Thu, Apr 22, 2021 at 10:08 AM Joseph Myers <joseph@codesourcery.com> wrote:
>
> On Wed, 21 Apr 2021, H.J. Lu via Libc-alpha wrote:
>
> > On Wed, Apr 21, 2021 at 3:45 PM Joseph Myers <joseph@codesourcery.com> wrote:
> > >
> > > With GCC mainline, I'm seeing glibc testsuite build failures for x86_64
> > > and x86:
> > >
> > > ../sysdeps/x86/tst-cpu-features-supports.c: In function 'do_test':
> > > ../sysdeps/x86/tst-cpu-features-supports.c:62:3: error: parameter to builtin not valid: amx_bf16
> > > 62 | fails += CHECK_SUPPORTS (amx_bf16, AMX_BF16);
> > > | ^~~~~
> > > ../sysdeps/x86/tst-cpu-features-supports.c:63:3: error: parameter to builtin not valid: amx_int8
> > > 63 | fails += CHECK_SUPPORTS (amx_int8, AMX_INT8);
> > > | ^~~~~
> > > ../sysdeps/x86/tst-cpu-features-supports.c:64:3: error: parameter to builtin not valid: amx_tile
> > > 64 | fails += CHECK_SUPPORTS (amx_tile, AMX_TILE);
> > > | ^~~~~
> > >
> > > https://sourceware.org/pipermail/libc-testresults/2021q2/007827.html
> > >
> > > The code in question is inside a conditional
> > >
> > > #if __GNUC_PREREQ (11, 1)
> > >
> > > so wasn't compiled until the GCC version number was increased to 12 (and
> > > would probably start failing with GCC 11 branch as soon as 11.1 is
> > > released).
> > >
> >
> > Try this.
>
> That patch fixes the build failures for me.
>
> (Note: the fix ought to go on 2.33 branch as well, otherwise that branch
> will fail to build the testsuite with GCC 11.1 release.)
I will check it into master branch and backport it to 2.33 branch.
Thanks.
--
H.J.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-04-22 17:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-21 20:52 tst-cpu-features-supports.c build failures Joseph Myers
2021-04-22 1:43 ` [PATCH] x86: tst-cpu-features-supports.c: Update AMX check H.J. Lu
2021-04-22 17:07 ` Joseph Myers
2021-04-22 17:08 ` H.J. Lu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).