You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by sv...@apache.org on 2013/10/24 06:02:16 UTC
svn commit: r1535275 - in /subversion/branches/1.8.x: ./ STATUS configure.ac
Author: svn-role
Date: Thu Oct 24 04:02:16 2013
New Revision: 1535275
URL: http://svn.apache.org/r1535275
Log:
Merge r1534860 from trunk:
* r1534860
Remove -fwhole_program flag from optimized GCC builds.
Justification:
As described in http://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html,
-flto is superior and should not be combined with -fwhole-program.
As it turns out, -fwhole_program causes issues with llvm and other
sources indicated that the flag might not be particularly useful to
begin with.
Votes:
+1: stefan2, breser, brane
Modified:
subversion/branches/1.8.x/ (props changed)
subversion/branches/1.8.x/STATUS
subversion/branches/1.8.x/configure.ac (contents, props changed)
Propchange: subversion/branches/1.8.x/
------------------------------------------------------------------------------
Merged /subversion/trunk:r1534860
Modified: subversion/branches/1.8.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1535275&r1=1535274&r2=1535275&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Thu Oct 24 04:02:16 2013
@@ -146,14 +146,3 @@ Veto-blocked changes:
Approved changes:
=================
-
- * r1534860
- Remove -fwhole_program flag from optimized GCC builds.
- Justification:
- As described in http://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html,
- -flto is superior and should not be combined with -fwhole-program.
- As it turns out, -fwhole_program causes issues with llvm and other
- sources indicated that the flag might not be particularly useful to
- begin with.
- Votes:
- +1: stefan2, breser, brane
Modified: subversion/branches/1.8.x/configure.ac
URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/configure.ac?rev=1535275&r1=1535274&r2=1535275&view=diff
==============================================================================
--- subversion/branches/1.8.x/configure.ac (original)
+++ subversion/branches/1.8.x/configure.ac Thu Oct 24 04:02:16 2013
@@ -1089,7 +1089,6 @@ if test "$enable_optimization" = "yes";
SVN_CFLAGS_ADD_IFELSE([-O])])])])
SVN_CFLAGS_ADD_IFELSE([-Wno-clobbered])
SVN_CFLAGS_ADD_IFELSE([-flto])
- SVN_CFLAGS_ADD_IFELSE([-fwhole-program])
fi
fi
if test -z ["`echo $CXXUSERFLAGS' ' | $EGREP -- '-O[^ ]* '`"]; then
@@ -1106,7 +1105,6 @@ if test "$enable_optimization" = "yes";
SVN_CXXFLAGS_ADD_IFELSE([-O])])])])
SVN_CXXFLAGS_ADD_IFELSE([-Wno-clobbered])
SVN_CXXFLAGS_ADD_IFELSE([-flto])
- SVN_CXXFLAGS_ADD_IFELSE([-fwhole-program])
fi
fi
elif test "$enable_optimization" = "no"; then
Propchange: subversion/branches/1.8.x/configure.ac
------------------------------------------------------------------------------
Merged /subversion/trunk/configure.ac:r1534860