Skip to content

[Dependency Scanning] Remove coarse-grained synchronization on all dependency scan queries #1319

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
Apr 10, 2023

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Mar 28, 2023

In the InterModuleDependencyOracle, which may be shared across different instances of the Driver, all dependency scanning queries are executed synchronously on a shared dispatch queue. This is no longer necessary with the scanner guarding its own shared state as of swiftlang/swift#64615.

@artemcm artemcm requested review from xymus and nkcsgexi March 28, 2023 16:45
@artemcm
Copy link
Contributor Author

artemcm commented Mar 28, 2023

@swift-ci test

@artemcm
Copy link
Contributor Author

artemcm commented Mar 28, 2023

We need swiftlang/swift#64615 in a nightly toolchain in order for the tests to pass here.

@artemcm
Copy link
Contributor Author

artemcm commented Mar 29, 2023

@swift-ci test

1 similar comment
@artemcm
Copy link
Contributor Author

artemcm commented Apr 5, 2023

@swift-ci test

…pendency scan queries

In the `InterModuleDependencyOracle`, which may be shared across different instances of the `Driver`, all dependency scanning queries are executed synchronously on a shared dispatch queue.
This is no longer necessary with the scanner guarding its own shared state as of swiftlang/swift#64615.
@artemcm artemcm force-pushed the NoCoarseDepScanSync branch from 3991b48 to 2a9b7f7 Compare April 7, 2023 18:21
@artemcm
Copy link
Contributor Author

artemcm commented Apr 7, 2023

@swift-ci test

@artemcm
Copy link
Contributor Author

artemcm commented Apr 10, 2023

@swift-ci test Windows platform

@artemcm artemcm merged commit 82ef3e2 into swiftlang:main Apr 10, 2023
@artemcm artemcm deleted the NoCoarseDepScanSync branch April 10, 2023 21:03
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.

3 participants