diff options
| -rw-r--r-- | ld/ChangeLog | 24 | ||||
| -rw-r--r-- | ld/ldctor.c | 223 | ||||
| -rw-r--r-- | ld/scripttempl/elf.sc | 73 |
3 files changed, 271 insertions, 49 deletions
diff --git a/ld/ChangeLog b/ld/ChangeLog index a38b01c..ce8d936 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,27 @@ +Fri Aug 14 15:34:29 1998 Ian Lance Taylor <ian@cygnus.com> + + * ldgram.y: Include "ldctor.h". + (statement): Accept SORT around CONSTRUCTORS. + * ldctor.c: Include <ctype.h>. + (constructors_sorted): New global variable. + (ctor_prio, ctor_cmp): New static functions. + (ldctor_build_sets): Sort constructors if requested. + * ldctor.h (constructors_sorted): Declare. + * ldlang.c (print_statement): Print sorted CONSTRUCTORS + correctly. + * scripttempl/elf.sc: Add sort around CONSTRUCTORS. + * ld.texinfo (Output Section Keywords): Document SORT + (CONSTRUCTORS). + +start-sanitize-armelf +Thu Aug 13 12:20:39 1998 Catherine Moore <clm@cygnus.com> + + * emulparams/armelf.sh: Define TEMPLATE_NAME to armelf. + * scripttempl/elfarm.sc: Include .glue_7t and .glue7 + sections. + * emultempl/armelf.em: New file. + +end-sanitize-armelf Thu Aug 13 12:52:45 1998 H.J. Lu <hjl@gnu.org> * Makefile.am (ld.dvi): Use " instead of ' for MAKEINFO. diff --git a/ld/ldctor.c b/ld/ldctor.c index 8ba866c..ab810cd 100644 --- a/ld/ldctor.c +++ b/ld/ldctor.c @@ -1,5 +1,6 @@ /* ldctor.c -- constructor support routines - Copyright (C) 1991, 92, 93, 94 Free Software Foundation, Inc. + Copyright (C) 1991, 92, 93, 94, 95, 96, 97, 1998 + Free Software Foundation, Inc. By Steve Chamberlain <sac@cygnus.com> This file is part of GLD, the Gnu Linker. @@ -15,13 +16,16 @@ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License -along with GLD; see the file COPYING. If not, write to -the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA. */ +along with GLD; see the file COPYING. If not, write to the Free +Software Foundation, 59 Temple Place - Suite 330, Boston, MA +02111-1307, USA. */ #include "bfd.h" #include "sysdep.h" #include "bfdlink.h" +#include <ctype.h> + #include "ld.h" #include "ldexp.h" #include "ldlang.h" @@ -30,31 +34,20 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA. */ #include "ldmain.h" #include "ldctor.h" +static int ctor_prio PARAMS ((const char *)); +static int ctor_cmp PARAMS ((const PTR, const PTR)); + /* The list of statements needed to handle constructors. These are invoked by the command CONSTRUCTORS in the linker script. */ lang_statement_list_type constructor_list; -/* We keep a list of these structures for each sets we build. */ - -struct set_info -{ - struct set_info *next; /* Next set. */ - struct bfd_link_hash_entry *h; /* Hash table entry. */ - bfd_reloc_code_real_type reloc; /* Reloc to use for an entry. */ - size_t count; /* Number of elements. */ - struct set_element *elements; /* Elements in set. */ -}; - -struct set_element -{ - struct set_element *next; /* Next element. */ - asection *section; /* Section of value. */ - bfd_vma value; /* Value. */ -}; +/* Whether the constructors should be sorted. Note that this is + global for the entire link; we assume that there is only a single + CONSTRUCTORS command in the linker script. */ +boolean constructors_sorted; /* The sets we have seen. */ - -static struct set_info *sets; +struct set_info *sets; /* Add an entry to a set. H is the entry in the linker hash table. RELOC is the relocation to use for an entry in the set. SECTION @@ -64,9 +57,10 @@ static struct set_info *sets; function will construct the sets. */ void -ldctor_add_set_entry (h, reloc, section, value) +ldctor_add_set_entry (h, reloc, name, section, value) struct bfd_link_hash_entry *h; bfd_reloc_code_real_type reloc; + const char *name; asection *section; bfd_vma value; { @@ -92,7 +86,7 @@ ldctor_add_set_entry (h, reloc, section, value) { if (p->reloc != reloc) { - einfo ("%P%X: Different relocs used in set %s\n", h->root.string); + einfo (_("%P%X: Different relocs used in set %s\n"), h->root.string); return; } @@ -109,7 +103,7 @@ ldctor_add_set_entry (h, reloc, section, value) && strcmp (bfd_get_target (section->owner), bfd_get_target (p->elements->section->owner)) != 0) { - einfo ("%P%X: Different object file formats composing set %s\n", + einfo (_("%P%X: Different object file formats composing set %s\n"), h->root.string); return; } @@ -117,6 +111,7 @@ ldctor_add_set_entry (h, reloc, section, value) e = (struct set_element *) xmalloc (sizeof (struct set_element)); e->next = NULL; + e->name = name; e->section = section; e->value = value; @@ -127,6 +122,81 @@ ldctor_add_set_entry (h, reloc, section, value) ++p->count; } +/* Get the priority of a g++ global constructor or destructor from the + symbol name. */ + +static int +ctor_prio (name) + const char *name; +{ + /* The name will look something like _GLOBAL_$I$65535$test02__Fv. + There might be extra leading underscores, and the $ characters + might be something else. The I might be a D. */ + + while (*name == '_') + ++name; + + if (strncmp (name, "GLOBAL_", sizeof "GLOBAL_" - 1) != 0) + return -1; + + name += sizeof "GLOBAL_" - 1; + + if (name[0] != name[2]) + return -1; + if (name[1] != 'I' && name[1] != 'D') + return -1; + if (! isdigit ((unsigned char) name[3])) + return -1; + + return atoi (name + 3); +} + +/* This function is used to sort constructor elements by priority. It + is called via qsort. */ + +static int +ctor_cmp (p1, p2) + const PTR p1; + const PTR p2; +{ + const struct set_element **pe1 = (const struct set_element **) p1; + const struct set_element **pe2 = (const struct set_element **) p2; + const char *n1; + const char *n2; + int prio1; + int prio2; + int ret; + + n1 = (*pe1)->name; + if (n1 == NULL) + n1 = ""; + n2 = (*pe2)->name; + if (n2 == NULL) + n2 = ""; + + /* We need to sort in reverse order by priority. When two + constructors have the same priority, we should maintain their + current relative position. */ + + prio1 = ctor_prio (n1); + prio2 = ctor_prio (n2); + + /* We sort in reverse order because that is what g++ expects. */ + if (prio1 < prio2) + return 1; + else if (prio1 > prio2) + return -1; + + /* Force a stable sort. */ + + if (pe1 < pe2) + return -1; + else if (pe1 > pe2) + return 1; + else + return 0; +} + /* This function is called after the first phase of the link and before the second phase. At this point all set information has been gathered. We now put the statements to build the sets @@ -135,24 +205,70 @@ ldctor_add_set_entry (h, reloc, section, value) void ldctor_build_sets () { + static boolean called; lang_statement_list_type *old; + boolean header_printed; struct set_info *p; + /* The emulation code may call us directly, but we only want to do + this once. */ + if (called) + return; + called = true; + + if (constructors_sorted) + { + for (p = sets; p != NULL; p = p->next) + { + int c, i; + struct set_element *e; + struct set_element **array; + + if (p->elements == NULL) + continue; + + c = 0; + for (e = p->elements; e != NULL; e = e->next) + ++c; + + array = (struct set_element **) xmalloc (c * sizeof *array); + + i = 0; + for (e = p->elements; e != NULL; e = e->next) + { + array[i] = e; + ++i; + } + + qsort (array, c, sizeof *array, ctor_cmp); + + e = array[0]; + p->elements = e; + for (i = 0; i < c - 1; i++) + array[i]->next = array[i + 1]; + array[i]->next = NULL; + + free (array); + } + } + old = stat_ptr; stat_ptr = &constructor_list; lang_list_init (stat_ptr); + header_printed = false; for (p = sets; p != (struct set_info *) NULL; p = p->next) { struct set_element *e; reloc_howto_type *howto; - int size; + int reloc_size, size; /* If the symbol is defined, we may have been invoked from collect, and the sets may already have been built, so we do not do anything. */ - if (p->h->type == bfd_link_hash_defined) + if (p->h->type == bfd_link_hash_defined + || p->h->type == bfd_link_hash_defweak) continue; /* For each set we build: @@ -170,7 +286,7 @@ ldctor_build_sets () { if (link_info.relocateable) { - einfo ("%P%X: %s does not support reloc %s for set %s\n", + einfo (_("%P%X: %s does not support reloc %s for set %s\n"), bfd_get_target (output_bfd), bfd_get_reloc_code_name (p->reloc), p->h->root.string); @@ -183,7 +299,7 @@ ldctor_build_sets () p->reloc); if (howto == NULL) { - einfo ("%P%X: %s does not support reloc %s for set %s\n", + einfo (_("%P%X: %s does not support reloc %s for set %s\n"), bfd_get_target (p->elements->section->owner), bfd_get_reloc_code_name (p->reloc), p->h->root.string); @@ -191,28 +307,67 @@ ldctor_build_sets () } } - switch (bfd_get_reloc_size (howto)) + reloc_size = bfd_get_reloc_size (howto); + switch (reloc_size) { case 1: size = BYTE; break; case 2: size = SHORT; break; case 4: size = LONG; break; - case 8: size = QUAD; break; + case 8: + if (howto->complain_on_overflow == complain_overflow_signed) + size = SQUAD; + else + size = QUAD; + break; default: - einfo ("%P%X: Unsupported size %d for set %s\n", + einfo (_("%P%X: Unsupported size %d for set %s\n"), bfd_get_reloc_size (howto), p->h->root.string); size = LONG; break; } + lang_add_assignment (exp_assop ('=', ".", + exp_unop (ALIGN_K, + exp_intop (reloc_size)))); lang_add_assignment (exp_assop ('=', p->h->root.string, exp_nameop (NAME, "."))); lang_add_data (size, exp_intop ((bfd_vma) p->count)); for (e = p->elements; e != (struct set_element *) NULL; e = e->next) { + if (config.map_file != NULL) + { + int len; + + if (! header_printed) + { + minfo (_("\nSet Symbol\n\n")); + header_printed = true; + } + + minfo ("%s", p->h->root.string); + len = strlen (p->h->root.string); + + if (len >= 19) + { + print_nl (); + len = 0; + } + while (len < 20) + { + print_space (); + ++len; + } + + if (e->name != NULL) + minfo ("%T\n", e->name); + else + minfo ("%G\n", e->section->owner, e->section, e->value); + } + if (link_info.relocateable) - lang_add_reloc (p->reloc, howto, e->section, - (const char *) NULL, exp_intop (e->value)); + lang_add_reloc (p->reloc, howto, e->section, e->name, + exp_intop (e->value)); else lang_add_data (size, exp_relop (e->section, e->value)); } diff --git a/ld/scripttempl/elf.sc b/ld/scripttempl/elf.sc index 5fceace..5cdef33 100644 --- a/ld/scripttempl/elf.sc +++ b/ld/scripttempl/elf.sc @@ -24,6 +24,7 @@ # EMBEDDED - whether this is for an embedded system. # SHLIB_TEXT_START_ADDR - if set, add to SIZEOF_HEADERS to set # start address of shared library. +# INPUT_FILES - INPUT command of files to always include # # When adding sections, do note that the names of some sections are used # when specifying the start address of the next. @@ -34,6 +35,7 @@ test -z "${BIG_OUTPUT_FORMAT}" && BIG_OUTPUT_FORMAT=${OUTPUT_FORMAT} test -z "${LITTLE_OUTPUT_FORMAT}" && LITTLE_OUTPUT_FORMAT=${OUTPUT_FORMAT} if [ -z "$MACHINE" ]; then OUTPUT_ARCH=${ARCH}; else OUTPUT_ARCH=${ARCH}:${MACHINE}; fi test -z "${ELFSIZE}" && ELFSIZE=32 +test -z "${ALIGNMENT}" && ALIGNMENT="${ELFSIZE} / 8" test "$LD_FLAG" = "N" && DATA_ADDR=. INTERP=".interp ${RELOCATING-0} : { *(.interp) }" PLT=".plt ${RELOCATING-0} : { *(.plt) }" @@ -56,10 +58,12 @@ ${RELOCATING+${LIB_SEARCH_DIRS}} ${RELOCATING+/* Do we need any of these for elf? __DYNAMIC = 0; ${STACKZERO+${STACKZERO}} ${SHLIB_PATH+${SHLIB_PATH}} */} ${RELOCATING+${EXECUTABLE_SYMBOLS}} +${RELOCATING+${INPUT_FILES}} ${RELOCATING- /* For some reason, the Solaris linker makes bad executables if gld -r is used and the intermediate file has sections starting at non-zero addresses. Could be a Solaris ld bug, could be a GNU ld bug. But for now assigning the zero vmas works. */} + SECTIONS { /* Read-only sections, merged into text segment: */ @@ -75,17 +79,41 @@ SECTIONS .gnu.version_d ${RELOCATING-0} : { *(.gnu.version_d) } .gnu.version_r ${RELOCATING-0} : { *(.gnu.version_r) } .rel.text ${RELOCATING-0} : - { *(.rel.text) ${RELOCATING+*(.rel.gnu.linkonce.t*)} } + { + *(.rel.text) + ${RELOCATING+*(.rel.text.*)} + ${RELOCATING+*(.rel.gnu.linkonce.t*)} + } .rela.text ${RELOCATING-0} : - { *(.rela.text) ${RELOCATING+*(.rela.gnu.linkonce.t*)} } + { + *(.rela.text) + ${RELOCATING+*(.rela.text.*)} + ${RELOCATING+*(.rela.gnu.linkonce.t*)} + } .rel.data ${RELOCATING-0} : - { *(.rel.data) ${RELOCATING+*(.rel.gnu.linkonce.d*)} } + { + *(.rel.data) + ${RELOCATING+*(.rel.data.*)} + ${RELOCATING+*(.rel.gnu.linkonce.d*)} + } .rela.data ${RELOCATING-0} : - { *(.rela.data) ${RELOCATING+*(.rela.gnu.linkonce.d*)} } + { + *(.rela.data) + ${RELOCATING+*(.rela.data.*)} + ${RELOCATING+*(.rela.gnu.linkonce.d*)} + } .rel.rodata ${RELOCATING-0} : - { *(.rel.rodata) ${RELOCATING+*(.rel.gnu.linkonce.r*)} } + { + *(.rel.rodata) + ${RELOCATING+*(.rel.rodata.*)} + ${RELOCATING+*(.rel.gnu.linkonce.r*)} + } .rela.rodata ${RELOCATING-0} : - { *(.rela.rodata) ${RELOCATING+*(.rela.gnu.linkonce.r*)} } + { + *(.rela.rodata) + ${RELOCATING+*(.rela.rodata.*)} + ${RELOCATING+*(.rela.gnu.linkonce.r*)} + } .rel.got ${RELOCATING-0} : { *(.rel.got) } .rela.got ${RELOCATING-0} : { *(.rela.got) } .rel.ctors ${RELOCATING-0} : { *(.rel.ctors) } @@ -100,12 +128,13 @@ SECTIONS .rela.bss ${RELOCATING-0} : { *(.rela.bss) } .rel.plt ${RELOCATING-0} : { *(.rel.plt) } .rela.plt ${RELOCATING-0} : { *(.rela.plt) } - .init ${RELOCATING-0} : { *(.init) } =${NOP-0} + .init ${RELOCATING-0} : { KEEP (*(.init)) } =${NOP-0} ${DATA_PLT-${PLT}} .text ${RELOCATING-0} : { ${RELOCATING+${TEXT_START_SYMBOLS}} *(.text) + ${RELOCATING+*(.text.*)} *(.stub) /* .gnu.warning sections are handled specially by elf32.em. */ *(.gnu.warning) @@ -114,8 +143,13 @@ SECTIONS } =${NOP-0} ${RELOCATING+_etext = .;} ${RELOCATING+PROVIDE (etext = .);} - .fini ${RELOCATING-0} : { *(.fini) } =${NOP-0} - .rodata ${RELOCATING-0} : { *(.rodata) ${RELOCATING+*(.gnu.linkonce.r*)} } + .fini ${RELOCATING-0} : { KEEP (*(.fini)) } =${NOP-0} + .rodata ${RELOCATING-0} : + { + *(.rodata) + ${RELOCATING+*(.rodata.*)} + ${RELOCATING+*(.gnu.linkonce.r*)} + } .rodata1 ${RELOCATING-0} : { *(.rodata1) } ${RELOCATING+${OTHER_READONLY_SECTIONS}} @@ -128,23 +162,32 @@ SECTIONS { ${RELOCATING+${DATA_START_SYMBOLS}} *(.data) + ${RELOCATING+*(.data.*)} ${RELOCATING+*(.gnu.linkonce.d*)} - ${CONSTRUCTING+CONSTRUCTORS} + ${CONSTRUCTING+SORT(CONSTRUCTORS)} } .data1 ${RELOCATING-0} : { *(.data1) } ${RELOCATING+${OTHER_READWRITE_SECTIONS}} .ctors ${RELOCATING-0} : { ${CONSTRUCTING+${CTOR_START}} - *(SORT(.ctors.*)) - *(.ctors) + /* gcc uses crtbegin.o to find the start of the constructors, so + we make sure it is first. Because this is a wildcard, it + doesn't matter if the user does not actually link against + crtbegin.o; the linker won't look for a file to match a + wildcard. The wildcard also means that it doesn't matter which + directory crtbegin.o is in. */ + KEEP (*crtbegin.o(.ctors)) + KEEP (*(SORT(.ctors.*))) + KEEP (*(.ctors)) ${CONSTRUCTING+${CTOR_END}} } .dtors ${RELOCATING-0} : { ${CONSTRUCTING+${DTOR_START}} - *(SORT(.dtors.*)) - *(.dtors) + KEEP (*crtbegin.o(.dtors)) + KEEP (*(SORT(.dtors.*))) + KEEP (*(.dtors)) ${CONSTRUCTING+${DTOR_END}} } ${DATA_PLT+${PLT}} @@ -167,7 +210,7 @@ SECTIONS *(.bss) *(COMMON) } - ${RELOCATING+. = ALIGN(${ELFSIZE} / 8);} + ${RELOCATING+. = ALIGN(${ALIGNMENT});} ${RELOCATING+_end = . ;} ${RELOCATING+PROVIDE (end = .);} |