This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: C++ PATCH: cleanup DEBUG_BINDING_LEVELS code
- From: Mark Mitchell <mark at codesourcery dot com>
- To: Gabriel Dos Reis <gdr at integrable-solutions dot net>
- Cc: gcc-patches at gcc dot gnu dot org, jason at redhat dot com
- Date: 29 May 2003 09:30:52 -0700
- Subject: Re: C++ PATCH: cleanup DEBUG_BINDING_LEVELS code
- References: <m3llwppvsg.fsf@uniton.integrable-solutions.net>
> While working on the name-lookup performance issue, I came to turn
> on DEBUG_BINDING_LEVELS and realize that nobody has been turning it on
> for some while. This patch (which was dormant in my local tree)
> cleans up the DEBUG_BINDING_LEVELS code so as to make it bootstrap
> again with the -Werror option.
Why can't we just use this idiom:
#ifndef DEBUG_BINDING_LEVELS
#define DEBUG_BINDING_LEVELS 0
#else
#define DEBUG_BINDING_LEVELS 1
#endif
...
if (DEBUG_BINDING_LEVELS) {
...
}
Why do we need another macro?
--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com