aaron.ballman added a comment.

The examples and test code you use seem to imply that the isDefinition() 
matcher may be all you need; is isDefinition() insufficient for some reason? If 
so, can you expand the test cases to cover that particular usage? Also, 
FunctionDecl::getBody() does more work than FunctionDecl::hasBody() -- it would 
be better to implement the AST matcher in terms of FunctionDecl::hasBody() 
instead.


http://reviews.llvm.org/D16215



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

Reply via email to