<div dir="ltr">Doug, I'll not submit before I get an ok from you :)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 26, 2013 at 6:05 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Sorry, I haven't commented because it looks like it's going to be less and less relevant to the analyzer. As much as I don't like having two similar data structures floating around, Edwin's right that we don't need to try to optimize for two different use cases if it means that we won't get optimal behavior in either.<br>

<br>
  This design is at least not incompatible with what the analyzer needs, in case we can ever unify them in the future. We may want to rename the analyzer's class StmtParentMap, and migrate the other users (migrators/translators) to AST's ParentMap.<br>

<br>
<a href="http://llvm-reviews.chandlerc.com/D267" target="_blank">http://llvm-reviews.chandlerc.com/D267</a><br>
</blockquote></div><br></div>