[PATCH] D17163: [ASTMatchers] Add matcher hasAnyName.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 12 05:28:19 PST 2016


alexfh added inline comments.

================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:644
@@ -643,2 +643,3 @@
   explicit HasNameMatcher(StringRef Name);
+  explicit HasNameMatcher(ArrayRef<const StringRef *> Names);
 
----------------
bkramer wrote:
> alexfh wrote:
> > Why not `ArrayRef<StringRef>`?
> That's an artifact of how llvm::VariadicFunction works. It gives you an array ref of pointers to the arguments.
Thanks! Good to know. Maybe a add a comment or is it just me who doesn't know this?


http://reviews.llvm.org/D17163





More information about the cfe-commits mailing list