On Tue, Feb 5, 2013 at 3:27 PM, Edwin Vane <[email protected]> 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? :)


>
> http://llvm-reviews.chandlerc.com/D267
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to