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
build: expand the swift components and create install rules #21270
base: main
Are you sure you want to change the base?
Conversation
Replace the component system with the LLVM installation mechanism. This allows for a single unified toolchain build. Preserve the components by remapping the swift components to their substiutients during the configure phase.
Something that I was playing with, it may be interesting to finish off and merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to review this. Also can you please not do inline changes for this. Why not just hide these changes in the underlying component system.
@@ -454,12 +454,14 @@ include(CMakePushCheckState) | |||
print_versions() | |||
|
|||
include(SwiftComponents) | |||
|
|||
swift_expand_components() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you hide this in configure_components?
It just seems that having this be mapped away is easier to deal with the CMake logic overall. Really, the component mapping doesn't change very often, and most of that can be abstracted over with caches, so this seems better for the regular read/modify cycle of the CMake. |
Replace the component system with the LLVM installation mechanism. This allows
for a single unified toolchain build. Preserve the components by remapping the
swift components to their substiutients during the configure phase.
Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.
Resolves SR-NNNN.