Skip to content

[ExplicitModule] Support direct cc1 flag for all explicit module build job #1550

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

Conversation

cachemeifyoucan
Copy link
Contributor

Update swift-driver to support directly instantiate clang importer in all jobs with cc1 flags. In the new mode, swift dependency scanner will return all the cc1 flags to the actual compilation doesn't need to run clang driver again. In this mode, swift-driver should avoid forwarding Xcc command-line options since those are expected to be clang driver args, not cc1 args.

@cachemeifyoucan
Copy link
Contributor Author

Support for swiftlang/swift#71118

@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please test

…d jobs

Update swift-driver to support directly instantiate clang importer in
all jobs with cc1 flags. In the new mode, swift dependency scanner will
return all the cc1 flags to the actual compilation doesn't need to run
clang driver again. In this mode, swift-driver should avoid forwarding
Xcc command-line options since those are expected to be clang driver
args, not cc1 args.
@cachemeifyoucan cachemeifyoucan force-pushed the eng/PR-support-direct-cc1-scan branch from fe2a7ce to e31e98f Compare March 14, 2024 16:39
@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please test

@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please test windows platform

1 similar comment
@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please test windows platform

@cachemeifyoucan
Copy link
Contributor Author

ping

1 similar comment
@cachemeifyoucan
Copy link
Contributor Author

ping

@cachemeifyoucan cachemeifyoucan merged commit 27f5efd into swiftlang:main Mar 21, 2024
3 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