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

Kai Wasserbäch kai at dev.carbon-project.org
Wed Nov 15 16:45:30 UTC 2017


Doesn't the meson.build file need the same change?

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)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171115/73919010/attachment.sig>


More information about the mesa-dev mailing list