<div dir="ltr">On Sat, May 18, 2013 at 1:55 AM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</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>
  > Enhancements for the DynTypedNode/DynTypedMatcher system.<br>
  > - Added DynNodeType as a standalone way to represent node types and their hierarchy.<br>
  > - Added conversion routines and checks in Matcher<T> that take a DynTypedMatcher.<br>
  > - Fixed MatchFinder::addMatcherDynamic() to validate the input matcher.<br>
  > - Added type information on the error messages for the marshallers.<br>
  > - Allows future work on Polymorphic/overloaded matchers. We should be able to disambiguate at runtime and choose the appropriate overload.<br>
<br>
<br>
  Could you please split these up into separate patches so that they are easier to review?<br></blockquote><div><br></div><div style>+1. In general, always pull adding new headers / functionality to one of clang's core modules (like AST) into their own change.</div>
<div style><br></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/D815" target="_blank">http://llvm-reviews.chandlerc.com/D815</a><br>
</blockquote></div><br></div></div>