Skip to content

[6.2] Fix LifetimeDependenceScopeFixup: extend scopes enclosing coroutines #81208

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
May 1, 2025

Conversation

atrick
Copy link
Contributor

@atrick atrick commented Apr 30, 2025

When a coroutine is extended because of a dependent lifetime, extend all scopes
that enclose that coroutine even if the coroutine itself has no lifetime
dependencies.

Fixes rdar://150275147 (Invalid SIL after lifetime dependence fixup involving
coroutines)

(cherry picked from commit 4a65be8)

mainPR: #81207

When a coroutine is extended because of a dependent lifetime, extend all scopes
that enclose that coroutine even if the coroutine itself has no lifetime
dependencies.

Fixes rdar://150275147 (Invalid SIL after lifetime dependence fixup involving
coroutines)

(cherry picked from commit 4a65be8)
@atrick atrick requested a review from a team as a code owner April 30, 2025 18:41
@atrick atrick added 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2 labels Apr 30, 2025
@atrick atrick requested a review from tbkka April 30, 2025 18:42
@atrick atrick enabled auto-merge April 30, 2025 18:42
@atrick
Copy link
Contributor Author

atrick commented Apr 30, 2025

@swift-ci test

@atrick atrick merged commit 7708ef5 into swiftlang:release/6.2 May 1, 2025
5 checks passed
@atrick atrick deleted the 62-extend-coroutine branch May 1, 2025 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 6.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants