[PATCH] D34937: Suppressing Reference Counting Diagnostics for Functions Containing 'rc_ownership_trusted_implementation' Annotate Attribute

Malhar Thakkar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 17 10:43:09 PDT 2017


malhar1995 added a comment.

In https://reviews.llvm.org/D34937#811500, @dcoughlin wrote:

> Looks good to me, other than a nit on indentation and a request for a little bit more info in a comment!


Currently, I have added support for generic annotations for __isl_take and __isl_give and I have also added a generic diagnostic note when the function returns an object which is neither Core-Foundation nor Objective-C.
For emitting a generic diagnostic note, I have added an object kind to the enum ObjKind in ObjCRetainCount.h.

Now, for adding the aforementioned support, I have made some changes to the patch which I last submitted.

So, should I submit another patch on the same revision after modifying the title, summary, etc. or should I create another revision for that?


Repository:
  rL LLVM

https://reviews.llvm.org/D34937





More information about the cfe-commits mailing list