Build-script: propagate the build type to the xctest build script

Author: jamala

utils/build-script

@@ -594,6 +594,7 @@ class BuildScriptInvocation(object):
             "--swift-stdlib-build-type", args.swift_stdlib_build_variant,
             "--lldb-build-type", args.lldb_build_variant,
             "--foundation-build-type", args.foundation_build_variant,
+            "--xctest-build-type", args.build_variant,
             "--llvm-enable-assertions", str(args.llvm_assertions).lower(),
             "--swift-enable-assertions", str(args.swift_assertions).lower(),
             "--swift-stdlib-enable-assertions", str(

utils/build-script-impl

@@ -85,6 +85,7 @@ KNOWN_SETTINGS=(
     lldb-build-type             "Debug"          "the CMake build variant for LLDB"
     llbuild-build-type          "Debug"          "the CMake build variant for llbuild"
     foundation-build-type       "Debug"          "the build variant for Foundation"
+    xctest-build-type           "Debug"          "the build variant for xctest"
     llbuild-enable-assertions   "1"              "enable assertions in llbuild"
     enable-asan                 ""               "enable Address Sanitizer"
     enable-ubsan                ""               "enable Undefined Behavior Sanitizer"
@@ -2104,13 +2105,19 @@ for host in "${ALL_HOSTS[@]}"; do
                     copy_swift_stdlib_tool_substitute "$(build_directory_bin ${host} swift)/swift-stdlib-tool"
                 fi
 
-                # FIXME: Use XCTEST_BUILD_TYPE (which is never properly
-                #        set) to build either --debug or --release.
+                # Use XCTEST_BUILD_TYPE to build either --debug or --release.
+                if [[ "${XCTEST_BUILD_TYPE}" ==  "Debug" ]] ; then
+                    XCTEST_BUILD_ARGS="--debug"
+                else
+                    XCTEST_BUILD_ARGS="--release"
+                fi
+
                 call "${XCTEST_SOURCE_DIR}"/build_script.py \
                     --swiftc="${SWIFTC_BIN}" \
                     --build-dir="${XCTEST_BUILD_DIR}" \
                     --foundation-build-dir="${FOUNDATION_BUILD_DIR}/Foundation" \
-                    $LIBDISPATCH_BUILD_ARGS
+                    $LIBDISPATCH_BUILD_ARGS \
+                    $XCTEST_BUILD_ARGS
 
                 # XCTest builds itself and doesn't rely on cmake
                 continue