<div dir="ltr">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 class="gmail_extra"><div class="gmail_quote">

<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>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><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">
<br>
Regards,<br>
Manish<br>
<br>
<br>
<br>
</blockquote></div><br></div></div>