Skip to content

Commit 914a6fd

Browse files
committed
[CMake] Fix weiredness in swift runtime linking for SourceKit libraries
SourceKit components are never built while bootstrapping stages. So it doen't need to have stage 0 or 1 handling.
1 parent 9017ef5 commit 914a6fd

File tree

1 file changed

+10
-22
lines changed

1 file changed

+10
-22
lines changed

tools/SourceKit/cmake/modules/AddSwiftSourceKit.cmake

Lines changed: 10 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -154,9 +154,6 @@ function(add_sourcekit_swift_runtime_link_flags target path HAS_SWIFT_MODULES)
154154
endif()
155155

156156
if(SWIFT_SWIFT_PARSER)
157-
# Make sure we can find the early SwiftSyntax libraries.
158-
target_link_directories(${target} PRIVATE "${SWIFT_PATH_TO_EARLYSWIFTSYNTAX_BUILD_DIR}/lib/swift/host")
159-
160157
# Add rpath to the host Swift libraries.
161158
if (NOT SWIFT_HOST_VARIANT_SDK IN_LIST SWIFT_DARWIN_PLATFORMS)
162159
file(RELATIVE_PATH relative_hostlib_path "${path}" "${SWIFTLIB_DIR}/host")
@@ -166,25 +163,16 @@ function(add_sourcekit_swift_runtime_link_flags target path HAS_SWIFT_MODULES)
166163
# For the "end step" of bootstrapping configurations on Darwin, need to be
167164
# able to fall back to the SDK directory for libswiftCore et al.
168165
if (BOOTSTRAPPING_MODE MATCHES "BOOTSTRAPPING.*")
169-
if (NOT "${bootstrapping}" STREQUAL "1")
170-
if(SWIFT_HOST_VARIANT_SDK IN_LIST SWIFT_DARWIN_PLATFORMS)
171-
target_link_directories(${target} PRIVATE "${sdk_dir}")
172-
173-
# Include the abi stable system stdlib in our rpath.
174-
set(swift_runtime_rpath "/usr/lib/swift")
175-
176-
# Add in the toolchain directory so we can grab compatibility libraries
177-
get_filename_component(TOOLCHAIN_BIN_DIR ${SWIFT_EXEC_FOR_SWIFT_MODULES} DIRECTORY)
178-
get_filename_component(TOOLCHAIN_LIB_DIR "${TOOLCHAIN_BIN_DIR}/../lib/swift/${SWIFT_SDK_${SWIFT_HOST_VARIANT_SDK}_LIB_SUBDIR}" ABSOLUTE)
179-
target_link_directories(${target} PUBLIC ${TOOLCHAIN_LIB_DIR})
180-
else()
181-
get_filename_component(swift_bin_dir ${SWIFT_EXEC_FOR_SWIFT_MODULES} DIRECTORY)
182-
get_filename_component(swift_dir ${swift_bin_dir} DIRECTORY)
183-
set(host_lib_dir "${swift_dir}/lib/swift/${SWIFT_SDK_${SWIFT_HOST_VARIANT_SDK}_LIB_SUBDIR}")
184-
target_link_directories(${target} PUBLIC ${host_lib_dir})
185-
186-
list(APPEND RPATH_LIST "${host_lib_dir}")
187-
endif()
166+
if(SWIFT_HOST_VARIANT_SDK IN_LIST SWIFT_DARWIN_PLATFORMS)
167+
target_link_directories(${target} PRIVATE "${sdk_dir}")
168+
169+
# Include the abi stable system stdlib in our rpath.
170+
set(swift_runtime_rpath "/usr/lib/swift")
171+
172+
# Add in the toolchain directory so we can grab compatibility libraries
173+
get_filename_component(TOOLCHAIN_BIN_DIR ${SWIFT_EXEC_FOR_SWIFT_MODULES} DIRECTORY)
174+
get_filename_component(TOOLCHAIN_LIB_DIR "${TOOLCHAIN_BIN_DIR}/../lib/swift/${SWIFT_SDK_${SWIFT_HOST_VARIANT_SDK}_LIB_SUBDIR}" ABSOLUTE)
175+
target_link_directories(${target} PUBLIC ${TOOLCHAIN_LIB_DIR})
188176
endif()
189177
endif()
190178
endif()

0 commit comments

Comments
 (0)