Revert "[Concurrency] Build C only libdispatch before Swift on non-Darwin pla… (#35738)" (#35789)

This reverts commit 74e59ab.

Author: drexin

stdlib/public/Concurrency/CMakeLists.txt

@@ -19,17 +19,16 @@ set(LLVM_OPTIONAL_SOURCES
 set(swift_concurrency_link_libraries
   swiftCore)
 
-set(swift_concurrency_link_flags "${SWIFT_RUNTIME_SWIFT_LINK_FLAGS}")
-
 if(SWIFT_CONCURRENCY_USES_DISPATCH)
   if(NOT CMAKE_SYSTEM_NAME STREQUAL Darwin)
     include_directories(AFTER
                           ${SWIFT_PATH_TO_LIBDISPATCH_SOURCE})
-    link_directories(AFTER
-                        ${SWIFT_PATH_TO_LIBDISPATCH_BUILD})
 
-    list(APPEND swift_concurrency_link_flags
-      "-ldispatch")
+    # FIXME: we can't rely on libdispatch having been built for the
+    # target at this point in the process.  Currently, we're relying
+    # on soft-linking.
+    #list(APPEND swift_concurrency_link_libraries
+    #  dispatch)
   endif()
 endif()
 
@@ -68,6 +67,6 @@ add_swift_target_library(swift_Concurrency ${SWIFT_STDLIB_LIBRARY_BUILD_TYPES} I
     ${SWIFT_STANDARD_LIBRARY_SWIFT_FLAGS}
     -parse-stdlib
     -Xfrontend -enable-experimental-concurrency
-  LINK_FLAGS "${swift_concurrency_link_flags}"
+  LINK_FLAGS "${SWIFT_RUNTIME_SWIFT_LINK_FLAGS}"
   INSTALL_IN_COMPONENT stdlib
 )

utils/build-presets.ini

@@ -824,7 +824,6 @@ long-test
 stress-test
 test-optimized
 foundation
-clibdispatch
 libdispatch
 indexstore-db
 sourcekit-lsp
@@ -967,7 +966,6 @@ release
 test
 validation-test
 foundation
-clibdispatch
 libdispatch
 indexstore-db
 sourcekit-lsp
@@ -1067,7 +1065,6 @@ swiftpm
 swift-driver
 xctest
 foundation
-clibdispatch
 libdispatch
 swiftsyntax
 indexstore-db
@@ -1596,7 +1593,6 @@ mixin-preset=
     mixin_swiftpm_base
     mixin_linux_install_components_with_clang
 
-clibdispatch
 libdispatch
 libicu
 foundation
@@ -1845,7 +1841,6 @@ llbuild
 swiftpm
 xctest
 foundation
-clibdispatch
 libdispatch
 indexstore-db
 sourcekit-lsp
@@ -2461,7 +2456,6 @@ install-symroot=%(install_symroot)s
 mixin-preset=source_compat_suite_base
 build-subdir=compat_linux
 foundation
-clibdispatch
 libdispatch
 xctest
 install-foundation
@@ -2517,7 +2511,6 @@ no-assertions
 lldb
 foundation
 libicu
-clibdispatch
 libdispatch
 build-ninja
 

utils/build-script

@@ -318,10 +318,6 @@ def apply_default_arguments(toolchain, args):
         args.stdlib_deployment_targets = [
             target.name for target in stdlib_targets]
 
-    if (args.build_swift and
-            platform.system() != "Darwin"):
-        args.build_clibdispatch = True
-
     # SwiftPM and XCTest have a dependency on Foundation.
     # On OS X, Foundation is built automatically using xcodebuild.
     # On Linux, we must ensure that it is built manually.
@@ -628,7 +624,6 @@ class BuildScriptInvocation(object):
             (args.build_llbuild, "llbuild"),
             (args.build_libcxx, "libcxx"),
             (args.build_libdispatch, "libdispatch"),
-            (args.build_clibdispatch, "clibdispatch"),
             (args.build_libicu, "libicu")
         ]
         for (should_build, string_name) in conditional_subproject_configs:
@@ -870,8 +865,6 @@ class BuildScriptInvocation(object):
             impl_product_classes.append(products.LibCXX)
         if self.args.build_libicu:
             impl_product_classes.append(products.LibICU)
-        if self.args.build_clibdispatch:
-            impl_product_classes.append(products.CLibDispatch)
         if self.args.build_swift:
             impl_product_classes.append(products.Swift)
         if self.args.build_lldb:

utils/build-script-impl

@@ -230,7 +230,6 @@ KNOWN_SETTINGS=(
 )
 
 components=(
-  clibdispatch
   cmark
   foundation
   libcxx
@@ -1163,7 +1162,6 @@ STRESSTEST_PACKAGE_DIR="${WORKSPACE}/swift-stress-tester"
 XCTEST_SOURCE_DIR="${WORKSPACE}/swift-corelibs-xctest"
 FOUNDATION_SOURCE_DIR="${WORKSPACE}/swift-corelibs-foundation"
 FOUNDATION_STATIC_SOURCE_DIR="${WORKSPACE}/swift-corelibs-foundation"
-CLIBDISPATCH_SOURCE_DIR="${WORKSPACE}/swift-corelibs-libdispatch"
 LIBDISPATCH_SOURCE_DIR="${WORKSPACE}/swift-corelibs-libdispatch"
 LIBDISPATCH_STATIC_SOURCE_DIR="${WORKSPACE}/swift-corelibs-libdispatch"
 LIBICU_SOURCE_DIR="${WORKSPACE}/icu"
@@ -1177,7 +1175,6 @@ PRODUCTS=(llvm)
 [[ "${SKIP_BUILD_CMARK}" ]] || PRODUCTS+=(cmark)
 [[ "${SKIP_BUILD_LIBCXX}" ]] || PRODUCTS+=(libcxx)
 [[ "${SKIP_BUILD_LIBICU}" ]] || PRODUCTS+=(libicu)
-[[ "${SKIP_BUILD_CLIBDISPATCH}" ]] || PRODUCTS+=(clibdispatch)
 [[ "${SKIP_BUILD_SWIFT}" ]] || PRODUCTS+=(swift)
 [[ "${SKIP_BUILD_LLDB}" ]] || PRODUCTS+=(lldb)
 [[ "${SKIP_BUILD_LIBDISPATCH}" ]] || PRODUCTS+=(libdispatch)
@@ -1262,7 +1259,7 @@ function build_directory_bin() {
             foundation|foundation_static)
                 echo "${root}/${FOUNDATION_BUILD_TYPE}/bin"
                 ;;
-            clibdispatch|libdispatch|libdispatch_static)
+            libdispatch|libdispatch_static)
                 echo "${root}/${LIBDISPATCH_BUILD_TYPE}/bin"
                 ;;
             libicu)
