<div dir="ltr">Some of my measurements were wrong (testing a assertion enabled release build against a real opt build), but it gets even better:<br><div>For one of our large TUs, this brings non-module compile time down from 36 -> 30 seconds.</div></div><br><div class="gmail_quote">On Wed, Mar 25, 2015 at 3:58 PM Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think it's worth a 2-3% memory increase to remove this as a source of pathological performance problems. See also <a href="https://llvm.org/bugs/show_bug.cgi?id=21264" target="_blank">https://llvm.org/bugs/show_<u></u>bug.cgi?id=21264</a> which would also be fixed by this change (that bug suggests an alternative, much more complex, strategy if the 2-3% memory increase isn't acceptable).<br>
<br>
<br>
<a href="http://reviews.llvm.org/D8619" target="_blank">http://reviews.llvm.org/D8619</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>