<div dir="ltr">Ping</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 11, 2013 at 8:48 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Looking for a decision whether this is good as a first step, or what I should address before this can go in.<div>
<br></div><div>Thanks!</div><span class="HOEnZb"><font color="#888888"><div>/Manuel</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Feb 5, 2013 at 4:25 PM, Vane, Edwin <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">







<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Oops. I see it there now. Got lost in the context.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Manuel Klimek [mailto:<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>]
<br>
<b>Sent:</b> Tuesday, February 05, 2013 9:37 AM<br>
<b>To:</b> <a href="mailto:reviews%2BD267%2Bpublic%2B2ab3cabee59f81bc@llvm-reviews.chandlerc.com" target="_blank">reviews+D267+public+2ab3cabee59f81bc@llvm-reviews.chandlerc.com</a><br>
<b>Cc:</b> <a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a>; Jordan Rose; Vane, Edwin; <a href="mailto:joey.gouly@arm.com" target="_blank">joey.gouly@arm.com</a><br>
<b>Subject:</b> Re: [PATCH] First step towards adding a parent map to the ASTContext.<u></u><u></u></span></p><div><div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On Tue, Feb 5, 2013 at 3:27 PM, Edwin Vane <<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>> wrote:<u></u><u></u></p>
<div>
<div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal"><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()?<u></u><u></u></p>
</blockquote>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Yes, and that's part of the patch. At least if I didn't mess anything up? :)<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal"><br>
<a href="http://llvm-reviews.chandlerc.com/D267" target="_blank">http://llvm-reviews.chandlerc.com/D267</a><u></u><u></u></p>
</blockquote>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div></div></div>
</div>

</blockquote></div><br></div>
</div></div></blockquote></div><br></div>