@@ -1380,7 +1377,7 @@ function cmake_config_opt() {
             foundation|foundation_static)
                 echo "--config ${FOUNDATION_BUILD_TYPE}"
                 ;;
-            clibdispatch|libdispatch|libdispatch_static)
+            libdispatch|libdispatch_static)
                 echo "--config ${LIBDISPATCH_BUILD_TYPE}"
                 ;;
             libicu)
@@ -1767,6 +1764,7 @@ for host in "${ALL_HOSTS[@]}"; do
                 ;;
 
             swift)
+
                 if [[ "${ANDROID_API_LEVEL}" ]]; then
                     cmake_options=(
                         "${cmake_options[@]}"
@@ -2322,18 +2320,9 @@ for host in "${ALL_HOSTS[@]}"; do
                 )
 
                 ;;
-            clibdispatch|libdispatch|libdispatch_static)
+            libdispatch|libdispatch_static)
                 LIBDISPATCH_BUILD_DIR=$(build_directory ${host} ${product})
                 SWIFT_BUILD_PATH="$(build_directory ${host} swift)"
-                ENABLE_SWIFT=YES
-                if [[ "${product}" = "clibdispatch" ]]; then
-                  if [[ "$(uname -s)" = "Darwin" ]]; then
-                    continue # We don't build clibdispatch on Darwin
-                  fi
-                  ENABLE_SWIFT=NO
-                  LIBDISPATCH_BUILD_DIR=$(build_directory ${host} libdispatch)
-                  build_dir=${LIBDISPATCH_BUILD_DIR}
-                fi
 
                 case "${host}" in
                 macosx-*)
@@ -2349,7 +2338,7 @@ for host in "${ALL_HOSTS[@]}"; do
                   call rm -rf "${LIBDISPATCH_BUILD_DIR}"
 
                   cmake_options=(
-                    -DENABLE_SWIFT="${ENABLE_SWIFT}"
+                    -DENABLE_SWIFT=YES
                     ${cmake_options[@]}
                     -DCMAKE_BUILD_TYPE:STRING="${LIBDISPATCH_BUILD_TYPE}"
                     -DCMAKE_C_COMPILER:PATH="${CLANG_BIN}/clang"
@@ -2517,13 +2506,6 @@ for host in "${ALL_HOSTS[@]}"; do
         if [[ "${product}" = "llvm" ]] && [[ "${BUILD_LLVM}" = "1" ]] && [[ "$(uname -s)" = "Darwin" ]]; then
             copy_embedded_compiler_rt_builtins_from_darwin_host_toolchain "$(build_directory_bin ${host} llvm)/.."
         fi
-
-        # We need to copy libdispatch.so into the swift lib directory
-        # so the tests can find it
-        if [[ "${product}" = "clibdispatch" ]] && [[ "uname -s" != "Darwin" ]]; then
-            clibdispatch_libdir="$(build_directory ${host} swift)/lib/swift/${SWIFT_HOST_VARIANT}/${SWIFT_HOST_VARIANT_ARCH}"
-            call cp "${build_dir}/libdispatch.so" "${clibdispatch_libdir}"
-        fi
     done
 done
 # END OF BUILD PHASE
@@ -2780,17 +2762,13 @@ for host in "${ALL_HOSTS[@]}"; do
             foundation_static)
               continue
             ;;
