Skip to content

[Dependency Scanning] Do not include PCH-handling command-line flags on the scanning jobs #841

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
Sep 20, 2021

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Sep 16, 2021

These jobs do not require handling of the module's PCH.
If we do emit these flags, the underlying Clang invocation will have expectations that may not be met (such as going looking for existing PCH files, etc.).

…on the scanning jobs

These jobs do not require handling of the module's PCH.
@artemcm artemcm requested a review from nkcsgexi September 16, 2021 22:45
@artemcm
Copy link
Contributor Author

artemcm commented Sep 16, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 16, 2021

@swift-ci please test

5 similar comments
@artemcm
Copy link
Contributor Author

artemcm commented Sep 16, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 16, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 17, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 20, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 20, 2021

@swift-ci please test

@artemcm artemcm merged commit f1761dc into swiftlang:main Sep 20, 2021
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