On 15.11.2017 17:45, Kai Wasserbäch wrote:
Doesn't the meson.build file need the same change?

I don't really know Meson, but the master build file looks to me like Meson always builds with C++11.

Cheers,
Nicolai


Nicolai Hähnle wrote on 15.11.2017 12:55:
From: Nicolai Hähnle <nicolai.haeh...@amd.com>

It is required for LLVM anyway.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103658
Fixes: 7f33e94e43a6 ("amd/addrlib: update to latest version")
---
  src/amd/Makefile.addrlib.am | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/amd/Makefile.addrlib.am b/src/amd/Makefile.addrlib.am
index 46689637f9b..90dfe963445 100644
--- a/src/amd/Makefile.addrlib.am
+++ b/src/amd/Makefile.addrlib.am
@@ -26,15 +26,15 @@ addrlib_libamdgpu_addrlib_la_CPPFLAGS = \
        -I$(srcdir)/common \
        -I$(srcdir)/addrlib \
        -I$(srcdir)/addrlib/core \
        -I$(srcdir)/addrlib/inc/chip/gfx9 \
        -I$(srcdir)/addrlib/inc/chip/r800 \
        -I$(srcdir)/addrlib/gfx9/chip \
        -I$(srcdir)/addrlib/r800/chip \
        -DBRAHMA_BUILD=1
addrlib_libamdgpu_addrlib_la_CXXFLAGS = \
-       $(VISIBILITY_CXXFLAGS)
+       $(VISIBILITY_CXXFLAGS) $(CXX11_CXXFLAGS)
noinst_LTLIBRARIES += $(ADDRLIB_LIBS) addrlib_libamdgpu_addrlib_la_SOURCES = $(ADDRLIB_FILES)


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

Reply via email to