This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
PATCH: PR middle-end/20218: Can't use __attribute__ ((visibility ("hidden"))) to hide a symbol
- From: "H. J. Lu" <hjl at lucon dot org>
- To: gcc-patches at gcc dot gnu dot org
- Date: Wed, 4 May 2005 21:52:46 -0700
- Subject: PATCH: PR middle-end/20218: Can't use __attribute__ ((visibility ("hidden"))) to hide a symbol
Here is the updated patch. It works for me on ia32, ia64 and x86_64.
H.J.
---
2005-05-04 H.J. Lu <hongjiu.lu@intel.com>
PR middle-end/20218
* config/elfos.h (ASM_OUTPUT_EXTERNAL): New.
(TARGET_ASM_FILE_END): New.
* config/i386/i386-protos.h (ix86_elf_file_end): New.
* config/i386/i386.c (ix86_elf_file_end): New.
* config/i386/linux.h (TARGET_ASM_FILE_END): Defined.
* config/i386/linux64.h (TARGET_ASM_FILE_END): Likewise.
* config/ia64/hpux.h (TARGET_ASM_FILE_END): Removed.
* config/ia64/ia64.c (ia64_hpux_add_extern_decl): Removed.
(ia64_hpux_file_end): Renamed to ...
(ia64_file_end): This. Handle symbol with non-default
visibility.
(TARGET_ASM_FILE_END): Defined.
(ia64_asm_output_external): Rewritten.
(extern_func_list): Removed.
(extern_func_head): Likewise.
* output.h (default_elf_asm_output_external): New.
(default_elf_asm_output_external_1): New.
(default_elf_asm_file_end): New.
(maybe_assemble_visibility): New.
* tree.h (extern_symbol_list): New.
(extern_symbol_head): New.
* varasm.c (maybe_assemble_visibility): Make it extern and
return int.
(extern_symbol_head): New.
(default_elf_asm_output_external_1): New.
(default_elf_asm_output_external): New.
(default_elf_asm_file_end): New.
--- gcc/config/elfos.h.global 2005-04-14 14:57:02.000000000 -0700
+++ gcc/config/elfos.h 2005-05-04 20:35:06.000000000 -0700
@@ -487,3 +487,17 @@ Boston, MA 02111-1307, USA. */
fprintf ((FILE), "\"\n"); \
} \
while (0)
+
+/* A C statement (sans semicolon) to output to the stdio stream STREAM
+ any text necessary for declaring the name of an external symbol
+ named NAME whch is referenced in this compilation but not defined.
+ It is needed to properly support non-default visibility. */
+
+#ifndef ASM_OUTPUT_EXTERNAL
+#define ASM_OUTPUT_EXTERNAL(FILE, DECL, NAME) \
+ default_elf_asm_output_external (FILE, DECL, NAME)
+#endif
+
+#ifndef TARGET_ASM_FILE_END
+#define TARGET_ASM_FILE_END default_elf_asm_file_end
+#endif
--- gcc/config/i386/i386-protos.h.global 2005-04-24 11:06:02.000000000 -0700
+++ gcc/config/i386/i386-protos.h 2005-05-04 20:35:06.000000000 -0700
@@ -28,6 +28,7 @@ extern int ix86_frame_pointer_required (
extern void ix86_setup_frame_addresses (void);
extern void ix86_file_end (void);
+extern void ix86_elf_file_end (void);
extern HOST_WIDE_INT ix86_initial_elimination_offset (int, int);
extern void ix86_expand_prologue (void);
extern void ix86_expand_epilogue (int);
--- gcc/config/i386/i386.c.global 2005-05-04 20:35:05.000000000 -0700
+++ gcc/config/i386/i386.c 2005-05-04 20:35:06.000000000 -0700
@@ -4103,6 +4103,13 @@ ix86_file_end (void)
file_end_indicate_exec_stack ();
}
+void
+ix86_elf_file_end (void)
+{
+ ix86_file_end ();
+ default_elf_asm_file_end ();
+}
+
/* Emit code for the SET_GOT patterns. */
const char *
--- gcc/config/i386/linux.h.global 2004-11-28 17:04:42.000000000 -0800
+++ gcc/config/i386/linux.h 2005-05-04 20:35:06.000000000 -0700
@@ -185,3 +185,6 @@ Boston, MA 02111-1307, USA. */
/* This macro may be overridden in i386/k*bsd-gnu.h. */
#define REG_NAME(reg) reg
+
+#undef TARGET_ASM_FILE_END
+#define TARGET_ASM_FILE_END ix86_elf_file_end
--- gcc/config/i386/linux64.h.global 2004-11-28 17:04:42.000000000 -0800
+++ gcc/config/i386/linux64.h 2005-05-04 20:35:06.000000000 -0700
@@ -73,3 +73,6 @@ Boston, MA 02111-1307, USA. */
/* This macro may be overridden in i386/k*bsd-gnu.h. */
#define REG_NAME(reg) reg
+
+#undef TARGET_ASM_FILE_END
+#define TARGET_ASM_FILE_END ix86_elf_file_end
--- gcc/config/ia64/hpux.h.global 2005-03-17 13:05:06.000000000 -0800
+++ gcc/config/ia64/hpux.h 2005-05-04 20:35:06.000000000 -0700
@@ -147,10 +147,6 @@ do { \
definitions, so do not use them in gthr-posix.h. */
#define GTHREAD_USE_WEAK 0
-/* Put out the needed function declarations at the end. */
-
-#define TARGET_ASM_FILE_END ia64_hpux_file_end
-
#undef CTORS_SECTION_ASM_OP
#define CTORS_SECTION_ASM_OP "\t.section\t.init_array,\t\"aw\",\"init_array\""
--- gcc/config/ia64/ia64.c.global 2005-04-28 16:11:38.000000000 -0700
+++ gcc/config/ia64/ia64.c 2005-05-04 21:07:52.000000000 -0700
@@ -234,6 +234,7 @@ static void bundling (FILE *, int, rtx,
static void ia64_output_mi_thunk (FILE *, tree, HOST_WIDE_INT,
HOST_WIDE_INT, tree);
static void ia64_file_start (void);
+static void ia64_file_end (void);
static void ia64_select_rtx_section (enum machine_mode, rtx,
unsigned HOST_WIDE_INT);
@@ -245,10 +246,6 @@ static void ia64_rwreloc_select_rtx_sect
unsigned HOST_WIDE_INT)
ATTRIBUTE_UNUSED;
static unsigned int ia64_section_type_flags (tree, const char *, int);
-static void ia64_hpux_add_extern_decl (tree decl)
- ATTRIBUTE_UNUSED;
-static void ia64_hpux_file_end (void)
- ATTRIBUTE_UNUSED;
static void ia64_init_libfuncs (void)
ATTRIBUTE_UNUSED;
static void ia64_hpux_init_libfuncs (void)
@@ -360,6 +357,9 @@ static const struct attribute_spec ia64_
#undef TARGET_ASM_FILE_START
#define TARGET_ASM_FILE_START ia64_file_start
+#undef TARGET_ASM_FILE_END
+#define TARGET_ASM_FILE_END ia64_file_end
+
#undef TARGET_RTX_COSTS
#define TARGET_RTX_COSTS ia64_rtx_costs
#undef TARGET_ADDRESS_COST
@@ -4493,43 +4493,24 @@ ia64_secondary_reload_class (enum reg_cl
the Intel assembler does not support undefined externals. */
void
-ia64_asm_output_external (FILE *file, tree decl, const char *name)
+ia64_asm_output_external (FILE *file ATTRIBUTE_UNUSED, tree decl,
+ const char *name)
{
- int save_referenced;
-
- /* GNU as does not need anything here, but the HP linker does need
- something for external functions. */
-
- if (TARGET_GNU_AS
- && (!TARGET_HPUX_LD
- || TREE_CODE (decl) != FUNCTION_DECL
- || strstr (name, "__builtin_") == name))
- return;
-
/* ??? The Intel assembler creates a reference that needs to be satisfied by
the linker when we do this, so we need to be careful not to do this for
builtin functions which have no library equivalent. Unfortunately, we
can't tell here whether or not a function will actually be called by
expand_expr, so we pull in library functions even if we may not need
them later. */
- if (! strcmp (name, "__builtin_next_arg")
- || ! strcmp (name, "alloca")
- || ! strcmp (name, "__builtin_constant_p")
- || ! strcmp (name, "__builtin_args_info"))
+ if (TREE_CODE (decl) == FUNCTION_DECL
+ && ((TARGET_GNU_AS && strstr (name, "__builtin_") == name)
+ || (!strcmp (name, "__builtin_next_arg")
+ || ! strcmp (name, "alloca")
+ || ! strcmp (name, "__builtin_constant_p")
+ || ! strcmp (name, "__builtin_args_info"))))
return;
- if (TARGET_HPUX_LD)
- ia64_hpux_add_extern_decl (decl);
- else
- {
- /* assemble_name will set TREE_SYMBOL_REFERENCED, so we must save and
- restore it. */
- save_referenced = TREE_SYMBOL_REFERENCED (DECL_ASSEMBLER_NAME (decl));
- if (TREE_CODE (decl) == FUNCTION_DECL)
- ASM_OUTPUT_TYPE_DIRECTIVE (file, name, "function");
- (*targetm.asm_out.globalize_label) (file, name);
- TREE_SYMBOL_REFERENCED (DECL_ASSEMBLER_NAME (decl)) = save_referenced;
- }
+ default_elf_asm_output_external_1 (decl);
}
/* Parse the -mfixed-range= option string. */
@@ -7902,36 +7883,16 @@ ia64_hpux_function_arg_padding (enum mac
return DEFAULT_FUNCTION_ARG_PADDING (mode, type);
}
-/* Linked list of all external functions that are to be emitted by GCC.
- We output the name if and only if TREE_SYMBOL_REFERENCED is set in
- order to avoid putting out names that are never really used. */
-
-struct extern_func_list GTY(())
-{
- struct extern_func_list *next;
- tree decl;
-};
-
-static GTY(()) struct extern_func_list *extern_func_head;
-
-static void
-ia64_hpux_add_extern_decl (tree decl)
-{
- struct extern_func_list *p = ggc_alloc (sizeof (struct extern_func_list));
-
- p->decl = decl;
- p->next = extern_func_head;
- extern_func_head = p;
-}
-
/* Print out the list of used global functions. */
static void
-ia64_hpux_file_end (void)
+ia64_file_end (void)
{
- struct extern_func_list *p;
+ struct extern_symbol_list *p;
+ int hpld = TARGET_HPUX_LD;
+ int gas = TARGET_GNU_AS;
- for (p = extern_func_head; p; p = p->next)
+ for (p = extern_symbol_head; p; p = p->next)
{
tree decl = p->decl;
tree id = DECL_ASSEMBLER_NAME (decl);
@@ -7939,18 +7900,28 @@ ia64_hpux_file_end (void)
gcc_assert (id);
if (!TREE_ASM_WRITTEN (decl) && TREE_SYMBOL_REFERENCED (id))
- {
+ {
+ int vis = ((*targetm.binds_local_p) (decl)
+ && maybe_assemble_visibility (decl));
const char *name = XSTR (XEXP (DECL_RTL (decl), 0), 0);
+ /* GNU as does not need anything here, but the HP linker
+ does need something for external functions. */
+ if ((hpld || !gas)
+ && TREE_CODE (decl) == FUNCTION_DECL)
+ {
+ ASM_OUTPUT_TYPE_DIRECTIVE (asm_out_file, name,
+ "function");
+ (*targetm.asm_out.globalize_label) (asm_out_file, name);
+ }
+ else if (vis && !gas)
+ (*targetm.asm_out.globalize_label) (asm_out_file, name);
+
TREE_ASM_WRITTEN (decl) = 1;
- (*targetm.asm_out.globalize_label) (asm_out_file, name);
- fputs (TYPE_ASM_OP, asm_out_file);
- assemble_name (asm_out_file, name);
- fprintf (asm_out_file, "," TYPE_OPERAND_FMT "\n", "function");
}
}
- extern_func_head = 0;
+ extern_symbol_head = 0;
}
/* Set SImode div/mod functions, init_integral_libfuncs only initializes
--- gcc/output.h.global 2005-04-28 16:11:31.000000000 -0700
+++ gcc/output.h 2005-05-04 20:35:06.000000000 -0700
@@ -543,6 +543,12 @@ extern void default_file_start (void);
extern void file_end_indicate_exec_stack (void);
extern bool default_valid_pointer_mode (enum machine_mode);
+extern void default_elf_asm_output_external (FILE *file, tree,
+ const char *);
+extern void default_elf_asm_output_external_1 (tree);
+extern void default_elf_asm_file_end (void);
+extern int maybe_assemble_visibility (tree);
+
extern int default_address_cost (rtx);
/* When performing hot/cold basic block partitioning, insert note in
--- gcc/tree.h.global 2005-05-04 11:20:53.000000000 -0700
+++ gcc/tree.h 2005-05-04 20:35:06.000000000 -0700
@@ -3769,6 +3769,17 @@ extern void process_pending_assemble_ext
extern void finish_aliases_1 (void);
extern void finish_aliases_2 (void);
+/* Linked list of all external symbols that are to be emitted by
+ GCC. */
+
+struct extern_symbol_list GTY(())
+{
+ struct extern_symbol_list *next;
+ tree decl;
+};
+
+extern GTY(()) struct extern_symbol_list *extern_symbol_head;
+
/* In stmt.c */
extern void expand_computed_goto (tree);
extern bool parse_output_constraint (const char **, int, int, int,
--- gcc/varasm.c.global 2005-05-04 20:35:05.000000000 -0700
+++ gcc/varasm.c 2005-05-04 20:35:06.000000000 -0700
@@ -123,7 +123,6 @@ static unsigned HOST_WIDE_INT array_size
static unsigned min_align (unsigned, unsigned);
static void output_constructor (tree, unsigned HOST_WIDE_INT, unsigned int);
static void globalize_decl (tree);
-static void maybe_assemble_visibility (tree);
static int in_named_entry_eq (const void *, const void *);
static hashval_t in_named_entry_hash (const void *);
static void initialize_cold_section_name (void);
@@ -4664,13 +4663,18 @@ default_assemble_visibility (tree decl,
/* A helper function to call assemble_visibility when needed for a decl. */
-static void
+int
maybe_assemble_visibility (tree decl)
{
enum symbol_visibility vis = DECL_VISIBILITY (decl);
if (vis != VISIBILITY_DEFAULT)
- targetm.asm_out.visibility (decl, vis);
+ {
+ targetm.asm_out.visibility (decl, vis);
+ return 1;
+ }
+ else
+ return 0;
}
/* Returns 1 if the target configuration supports defining public symbols
@@ -5496,4 +5500,64 @@ file_end_indicate_exec_stack (void)
named_section_flags (".note.GNU-stack", flags);
}
+struct extern_symbol_list *extern_symbol_head;
+
+void
+default_elf_asm_output_external_1 (tree decl)
+{
+ struct extern_symbol_list *p
+ = ggc_alloc (sizeof (struct extern_symbol_list));
+
+ p->decl = decl;
+ p->next = extern_symbol_head;
+ extern_symbol_head = p;
+}
+
+/* Emit text to declare externally defined symbols. It is needed to
+ properly support non-default visibility. */
+
+void
+default_elf_asm_output_external (FILE *file ATTRIBUTE_UNUSED,
+ tree decl,
+ const char *name)
+{
+ /* Ignore builtin functions. */
+ if (TREE_CODE (decl) == FUNCTION_DECL
+ && strstr (name, "__builtin_") == name)
+ return;
+ else
+ default_elf_asm_output_external_1 (decl);
+}
+
+/* Print out the list of referenced global symbols with non-default
+ visibility. */
+
+void
+default_elf_asm_file_end (void)
+{
+ struct extern_symbol_list *p;
+
+ for (p = extern_symbol_head; p; p = p->next)
+ {
+ tree decl = p->decl;
+ tree id = DECL_ASSEMBLER_NAME (decl);
+
+ if (!id)
+ abort ();
+
+ /* We output the name if and only if TREE_SYMBOL_REFERENCED is
+ set in order to avoid putting out names that are never really
+ used. */
+ if (targetm.binds_local_p (decl)
+ && !TREE_ASM_WRITTEN (decl)
+ && TREE_SYMBOL_REFERENCED (id))
+ {
+ maybe_assemble_visibility (decl);
+ TREE_ASM_WRITTEN (decl) = 1;
+ }
+ }
+
+ extern_symbol_head = 0;
+}
+
#include "gt-varasm.h"