This reverts commit 2860f1780ca92c779ee0a2c545ae1b9c4818dc53.

Conflicts:
        configure.ac

Do not use -fgnu89-inline as it can emit duplicate symbols for
inline functions that are declared in multiple object files
being linked together.

For example on clang 5.0 targetting iOS
(clang advertises itself as gcc 4.2, the last GPL version of gcc)
---
 configure.ac | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/configure.ac b/configure.ac
index 7f63a04..fa6f4fb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -396,7 +396,7 @@ if test x$ac_cv_c_compiler_gnu = xyes ; then
        if test x$enable_gcc_werror = "xyes" ; then
                CFLAGS="$CFLAGS -Werror "
                CXXFLAGS="$CXXFLAGS -Werror "
-               fi
+       fi
 
        XIPH_ADD_CFLAGS([-Wdeclaration-after-statement])
        XIPH_ADD_CFLAGS([-D_FORTIFY_SOURCE=2])
@@ -409,18 +409,14 @@ if test x$ac_cv_c_compiler_gnu = xyes ; then
                CPPFLAGS="$CPPFLAGS -DFLAC__USE_VISIBILITY_ATTR"
                CFLAGS="$CFLAGS -fvisibility=hidden"
                CXXFLAGS="$CXXFLAGS -fvisibility=hidden"
-               fi
-
-       if test "x$GCC_MAJOR_VERSION$GCC_MINOR_VERSION" = "x42" ; then
-               XIPH_ADD_CFLAGS([-fgnu89-inline])
-               fi
+       fi
 
        if test "x$asm_optimisation" = "xyes" ; then
                XIPH_ADD_CFLAGS([-msse2])
-               fi
-
        fi
 
+fi
+
 XIPH_ADD_CFLAGS([-Wextra])
 
 if test x$enable_stack_smash_protection = "xyes" ; then
-- 
1.8.3.4 (Apple Git-47)
_______________________________________________
flac-dev mailing list
flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to