Skip to content

Sema: Adjust result builder operation lookup for MemberImportVisibility #79902

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

tshortli
Copy link
Contributor

To avoid spurious diagnostics about unavailable operations when checking uses of functions that take result builder closures, lookup of result builder operations needs to ignore the restrictions of the MemberImportVisibility feature. The result builder transform should simply use the operations that are found and allow later checks to diagnose the use of inaccessible builder operations.

Resolves rdar://144100445.

@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

To avoid spurious diagnostics about unavailable operations when checking uses
of functions that take result builder closures, lookup of result builder
operations needs to ignore the restrictions of the `MemberImportVisibility`
feature. The result builder transform should simply use the operations that are
found and allow later checks to diagnose the use of inaccessible builder
operations.

Resolves rdar://144100445.
@tshortli tshortli force-pushed the result-builder-member-import-visibility branch from 4c94849 to 7a514c9 Compare March 11, 2025 03:55
@tshortli tshortli enabled auto-merge March 11, 2025 03:55
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

@tshortli
Copy link
Contributor Author

@swift-ci please smoke test macOS

@tshortli tshortli merged commit e68d8ed into swiftlang:main Mar 11, 2025
3 checks passed
@tshortli tshortli deleted the result-builder-member-import-visibility branch March 11, 2025 18:40
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