diff options
author | Lasse Collin <lasse.collin@tukaani.org> | 2023-09-26 17:24:15 +0300 |
---|---|---|
committer | Lasse Collin <lasse.collin@tukaani.org> | 2023-09-26 17:24:15 +0300 |
commit | 092d21db2e5eea19fe079264ce48c178989c7606 (patch) | |
tree | d19242706c4647e9a68fbadc7e374695659c2cd7 | |
parent | Build: Fix __attribute__((ifunc(...))) detection with clang -Wall. (diff) | |
download | xz-092d21db2e5eea19fe079264ce48c178989c7606.tar.xz |
Build: Update the comment about -Werror usage in checks.
Diffstat (limited to '')
-rw-r--r-- | configure.ac | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/configure.ac b/configure.ac index cf34c739..4d298430 100644 --- a/configure.ac +++ b/configure.ac @@ -832,8 +832,14 @@ AC_C_BIGENDIAN # __attribute__((__constructor__)) can be used for one-time initializations. # Use -Werror because some compilers accept unknown attributes and just -# give a warning. If it works this should give no warnings, even -# clang -Weverything should be fine. +# give a warning. +# +# FIXME? Unfortunately -Werror can cause trouble if CFLAGS contains options +# that produce warnings for unrelated reasons. For example, GCC and Clang +# support -Wunused-macros which will warn about "#define _GNU_SOURCE 1" +# which will be among the #defines that Autoconf inserts to the beginning of +# the test program. There seems to be no nice way to prevent Autoconf from +# inserting the any defines to the test program. AC_MSG_CHECKING([if __attribute__((__constructor__)) can be used]) have_func_attribute_constructor=no OLD_CFLAGS="$CFLAGS" |