This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] ARM: Don't use weak linkage when comdat groups are available
- From: Julian Brown <julian at codesourcery dot com>
- To: Julian Brown <julian at codesourcery dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Paul Brook <paul at codesourcery dot com>
- Date: Mon, 16 May 2005 13:43:26 +0100
- Subject: Re: [PATCH] ARM: Don't use weak linkage when comdat groups are available
- References: <428894A7.1090402@codesourcery.com>
Julian Brown wrote:
ChangeLog:
* config/arm/arm.h (MAKE_DECL_ONE_ONLY): Redefine macro to stop
DECL_WEAK being used for symbols with vague linkage when
HAVE_COMDAT_GROUP is true.
...and with the patch...
Julian
Only in ./CVS: Entries.Log
diff -urp ../gcc-nopatch/gcc/config/arm/arm.h ./gcc/config/arm/arm.h
--- ../gcc-nopatch/gcc/config/arm/arm.h 2005-05-14 00:25:01.000000000 +0100
+++ ./gcc/config/arm/arm.h 2005-05-13 22:22:03.000000000 +0100
@@ -2247,6 +2247,18 @@ extern int making_const_table;
/* The arm5 clz instruction returns 32. */
#define CLZ_DEFINED_VALUE_AT_ZERO(MODE, VALUE) ((VALUE) = 32, 1)
+/* Don't use weak for entities with vague linkage when HAVE_COMDAT_GROUP
+ is true. */
+#undef MAKE_DECL_ONE_ONLY
+#define MAKE_DECL_ONE_ONLY(DECL) \
+ do \
+ { \
+ if (!HAVE_COMDAT_GROUP) \
+ DECL_WEAK(DECL) = 1; \
+ } \
+ while (0)
+
+
#undef ASM_APP_OFF
#define ASM_APP_OFF (TARGET_THUMB ? "\t.code\t16\n" : "")