Skip to content

Commit 9e94f0f

Browse files
committed
Enable SWIFT_NATIVE_SWIFT_TOOLS_PATH on macOS
* Remove unneeded quotes in extra_flags * Use proper build_arch value on macOS
1 parent d65f3fa commit 9e94f0f

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

.github/workflows/build-toolchain.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -436,7 +436,7 @@ jobs:
436436
"cxxflags": "${{ steps.context.outputs.DARWIN_CMAKE_CXX_FLAGS }}",
437437
"swiftflags": "${{ steps.context.outputs.DARWIN_CMAKE_Swift_FLAGS }}",
438438
"cmake_linker_flags": "-D CMAKE_EXE_LINKER_FLAGS=\"${{ steps.context.outputs.DARWIN_CMAKE_EXE_LINKER_FLAGS }}\" -D CMAKE_SHARED_LINKER_FLAGS=\"${{ steps.context.outputs.DARWIN_CMAKE_SHARED_LINKER_FLAGS }}\"",
439-
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=x86_64 -D CMAKE_OSX_DEPLOYMENT_TARGET=\"15.0\" -D CMAKE_OSX_ARCHITECTURES=x86_64"
439+
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=x86_64 -D CMAKE_OSX_DEPLOYMENT_TARGET=15.0 -D CMAKE_OSX_ARCHITECTURES=x86_64"
440440
},
441441
{
442442
"arch": "arm64",
@@ -447,7 +447,7 @@ jobs:
447447
"cxxflags": "${{ steps.context.outputs.DARWIN_CMAKE_CXX_FLAGS }}",
448448
"swiftflags": "${{ steps.context.outputs.DARWIN_CMAKE_Swift_FLAGS }}",
449449
"cmake_linker_flags": "-D CMAKE_EXE_LINKER_FLAGS=${{ steps.context.outputs.DARWIN_CMAKE_EXE_LINKER_FLAGS }} -D CMAKE_SHARED_LINKER_FLAGS=${{ steps.context.outputs.DARWIN_CMAKE_SHARED_LINKER_FLAGS }}",
450-
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=arm64 -D CMAKE_OSX_DEPLOYMENT_TARGET=\"15.0\" -D CMAKE_OSX_ARCHITECTURES=arm64"
450+
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=arm64 -D CMAKE_OSX_DEPLOYMENT_TARGET=15.0 -D CMAKE_OSX_ARCHITECTURES=arm64"
451451
}
452452
]
453453
}
@@ -462,7 +462,7 @@ jobs:
462462
"cxx": "clang++",
463463
"cxxflags": "${{ steps.context.outputs.DARWIN_CMAKE_CXX_FLAGS }}",
464464
"swiftflags": "${{ steps.context.outputs.DARWIN_CMAKE_Swift_FLAGS }}",
465-
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=arm64 -D CMAKE_OSX_DEPLOYMENT_TARGET=\"15.0\" -D CMAKE_OSX_ARCHITECTURES=arm64"
465+
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=arm64 -D CMAKE_OSX_DEPLOYMENT_TARGET=15.0 -D CMAKE_OSX_ARCHITECTURES=arm64"
466466
}
467467
]
468468
}
@@ -477,7 +477,7 @@ jobs:
477477
"cxx": "clang++",
478478
"cxxflags": "${{ steps.context.outputs.DARWIN_CMAKE_CXX_FLAGS }}",
479479
"swiftflags": "${{ steps.context.outputs.DARWIN_CMAKE_Swift_FLAGS }}",
480-
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=x86_64 -D CMAKE_OSX_DEPLOYMENT_TARGET=\"15.0\" -D CMAKE_OSX_ARCHITECTURES=x86_64"
480+
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=x86_64 -D CMAKE_OSX_DEPLOYMENT_TARGET=15.0 -D CMAKE_OSX_ARCHITECTURES=x86_64"
481481
},
482482
{
483483
"arch": "arm64",
@@ -487,7 +487,7 @@ jobs:
487487
"cxx": "clang++",
488488
"cxxflags": "${{ steps.context.outputs.DARWIN_CMAKE_CXX_FLAGS }}",
489489
"swiftflags": "${{ steps.context.outputs.DARWIN_CMAKE_Swift_FLAGS }}",
490-
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=arm64 -D CMAKE_OSX_DEPLOYMENT_TARGET=\"15.0\" -D CMAKE_OSX_ARCHITECTURES=arm64"
490+
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=arm64 -D CMAKE_OSX_DEPLOYMENT_TARGET=15.0 -D CMAKE_OSX_ARCHITECTURES=arm64"
491491
}
492492
]
493493
}
@@ -578,7 +578,7 @@ jobs:
578578
uses: ./.github/workflows/swift-toolchain.yml
579579
with:
580580
build_os: Darwin
581-
build_arch: aarch64
581+
build_arch: arm64
582582
build_matrix: ${{ needs.context.outputs.darwin_build_matrix }}
583583
host_matrix: ${{ needs.context.outputs.darwin_host_matrix }}
584584
target_matrix: ${{ needs.context.outputs.darwin_target_matrix }}

