This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[patch] un-#ifdef GATHER_STATISTICS

From: Steven Bosscher <stevenb dot gcc at gmail dot com>
To: GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Richard Guenther <richard dot guenther at gmail dot com>
Date: Mon, 23 Jul 2012 19:01:06 +0200
Subject: [patch] un-#ifdef GATHER_STATISTICS

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?

Ciao!
Steven

Attachment: unifdef_mem_stat.diff.txt
Description: Text document

Follow-Ups:
- Re: [patch] un-#ifdef GATHER_STATISTICS
  - From: Richard Guenther
- Re: [patch] un-#ifdef GATHER_STATISTICS
  - From: Markus Trippelsdorf
- Re: [patch] un-#ifdef GATHER_STATISTICS
  - From: Jason Merrill

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]