[cfe-commits] r143922 - in /cfe/trunk: include/clang/AST/APValue.h lib/AST/APValue.cpp lib/AST/ExprConstant.cpp lib/CodeGen/CGExprConstant.cpp test/SemaCXX/constant-expression-cxx11.cpp

[Richard Smith]

On Tue, Mar 6, 2012 at 3:37 PM, Daniel Dunbar <daniel at zuster.org> 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120306/31abf789/attachment.html>