-            clibdispatch|libdispatch)
+            libdispatch)
                 if [[ "${SKIP_TEST_LIBDISPATCH}" ]]; then
                     continue
                 fi
 
                 case "${host}" in
                 macosx-*)
-                  if [[ "${product}" = clibdispatch ]]; then
-                    # we don't build clibdispatch on Darwin
-                    continue
-                  fi
                   LIBDISPATCH_BUILD_DIR=$(build_directory ${host} ${product})
                   echo "--- Running tests for ${product} ---"
                   with_pushd "${LIBDISPATCH_BUILD_DIR}" \
@@ -2799,9 +2777,6 @@ for host in "${ALL_HOSTS[@]}"; do
                   continue
                 ;;
                 *)
-                  if [[ "${product}" = clibdispatch ]]; then
-                    product=libdispatch
-                  fi
                   results_targets=( "test" )
                   executable_target=""
                 ;;
@@ -2976,21 +2951,13 @@ for host in "${ALL_HOSTS[@]}"; do
                 fi
 
                 ;;
-            clibdispatch|libdispatch|libdispatch_static)
-                if [[ "${product}" = clibdispatch ]]; then
-                  if [[ -z "${INSTALL_CLIBDISPATCH}" ]]; then
-                    continue
-                  fi
-                elif [[ -z "${INSTALL_LIBDISPATCH}" ]]; then
+            libdispatch|libdispatch_static)
+                if [[ -z "${INSTALL_LIBDISPATCH}" ]] ; then
                     continue
                 fi
 
                 case "${host}" in
                 macosx-*)
-                  if [[ "${product}" = clibdispatch ]]; then
-                    # we don't build clibdispatch on Darwin
-                    continue
-                  fi
                   if [[ -z "${INSTALL_DESTDIR}" ]] ; then
                       echo "--install-destdir is required to install products."
                       exit 1
@@ -3009,9 +2976,6 @@ for host in "${ALL_HOSTS[@]}"; do
                   continue
                 ;;
                 *)
-                  if [[ "${product}" = clibdispatch ]]; then
-                    product=libdispatch
-                  fi
                 ;;
                 esac
                 ;;

utils/build_swift/build_swift/driver_arguments.py

@@ -652,9 +652,6 @@ def create_argument_parser():
     option('--libdispatch', toggle_true('build_libdispatch'),
            help='build libdispatch')
 
-    option('--clibdispatch', toggle_true('build_clibdispatch'),
-           help='build clibdispatch')
-
     option('--libicu', toggle_true('build_libicu'),
            help='build libicu')
 

utils/build_swift/tests/expected_options.py

@@ -70,7 +70,6 @@
     'build_ios_simulator': False,
     'build_jobs': multiprocessing.cpu_count(),
     'build_libdispatch': False,
-    'build_clibdispatch': False,
     'build_libicu': False,
     'build_linux': True,
     'build_llbuild': False,
@@ -523,7 +522,6 @@ class BuildScriptImplOption(_BaseOption):
     EnableOption('--only-executable-test'),
     EnableOption('--only-non-executable-test'),
     EnableOption('--libdispatch', dest='build_libdispatch'),
-    EnableOption('--clibdispatch', dest='build_clibdispatch'),
     EnableOption('--libicu', dest='build_libicu'),
     EnableOption('--indexstore-db', dest='build_indexstoredb'),
     EnableOption('--test-indexstore-db-sanitize-all',

utils/swift_build_support/swift_build_support/products/__init__.py

@@ -11,7 +11,6 @@
 # ----------------------------------------------------------------------------
 
 from .benchmarks import Benchmarks
-from .clibdispatch import CLibDispatch
 from .cmark import CMark
 from .foundation import Foundation
 from .indexstoredb import IndexStoreDB
@@ -40,7 +39,6 @@
     'Ninja',
     'Foundation',
     'LibCXX',
-    'CLibDispatch',
     'LibDispatch',
     'LibICU',
     'LLBuild',

utils/swift_build_support/swift_build_support/products/clibdispatch.py

@@ -10,7 +10,6 @@
 #
 # ----------------------------------------------------------------------------
 
-from . import clibdispatch
 from . import cmark
 from . import libcxx
 from . import libicu
@@ -148,5 +147,4 @@ def get_dependencies(cls):
         return [cmark.CMark,
                 llvm.LLVM,
                 libcxx.LibCXX,
-                libicu.LibICU,
-                clibdispatch.CLibDispatch]
+                libicu.LibICU]