<div dir="ltr">On Mon, Jun 10, 2013 at 6:36 PM, Samuel Benzaquen <span dir="ltr"><<a href="mailto:sbenza@google.com" target="_blank" class="cremed">sbenza@google.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"><div dir="ltr"><div><div class="h5">On Mon, Jun 10, 2013 at 5:34 AM, Manish Verma <span dir="ltr"><<a href="mailto:manish.verma@arm.com" target="_blank" class="cremed">manish.verma@arm.com</a>></span> wrote:<br>
</div></div><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Samuel,<br>
<br>
Thank you for looking at this issue.<br>
<div><br>
> I see a few solutions for this:<br>
>  1. For any non-template matcher, move its code to an ASTMatchers.cpp<br>
> to be created. This would take away a big chunk of the size of<br>
> Registry.cpp. I don't know if this will just move the problem to<br>
> ASTMatchers.cpp instead.<br>
>  2. Split Registry.cpp in smaller compilation units, each with a<br>
> subset of the matchers. It should be easier than (1).<br>
<br>
</div>I think it would be better to implement option (2) while you look<br>
for a better solution. We are happy to do the testing at our end,<br>
as none of build-bots are testing this configuration.<br></blockquote><div><br></div></div></div><div>I found what appears to be a solution for the problem without having to break the file.<br></div><div>Removed a bunch of template instantiations from the marshaller functions. In particular, std::list<Matcher<T>> and std::vector<Matcher<T>*> for each node T. Just that one change seemed to take the unit below the limit.</div>


<div>Sent the change on <a href="http://llvm-reviews.chandlerc.com/D948" target="_blank" class="cremed">http://llvm-reviews.chandlerc.com/D948</a>. If you want, please patch the change and let me know if it fixes the issue. I don't have that environment, so I can't test it there.</div>
</div></div></div></blockquote><div><br></div><div style>The change was submitted.</div><div style>I found more opportunities to reduce the number of symbols changing some things in ASTMatchers.h, but that will take a little more work. We can do it if necessary.</div>
<div style>_Sam</div><div style><br></div></div></div></div>