From d27593cce60720b936aa484d007c489b189b412a Mon Sep 17 00:00:00 2001 From: Richard Kreckel Date: Fri, 29 Apr 2005 22:10:55 +0000 Subject: [PATCH] * m4/cc.m4: Emit a warning if g++ is used and optimization turned off. --- ChangeLog | 5 +++++ m4/cc.m4 | 15 +++++++++++++-- 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 51a4f89..857fbee 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2005-04-29 Richard B. Kreckel + Ralf Wildenhues + + * m4/cc.m4: Emit a warning if g++ is used and optimization turned off. + 2005-04-24 Richard B. Kreckel Make GCC compiler flags default to -O diff --git a/m4/cc.m4 b/m4/cc.m4 index 0f071e5..9a8326c 100644 --- a/m4/cc.m4 +++ b/m4/cc.m4 @@ -18,14 +18,25 @@ fi dnl Wrapper around AC_PROG_CXX setting CXXFLAGS to plain "-O" as opposed to -dnl "-g -O2" for the GNU compiler (unless CXXFLAGS was set before). +dnl "-g -O2" for the GNU compiler (unless CXXFLAGS was set before). Also +dnl emits a warning if G++ is used and optimization turned off. AC_DEFUN([CL_PROG_CXX], [cl_test_CXXFLAGS=${CXXFLAGS+set} # Make sure this macro does not come after AC_PROG_CXX. # Otherwise CXXFLAGS would already be set. AC_BEFORE([$0],[AC_PROG_CXX])dnl AC_PROG_CXX([$1]) -if test "$cl_test_CXXFLAGS" != set && test "$ac_compiler_gnu" = yes; then +if test "$ac_compiler_gnu" = yes; then + if test "$cl_test_CXXFLAGS" != set; then + # User has not set CXXFLAGS. CXXFLAGS="-O" + else + # Warn if optimization has been turned off with GCC. + # Optimization is used for module ordering. + case $CXXFLAGS in + [ *\ -O | -O | -O\ * | *\ -O\ * | -O[!0]* | *\ -O[!0]*) ;; ] + *) AC_MSG_WARN([Optimization turned off. I recommend you unset CXXFLAGS.]);; + esac + fi fi ])