Skip to content

Commit 7f02653

Browse files
authored
Merge pull request #41327 from gottesmm/pr-5939707c6d758909b99bd67cd865a5f1adf02128
Rather than turning off sil-verify-all on all stdlib builds... Re-enable it only on the macOS x86_64 platform.
2 parents 1b9fe44 + 392e02d commit 7f02653

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

stdlib/cmake/modules/SwiftSource.cmake

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -459,8 +459,8 @@ function(_compile_swift_files
459459

460460
if(SWIFT_SIL_VERIFY_ALL_MACOS_ONLY)
461461
# Only add if we have a macOS build triple
462-
if (STREQUAL "${SWIFTFILE_SDK}" "OSX" AND
463-
STREQUAL "${SWIFTFILE_ARCHITECTURE}" "x86_64")
462+
if ("${SWIFTFILE_SDK}" STREQUAL "OSX" AND
463+
"${SWIFTFILE_ARCHITECTURE}" STREQUAL "x86_64")
464464
list(APPEND swift_flags "-Xfrontend" "-sil-verify-all")
465465
endif()
466466
endif()

utils/build-presets.ini

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1601,6 +1601,9 @@ skip-build-tvos
16011601
skip-test-tvos
16021602
skip-test-tvos-host
16031603

1604+
# Run the SIL verifier after each transform when building swift files
1605+
sil-verify-all-macos-only
1606+
16041607
# Don't run host tests for iOS, tvOS and watchOS platforms to make the build
16051608
# faster.
16061609
skip-test-ios-host

0 commit comments

Comments
 (0)