[PATCH] First revision of the dynamic ASTMatcher library

Dmitri Gribenko gribozavr at gmail.com
Tue Apr 23 13:46:17 PDT 2013



================
Comment at: include/clang/ASTMatchers/Dynamic/Parser.h:65
@@ +64,3 @@
+    processMatcherToken(llvm::StringRef MatcherName,
+                        const llvm::ArrayRef<VariantValue> &Args,
+                        const TokenInfo &Info) = 0;
----------------
ArrayRef is a value type that is conceptually const.  So this (and others) should be 'ArrayRef<VariantValue> Args'.


http://llvm-reviews.chandlerc.com/D714



More information about the cfe-commits mailing list