[PATCH] D135273: [Clang][ObjC] Add optionality to property attribute strings.

[David Chisnall via Phabricator via cfe-commits]

theraven added a comment.

In D135273#3936297 <https://reviews.llvm.org/D135273#3936297>, @al45tair wrote:

> @theraven Any chance you could glance over this and reassure us that it isn't going to break the GNU runtime if we do this? (We're adding an extra attribute in the property attribute string so that we can detect `@optional` properties in ObjC protocols at runtime.)

It shouldn't, we ignore any unknown property attributes.  I'd be more concerned about code outside the runtime.  Lots of things parse encoding strings badly, but the property APIs make it much easier to query known attributes and so I think that's a lot lower risk than changing anything in encoding strings.  It's a shame to use ?, since that is unknown type in type encodings and that may confuse some parsers..


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135273/new/

https://reviews.llvm.org/D135273