rjmccall added inline comments.

================
Comment at: lib/CodeGen/CGBlocks.cpp:2582
 /// yet; if a cleanup is required for the variable itself, that needs
 /// to be done externally.
+void CodeGenFunction::enterByrefCleanup(CleanupKind Kind, Address Addr,
----------------
ahatanak wrote:
> rjmccall wrote:
> > Please document the meaning of the parameters here.
> One more question: I think documentation comments for public APIs are 
> normally in the header file. Should we move these comments to 
> CodeGenFunction.h? 
It'd be better, yeah.  Having it be on a unique implementation is also okay 
because it's not hard to find, but yeah, header is better.


Repository:
  rC Clang

https://reviews.llvm.org/D49718



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

Reply via email to