silgen: allow borrow of subscript for noncopyables #83959
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
016621c to
c6a9706
Compare
|
@swift-ci smoke test |
1 similar comment
|
@swift-ci smoke test |
|
@swift-ci test source compatibility |
d6ee140 to
5dc9963
Compare
|
@swift-ci test source compatibility |
|
@swift-ci please test |
If a subscript uses a read accessor to yield a noncopyable value, we'd emit an `end_apply` that's too tightly scoped to allow for a subsequent borrowing access on the yielded value. resolves rdar://159079818
We were not able to use an existential as the base of an access that strictly borrows the existential, because SILGen's RValue emission would establish a fresh evaluation scope just for the existential's opening, and then copy the opened value out. This is problematic for noncopyable existentials. So this patch moves & adds FormalEvaluationScope's around so they're broad enough to enable a borrow of an existential. The idea behind this refactoring is to establish top-level FormalEvaluationScopes when initially creating RValue's for Expr's in SILGen. Any more-tightly scoped operations will already establish their own nested scope, so this is mostly adding safe-guards. I've limited the existentials fix to noncopyables for now. part of rdar://159079818
5dc9963 to
16c5c36
Compare
|
Full test passed for all but the one test I needed to update, which I've done now. @swift-ci smoke test |
@swift-ci smoke test |
jckarter
approved these changes
Aug 29, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If a subscript uses a read accessor to yield a noncopyable value, we'd emit an
end_applythat's too tightly scoped to allow for a subsequent borrowing access on the yielded value.resolves rdar://159079818