On Thu, Nov 9, 2017 at 1:58 PM, Dylan Baker <dy...@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.ba...@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 <matts...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to