[patch] un-#ifdef GATHER_STATISTICS
Uros Bizjak
ubizjak@gmail.com
Tue Jul 24 13:09:00 GMT 2012
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.
More information about the Gcc-patches
mailing list