alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
LG
================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:80
@@ +79,3 @@
+ ResultT operator()(ArrayRef<ArgT> Args) const {
+ std::vector<const ArgT *> InnerArgs;
+ for (const ArgT &Arg : Args)
----------------
It's unfortunate that we need to create an array of the argument pointers here,
but it seems there's no larger common denominator of the two ways this
functions can be called.
One nit though: SmallVector will be a better container here.
http://reviews.llvm.org/D18275
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
