xazax.hun marked 3 inline comments as done.
xazax.hun added inline comments.
================
Comment at: clang/include/clang/Basic/Attr.td:3445
+ let Spellings = [Clang<"acquire_handle">];
+ let Subjects = SubjectList<[Function, ParmVar], ErrorDiag>;
+ let Documentation = [AcquireHandleDocs];
----------------
aaron.ballman wrote:
> What about function-like interfaces such as lambdas, blocks, or other
> callable objects that are not a `FunctionDecl`?
Good point!
================
Comment at: clang/include/clang/Basic/Attr.td:3450
+def UseHandle : InheritableParamAttr {
+ let Spellings = [Clang<"use_handle">];
+ let Documentation = [UseHandleDocs];
----------------
aaron.ballman wrote:
> Should this have a subject limiting it to parameters?
My understanding was that the subject list is "inherited" from
InheritableParamAttr, which already has it limited to parameters. Is this not
the case?
================
Comment at: clang/include/clang/Basic/AttrDocs.td:4474
+ let Content = [{
+If this annotation is on a function it is assumed to return a new handle.
+In case this annotation is on an output parameter, the function is assumed
----------------
aaron.ballman wrote:
> What is a "handle"? I think some introduction docs are needed.
Good point. Do you prefer to copy and paste the introduction to all attributes
or is it enough to only have it for one of them?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70469/new/
https://reviews.llvm.org/D70469
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
