Skip to content

Conversation

gottesmm
Copy link
Contributor

@gottesmm gottesmm commented Jul 16, 2025

Description: We were going to try and take this fix for nonisolated(unsafe) closures for 6.2... but decided to delay. This just removes it from 6.2.

Scope/Impact: nonisolated(unsafe) will not be inferred onto anonymous closure literals if the closure has only nonisolated(unsafe) captures.
Risk: Low. This didn't make it out into the world, so no risk.
Testing: CI testing
Reviewed by: N/A
Original PR: N/A
Radar: N/A

gottesmm added 3 commits July 16, 2025 09:58
…nisolated(unsafe) onto named closures."

This reverts commit 4ccb39e.
…its captures are nonisolated(unsafe)."

This reverts commit 6ce93c8.
@gottesmm gottesmm requested a review from a team as a code owner July 16, 2025 16:59
@gottesmm
Copy link
Contributor Author

@swift-ci test

@xedin
Copy link
Contributor

xedin commented Jul 16, 2025

@gottesmm this is about nonisolated(unsafe) though right?

@gottesmm gottesmm changed the title [6.2] Revert nonisolated(nonsending) closure inference [6.2] Revert nonisolated(unsafe) closure inference Jul 16, 2025
@gottesmm
Copy link
Contributor Author

@xedin yes. Sorry! My brain failed.

@gottesmm gottesmm enabled auto-merge July 16, 2025 20:10
@gottesmm
Copy link
Contributor Author

Timed out in LLVM

@gottesmm
Copy link
Contributor Author

@swift-ci test macOS platform

@gottesmm gottesmm merged commit 7f9eb4c into swiftlang:release/6.2 Jul 17, 2025
5 checks passed
@gottesmm gottesmm deleted the release/6.2-revert branch July 17, 2025 23:07
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