aboutsummaryrefslogtreecommitdiff
path: root/sim/m4
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2021-06-18 02:14:52 -0400
committerMike Frysinger <vapier@gentoo.org>2021-06-18 10:15:15 -0400
commit982c3a65ca5dbea8d6de3afd934f0a06fef54383 (patch)
tree416ece9ed53e3350c23a09b381139bdc4cc9c6d7 /sim/m4
parent3f8414df7a533cccd39bf19c0e311975d9ededde (diff)
downloadfsf-binutils-gdb-982c3a65ca5dbea8d6de3afd934f0a06fef54383.zip
fsf-binutils-gdb-982c3a65ca5dbea8d6de3afd934f0a06fef54383.tar.gz
fsf-binutils-gdb-982c3a65ca5dbea8d6de3afd934f0a06fef54383.tar.bz2
sim: move -Werror disabling to Makefile
For the ports that still don't build with -Werror, rather than disable the flag at configure time, do it at make time. This will allow us to unify these tests in the common sim configure script.
Diffstat (limited to 'sim/m4')
-rw-r--r--sim/m4/sim_ac_option_warnings.m413
1 files changed, 5 insertions, 8 deletions
diff --git a/sim/m4/sim_ac_option_warnings.m4 b/sim/m4/sim_ac_option_warnings.m4
index c831fa8..5a0fe30 100644
--- a/sim/m4/sim_ac_option_warnings.m4
+++ b/sim/m4/sim_ac_option_warnings.m4
@@ -15,7 +15,6 @@ dnl along with this program. If not, see <http://www.gnu.org/licenses/>.
dnl
dnl --enable-build-warnings is for developers of the simulator.
dnl it enables extra GCC specific warnings.
-dnl arg[1] Enable -Werror by default? ("yes" or "no")
AC_DEFUN([SIM_AC_OPTION_WARNINGS],
[
AC_ARG_ENABLE(werror,
@@ -26,17 +25,15 @@ AC_ARG_ENABLE(werror,
*) AC_MSG_ERROR(bad value ${enableval} for --enable-werror) ;;
esac])
-# Enable -Werror by default when using gcc
+dnl Enable -Werror by default when using gcc
if test "${GCC}" = yes -a -z "${ERROR_ON_WARNING}" ; then
- ERROR_ON_WARNING=yes
+ ERROR_ON_WARNING=yes
fi
WERROR_CFLAGS=""
-m4_if(m4_default([$1], [yes]), [yes], [dnl
- if test "${ERROR_ON_WARNING}" = yes ; then
- WERROR_CFLAGS="-Werror"
- fi
-])dnl
+if test "${ERROR_ON_WARNING}" = yes ; then
+ WERROR_CFLAGS="-Werror"
+fi
build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \
-Wpointer-sign \