rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.

One minor request (which will be dead code in your patch), but otherwise LGTM.



================
Comment at: clang/include/clang/Basic/IdentifierTable.h:754
+  /// If this selector is the specific keyword selector described by Names.
+  bool isKeywordSelector(ArrayRef<StringRef> Names);
+
----------------
Thanks.  Could you go ahead and add `isUnarySelector(StringRef)` for 
consistency?


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

https://reviews.llvm.org/D27165



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to