[PATCH] D28260: Add an argumentsAre matcher

2017-01-10 Thread Matt Kulukundis via Phabricator via cfe-commits
fowles added a subscriber: jdennett. fowles added a comment. @jdennett wanted this matcher for something he is working on and I had some free cycles to write it up. Unfortunately, I am about to leave on an extended vacation, so I will not be able to follow up with this patch for 2 months at th

[PATCH] D28260: Add an argumentsAre matcher

2017-01-10 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Given that this can't be expressed as a dynamic matcher, I'm wondering what the use case is for the matcher. Do you have a specific need for this functionality, or do you see this being generally useful? https://reviews.llvm.org/D28260

[PATCH] D28260: Add an argumentsAre matcher

2017-01-10 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman requested changes to this revision. aaron.ballman added a reviewer: aaron.ballman. aaron.ballman added a comment. This revision now requires changes to proceed. You should also regenerate the HTML matcher documentation as part of this patch. Comment at: include/cl

[PATCH] D28260: Add an argumentsAre matcher

2017-01-09 Thread Manuel Klimek via Phabricator via cfe-commits
klimek added a reviewer: bkramer. klimek added a comment. +benjamin https://reviews.llvm.org/D28260 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D28260: Add an argumentsAre matcher

2017-01-03 Thread Matt Kulukundis via Phabricator via cfe-commits
fowles created this revision. fowles added a reviewer: klimek. fowles added a subscriber: cfe-commits. Add an argumentsAre matcher https://reviews.llvm.org/D28260 Files: include/clang/ASTMatchers/ASTMatchers.h unittests/ASTMatchers/ASTMatchersTraversalTest.cpp Index: unittests/ASTMatchers