[Mesa-dev] [PATCH 3/5] radv: correct variable name VISIBILITY_{, C}FLAGS

[Dave Airlie]

Oops,

Reviewed-by: Dave Airlie <airlied at redhat.com>

On 13 October 2016 at 04:05, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
>
> The letter C was missing, thus in turn all the internal symbols were
> exported.
>
> As a result we hide ~150 symbols and cut ~36K from libvulkan_radeon.so.
>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> My current testing involves static linking LLVM (which exports the
> world) so the final binary might have some spurous exports + is quite
> large. One step at a time... we'll get to the rest in due time.
> ---
>  src/amd/vulkan/Makefile.am | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/amd/vulkan/Makefile.am b/src/amd/vulkan/Makefile.am
> index 6eaecd8..b6f3d00 100644
> --- a/src/amd/vulkan/Makefile.am
> +++ b/src/amd/vulkan/Makefile.am
> @@ -49,7 +49,8 @@ AM_CPPFLAGS = \
>         -I$(top_srcdir)/src/gallium/auxiliary \
>         -I$(top_srcdir)/src/gallium/include
>
> -AM_CFLAGS = $(VISIBILITY_FLAGS) \
> +AM_CFLAGS = \
> +       $(VISIBILITY_CFLAGS) \
>         $(PTHREAD_CFLAGS) \
>         $(LLVM_CFLAGS)
>
> --
> 2.10.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev