swift
[5.2] [Typechecker] Fix _modify for wrapped properties with observers
#30994
Merged

[5.2] [Typechecker] Fix _modify for wrapped properties with observers #30994

tkremenek merged 10 commits into swiftlang:swift-5.2-branch from fix/SR-12178_5.2
theblixguy
theblixguy [Typechecker] Wrapped properties with observers should not have ReadW…
f087a688
theblixguy [Typechecker] Supress _modify synthesis if the wrapped property has o…
035bc16e
theblixguy [Test] Add an executable test
00b7f80a
theblixguy [Typechecker] Add an hasObservers() convenience method and simplify s…
4b6350e4
theblixguy [Typechecker] Simplify StorageImplInfo code in finishPropertyWrapperI…
47742772
theblixguy [Test] Add a SILGen test to make sure the observers are correctly called
8a049d02
theblixguy theblixguy requested a review 5 years ago
theblixguy
shahmishal
swift-ci
swift-ci
theblixguy [Typechecker] Add missing 'storageReadWriteImpl' variable in 'synthes…
fa1370ea
theblixguy
swift-ci
swift-ci
CodaFi
swift-ci
swift-ci
theblixguy
swift-ci
swift-ci
theblixguy [Typechecker] Tweak property wrapper mutability computation a bit
57bc6195
theblixguy [Test] Update a SILGen test which relies on oldValue not being fetched
a422a1e2
theblixguy
swift-ci
swift-ci
theblixguy [Typechecker] Return 'None' if TargetImpl isn't Wrapper or WrapperSto…
30ad8d07
theblixguy
swift-ci
swift-ci
tkremenek
tkremenek approved these changes on 2020-04-16
tkremenek tkremenek merged 49e70edc into swift-5.2-branch 5 years ago
theblixguy theblixguy deleted the fix/SR-12178_5.2 branch 5 years ago
stuffmc
theblixguy
AnthonyLatsis AnthonyLatsis added swift 5.2
AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick
AnthonyLatsis AnthonyLatsis removed swift 5.2

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone