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

Author: Gert Wollny <gw.foss...@gmail.com>
Date:   Wed Oct 18 17:05:27 2017 +0200

clover: use the unified check for c++11 instead of the gcc version number

So far clover based its test for compiler support on the version of gcc,
while in reality support for c++11 is required. This patch replaces the
version check by the check unified for all modules that require c++11.

Reviewed-by: Emil Velikov <emil.veli...@collabora.com>

---

 configure.ac                                  | 4 ++--
 src/gallium/state_trackers/clover/Makefile.am | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/configure.ac b/configure.ac
index 3dd9b9a4ce..d362dfb15b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2294,8 +2294,8 @@ if test "x$enable_opencl" = xyes; then
         AC_MSG_ERROR([cannot enable OpenCL without Gallium])
     fi
 
-    if test $GCC_VERSION_MAJOR -lt 4 -o $GCC_VERSION_MAJOR -eq 4 -a 
$GCC_VERSION_MINOR -lt 7; then
-        AC_MSG_ERROR([gcc >= 4.7 is required to build clover])
+    if test "x$HAVE_CXX11" != "xyes"; then
+       AC_MSG_ERROR([clover requires c++11 support])
     fi
 
     if test "x$have_libclc" = xno; then
diff --git a/src/gallium/state_trackers/clover/Makefile.am 
b/src/gallium/state_trackers/clover/Makefile.am
index 321393536d..7167bc1c5c 100644
--- a/src/gallium/state_trackers/clover/Makefile.am
+++ b/src/gallium/state_trackers/clover/Makefile.am
@@ -31,14 +31,14 @@ endif
 noinst_LTLIBRARIES = libclover.la libcltgsi.la libclllvm.la
 
 libcltgsi_la_CXXFLAGS = \
-       -std=c++11 \
+       $(CXX11_CXXFLAGS) \
        $(CLOVER_STD_OVERRIDE) \
        $(VISIBILITY_CXXFLAGS)
 
 libcltgsi_la_SOURCES = $(TGSI_SOURCES)
 
 libclllvm_la_CXXFLAGS = \
-       -std=c++11 \
+       $(CXX11_CXXFLAGS) \
        $(VISIBILITY_CXXFLAGS) \
        $(LLVM_CXXFLAGS) \
        $(CLOVER_STD_OVERRIDE) \
@@ -51,7 +51,7 @@ libclllvm_la_CXXFLAGS = \
 libclllvm_la_SOURCES = $(LLVM_SOURCES)
 
 libclover_la_CXXFLAGS = \
-       -std=c++11 \
+       $(CXX11_CXXFLAGS) \
        $(CLOVER_STD_OVERRIDE) \
        $(VISIBILITY_CXXFLAGS)
 

_______________________________________________
mesa-commit mailing list
mesa-commit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to