amd/blis
1
0
Fork 0
mirror of https://github.com/amd/blis.git synced 2026-05-12 18:15:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use -fvisibility=[...] with clang on Linux/BSD/OSX.

Browse Source 
Details:
- Modified common.mk to use the -fvisibility=[hidden|default] option
  when compiling with clang on non-Windows platforms (Linux, BSD, OS X,
  etc.). Thanks to Isuru Fernando for pointing out this option works
  with clang on these OSes.
This commit is contained in:
Field G. Van Zee
2019-03-15 13:57:49 -05:00
committed by Devrajegowda, Kiran
parent 40b392de3f
commit 9ea223df2d
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
common.mk
View File

@@ -643,6 +643,8 @@ endif
endif
# Determine default export behavior / visibility of symbols for icc.
# NOTE: The Windows branches have been omitted since we currently make no
# effort to support Windows builds via icc (only gcc/clang via AppVeyor).
ifeq ($(CC_VENDOR),icc)
ifeq ($(ENABLE_EXPORT_ALL),yes)
# Export all symbols by default.
@@ -669,13 +671,12 @@ CMISCFLAGS :=
endif
else # ifeq ($(IS_WIN),no)
ifeq ($(ENABLE_EXPORT_ALL),yes)
# NOTE: Not sure if clang on Linux/BSD/OSX supports exporting all symbols by
# default.
CMISCFLAGS :=
# Export all symbols by default.
CMISCFLAGS := -fvisibility=default
else
# NOTE: Not sure if clang on Linux/BSD/OSX supports hiding all symbols by
# default.
CMISCFLAGS :=
# Hide all symbols by default and export only those that have been annotated
# as needing to be exported.
CMISCFLAGS := -fvisibility=hidden
endif
endif
endif