Some of my measurements were wrong (testing a assertion enabled release
build against a real opt build), but it gets even better:
For one of our large TUs, this brings non-module compile time down from 36
-> 30 seconds.

On Wed, Mar 25, 2015 at 3:58 PM Richard Smith <[email protected]> wrote:

> I think it's worth a 2-3% memory increase to remove this as a source of
> pathological performance problems. See also https://llvm.org/bugs/show_
> bug.cgi?id=21264 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).
>
>
> http://reviews.llvm.org/D8619
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to