Skip to content

Commit 1ab968d

Browse files
authored
[BuildSystem] Stop building for i386-watch-simulator (#77692)
* [BuildSystem] Stop building for i386-watch-simulator In Xcode16 it is not supported.
1 parent 9eaad63 commit 1ab968d

File tree

10 files changed

+5
-56
lines changed

10 files changed

+5
-56
lines changed

cmake/modules/DarwinSDKs.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ set(SUPPORTED_IOS_SIMULATOR_ARCHS "x86_64;arm64")
33
set(SUPPORTED_TVOS_ARCHS "arm64")
44
set(SUPPORTED_TVOS_SIMULATOR_ARCHS "x86_64;arm64")
55
set(SUPPORTED_WATCHOS_ARCHS "armv7k;arm64_32")
6-
set(SUPPORTED_WATCHOS_SIMULATOR_ARCHS "i386;x86_64;arm64")
6+
set(SUPPORTED_WATCHOS_SIMULATOR_ARCHS "x86_64;arm64")
77
set(SUPPORTED_OSX_ARCHS "x86_64;arm64")
88
set(SUPPORTED_XROS_ARCHS "arm64;arm64e")
99
set(SUPPORTED_XROS_SIMULATOR_ARCHS "arm64")

cmake/modules/SwiftConfigureSDK.cmake

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,10 +91,6 @@ function(remove_sdk_unsupported_archs name os sdk_path deployment_version archit
9191
# 32-bit watchOS is not listed explicitly in SDK settings.
9292
message(STATUS "Assuming ${name} SDK at ${sdk_path} supports architecture ${arch}")
9393
list(APPEND architectures ${arch})
94-
elseif(arch STREQUAL "i386" AND os STREQUAL "watchsimulator")
95-
# 32-bit watchOS simulator is not listed explicitly in SDK settings.
96-
message(STATUS "Assuming ${name} SDK at ${sdk_path} supports architecture ${arch}")
97-
list(APPEND architectures ${arch})
9894
else()
9995
message(STATUS "${name} SDK at ${sdk_path} does not support architecture ${arch}")
10096
endif()

utils/build-script-impl

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -485,7 +485,6 @@ function verify_host_is_supported() {
485485
| appletvsimulator-x86_64 \
486486
| appletvsimulator-arm64 \
487487
| appletvos-arm64 \
488-
| watchsimulator-i386 \
489488
| watchsimulator-x86_64 \
490489
| watchsimulator-arm64 \
491490
| watchos-armv7k \
@@ -664,13 +663,6 @@ function set_build_options_for_host() {
664663
SWIFT_HOST_VARIANT_SDK="TVOS"
665664
cmake_osx_deployment_target=""
666665
;;
667-
watchsimulator-i386)
668-
SWIFT_HOST_TRIPLE="i386-apple-watchos${DARWIN_DEPLOYMENT_VERSION_WATCHOS}-simulator"
669-
llvm_target_arch="X86"
670-
671-
SWIFT_HOST_VARIANT_SDK="WATCHOS_SIMULATOR"
672-
cmake_osx_deployment_target=""
673-
;;
674666
watchsimulator-x86_64)
675667
SWIFT_HOST_TRIPLE="x86_64-apple-watchos${DARWIN_DEPLOYMENT_VERSION_WATCHOS}-simulator"
676668
llvm_target_arch="X86"

utils/build_swift/build_swift/driver_arguments.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1268,10 +1268,6 @@ def create_argument_parser():
12681268
option('--skip-test-ios-simulator',
12691269
toggle_false('test_ios_simulator'),
12701270
help='skip testing iOS simulator targets')
1271-
option('--skip-test-watchos-32bit-simulator',
1272-
toggle_false('test_watchos_32bit_simulator'),
1273-
default=False,
1274-
help='skip testing watchOS 32 bit simulator targets')
12751271
option('--skip-test-ios-host',
12761272
toggle_false('test_ios_host'),
12771273
help='skip testing iOS device targets on the host machine (the '

utils/build_swift/tests/expected_options.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,6 @@
292292
'test_cygwin': False,
293293
'test_freebsd': False,
294294
'test_ios': False,
295-
'test_watchos_32bit_simulator': False,
296295
'test_ios_host': False,
297296
'test_ios_simulator': False,
298297
'test_linux': False,
@@ -736,8 +735,6 @@ class BuildScriptImplOption(_BaseOption):
736735
DisableOption('--skip-test-cygwin', dest='test_cygwin'),
737736
DisableOption('--skip-test-freebsd', dest='test_freebsd'),
738737
DisableOption('--skip-test-ios', dest='test_ios'),
739-
DisableOption('--skip-test-watchos-32bit-simulator',
740-
dest='test_watchos_32bit_simulator'),
741738
DisableOption('--skip-test-ios-host', dest='test_ios_host'),
742739
DisableOption('--skip-test-ios-simulator', dest='test_ios_simulator'),
743740
DisableOption('--skip-test-linux', dest='test_linux'),

utils/sil-opt-verify-all-modules.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,6 @@ def get_verify_resource_dir_modules_commands(
6363
('iphonesimulator', 'x86_64', 'x86_64-apple-ios7.0'),
6464
('macosx', 'x86_64', 'x86_64-apple-macosx10.9'),
6565
('watchos', 'armv7k', 'armv7k-apple-watchos2.0'),
66-
('watchsimulator', 'i386', 'i386-apple-watchos2.0'),
6766
]
6867

6968
commands = []

utils/swift_build_support/swift_build_support/host_specific_configuration.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -323,9 +323,6 @@ def __platforms_to_skip_test(self, args, stage_dependent_args):
323323

324324
def __platforms_archs_to_skip_test(self, args, stage_dependent_args, host_target):
325325
platforms_archs_to_skip_test = set()
326-
if not stage_dependent_args.test_watchos_32bit_simulator:
327-
platforms_archs_to_skip_test.add(
328-
StdlibDeploymentTarget.AppleWatchSimulator.i386)
329326
if host_target == StdlibDeploymentTarget.OSX.x86_64.name:
330327
platforms_archs_to_skip_test.add(
331328
StdlibDeploymentTarget.iOSSimulator.arm64)
@@ -341,8 +338,6 @@ def __platforms_archs_to_skip_test(self, args, stage_dependent_args, host_target
341338
StdlibDeploymentTarget.iOSSimulator.x86_64)
342339
platforms_archs_to_skip_test.add(
343340
StdlibDeploymentTarget.AppleTVSimulator.x86_64)
344-
platforms_archs_to_skip_test.add(
345-
StdlibDeploymentTarget.AppleWatchSimulator.i386)
346341
platforms_archs_to_skip_test.add(
347342
StdlibDeploymentTarget.AppleWatchSimulator.x86_64)
348343

utils/swift_build_support/swift_build_support/products/cmake_product.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -303,13 +303,6 @@ def host_cmake_options(self, host_target):
303303
swift_host_variant_sdk = 'TVOS'
304304
cmake_osx_deployment_target = None
305305

306-
elif host_target == 'watchsimulator-i386':
307-
swift_host_triple = 'i386-apple-watchos{}-simulator'.format(
308-
self.args.darwin_deployment_version_watchos)
309-
llvm_target_arch = 'X86'
310-
swift_host_variant_sdk = 'WATCHOS_SIMULATOR'
311-
cmake_osx_deployment_target = None
312-
313306
elif host_target == 'watchsimulator-x86_64':
314307
swift_host_triple = 'x86_64-apple-watchos{}-simulator'.format(
315308
self.args.darwin_deployment_version_watchos)

utils/swift_build_support/swift_build_support/targets.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ class StdlibDeploymentTarget(object):
258258
sdk_name="WATCHOS")
259259

260260
AppleWatchSimulator = DarwinPlatform("watchsimulator",
261-
archs=["i386", "x86_64", "arm64"],
261+
archs=["x86_64", "arm64"],
262262
sdk_name="WATCHOS_SIMULATOR",
263263
is_simulator=True)
264264

utils/swift_build_support/tests/test_host_specific_configuration.py

Lines changed: 3 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -191,9 +191,9 @@ def test(self):
191191
'build_watchos_device')
192192
test_should_skip_building_watchos_sim =\
193193
generate_should_skip_building_platform(
194-
'watchsimulator-i386',
194+
'watchsimulator-x86_64',
195195
'WATCHOS_SIMULATOR',
196-
'swift-test-stdlib-watchsimulator-i386',
196+
'swift-test-stdlib-watchsimulator-x86_64',
197197
'build_watchos_simulator')
198198

199199
def generate_should_build_full_targets_when_test(test_arg_name):
@@ -290,28 +290,10 @@ def test(self):
290290
# NOTE: test_watchos_host is not supported in open-source Swift
291291
test_should_skip_testing_watchos_sim =\
292292
generate_should_skip_testing_platform(
293-
'watchsimulator-i386',
293+
'watchsimulator-x86_64',
294294
'build_watchos_simulator',
295295
'test_watchos_simulator')
296296

297-
def test_should_skip_testing_32bit_watchos(self):
298-
host_target = 'watchsimulator-i386'
299-
args = self.default_args()
300-
args.build_watchos_simulator = True
301-
args.test_watchos_simulator = True
302-
args.test_watchos_32bit_simulator = False
303-
args.host_target = host_target
304-
args.stdlib_deployment_targets = [host_target]
305-
args.build_stdlib_deployment_targets = 'all'
306-
307-
before = HostSpecificConfiguration(host_target, args)
308-
self.assertEqual(len(before.swift_test_run_targets), 0)
309-
310-
args.test_watchos_32bit_simulator = True
311-
after = HostSpecificConfiguration(host_target, args)
312-
self.assertIn('check-swift-watchsimulator-i386',
313-
after.swift_test_run_targets)
314-
315297
def generate_should_allow_testing_only_host(
316298
host_target, build_arg_name, test_arg_name, host_test_arg_name):
317299
def test(self):
@@ -681,7 +663,6 @@ def default_args(self):
681663
test_freebsd=False,
682664
test_ios_host=False,
683665
test_ios_simulator=False,
684-
test_watchos_32bit_simulator=True,
685666
test_linux=False,
686667
test_optimize_for_size=False,
687668
test_optimize_none_with_implicit_dynamic=False,

0 commit comments

Comments
 (0)