Skip to content

Conversation

@benrimmington
Copy link
Contributor

Follow-up to: #34150

@benrimmington
Copy link
Contributor Author

Cc: @hborla, @xedin

@benrimmington
Copy link
Contributor Author

@swift-ci Please smoke test

Copy link
Member

@hborla hborla left a comment

Choose a reason for hiding this comment

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

Thank you!

@benrimmington benrimmington merged commit ac06148 into swiftlang:main Oct 2, 2020
@benrimmington benrimmington deleted the type-checker-doc branch October 2, 2020 05:47
ainu-bot added a commit to google/swift that referenced this pull request Oct 2, 2020
* 'master' of github.com:apple/swift:
  [Docs] Update the documentation contents and index (swiftlang#34157)
  [SILGen] Don't use assign_by_wrapper for local property wrappers if there is an initial wrapped value.
  [Property Wrappers] Ban non-member property wrappers with observers, just like we do for regular non-member properties.
  [Test] Start to add SILGen tests for local property wrappers.
  [SILGen] Add a new CaptureEmission kind specifically for emitting captured local variables for the assign_by_wrapper setter.
  [Name Lookup] Remove property wrapper name lookup flags and generalize unqualified lookup of auxiliary decl names.
  [Property Wrappers] Generalize a few property wrapper decl context checks to check for type context instead of local context.
  [Property Wrappers] Add a VarDecl helper method for visiting synthesized property wrapper vars.
  [Property Wrappers] Allow property wrappers on local variables.
  [Property Wrappers] Make sure captures are computed for synthesized property wrapper accessors.
  [SILGen] Only use assign_by_wrapper for wrapped instance properties inside an initializer, and for wrapped local variables.
  [SILGen] Teach SIlGen to emit local property wrappers
  [NameLookup] Teach unqualified lookup to resolve backing property wrapper and projected value references
  [Parser] Don't resolve decl references in the parser if the declaration has a custom attribute.
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