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/RFA] Make *_DEBUGGING_INFO defines consistent

From: Jason R Thorpe <thorpej at wasabisystems dot com>
To: gcc-patches at gcc dot gnu dot org
Date: Thu, 29 Aug 2002 09:16:48 -0700
Subject: [PATCH/RFA] Make *_DEBUGGING_INFO defines consistent
Organization: Wasabi Systems, Inc.

The vast majority of places that define *_DEBUGGING_INFO do it like so:

#define DBX_DEBUGGING_INFO

However, a few define with a value, e.g.:

#define DBX_DEBUGGING_INFO 1

Defining with a value is unnecessary, and causes "redefined" warnings on
e.g. the mips-netbsd target (because elfos.h is included before mips/elf.h).

The following patch makes them all consistent (i.e. no value).  OK to
commit?

	* config/elfos.h (DWARF_DEBUGGING_INFO)
	(DWARF2_DEBUGGING_INFO): Define with no value.
	* config/tm-dwarf2.h (DWARF2_DEBUGGING_INFO): Likewise.
	* config/arm/aout.h (DBX_DEBUGGING_INFO): Likewise.
	* config/arm/netbsd.h (DBX_DEBUGGING_INFO): Likewise.
	* config/i386/sco5.h (DWARF2_DEBUGGING_INFO)
	(DWARF_DEBUGGING_INFO, SDB_DEBUGGING_INFO)
	(DBX_DEBUGGING_INFO): Likewise.
	* config/ip2k/ip2k.h (DWARF2_DEBUGGING_INFO): Likewise.
	* config/mcore/mcore-elf.h (DWARF2_DEBUGGING_INFO)
	(DWARF_DEBUGGING_INFO): Likewise.
	* config/mcore/mcore-pe.h (DBX_DEBUGGING_INFO): Likewise.
	* config/sh/elf.h (DWARF2_DEBUGGING_INFO): Likewise.

-- 
        -- Jason R. Thorpe <thorpej@wasabisystems.com>

Attachment: debug-define-patch
Description: Text document

Follow-Ups:
- Re: [PATCH/RFA] Make *_DEBUGGING_INFO defines consistent
  - From: Mark Mitchell
- Re: [PATCH/RFA] Make *_DEBUGGING_INFO defines consistent
  - From: Zack Weinberg

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