[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 6 07:41:53 PST 2020


aaron.ballman added reviewers: aaron.ballman, rjmccall, erik.pilkington, MadCoder.
aaron.ballman added a comment.
Herald added a subscriber: dexonsmith.

Adding some more knowledgeable reviewers for comments on your RFC. I pointed out a few minor nits, but I'll hold off on a technical review until the ObjC-specific details are worked out and there is buy-in on the feature.



================
Comment at: clang/include/clang/AST/DeclObjC.h:2197
 
+  /// True if the protocol is tagged as objc_static_protocol
+  bool isStaticProtocol() const;
----------------
Comments should be properly punctuated (missing a full stop at the end of the sentence), here and elsewhere.


================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:2609
+                                         const ParsedAttr &AL) {
+  if (S.getLangOpts().ObjCRuntime.allowsStaticProtocols()) {
+    handleSimpleAttribute<ObjCStaticProtocolAttr>(S, D, AL);
----------------
This should be implemented via a custom `LangOpt` in `Attr.td`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75574





More information about the cfe-commits mailing list