Mesa (master): automake: strip LLVM_CXXFLAGS and LLVM_CPPFLAGS too

Maarten Lankhorst mlankhorst at kemper.freedesktop.org
Mon Nov 19 08:47:06 UTC 2012


Module: Mesa
Branch: master
Commit: ddb901fbf4489ffcd85d3320f23913eb1d4fbdfe
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ddb901fbf4489ffcd85d3320f23913eb1d4fbdfe

Author: Maarten Lankhorst <maarten.lankhorst at canonical.com>
Date:   Mon Nov 19 09:43:29 2012 +0100

automake: strip LLVM_CXXFLAGS and LLVM_CPPFLAGS too

It seems that -NDEBUG and other flags might still be leaked through
those variables, so strip those off there as well.

NOTE: This is a candidate for the 9.0 branch.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst at canonical.com>

---

 configure.ac |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure.ac b/configure.ac
index 228f464..f484dc4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1675,8 +1675,8 @@ if test "x$enable_gallium_llvm" = xyes; then
 	fi
 	LLVM_LDFLAGS=`$LLVM_CONFIG --ldflags`
 	LLVM_BINDIR=`$LLVM_CONFIG --bindir`
-	LLVM_CXXFLAGS=`$LLVM_CONFIG --cxxflags`
-	LLVM_CPPFLAGS=`$LLVM_CONFIG --cppflags`
+	LLVM_CXXFLAGS=`$LLVM_CONFIG --cxxflags|sed -e 's/-DNDEBUG\>//g' -e 's/-pedantic//g' -e 's/-Wcovered-switch-default//g'`
+	LLVM_CPPFLAGS=`$LLVM_CONFIG --cppflags|sed -e 's/-DNDEBUG\>//g' -e 's/-pedantic//g' -e 's/-Wcovered-switch-default//g'`
 	LLVM_INCLUDEDIR=`$LLVM_CONFIG --includedir`
 	LLVM_LIBDIR=`$LLVM_CONFIG --libdir`
 	DEFINES="${DEFINES} -DHAVE_LLVM=`echo $LLVM_VERSION | sed -e 's/\([[0-9]]\)\.\([[0-9]]\)/0x0\10\2/g'`"




More information about the mesa-commit mailing list