On Tue, Mar 6, 2012 at 3:37 PM, Daniel Dunbar <[email protected]> wrote:

> Hi Richard,
>
> I've been looking into compile time performance, and I'm worried about
> this code.
>
> It looks like it is doing a lot of work for non-C++ code, and I see a
> pretty big compile time speedup (2%, which is pretty big for one
> thing) from disabling this code.
>

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).

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.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to