[Mesa-dev] [PATCH] autotools: Set C++ visibility flags on Intel

[Matt Turner]

On Thu, Nov 9, 2017 at 1:58 PM, Dylan Baker <dylan at pnwbakers.com> wrote:
> These flags are set for C sources, but not C++. This causes symbol
> visibility leaks from the C++ parts of the Intel compiler.
>
> fixes: 700bebb958e93f4d ("i965: Move the back-end compiler to src/intel/compiler")

Fixes

> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> ---
>
>  src/intel/Makefile.am | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/intel/Makefile.am b/src/intel/Makefile.am
> index a34e3014497..6a5e393f175 100644
> --- a/src/intel/Makefile.am
> +++ b/src/intel/Makefile.am
> @@ -46,6 +46,9 @@ AM_CFLAGS = \
>         $(VISIBILITY_CFLAGS) \
>         $(WNO_OVERRIDE_INIT)
>
> +AM_CXXFLAGS = \
> +       $(VISIBILITY_CXXFLAGS)
> +


Reviewed-by: Matt Turner <mattst88 at gmail.com>