Turn on -Wimplicit-fallthrough by default if available

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
This commit is contained in:
Florian Weimer 2024-08-09 15:34:53 +02:00
parent 5662433c38
commit 4331769c0f
3 changed files with 36 additions and 1 deletions

View File

@ -888,7 +888,7 @@ host-test-program-cmd = $(host-built-program-cmd)
endif endif
# Extra flags to pass to GCC. # Extra flags to pass to GCC.
+gccwarn := -Wall -Wwrite-strings -Wundef +gccwarn := -Wall -Wwrite-strings -Wundef $(cc-option-wimplicit-fallthrough)
ifeq ($(enable-werror),yes) ifeq ($(enable-werror),yes)
+gccwarn += -Werror +gccwarn += -Werror
endif endif

26
configure vendored
View File

@ -7465,6 +7465,32 @@ if test $libc_cv_cc_loop_to_function = yes; then
fi fi
{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking for -Wimplicit-fallthrough" >&5
printf %s "checking for -Wimplicit-fallthrough... " >&6; }
if test ${libc_cv_cc_wimplicit_fallthrough+y}
then :
printf %s "(cached) " >&6
else case e in #(
e) if { ac_try='${CC-cc} -Werror -Wimplicit-fallthrough -xc /dev/null -S -o /dev/null'
{ { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
(eval $ac_try) 2>&5
ac_status=$?
printf "%s\n" "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
test $ac_status = 0; }; }
then :
libc_cv_cc_wimplicit_fallthrough=-Wimplicit-fallthrough
else case e in #(
e) libc_cv_cc_wimplicit_fallthrough= ;;
esac
fi
;;
esac
fi
{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: result: $libc_cv_cc_wimplicit_fallthrough" >&5
printf "%s\n" "$libc_cv_cc_wimplicit_fallthrough" >&6; }
config_vars="$config_vars
cc-option-wimplicit-fallthrough = $libc_cv_cc_wimplicit_fallthrough"
{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking for libgd" >&5 { printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking for libgd" >&5
printf %s "checking for libgd... " >&6; } printf %s "checking for libgd... " >&6; }
if test "$with_gd" != "no"; then if test "$with_gd" != "no"; then

View File

@ -1468,6 +1468,15 @@ if test $libc_cv_cc_loop_to_function = yes; then
fi fi
AC_SUBST(libc_cv_cc_loop_to_function) AC_SUBST(libc_cv_cc_loop_to_function)
AC_CACHE_CHECK([for -Wimplicit-fallthrough],
libc_cv_cc_wimplicit_fallthrough, [dnl
LIBC_TRY_CC_OPTION([-Werror -Wimplicit-fallthrough],
[libc_cv_cc_wimplicit_fallthrough=-Wimplicit-fallthrough],
[libc_cv_cc_wimplicit_fallthrough=])
])
LIBC_CONFIG_VAR([cc-option-wimplicit-fallthrough],
[$libc_cv_cc_wimplicit_fallthrough])
dnl Check whether we have the gd library available. dnl Check whether we have the gd library available.
AC_MSG_CHECKING(for libgd) AC_MSG_CHECKING(for libgd)
if test "$with_gd" != "no"; then if test "$with_gd" != "no"; then