diff options
| author | sezero <sezero@users.sourceforge.net> | 2019-10-08 20:25:55 +0300 |
|---|---|---|
| committer | Erik de Castro Lopo <erikd@mega-nerd.com> | 2019-10-10 18:31:26 +1100 |
| commit | d4a1b345dd16591ff6f17c67ee519afebe2f9792 (patch) | |
| tree | cc5c62eed8547201f31eb7d40a6ebcec6db7cf2c /m4 | |
| parent | ee70d969322105fa5f857d9d090302d9936b5df8 (diff) | |
| download | flac-d4a1b345dd16591ff6f17c67ee519afebe2f9792.tar.gz | |
configure.ac: allow several compiler flags for clang
The flac configury marked clang as 'not being gcc' and excluded
a lot of compiler switches, most importantly the visibility flags,
from being used with it. This was done possibly after a problem
reported at: https://github.com/erikd/libsndfile/issues/49 .
This patch does the following:
- m4/gcc_version.m4 (XIPH_GCC_VERSION): set GCC_MAJOR_VERSION and
GCC_MINOR_VERSION to 0 for non-gcc. Previously, they were left
unset.
- configure: the gcc version checks are, naturally, against non-
zero values, so, allow many compiler switches to be used with
clang without affecting real-gcc cases.
- configure: When setting CFLAGS="-O3 -funroll-loops", also set
CXXFLAGS="-O3". Prevents g++ warnings with _FORTIFY_SOURCE, i.e.:
'_FORTIFY_SOURCE requires compiling with optimization (-O)'
Tested compilation using gcc-7.3.1 and clang-5.0.2 on x86_64-linux,
and gcc-4.4.7 and clang-3.4.2 on an i686-linux. Also tested cross-
compiling for Mac OS X using clang-5.0.2.
Diffstat (limited to 'm4')
| -rw-r--r-- | m4/gcc_version.m4 | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/m4/gcc_version.m4 b/m4/gcc_version.m4 index 1c53086f..e6aaa603 100644 --- a/m4/gcc_version.m4 +++ b/m4/gcc_version.m4 @@ -22,6 +22,9 @@ if test "x$ac_cv_c_compiler_gnu" = "xyes" ; then GCC_MAJOR_VERSION=`echo $GCC_VERSION | cut -d. -f 1` GCC_MINOR_VERSION=`echo $GCC_VERSION | cut -d. -f 2` +else + GCC_MAJOR_VERSION=0 + GCC_MINOR_VERSION=0 fi AC_SUBST(GCC_VERSION) |