aaron.ballman added inline comments.
================
Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:21
@@ +20,3 @@
+llvm::Optional<ast_matchers::internal::Matcher<NamedDecl>>
+getHasAnyName(const std::vector<std::string> &Names) {
+ llvm::Optional<ast_matchers::internal::Matcher<NamedDecl>> HasNameMatcher;
----------------
Looking at `VariadicFunction`, it appears that it already works if you pass an
`ArrayRef<T>` to the `operator()()` overload. See ASTMatchersInternal.h:81. So
I still think the matcher can be used directly, just with changing the type of
the object passed to the functor.
Repository:
rL LLVM
https://reviews.llvm.org/D22208
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
