aaron.ballman added inline comments.
================ Comment at: include/clang/Basic/AttrDocs.td:3489 + +``noderef`` is currently only supported for C style pointers and arrays and not usable for +references or Objective-C pointers. ---------------- I would drop the "C style" and just say it's only supported for pointers and arrays. ================ Comment at: include/clang/Basic/AttrDocs.td:3490 +``noderef`` is currently only supported for C style pointers and arrays and not usable for +references or Objective-C pointers. + ---------------- I'd clarify this a little bit to Objective-C object pointers. ================ Comment at: include/clang/Basic/DiagnosticSemaKinds.td:9436 +def warn_dereference_of_noderef_type : Warning< + "dereferencing %0; was declared with a `noderef` type">, InGroup<NoDeref>; +def warn_dereference_of_noderef_type_no_decl : Warning< ---------------- Please use single quotes instead of backticks in the quoting around `noderef` in these diagnostics. Repository: rC Clang https://reviews.llvm.org/D49511 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits