[Mesa-dev] [PATCH] gallium/auxiliary: Fix build with newer LLVM.

[Johannes Obermayr]

Am Dienstag, 20. November 2012, 06:41:22 schrieb Dave Airlie:
> > $ llvm-config --cxxflags
> >
> > -I/usr/include -fmessage-length=0 -O2 -Wall -D_FORTIFY_SOURCE=2
> > -fstack-protector -funwind-tables -fasynchronous-unwind-tables -g -fPIC
> > -fvisibility-inlines-hidden -O2 -g -D_GNU_SOURCE -Wall -W
> > -Wno-unused-parameter -Wwrite-strings -pedantic -Wno-long-long
> > -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
> 
> And this is why we don't include LLVM CXX flags :-)
> 
> notice the -g -O2 -Wall bits? not something that pkg-config wants in its flags.
> 
> Dave.

Does that mean the required patch does not go mainline and you want to remove LLVM_CXXFLAGS from these files:

$ grep -r -n -e "LLVM_CXXFLAGS"
configure.ac:1491:AC_SUBST([LLVM_CXXFLAGS])
configure.ac:1678:      LLVM_CXXFLAGS=`$LLVM_CONFIG --cxxflags|sed -e 's/-DNDEBUG\>//g' -e 's/-pedantic//g' -e 's/-Wcovered-switch-default//g'`
src/gallium/drivers/radeon/Makefile:13:CXXFLAGS+= $(LLVM_CXXFLAGS)
src/gallium/drivers/r600/Makefile.am:37:        $(LLVM_CXXFLAGS)
src/gallium/auxiliary/Makefile:16:CXXFLAGS += $(LLVM_CXXFLAGS)
configs/current.in:34:LLVM_CXXFLAGS = @LLVM_CXXFLAGS@