.github/workflows/swift-toolchain.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -829,7 +829,7 @@ jobs:
829829
path: ${{ github.workspace }}/BinaryCache/Library/cmark-gfm-${{ inputs.swift_cmark_version }}/usr
830830
- uses: thebrowsercompany/gha-download-tar-artifact@717214c9da2a52b3407a5b0a1f31c00b8fde5681 # main
831831
with:
832-
name: early-swift-driver-${{ matrix.os }}-${{ matrix.arch }}
832+
name: early-swift-driver-${{ inputs.build_os }}-${{ inputs.build_arch }}
833833
path: ${{ github.workspace }}/BinaryCache/swift-driver
834834

835835
- uses: actions/checkout@v4
@@ -943,13 +943,13 @@ jobs:
943943
run: |
944944
$CxxFlags = "${{ matrix.cxxflags }}"
945945
$SwiftFlags = ""
946+
$EXTRA_FLAGS = @()
946947
947948
if ( "${{ matrix.os }}" -eq "Windows" ) {
948949
$SWIFTC = cygpath -m (Get-Command swiftc).Source
949950
# Use toolchain clang to avoid broken __prefetch intrinsic on arm64 in Clang 18.
950951
# TODO: Use llvm-19 when available. See https://github.com/compnerd/swift-build/issues/846
951952
$CLANG_LOCATION = Split-Path (Get-Command swiftc).Source
952-
$EXTRA_FLAGS = @("-D", "SWIFT_NATIVE_SWIFT_TOOLS_PATH=${{ github.workspace }}/BinaryCache/0/bin")
953953
if ( "${{ matrix.arch }}" -eq "arm64" ) {
954954
$EXTRA_FLAGS += "${{ matrix.extra_flags }}".Split()
955955
$EXTRA_FLAGS += @("-D", "CMAKE_SYSTEM_NAME=Windows")
@@ -986,7 +986,7 @@ jobs:
986986
$CXX = Join-Path $CLANG_LOCATION "clang++"
987987
$CACHE = "${{ github.workspace }}/SourceCache/swift/cmake/caches/Darwin-${{ matrix.arch }}.cmake"
988988
$SDKROOT = xcrun --sdk macosx --show-sdk-path
989-
$EXTRA_FLAGS = "${{ matrix.extra_flags }}".Split()
989+
$EXTRA_FLAGS += "${{ matrix.extra_flags }}".Split()
990990
$EXTRA_FLAGS += @("-D", "CMAKE_SYSTEM_NAME=Darwin")
991991
# TODO: Use early-swift-driver on Windows too.
992992
$EXTRA_FLAGS += @("-D", "SWIFT_EARLY_SWIFT_DRIVER_BUILD=${{ github.workspace }}/BinaryCache/swift-driver/bin")
@@ -1035,6 +1035,7 @@ jobs:
10351035
-D SWIFT_ENABLE_EXPERIMENTAL_OBSERVATION=YES `
10361036
-D SWIFT_ENABLE_EXPERIMENTAL_STRING_PROCESSING=YES `
10371037
-D SWIFT_ENABLE_SYNCHRONIZATION=YES `
1038+
-D SWIFT_NATIVE_SWIFT_TOOLS_PATH=${{ github.workspace }}/BinaryCache/0/bin `
10381039
-D SWIFT_PATH_TO_LIBDISPATCH_SOURCE=${{ github.workspace }}/SourceCache/swift-corelibs-libdispatch `
10391040
-D SWIFT_PATH_TO_SWIFT_SYNTAX_SOURCE=${{ github.workspace }}/SourceCache/swift-syntax `
10401041
-D SWIFT_PATH_TO_STRING_PROCESSING_SOURCE=${{ github.workspace }}/SourceCache/swift-experimental-string-processing `

0 commit comments

Comments
 (0)