<div dir="ltr">On Tue, Feb 5, 2013 at 3:27 PM, Edwin Vane <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  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.<br>

<br>
  Question: does MatchASTVisitor::matchesAncestorOf() need fixing to use the new ASTContext::getParents()?<br></blockquote><div><br></div><div style>Yes, and that's part of the patch. At least if I didn't mess anything up? :)</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://llvm-reviews.chandlerc.com/D267" target="_blank">http://llvm-reviews.chandlerc.com/D267</a><br>
</blockquote></div><br></div></div>