[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