rnk added inline comments. ================ Comment at: include/clang/AST/ASTConsumer.h:58-64 @@ -57,5 +57,9 @@ /// \brief This callback is invoked each time an inline method definition is /// completed. virtual void HandleInlineMethodDefinition(CXXMethodDecl *D) {} + /// \brief This callback is invoked each time an inline friend function + /// definition is completed. + virtual void HandleInlineFriendFunctionDefinition(FunctionDecl *D) {} + ---------------- sberg wrote: > rnk wrote: > > I'm pretty sure we can relax HandleInlineMethodDefinition to take a > > FunctionDecl and then we don't need the extra AST consumer callback. > ...and then also rename it from HandleInlineMethodDefinition to, > say,HandleInlineFunctionDefinition? Or better stick with the (then no longer > accurate) name? Renaming would be good if you're up for it. I have a feeling that nobody outside of Clang is overriding this ASTConsumer callback. It's purpose is very specific to dllexport.
http://reviews.llvm.org/D15267 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits