Skip to content

Add regression test for redundant strong_retain/strong_release pair #89473

Merged
airspeedswift merged 1 commit into
swiftlang:mainfrom
airspeedswift:arc-closure-spec-regression-82155
May 28, 2026
Merged

Add regression test for redundant strong_retain/strong_release pair #89473
airspeedswift merged 1 commit into
swiftlang:mainfrom
airspeedswift:arc-closure-spec-regression-82155

Conversation

@airspeedswift
Copy link
Copy Markdown
Member

Locks in that closure specialization no longer leaves a redundant strong_retain/strong_release pair around a non-escaping closure call that captures a borrowed class reference. Fixed between Swift 6.3 and 6.4; test guards against future regression.

Resolves #82155

Locks in that closure specialization no longer leaves a redundant
strong_retain/strong_release pair around a non-escaping closure call
that captures a borrowed class reference. Fixed between Swift 6.3 and
6.4; test guards against future regression.
@airspeedswift airspeedswift requested a review from eeckstein as a code owner May 27, 2026 16:21
@airspeedswift
Copy link
Copy Markdown
Member Author

@swift-ci please smoke test

Copy link
Copy Markdown
Contributor

@eeckstein eeckstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@airspeedswift airspeedswift merged commit 367b88e into swiftlang:main May 28, 2026
3 checks passed
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.

Unnecessary retain/release pair around closure

2 participants