Skip to content

On 'canImport' of Clang submodules of modules with an umbrella header, attempt to infer the submodule #72129

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
Mar 15, 2024

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Mar 6, 2024

Resolves rdar://124163264

@cachemeifyoucan
Copy link
Contributor

Can we add a test case to make sure canImport of submodule works?

@artemcm
Copy link
Contributor Author

artemcm commented Mar 6, 2024

Can we add a test case to make sure canImport of submodule works?

Yup, working on it.

@nkcsgexi nkcsgexi requested a review from beccadax March 6, 2024 22:39
@artemcm artemcm force-pushed the canImportSubmoduleInfer branch from a46fced to dbf220b Compare March 8, 2024 19:37
@artemcm
Copy link
Contributor Author

artemcm commented Mar 8, 2024

@swift-ci test

@artemcm
Copy link
Contributor Author

artemcm commented Mar 13, 2024

@swift-ci smoke test

@artemcm artemcm force-pushed the canImportSubmoduleInfer branch from dbf220b to 407a75e Compare March 13, 2024 22:45
@artemcm
Copy link
Contributor Author

artemcm commented Mar 13, 2024

@swift-ci test

…, attempt to infer the submodule

Resolves rdar://124163264
@artemcm artemcm force-pushed the canImportSubmoduleInfer branch from 407a75e to 0808a1d Compare March 14, 2024 22:12
@artemcm
Copy link
Contributor Author

artemcm commented Mar 14, 2024

@swift-ci test

@artemcm
Copy link
Contributor Author

artemcm commented Mar 15, 2024

@swift-ci 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