On Sat, Mar 01, 2014 at 11:51:24PM +0100, Bruno Jiménez wrote:
> Recently, llvm has changed to use c++11, so we also should use it
> ---
>  src/gallium/state_trackers/clover/Makefile.am | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/gallium/state_trackers/clover/Makefile.am 
> b/src/gallium/state_trackers/clover/Makefile.am
> index ece2b38..cc9311c 100644
> --- a/src/gallium/state_trackers/clover/Makefile.am
> +++ b/src/gallium/state_trackers/clover/Makefile.am
> @@ -37,7 +37,7 @@ libcltgsi_la_SOURCES = \
>       tgsi/compiler.cpp
>  
>  libclllvm_la_CXXFLAGS = \
> -     -std=c++98 \
> +     -std=c++11 \
>       $(VISIBILITY_CXXFLAGS) \
>       $(LLVM_CPPFLAGS) \
>       $(DEFINES) \

I just sent out a patch which fixes this and doesn't break older versions:
http://lists.freedesktop.org/archives/mesa-dev/2014-March/055191.html

> @@ -49,7 +49,7 @@ libclllvm_la_SOURCES = \
>       llvm/invocation.cpp
>  
>  libclover_la_CXXFLAGS = \
> -     -std=c++0x \
> +     -std=c++11 \
>       $(VISIBILITY_CXXFLAGS)
>  

Can you resubmit this hunk as a separate patch.

Thanks,
Tom

>  libclover_la_LIBADD = \
> -- 
> 1.9.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to