[cfe-commits] PATCH: flesh out RecursiveASTVisitor

Craig Silverstein csilvers2000 at yahoo.com
Wed May 26 11:15:34 PDT 2010


RecursiveASTVisitor.h does not yet completely recurse on all the AST
types it sees.  I've written a patch to add recursive calls for all
the Visit*Decl() visitor methods (except VisitObjC*Decl, since I don't
know objective C).

This patch is almost certainly not complete -- there are structures in
these various decls I should be recursing on but am not -- and
probably over-complete -- I likely recurse on some of the same data
multiple times, via different API calls.  There are also probably
bugs.

I tested by running over a program that triggers many, but not all, of
these Visitor callbacks.  I can verify that the ones I've tested do
not crash clang, at least on the inputs I gave them, and that the
output looks plausible.  Hopefully someone more expert in the clang
AST can look this over, tidy it up, and make it appropriate to commit.

The patch is at
   http://llvm.org/bugs/attachment.cgi?id=4933
It is part of
   http://llvm.org/bugs/show_bug.cgi?id=7161

craig


      



More information about the cfe-commits mailing list