Skip to content

[Dependency Scanning] Include initial PCM arguments on Clang module dependency details #39397

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 23, 2021

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Sep 22, 2021

Doing so will allow clients to know which Swift-specific PCM arguments are already captured from the scan that first discovered this module.
SwiftDriver, in particular, will be able to use this information to avoid batch re-scanning a given Clang module if the initial scan was sufficient for all possible sets of PCM arguments on Swift modules that depend on said Clang module.

…ependency details

Doing so will allow clients to know which Swift-specific PCM arguments are already captured from the scan that first discovered this module.
SwiftDriver, in particular, will be able to use this information to avoid re-scanning a given Clang module if the initial scan was sufficient for all possible sets of PCM arguments on Swift modules that depend on said Clang module.
@artemcm artemcm requested a review from nkcsgexi September 22, 2021 18:47
@artemcm
Copy link
Contributor Author

artemcm commented Sep 22, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 22, 2021

Corresponding driver PR that queries the newly-added libSwiftScan entry-points in order to avoid redundant batch re-scanning:
swiftlang/swift-driver#848

@artemcm
Copy link
Contributor Author

artemcm commented Sep 22, 2021

@swift-ci please test Windows platform

1 similar comment
@artemcm
Copy link
Contributor Author

artemcm commented Sep 22, 2021

@swift-ci please test Windows platform

Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@artemcm
Copy link
Contributor Author

artemcm commented Sep 23, 2021

@swift-ci please test Windows platform

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