[PATCH] First step towards adding a parent map to the ASTContext.
klimek at google.com
Tue Feb 5 06:37:20 PST 2013
On Tue, Feb 5, 2013 at 3:27 PM, Edwin Vane <edwin.vane at intel.com> wrote:
> I don't know anything about the analyzer but it sounds like it is what
> is driving the design. For tools this all looks good if only a little
> heavy-handed. Your suggestions for improvements that would make parent map
> construction not touch the whole AST would be welcome.
> Question: does MatchASTVisitor::matchesAncestorOf() need fixing to use
> the new ASTContext::getParents()?
Yes, and that's part of the patch. At least if I didn't mess anything up? :)
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits