Skip to content

Commit 9f9f91d

Browse files
authored
Merge pull request #71996 from edymtt/edymtt/ensure-visibility-hidden-is-set-for-compatibility-archives
Ensure that proper default visibility is set for compatibility archives
2 parents ede186c + 0ff811a commit 9f9f91d

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

stdlib/toolchain/CMakeLists.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,10 @@ endif()
4646
# runtime being patched only through public ABI.
4747
list(APPEND CXX_COMPILE_FLAGS "-DSWIFT_COMPATIBILITY_LIBRARY=1")
4848

49+
set(CMAKE_C_VISIBILITY_PRESET hidden)
50+
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
51+
set(CMAKE_VISIBILITY_INLINES_HIDDEN YES)
52+
4953
if(SWIFT_STDLIB_SUPPORT_BACK_DEPLOYMENT)
5054
add_subdirectory(legacy_layouts)
5155
add_subdirectory(Compatibility50)

stdlib/toolchain/Compatibility56/CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@ set(library_name "swiftCompatibility56")
22

33
include_directories("include/" "${SWIFT_STDLIB_SOURCE_DIR}")
44

5-
set(CMAKE_C_VISIBILITY_PRESET hidden)
6-
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
7-
set(CMAKE_VISIBILITY_INLINES_HIDDEN YES)
8-
95
add_compile_definitions(SWIFT_COMPATIBILITY56)
106
add_swift_target_library("${library_name}" STATIC
117
Overrides.cpp

stdlib/toolchain/CompatibilityPacks/CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@ set(library_name "swiftCompatibilityPacks")
22

33
include_directories("include/" "${SWIFT_STDLIB_SOURCE_DIR}")
44

5-
set(CMAKE_C_VISIBILITY_PRESET hidden)
6-
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
7-
set(CMAKE_VISIBILITY_INLINES_HIDDEN YES)
8-
95
add_compile_definitions(SWIFT_COMPATIBILITY_PACKS)
106
add_swift_target_library("${library_name}" STATIC
117
Metadata.cpp

0 commit comments

Comments
 (0)