Patchwork Remove and poison a few more old target macros

login
register
mail settings
Submitter Steven Bosscher
Date June 24, 2012, 2:24 p.m.
Message ID <CABu31nP7jSNq1cZtGuCcGDpGbcup+33T-DuF_YTSJH3=So2Mwg@mail.gmail.com>
Download mbox | patch
Permalink /patch/166886/
State New
Headers show

Comments

Steven Bosscher - June 24, 2012, 2:24 p.m.
On Sun, Jun 24, 2012 at 3:42 PM, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
> Like the ones before: they are unused.
> Will commit as an obvious cleanup in a few days' time.

...and one more...

        * system.h (USE_COMMON_FOR_ONE_ONLY): Poison.
        * defaults.h (USE_COMMON_FOR_ONE_ONLY): Do not provide default.
        * config/darwin.h (USE_COMMON_FOR_ONE_ONLY): Do not define.

Patch

Index: system.h
===================================================================
--- system.h    (revision 188856)
+++ system.h    (working copy)
@@ -888,7 +888,7 @@  extern void fancy_abort (const char *, int, const
        PREFERRED_OUTPUT_RELOAD_CLASS SYSTEM_INCLUDE_DIR                   \
        STANDARD_INCLUDE_DIR STANDARD_INCLUDE_COMPONENT                    \
        LINK_ELIMINATE_DUPLICATE_LDIRECTORIES MIPS_DEBUGGING_INFO          \
-       IDENT_ASM_OP
+       IDENT_ASM_OP USE_COMMON_FOR_ONE_ONLY

 /* Hooks that are no longer used.  */
  #pragma GCC poison LANG_HOOKS_FUNCTION_MARK LANG_HOOKS_FUNCTION_FREE  \
Index: defaults.h
===================================================================
--- defaults.h  (revision 188856)
+++ defaults.h  (working copy)
@@ -332,12 +332,6 @@  see the files COPYING3 and COPYING.RUNTIME respect
 # endif
 #endif

-/* Determines whether we may use common symbols to represent one-only
-   semantics (a.k.a. "vague linkage").  */
-#ifndef USE_COMMON_FOR_ONE_ONLY
-# define USE_COMMON_FOR_ONE_ONLY 1
-#endif
-
 /* By default we can assume that all global symbols are in one namespace,
    across all shared libraries.  */
 #ifndef MULTIPLE_SYMBOL_SPACES
Index: config/darwin.h
===================================================================
--- config/darwin.h     (revision 188856)
+++ config/darwin.h     (working copy)
@@ -464,13 +464,6 @@  extern GTY(()) int darwin_ms_struct;
    links to, so there's no need for weak-ness for that.  */
 #define GTHREAD_USE_WEAK 0

-/* The Darwin linker imposes two limitations on common symbols: they
-   can't have hidden visibility, and they can't appear in dylibs.  As
-   a consequence, we should never use common symbols to represent
-   vague linkage. */
-#undef USE_COMMON_FOR_ONE_ONLY
-#define USE_COMMON_FOR_ONE_ONLY 0
-
 /* The Darwin linker doesn't want coalesced symbols to appear in
    a static archive's table of contents. */
 #undef TARGET_WEAK_NOT_IN_ARCHIVE_TOC