From 6ec3ccd76f4c9d533ced58b48f95727782a6ab8f Mon Sep 17 00:00:00 2001 From: Nicholas Sielicki Date: Wed, 11 Sep 2024 13:12:13 -0700 Subject: [PATCH] feat(build): add -Wextra to "picky" compiler flags (#588) Signed-off-by: Nicholas Sielicki --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 85cccf5d3..6f5cb63b6 100644 --- a/configure.ac +++ b/configure.ac @@ -171,13 +171,13 @@ AC_DEFINE_UNQUOTED([OFI_NCCL_TRACE], [${trace}], [Defined to 1 unit test output AC_ARG_ENABLE([picky-compiler], [AS_HELP_STRING([--disable-picky-compiler], [Disable adding picky compiler flags.])]) AS_IF([test "${enable_picky_compiler}" != "no"], - [picky_compiler_flags="-Wall -Wc++-compat" + [picky_compiler_flags="-Wall -Wc++-compat -Wextra -Wno-unused-parameter" AC_MSG_NOTICE([Adding ${picky_compiler_flags} to CFLAGS.]) CFLAGS="${CFLAGS} ${picky_compiler_flags}" AS_UNSET([picky_compiler_flags])]) AC_ARG_ENABLE([werror], - [AS_HELP_STRING([--enable-werror], [Enable setting -Werror. Off by default, unless building from Git tree.])]) + [AS_HELP_STRING([--enable-werror], [(Developer) Enable setting -Werror. Off by default, unless building from Git tree.])]) werror_flags="-Werror" AS_IF([test -d "${srcdir}/.git" -a -z "${enable_werror}"], [AC_MSG_NOTICE([Found .git directory. Adding ${werror_flags} to CFLAGS.])