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

Emil Velikov emil.l.velikov at gmail.com
Wed Oct 12 18:05:31 UTC 2016


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



More information about the mesa-dev mailing list