[PATCH] First revision of the dynamic ASTMatcher library

Dmitri Gribenko gribozavr at gmail.com
Tue Apr 23 14:29:55 PDT 2013



================
Comment at: include/clang/ASTMatchers/Dynamic/Parser.h:11-12
@@ +10,4 @@
+// Simple matcher expression parser.
+// The parser understand matcher expressions of the form:
+//   MatcherName(Arg0, Arg1, ..., ArgN)
+// as well as simple types like strings.
----------------
Samuel Benzaquen wrote:
> Dmitri Gribenko wrote:
> > Manuel Klimek wrote:
> > > understand*s*
> > It would help a lot to give a formal definition of the grammar in documentation.
> Done. I am a little rusty on how to specify grammars, so it might be off a little bit.
This looks good for a start (I expect the grammar to become more complex as more features are added), but please wrap it into \code...\endcode so that it looks well in HTML pages.


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



More information about the cfe-commits mailing list