From b0c06375e168def92a1e95fc8122a032e3083624 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Tue, 5 Dec 2023 23:13:50 -0700 Subject: sim: warnings: add more flags Sync with the list of flags from gdbsupport, and add a few more of our own to catch recent issues. Comment out the C++-specific flags as we don't build with C++. --- sim/configure | 19 +++++++++++++++---- sim/m4/sim_ac_option_warnings.m4 | 26 ++++++++++++++++++++++---- 2 files changed, 37 insertions(+), 8 deletions(-) (limited to 'sim') diff --git a/sim/configure b/sim/configure index 73a638f..a7a914a 100755 --- a/sim/configure +++ b/sim/configure @@ -15821,16 +15821,27 @@ if test "${ERROR_ON_WARNING}" = yes ; then WERROR_CFLAGS="-Werror" fi -build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith +build_warnings="-Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter --Wno-error=maybe-uninitialized +-Wno-sign-compare -Wno-error=maybe-uninitialized +-Wno-error=deprecated-register -Wmissing-declarations +" +build_warnings="$build_warnings +-Wdeclaration-after-statement +-Wdeprecated-non-prototype +-Wimplicit-function-declaration +-Wimplicit-int +-Wincompatible-function-pointer-types +-Wincompatible-pointer-types +-Wmisleading-indentation +-Wmissing-parameter-type -Wmissing-prototypes --Wdeclaration-after-statement -Wmissing-parameter-type +-Wold-style-declaration +-Wold-style-definition -Wpointer-sign --Wold-style-declaration -Wold-style-definition " case "${host}" in diff --git a/sim/m4/sim_ac_option_warnings.m4 b/sim/m4/sim_ac_option_warnings.m4 index b66e0ec..e453d88 100644 --- a/sim/m4/sim_ac_option_warnings.m4 +++ b/sim/m4/sim_ac_option_warnings.m4 @@ -36,16 +36,34 @@ fi dnl The options we'll try to enable. dnl NB: Kept somewhat in sync with gdbsupport/warnings.m4. -build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith +build_warnings="-Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter --Wno-error=maybe-uninitialized +-Wno-sign-compare -Wno-error=maybe-uninitialized +dnl C++ -Wno-mismatched-tags +-Wno-error=deprecated-register +dnl C++ -Wsuggest-override +dnl C++ -Wdeprecated-copy +dnl C++ -Wdeprecated-copy-dtor +dnl C++ -Wredundant-move -Wmissing-declarations +dnl C++ -Wstrict-null-sentinel +" +dnl Some extra warnings we use in the sim. +build_warnings="$build_warnings +-Wdeclaration-after-statement +-Wdeprecated-non-prototype +-Wimplicit-function-declaration +-Wimplicit-int +-Wincompatible-function-pointer-types +-Wincompatible-pointer-types +-Wmisleading-indentation +-Wmissing-parameter-type -Wmissing-prototypes --Wdeclaration-after-statement -Wmissing-parameter-type +-Wold-style-declaration +-Wold-style-definition -Wpointer-sign --Wold-style-declaration -Wold-style-definition " case "${host}" in -- cgit v1.1