<div dir="ltr">*ping*</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 10 February 2014 14:59, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.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">Hi All,
<div><br></div><div>I've had a crack at implementing lazy loading for late parsed templates (used by MSVCCompat) as I've noticed that they add unnecessary overhead to compilation using large PCHs. This is especially true for small files compiled with large PCH files containing many templated functions - they were all being deserialized but most never got used.</div>

<div><br></div><div>A quick analysis on a file in a large code base shows the AST parsing time time dropping from 0.58s without the change and 0.37s with the change. This is with 699/7333 late parsed templated loaded and parsed from the PCH.</div>

<div><br></div><div>I've added the data structures to the ModuleFile class to follow convention, although I haven't added any new tests as I doubt if anyone's likely to use modules with late template parsing enabled. However, at least the option is there should someone want to use (and test) it with modules in the future. That said, all existing test pass with this patch.</div>

<div><br></div><div>Formatted and tested patch attached ready for review.</div><div><br></div><div>Cheers,</div><div>Will.</div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant Ltd.</b></span></div>
</div>
</div>