<div class="gmail_quote">On Tue, Mar 6, 2012 at 3:37 PM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org" target="_blank">daniel@zuster.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Hi Richard,<br>
<br>
I've been looking into compile time performance, and I'm worried about<br>
this code.<br>
<br>
It looks like it is doing a lot of work for non-C++ code, and I see a<br>
pretty big compile time speedup (2%, which is pretty big for one<br>
thing) from disabling this code.<br></blockquote><div><br></div><div>r152181 should make things a little better here, and r152193 should help a lot (I'm seeing nearly a 3% speedup from that change for clang -Wno-conversion -O0 combine.c -c -o /dev/null).</div>
<div><br></div><div>We still have some time wasted evaluating AST nodes for disabled diagnostics, but checking whether a diagnostic is enabled is currently too slow for there to be a trivial fix here.</div></div>