Skip to content

Add libclang to install_components_with_clang presets #79108

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

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

owenv
Copy link
Contributor

@owenv owenv commented Feb 3, 2025

Swift Build uses libclang to read serialized diagnostic files, and invoke the module dependency scanner. Add it to the toolchain build.

@owenv
Copy link
Contributor Author

owenv commented Feb 3, 2025

@swift-ci please build toolchain

@owenv
Copy link
Contributor Author

owenv commented Feb 3, 2025

@swift-ci please smoke test

@owenv
Copy link
Contributor Author

owenv commented Feb 3, 2025

Verified macOS and Linux toolchains have the new content in the right place. Windows had one lldb test timeout, but should not be impacted by this change.

@owenv owenv changed the title [Draft] Add libclang to install_components_with_clang presets Add libclang to install_components_with_clang presets Feb 3, 2025
@owenv owenv requested a review from shahmishal February 3, 2025 17:16
@owenv
Copy link
Contributor Author

owenv commented Feb 5, 2025

@swift-ci please build toolchain Windows

@owenv owenv requested a review from edymtt February 5, 2025 22:42
@owenv
Copy link
Contributor Author

owenv commented Feb 7, 2025

It turns out the Windows toolchains already ship libclang.dll, so I think this change is sufficient to ensure it's included in swift.org toolchains for all the supported platforms

@owenv
Copy link
Contributor Author

owenv commented Feb 7, 2025

@edymtt @shahmishal feel free to suggest any additional testing/investigation that should be done or reviewers who should be added before this lands - it's been a while since I took a close look at the presets

@owenv owenv requested a review from akyrtzi February 7, 2025 18:17
@owenv owenv merged commit 3787044 into swiftlang:main Feb 7, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants