Hello!

> This large patch makes GATHER_STATISTICS always take a value, 0
> (disabled) or 1 (enabled), and turns all related #ifdef code into
> conditional branches.

> This slightly increases the data section of cc1, but only marginally.
> There is no impact on compile time, because all tests are simply
> optimized away if GATHER_STATISTICS is 0.

> The benefit is that this improves the coverage of this code, and less
> #ifdef'ed code (which is IMHO a Good Thing).

> Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk?

This patch (r189803) regressed a bunch of tests on x86_64 [1], [2].

[1] http://gcc.gnu.org/ml/gcc-testresults/2012-07/msg02066.html
[2] http://gcc.gnu.org/ml/gcc-regression/2012-07/msg00177.html

Uros.

Reply via email to