On Monday, July 14, 2014 11:49:57 AM Ian Romanick wrote:
> From: Ian Romanick <ian.d.roman...@intel.com>
> 
> I already pass '-ggdb3 -O1' or '-ggdb3 -Og' for CFLAGS, and I don't want
> configure to change them for me.
> 
> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
> ---
>  configure.ac | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 4646212..04abcc0 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -308,10 +308,20 @@ AC_ARG_ENABLE([debug],
>  if test "x$enable_debug" = xyes; then
>      DEFINES="$DEFINES -DDEBUG"
>      if test "x$GCC" = xyes; then
> -        CFLAGS="$CFLAGS -g -O0"
> +        if ! echo "$CFLAGS" | grep -q -e '-g'; then
> +            CFLAGS="$CFLAGS -g"
> +        fi
> +        if ! echo "$CFLAGS" | grep -q -e '-O'; then
> +            CFLAGS="$CFLAGS -O0"
> +        fi
>      fi
>      if test "x$GXX" = xyes; then
> -        CXXFLAGS="$CXXFLAGS -g -O0"
> +        if ! echo "$CXXFLAGS" | grep -q -e '-g'; then
> +            CXXFLAGS="$CXXFLAGS -g"
> +        fi
> +        if ! echo "$CXXFLAGS" | grep -q -e '-O'; then
> +            CXXFLAGS="$CXXFLAGS -O0"
> +        fi
>      fi
>  fi
>  
> 

Seems reasonable to me.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to