From 8e2b9c800d10e1a7979e38c168a076e7b7403807 Mon Sep 17 00:00:00 2001 From: Jeff Law <jlaw@ventanamicro.com> Date: Sat, 16 Nov 2024 08:24:20 -0700 Subject: [PATCH] [committed] Fix compilation of testglue wrapper after c23 changes testglue.c (which is used for exit/abort wrappers in the testsuite) isn't c23 compatible. The testing harness tries to build testglue.c and use it, but doesn't report a failure if the build fails, instead it's just not used. As a result we get all kinds of failures on targets which depend on testglue to report back simulator status -- like tens of thousands of execution failures. This patch just adds -std=gnu17 to the command line to build testglue.c. There's other fallout from the c23 change.. My tester is chewing through things right now... Installing on the trunk. testsuite * lib/wrapper.exp (${tool}_maybe_build_wrapper): Pass -std=gnu17 flag to build testglue wrapper. --- gcc/testsuite/lib/wrapper.exp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/testsuite/lib/wrapper.exp b/gcc/testsuite/lib/wrapper.exp index 667002bb571a..8363181621d7 100644 --- a/gcc/testsuite/lib/wrapper.exp +++ b/gcc/testsuite/lib/wrapper.exp @@ -36,7 +36,7 @@ proc ${tool}_maybe_build_wrapper { filename args } { # passed to turn off all warnings. unset_currtarget_info wrap_compile_flags set_currtarget_info wrap_compile_flags \ - "$saved_wrap_compile_flags -w $flags" + "$saved_wrap_compile_flags -std=gnu17 -w $flags" set result [build_wrapper $filename] unset_currtarget_info wrap_compile_flags set_currtarget_info wrap_compile_flags "$saved_wrap_compile_flags" -- GitLab