-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[CMake] Make swift-frontend the primary Swift binary. #28003
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CMake] Make swift-frontend the primary Swift binary. #28003
Conversation
@swift-ci please test |
@swift-ci please test platform windows |
Build failed |
Build failed |
034daa8
to
409509f
Compare
@swift-ci please test |
@swift-ci please test platform Windows |
Build failed |
Build failed |
@swift-ci please test Windows platform |
How can I track down the results when we run these tests? |
https://ci-external.swift.org/view/Pull%20Request/job/swift-PR-windows/160/ (but I don't know why it wasn't reported back here). |
82ba0c4
to
28d1b23
Compare
@swift-ci please test |
Build failed |
28d1b23
to
cc23d4d
Compare
@swift-ci test |
@swift-ci test source compatibility |
Build failed |
Build failed |
@swift-ci please test |
Build failed |
Build failed |
@swift-ci please test |
1 similar comment
@swift-ci please test |
Build failed |
@swift-ci please test |
@swift-ci test macOS |
Build failed |
Build failed |
Build failed |
@swift-ci please test source compatibility |
@swift-ci please smoke test |
1 similar comment
@swift-ci please smoke test |
Build swift-frontend as the primary Swift binary, and have swift/swiftc/etc. symlink over to it. This is a step toward allowing swift-driver to replace the swift and swiftc binaries.
This obviates the need for "-frontend", although we still accept it for backward compatibility. Switch lit's %target-swift-frontend et al over to calling swift-frontend.
Hardlinking to a symlink is not permitted, so use "swift-frontend" instead of "swift" as the hard-link target. Also, update one Linux-specific test to check for swift-frontend.
9fbf83a
to
d49e4a2
Compare
Build failed |
Build failed |
@swift-ci please test source compatibility Debug |
2 similar comments
@swift-ci please test source compatibility Debug |
@swift-ci please test source compatibility Debug |
@swift-ci please test source compatibility debug |
Source-compatibility failures are actually UPASS's, so we'll get that cleaned up. Windows failure is unrelated. Time to merge! |
Hi @DougGregor – This broke unified builds:
|
[CMake] Fix unified builds after #28003
Build swift-frontend as the primary Swift binary, and have
swift
/swiftc
/etc. symlink over to it. This is a step toward allowingswift-driver to replace
swift
andswiftc
binaries.