[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