[PATCH] D69991: Implement __attribute__((objc_direct)), __attribute__((objc_direct_members))

Pierre Habouzit via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 14 14:36:30 PST 2019


MadCoder added inline comments.


================
Comment at: clang/lib/Sema/SemaExprObjC.cpp:1129
+  }
+}
+
----------------
rjmccall wrote:
> Is this a good idea to do for arbitrary diagnostics?  I feel like saying "direct" in the note when that has nothing to do with the conflict could be a little misleading.
fair enough. some of the mismatches will be because of directness so it can help in that instance but I can see how it's confusing.

fixed, and the fact that tests didn't need any updating probably proves you right ;)


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

https://reviews.llvm.org/D69991





More information about the cfe-commits mailing list