From 8fbb09e83e588193462a3c488466cb4d1a80ba98 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Thu, 6 Dec 2012 00:25:19 +0000 Subject: [PATCH] Extend --copy-dt-needed-entries to creating DSO ld/ PR ld/14915 * emultempl/elf32.em (gld${EMULATION_NAME}_after_open): Also check DT_NEEDED entries when creating shared object with --copy-dt-needed-entries. ld/testsuite/ PR ld/14915 * ld-elf/shared.exp (build_tests): Test --add-needed, --copy-dt-needed-entries, --no-add-needed and --no-copy-dt-needed-entries with -shared. Add tests for --no-add-needed and --no-copy-dt-needed-entries with -shared. --- ld/ChangeLog | 7 +++++++ ld/emultempl/elf32.em | 10 ++++++---- ld/testsuite/ChangeLog | 9 +++++++++ ld/testsuite/ld-elf/shared.exp | 29 +++++++++++++++++++++++++++++ 4 files changed, 51 insertions(+), 4 deletions(-) diff --git a/ld/ChangeLog b/ld/ChangeLog index 09c71111428..ec911a158c5 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,10 @@ +2012-12-05 H.J. Lu + + PR ld/14915 + * emultempl/elf32.em (gld${EMULATION_NAME}_after_open): Also + check DT_NEEDED entries when creating shared object with + --copy-dt-needed-entries. + 2012-12-05 Leif Ekblad * configure.tgt: Add x86_64-*-rdos. diff --git a/ld/emultempl/elf32.em b/ld/emultempl/elf32.em index 6c84b82d4ed..d30a0ad4045 100644 --- a/ld/emultempl/elf32.em +++ b/ld/emultempl/elf32.em @@ -1181,8 +1181,6 @@ gld${EMULATION_NAME}_after_open (void) special action by the person doing the link. Note that the needed list can actually grow while we are stepping through this loop. */ - if (!link_info.executable) - return; needed = bfd_elf_get_needed_list (link_info.output_bfd, &link_info); for (l = needed; l != NULL; l = l->next) { @@ -1191,9 +1189,13 @@ gld${EMULATION_NAME}_after_open (void) int force; /* If the lib that needs this one was --as-needed and wasn't - found to be needed, then this lib isn't needed either. */ + found to be needed, then this lib isn't needed either. Skip + the lib when creating a shared object unless we are copying + DT_NEEDED entres. */ if (l->by != NULL - && (bfd_elf_get_dyn_lib_class (l->by) & DYN_AS_NEEDED) != 0) + && ((bfd_elf_get_dyn_lib_class (l->by) & DYN_AS_NEEDED) != 0 + || (!link_info.executable + && bfd_elf_get_dyn_lib_class (l->by) & DYN_NO_ADD_NEEDED) != 0)) continue; /* If we've already seen this file, skip it. */ diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog index 5b1bc8fcfef..64632bc1f37 100644 --- a/ld/testsuite/ChangeLog +++ b/ld/testsuite/ChangeLog @@ -1,3 +1,12 @@ +2012-12-05 H.J. Lu + + PR ld/14915 + * ld-elf/shared.exp (build_tests): Test --add-needed, + --copy-dt-needed-entries, --no-add-needed and + --no-copy-dt-needed-entries with -shared. + Add tests for --no-add-needed and --no-copy-dt-needed-entries + with -shared. + 2012-12-06 Alan Modra * ld-elf/comm-data4.d, ld-elf/comm-data4.s: New test. diff --git a/ld/testsuite/ld-elf/shared.exp b/ld/testsuite/ld-elf/shared.exp index 2e679d9fc7c..eef264e1581 100644 --- a/ld/testsuite/ld-elf/shared.exp +++ b/ld/testsuite/ld-elf/shared.exp @@ -188,6 +188,21 @@ set build_tests { {"Build libneeded1c.o" "-r -nostdlib" "" {needed1c.c} {} "libneeded1c.o"} + {"Build libneeded1pic.o" + "-r -nostdlib" "-fPIC" + {needed1c.c} {} "libneeded1pic.o"} + {"Build needed1a.so with --add-needed" + "-shared tmpdir/libneeded1pic.o -Wl,--add-needed,-rpath=tmpdir,-z,defs -Ltmpdir -lneeded1a" "" + {dummy.c} {} "needed1a.so"} + {"Build needed1b.so with --copy-dt-needed-entries" + "-shared tmpdir/libneeded1pic.o -Wl,--copy-dt-needed-entries,-rpath=tmpdir,-z,defs -Ltmpdir -lneeded1a" "" + {dummy.c} {} "needed1b.so"} + {"Build needed1a.so with --no-add-needed" + "-shared tmpdir/libneeded1pic.o -Wl,--no-add-needed -Ltmpdir -lneeded1a" "" + {dummy.c} {} "needed1c.so"} + {"Build needed1b.so with --no-copy-dt-needed-entries" + "-shared tmpdir/libneeded1pic.o -Wl,--no-copy-dt-needed-entries -Ltmpdir -lneeded1a" "" + {dummy.c} {} "needed1d.so"} } run_cc_link_tests $build_tests @@ -352,6 +367,20 @@ if { [ regexp "'bar' is defined in DSO tmpdir/libneeded1b.so" $exec_output ] } { } { fail $testname } +set testname "--no-add-needed -shared" +set exec_output [run_host_cmd "$CC" "$gcc_gas_flag $gcc_ld_flag -shared tmpdir/libneeded1pic.o -Wl,--no-add-needed,-z,defs -Ltmpdir -lneeded1a"] +if { [ regexp "undefined reference to `bar'" $exec_output ] } { + pass $testname +} { + fail $testname +} +set testname "--no-copy-dt-needed-entries -shared" +set exec_output [run_host_cmd "$CC" "$gcc_gas_flag $gcc_ld_flag -shared tmpdir/libneeded1pic.o -Wl,--no-copy-dt-needed-entries,-z,defs -Ltmpdir -lneeded1a"] +if { [ regexp "undefined reference to `bar'" $exec_output ] } { + pass $testname +} { + fail $testname +} # Check to see if the C++ compiler works if { [which $CXX] == 0 } { -- 2.30.2