[Mesa-dev] [PATCH] configure.ac: do not let LLVM_CONFIG to define NDEBUG and disable assertions

Jose Fonseca jfonseca at vmware.com
Mon Jul 11 07:06:08 PDT 2011


Looks good. But this would be more correct:

  llvm-config --cppflags | sed 's/-DNDEBUG\>//g'

I'll do the same thing in scons. 

Jose

----- Original Message -----
> Re-enables assertions in src/mesa.
> ---
>  configure.ac |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index dde13c9..84710ed 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1806,7 +1806,7 @@ fi
>  if test "x$enable_gallium_llvm" = xyes; then
>      if test "x$LLVM_CONFIG" != xno; then
>  	LLVM_VERSION=`$LLVM_CONFIG --version`
> -	LLVM_CFLAGS=`$LLVM_CONFIG --cppflags`
> +	LLVM_CFLAGS=`$LLVM_CONFIG --cppflags|sed s/-DNDEBUG//g`
>  	LLVM_LIBS="`$LLVM_CONFIG --libs` -lstdc++"
>  
>  	LLVM_LDFLAGS=`$LLVM_CONFIG --ldflags`
> --
> 1.7.4.1
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 


More information about the mesa-dev mailing list