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



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

Reply via email to