[Mesa-dev] [PATCH] amd: build addrlib with C++11

[Nicolai Hähnle]

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.haehnle at 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)
>