Skip to content

Commit b73f2a8

Browse files
authored
build-script-impl: remove get_stdlib_targets_for_host as unused (#62238)
This function is declared, but not used anywhere. There are no references to it anywhere else in the codebase or documentation.
1 parent 38a10bb commit b73f2a8

File tree

1 file changed

+1
-21
lines changed

1 file changed

+1
-21
lines changed

utils/build-script-impl

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1207,26 +1207,6 @@ function host_has_darwin_symbols() {
12071207
esac
12081208
}
12091209

1210-
function get_stdlib_targets_for_host() {
1211-
# Don't build the stdlib in the Xcode train for host and cross-compilations host.
1212-
if [[ "${STDLIB_DEPLOYMENT_TARGETS[@]}" == "" ]]; then
1213-
return 0
1214-
fi
1215-
1216-
# FIXME: STDLIB_DEPLOYMENT_TARGETS argument assumed to apply when Host == Build
1217-
# Cross-compile Hosts are only built with their native standard libraries.
1218-
# To fix this, we would need to pass in a list of stdlib targets _per host_,
1219-
# and the SWIFT_SDKS parameters would need to be able to capture both the SDK
1220-
# and architecture of each stdlib target -- currently it only captures the SDK.
1221-
#
1222-
# We turn these targets in to SWIFT_SDKS in `calculate_targets_for_host()`
1223-
if [[ $(is_cross_tools_host $1) ]] ; then
1224-
echo "$1"
1225-
else
1226-
echo "${STDLIB_DEPLOYMENT_TARGETS[@]}"
1227-
fi
1228-
}
1229-
12301210
#
12311211
# Calculate source directories for each product.
12321212
#
@@ -3183,7 +3163,7 @@ for host in "${ALL_HOSTS[@]}"; do
31833163
fi
31843164

31853165
# We can only run tests built for the host machine, because
3186-
# cross-compiled hosts only build their native target. See: get_stdlib_targets_for_host()
3166+
# cross-compiled hosts only build their native target.
31873167
if [[ $(is_cross_tools_host ${host}) ]]; then
31883168
echo "--- Can't execute tests for ${host}, skipping... ---"
31893169
continue

0 commit comments

Comments
 (0)