On Mon, Jul 23, 2012 at 7:01 PM, Steven Bosscher <stevenb....@gmail.com> wrote: > 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).
Indeed. > Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk? Ok! Thanks, Richard. > Ciao! > Steven
