Skip to content

Commit 6d48585

Browse files
Pass -ivfsoverlay windows-vfs-overlay.yaml to swift-ide-test only when testing for Windows host platform
1 parent fb8bf53 commit 6d48585

File tree

1 file changed

+22
-22
lines changed

1 file changed

+22
-22
lines changed

test/lit.cfg

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -395,28 +395,6 @@ lit_config.note('Using resource dir: ' + test_resource_dir)
395395
test_clang_resource_dir = lit.util.executeCommand([config.clang, '-print-resource-dir'])[0].rstrip()
396396
lit_config.note('Using Clang resource dir: ' + test_clang_resource_dir)
397397

398-
config.swift_system_overlay_opt = ""
399-
config.clang_system_overlay_opt = ""
400-
config.windows_vfs_overlay_opt = ""
401-
if kIsWindows:
402-
# ucrt.modulemap currently requires -fbuiltin-headers-in-system-modules. -strict-implicit-module-context
403-
# is necessary for -Xcc arguments to be passed through ModuleInterfaceLoader.
404-
config.swift_system_overlay_opt = "-vfsoverlay {} -strict-implicit-module-context -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules".format(
405-
os.path.join(config.swift_obj_root, "stdlib", "windows-vfs-overlay.yaml")
406-
)
407-
# this variant is for extract-symbolgraph which doesn't accept all the same arugments as swiftc
408-
# so the extra ceremony lets us pass in the relevant pieces needed for Windows SDK access.
409-
config.windows_vfs_overlay_opt = "-Xcc -vfsoverlay -Xcc {} -Xcc -strict-implicit-module-context -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules".format(
410-
os.path.join(config.swift_obj_root, "stdlib", "windows-vfs-overlay.yaml")
411-
)
412-
config.clang_system_overlay_opt = "-Xcc -ivfsoverlay -Xcc {} -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules".format(
413-
os.path.join(config.swift_obj_root, "stdlib", "windows-vfs-overlay.yaml")
414-
)
415-
config.substitutions.append( ('%windows_vfs_overlay_opt', config.windows_vfs_overlay_opt) )
416-
stdlib_resource_dir_opt = config.resource_dir_opt
417-
sourcekitd_framework_dir = config.swift_lib_dir
418-
config.substitutions.append( ('%test-resource-dir', test_resource_dir) )
419-
420398
# Parse the variant triple.
421399
# FIXME: We ought to parse 'run_environment' separately from 'run_os'.
422400
(run_cpu, run_vendor, run_os, run_vers) = re.match('([^-]+)-([^-]+)-([^0-9]+)(.*)', config.variant_triple).groups()
@@ -479,6 +457,28 @@ if test_sdk_overlay_dir is not None:
479457
lit_config.note('Using SDK overlay dir: ' + test_sdk_overlay_dir)
480458
config.resource_dir_opt += (" %s" % sdk_overlay_dir_opt)
481459

460+
config.swift_system_overlay_opt = ""
461+
config.clang_system_overlay_opt = ""
462+
config.windows_vfs_overlay_opt = ""
463+
if kIsWindows and run_os in ['windows-msvc']:
464+
# ucrt.modulemap currently requires -fbuiltin-headers-in-system-modules. -strict-implicit-module-context
465+
# is necessary for -Xcc arguments to be passed through ModuleInterfaceLoader.
466+
config.swift_system_overlay_opt = "-vfsoverlay {} -strict-implicit-module-context -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules".format(
467+
os.path.join(config.swift_obj_root, "stdlib", "windows-vfs-overlay.yaml")
468+
)
469+
# this variant is for extract-symbolgraph which doesn't accept all the same arugments as swiftc
470+
# so the extra ceremony lets us pass in the relevant pieces needed for Windows SDK access.
471+
config.windows_vfs_overlay_opt = "-Xcc -vfsoverlay -Xcc {} -Xcc -strict-implicit-module-context -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules".format(
472+
os.path.join(config.swift_obj_root, "stdlib", "windows-vfs-overlay.yaml")
473+
)
474+
config.clang_system_overlay_opt = "-Xcc -ivfsoverlay -Xcc {} -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules".format(
475+
os.path.join(config.swift_obj_root, "stdlib", "windows-vfs-overlay.yaml")
476+
)
477+
config.substitutions.append( ('%windows_vfs_overlay_opt', config.windows_vfs_overlay_opt) )
478+
stdlib_resource_dir_opt = config.resource_dir_opt
479+
sourcekitd_framework_dir = config.swift_lib_dir
480+
config.substitutions.append( ('%test-resource-dir', test_resource_dir) )
481+
482482
# Default to Swift 4 for now.
483483
# Note that this accepts both `--param swift-version` (like the compiler flag)
484484
# and `--param swift_version` (like a lit configuration parameter).

0 commit comments

Comments
 (0)