Skip to content

Commit d6b43e4

Browse files
committed
build: clean up the mac catalyst flag handling in the host
The toolchain does not actually use any catalyst flavor, which resulted in these paths actually being dead. Cull them except for the one path which simply computes the target triple.
1 parent ced7dc1 commit d6b43e4

File tree

1 file changed

+3
-25
lines changed

1 file changed

+3
-25
lines changed

cmake/modules/AddSwift.cmake

Lines changed: 3 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,6 @@ function(_add_host_variant_c_compile_link_flags)
8989
""
9090
${ARGN})
9191

92-
get_maccatalyst_build_flavor(maccatalyst_build_flavor
93-
"${SWFIT_HOST_VARIANT_SDK}" "")
94-
9592
set(result ${${CFLAGS_RESULT_VAR_NAME}})
9693

9794
is_darwin_based_sdk("${SWIFT_HOST_VARIANT_SDK}" IS_DARWIN)
@@ -102,12 +99,9 @@ function(_add_host_variant_c_compile_link_flags)
10299
# MSVC, clang-cl, gcc don't understand -target.
103100
if(CMAKE_C_COMPILER_ID MATCHES "Clang" AND NOT SWIFT_COMPILER_IS_MSVC_LIKE)
104101
get_target_triple(target target_variant "${SWIFT_HOST_VARIANT_SDK}" "${SWIFT_HOST_VARIANT_ARCH}"
105-
MACCATALYST_BUILD_FLAVOR "${maccatalyst_build_flavor}"
102+
MACCATALYST_BUILD_FLAVOR ""
106103
DEPLOYMENT_VERSION "${DEPLOYMENT_VERSION}")
107104
list(APPEND result "-target" "${target}")
108-
if(target_variant)
109-
list(APPEND result "-target-variant" "${target_variant}")
110-
endif()
111105
endif()
112106

113107
set(_sysroot
@@ -133,21 +127,8 @@ function(_add_host_variant_c_compile_link_flags)
133127
# side effects are introduced should a new search path be added.
134128
list(APPEND result
135129
"-arch" "${SWIFT_HOST_VARIANT_ARCH}"
136-
"-F${SWIFT_SDK_${SWIFT_HOST_VARIANT_ARCH}_PATH}/../../../Developer/Library/Frameworks")
137-
138-
set(add_explicit_version TRUE)
139-
140-
# iOS-like and zippered libraries get their deployment version from the
141-
# target triple
142-
if(maccatalyst_build_flavor STREQUAL "ios-like" OR
143-
maccatalyst_build_flavor STREQUAL "zippered")
144-
set(add_explicit_version FALSE)
145-
endif()
146-
147-
if(add_explicit_version)
148-
list(APPEND result
149-
"-m${SWIFT_SDK_${SWIFT_HOST_VARIANT_SDK}_VERSION_MIN_NAME}-version-min=${DEPLOYMENT_VERSION}")
150-
endif()
130+
"-F${SWIFT_SDK_${SWIFT_HOST_VARIANT_ARCH}_PATH}/../../../Developer/Library/Frameworks"
131+
"-m${SWIFT_SDK_${SWIFT_HOST_VARIANT_SDK}_VERSION_MIN_NAME}-version-min=${DEPLOYMENT_VERSION}")
151132
endif()
152133

153134
if(CFLAGS_ANALYZE_CODE_COVERAGE)
@@ -432,9 +413,6 @@ function(_add_host_variant_link_flags target)
432413
"SHELL:-Xlinker -dead_strip")
433414
endif()
434415
endif()
435-
436-
get_maccatalyst_build_flavor(maccatalyst_build_flavor
437-
"${SWIFT_HOST_VARIANT_SDK}" "")
438416
endfunction()
439417

440418
# Add a single variant of a new Swift library.

0 commit comments

Comments
 (0)