Skip to content

Conversation

@rjmccall
Copy link
Contributor

@rjmccall rjmccall commented Jul 1, 2025

Also, I discovered that we don't apply nonisolated(nonsending) to function types in the new mode. That's one for a different patch.

Fixes rdar://154401813 and rdar://154137740

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 1, 2025

@swift-ci Please test

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

What

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test

SILGen thunks.

Also, I discovered that we don't apply nonisolated(nonsending) to
function types in the new mode. That's one for a different patch.

Fixes rdar://154401813
@rjmccall rjmccall force-pushed the nonisolated-nonsending-thunks branch from 51a9583 to c1d153c Compare July 2, 2025 02:01
@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test macOS

1 similar comment
@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test macOS

@xedin
Copy link
Contributor

xedin commented Jul 2, 2025

@swift-ci Please test macOS

1 similar comment
@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test macOS

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test macOS

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please test Windows

@rjmccall
Copy link
Contributor Author

rjmccall commented Jul 2, 2025

@swift-ci Please smoke test macOS

@phausler phausler merged commit 834b606 into swiftlang:main Jul 2, 2025
2 of 5 checks passed
@rjmccall rjmccall deleted the nonisolated-nonsending-thunks branch July 2, 2025 19:26
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.

5 participants