From: Theodore Ts'o <tytso@mit.edu>
Date: Sat, 18 Aug 2018 02:26:24 +0000 (-0400)
Subject: configure: don't enable LTO by default
X-Git-Tag: v1.45.0~48
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=139a30928a23f8bbc7148197bbe98ed0e0565cf1;p=tools%2Fe2fsprogs.git

configure: don't enable LTO by default

Not all compilers (or versions of compilers) use the same LTO options
or behave the same way with LTO.  In particular, using clang and the
current LTO options cause the build to fail.  We should probably fix
up the configure script to handle Clang and LTO, but for now, we won't
enable LTO unless the user explicitly passes --enable-lto to the
configure script.

Signed-off-by: Theodore Ts'o <tytso@mit.edu>
---

diff --git a/configure b/configure
index 6244cb9..98456b7 100755
--- a/configure
+++ b/configure
@@ -13755,7 +13755,7 @@ esac
 if test "${enable_lto+set}" = set; then :
   enableval=$enable_lto;
 else
-  enable_lto=probe
+  enable_lto=no
 fi
 
 if test "$enable_lto" = "yes" || test "$enable_lto" = "probe"; then
diff --git a/configure.ac b/configure.ac
index 6c464ba..0c2b242 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1301,7 +1301,7 @@ dnl Enable LTO for all packages
 dnl
 AC_ARG_ENABLE([lto],
 [  --enable-lto		  enable link time optimization],,
-enable_lto=probe)
+enable_lto=no)
 if test "$enable_lto" = "yes" || test "$enable_lto" = "probe"; then
 	AC_MSG_CHECKING([if C compiler supports LTO])
 	OLD_CFLAGS="$CFLAGS"