From bf470982f998b2b18545cc335f469a20e457acdd Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Sat, 9 Jan 2021 01:16:11 -0500 Subject: sim: enable -Werror by default for some arches We've had this off for a long time because the sim code was way too full of warnings for it to be feasible. However, I've cleaned things up significantly from when this was first merged, and we can start to turn this around. Change the macro to enable -Werror by default, and allow ports to opt out. New ports will get it automatically (and we can push back on them if they try to turn it off). Also turn it off for the few ports that still hit warnings for me. All the rest will get the new default, and we'll wait for feedback if/when new issues come up. --- sim/d10v/ChangeLog | 5 +++++ sim/d10v/configure | 5 ----- sim/d10v/configure.ac | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) (limited to 'sim/d10v') diff --git a/sim/d10v/ChangeLog b/sim/d10v/ChangeLog index a53becd..cad3063 100644 --- a/sim/d10v/ChangeLog +++ b/sim/d10v/ChangeLog @@ -1,3 +1,8 @@ +2021-01-09 Mike Frysinger + + * configure.ac (SIM_AC_OPTION_WARNINGS): Pass "no". + * configure: Regenerate. + 2021-01-08 Mike Frysinger * configure: Regenerate. diff --git a/sim/d10v/configure b/sim/d10v/configure index 692cd90..f0ce157 100755 --- a/sim/d10v/configure +++ b/sim/d10v/configure @@ -13684,11 +13684,6 @@ if test "${GCC}" = yes -a -z "${ERROR_ON_WARNING}" ; then fi WERROR_CFLAGS="" -if test "${ERROR_ON_WARNING}" = yes ; then -# NOTE: Disabled in the sim dir due to most sims generating warnings. -# WERROR_CFLAGS="-Werror" - true -fi build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ -Wpointer-sign \ diff --git a/sim/d10v/configure.ac b/sim/d10v/configure.ac index 5dffa14..1551df5 100644 --- a/sim/d10v/configure.ac +++ b/sim/d10v/configure.ac @@ -6,6 +6,6 @@ SIM_AC_COMMON SIM_AC_OPTION_ENDIAN SIM_AC_OPTION_ALIGNMENT(STRICT_ALIGNMENT,STRICT_ALIGNMENT) -SIM_AC_OPTION_WARNINGS +SIM_AC_OPTION_WARNINGS(no) SIM_AC_OUTPUT -- cgit v1.1