binutils: add support for non-pic handling and use it in the codesourcery based gcc...
authorFelix Fietkau <nbd@openwrt.org>
Mon, 19 Oct 2009 07:41:33 +0000 (07:41 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Mon, 19 Oct 2009 07:41:33 +0000 (07:41 +0000)
SVN-Revision: 18070

toolchain/binutils/patches/2.19.1/200-mips_non_pic.patch [new file with mode: 0644]
toolchain/gcc/patches/4.3.3+cs/001-no_mips_nonpic_default.patch [deleted file]

diff --git a/toolchain/binutils/patches/2.19.1/200-mips_non_pic.patch b/toolchain/binutils/patches/2.19.1/200-mips_non_pic.patch
new file mode 100644 (file)
index 0000000..2d3d8de
--- /dev/null
@@ -0,0 +1,202 @@
+--- a/bfd/elf32-mips.c
++++ b/bfd/elf32-mips.c
+@@ -1662,6 +1662,15 @@ static const struct ecoff_debug_swap mip
+ #define elf_backend_plt_readonly      1
+ #define elf_backend_plt_sym_val               _bfd_mips_elf_plt_sym_val
++/* Most MIPS ELF files do not contain a traditional PLT; only VxWorks
++   and non-PIC dynamic executables do.  These settings only affect
++   _bfd_elf_create_dynamic_sections, which is only called when we
++   do want a traditional PLT.  */
++#undef elf_backend_want_plt_sym
++#define elf_backend_want_plt_sym              1
++#undef elf_backend_plt_readonly
++#define elf_backend_plt_readonly              1
++
+ #define elf_backend_discard_info      _bfd_mips_elf_discard_info
+ #define elf_backend_ignore_discarded_relocs \
+                                       _bfd_mips_elf_ignore_discarded_relocs
+@@ -1686,6 +1695,8 @@ static const struct ecoff_debug_swap mip
+ #define bfd_elf32_bfd_print_private_bfd_data \
+                                       _bfd_mips_elf_print_private_bfd_data
++#define elf_backend_plt_sym_val               _bfd_mips_elf_plt_sym_val
++
+ /* Support for SGI-ish mips targets.  */
+ #define TARGET_LITTLE_SYM             bfd_elf32_littlemips_vec
+ #define TARGET_LITTLE_NAME            "elf32-littlemips"
+@@ -1789,6 +1800,7 @@ mips_vxworks_final_write_processing (bfd
+ #undef elf_backend_additional_program_headers
+ #undef elf_backend_modify_segment_map
+ #undef elf_backend_symbol_processing
++#undef elf_backend_plt_sym_val
+ /* NOTE: elf_backend_rela_normal is not defined for MIPS.  */
+ #include "elf32-target.h"
+--- a/bfd/elfxx-mips.c
++++ b/bfd/elfxx-mips.c
+@@ -677,6 +677,11 @@ static bfd *reldyn_sorting_bfd;
+ /* Nonzero if ABFD is using NewABI conventions.  */
+ #define NEWABI_P(abfd) (ABI_N32_P (abfd) || ABI_64_P (abfd))
++/* Nonzero if ABFD is a non-PIC object.  */
++#define NON_PIC_P(abfd) \
++  (((elf_elfheader (abfd)->e_flags & EF_MIPS_PIC) == 0) \
++   && ((elf_elfheader (abfd)->e_flags & EF_MIPS_CPIC) == EF_MIPS_CPIC))
++
+ /* The IRIX compatibility level we are striving for.  */
+ #define IRIX_COMPAT(abfd) \
+   (get_elf_backend_data (abfd)->elf_backend_mips_irix_compat (abfd))
+@@ -689,6 +694,9 @@ static bfd *reldyn_sorting_bfd;
+ #define MIPS_ELF_OPTIONS_SECTION_NAME(abfd) \
+   (NEWABI_P (abfd) ? ".MIPS.options" : ".options")
++/* The name of the section holding non-PIC to PIC call stubs.  */
++#define NON_PIC_TO_PIC_STUB_SECTION_NAME ".MIPS.pic_stubs"
++
+ /* True if NAME is the recognized name of any SHT_MIPS_OPTIONS section.
+    Some IRIX system files do not use MIPS_ELF_OPTIONS_SECTION_NAME.  */
+ #define MIPS_ELF_OPTIONS_SECTION_NAME_P(NAME) \
+@@ -7583,7 +7591,9 @@ _bfd_mips_elf_check_relocs (bfd *abfd, s
+             /* We need a stub, not a plt entry for the undefined
+                function.  But we record it as if it needs plt.  See
+-               _bfd_elf_adjust_dynamic_symbol.  */
++               _bfd_elf_adjust_dynamic_symbol.  Note that these relocations
++               are always used for PIC calls, even when using the new
++               non-PIC ABI.  */
+             h->needs_plt = 1;
+             h->type = STT_FUNC;
+           }
+@@ -7689,6 +7699,8 @@ _bfd_mips_elf_check_relocs (bfd *abfd, s
+       case R_MIPS_32:
+       case R_MIPS_REL32:
+       case R_MIPS_64:
++        if (h != NULL)
++          h->non_got_ref = TRUE;
+         /* In VxWorks executables, references to external symbols
+            are handled using copy relocs or PLT stubs, so there's
+            no need to add a .rela.dyn entry for this relocation.  */
+@@ -7744,11 +7756,21 @@ _bfd_mips_elf_check_relocs (bfd *abfd, s
+       case R_MIPS_GPREL16:
+       case R_MIPS_LITERAL:
+       case R_MIPS_GPREL32:
++        if (h != NULL
++            && (r_type == R_MIPS_GPREL16 || r_type == R_MIPS_GPREL32))
++          h->non_got_ref = TRUE;
++
+         if (SGI_COMPAT (abfd))
+           mips_elf_hash_table (info)->compact_rel_size +=
+             sizeof (Elf32_External_crinfo);
+         break;
++      case R_MIPS_HI16:
++      case R_MIPS_LO16:
++        if (h != NULL && strcmp (h->root.root.string, "_gp_disp") != 0)
++          h->non_got_ref = TRUE;
++        break;
++
+         /* This relocation describes the C++ object vtable hierarchy.
+            Reconstruct it for later use during GC.  */
+       case R_MIPS_GNU_VTINHERIT:
+@@ -7771,20 +7793,20 @@ _bfd_mips_elf_check_relocs (bfd *abfd, s
+       /* We must not create a stub for a symbol that has relocations
+        related to taking the function's address.  This doesn't apply to
+-       VxWorks, where CALL relocs refer to a .got.plt entry instead of
+-       a normal .got entry.  */
++       VxWorks or the non-PIC ABI, where CALL relocs refer to a
++       .got.plt entry instead of a normal .got entry.  */
+       if (!htab->is_vxworks && h != NULL)
+       switch (r_type)
+         {
+-        default:
+-          ((struct mips_elf_link_hash_entry *) h)->no_fn_stub = TRUE;
+-          break;
+         case R_MIPS16_CALL16:
+         case R_MIPS_CALL16:
+         case R_MIPS_CALL_HI16:
+         case R_MIPS_CALL_LO16:
+         case R_MIPS_JALR:
+           break;
++        default:
++          ((struct mips_elf_link_hash_entry *) h)->no_fn_stub = TRUE;
++          break;
+         }
+       /* See if this reloc would need to refer to a MIPS16 hard-float stub,
+@@ -12458,7 +12480,9 @@ _bfd_mips_elf_merge_private_bfd_data (bf
+         break;
+       }
+     }
+-  if (null_input_bfd)
++  /* Dynamic objects normally have no sections, and do not reach
++     here - but they might if used as DYNOBJ.  */
++  if (null_input_bfd || (ibfd->flags & DYNAMIC) != 0)
+     return TRUE;
+   ok = TRUE;
+--- a/bfd/elfxx-mips.h
++++ b/bfd/elfxx-mips.h
+@@ -63,6 +63,9 @@ extern bfd_boolean _bfd_mips_elf_finish_
+ extern bfd_boolean _bfd_mips_vxworks_finish_dynamic_symbol
+   (bfd *, struct bfd_link_info *, struct elf_link_hash_entry *,
+    Elf_Internal_Sym *);
++extern bfd_boolean _bfd_mips_nonpic_finish_dynamic_symbol
++  (bfd *, struct bfd_link_info *, struct elf_link_hash_entry *,
++   Elf_Internal_Sym *);
+ extern bfd_boolean _bfd_mips_elf_finish_dynamic_sections
+   (bfd *, struct bfd_link_info *);
+ extern void _bfd_mips_elf_final_write_processing
+@@ -153,6 +156,15 @@ extern const struct bfd_elf_special_sect
+ extern bfd_boolean _bfd_mips_elf_common_definition (Elf_Internal_Sym *);
++extern bfd_vma _bfd_mips_elf_plt_sym_val
++  (bfd_vma, const asection *, const arelent *);
++extern void _bfd_mips_elf_begin_write_processing
++  (bfd *abfd, struct bfd_link_info *link_info);
++extern bfd_boolean bfd_mips_elf_maybe_create_non_pic_to_pic_stubs_section
++  (struct bfd_link_info *);
++extern void _bfd_mips_post_process_headers
++  (bfd *abfd, struct bfd_link_info *link_info);
++
+ #define elf_backend_common_definition   _bfd_mips_elf_common_definition
+ #define elf_backend_name_local_section_symbols \
+   _bfd_mips_elf_name_local_section_symbols
+--- a/gas/config/tc-mips.c
++++ b/gas/config/tc-mips.c
+@@ -1874,6 +1874,12 @@ md_begin (void)
+       as_bad (_("-G may not be used in position-independent code"));
+       g_switch_value = 0;
+     }
++  else if (mips_abicalls)
++    {
++      if (g_switch_seen && g_switch_value != 0)
++      as_bad (_("-G may not be used with abicalls"));
++      g_switch_value = 0;
++    }
+   if (! bfd_set_arch_mach (stdoutput, bfd_arch_mips, file_mips_arch))
+     as_warn (_("Could not set architecture and machine"));
+@@ -11254,6 +11260,8 @@ struct option md_longopts[] =
+   {"mno-pdr", no_argument, NULL, OPTION_NO_PDR},
+ #define OPTION_MVXWORKS_PIC (OPTION_ELF_BASE + 12)
+   {"mvxworks-pic", no_argument, NULL, OPTION_MVXWORKS_PIC},
++#define OPTION_NON_PIC_ABICALLS (OPTION_ELF_BASE + 13)
++  {"mnon-pic-abicalls", no_argument, NULL, OPTION_NON_PIC_ABICALLS},
+ #endif /* OBJ_ELF */
+   {NULL, no_argument, NULL, 0}
+@@ -11672,6 +11680,11 @@ md_parse_option (int c, char *arg)
+     case OPTION_MVXWORKS_PIC:
+       mips_pic = VXWORKS_PIC;
+       break;
++
++    case OPTION_NON_PIC_ABICALLS:
++      mips_pic = NO_PIC;
++      mips_abicalls = TRUE;
++      break;
+ #endif /* OBJ_ELF */
+     default:
diff --git a/toolchain/gcc/patches/4.3.3+cs/001-no_mips_nonpic_default.patch b/toolchain/gcc/patches/4.3.3+cs/001-no_mips_nonpic_default.patch
deleted file mode 100644 (file)
index cf6503b..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
---- a/gcc/config/mips/linux.h
-+++ b/gcc/config/mips/linux.h
-@@ -75,8 +75,7 @@ along with GCC; see the file COPYING3.  
-         %{static:-static}}}"
- #undef SUBTARGET_ASM_SPEC
--#define SUBTARGET_ASM_SPEC \
-- "%{mabi=64: -64} %{mabicalls:%{fpic|fPIC|fpie|fPIE:-KPIC;:-mnon-pic-abicalls}}"
-+#define SUBTARGET_ASM_SPEC "%{mabi=64: -64} %{!mno-abicalls:-KPIC}"
- /* The MIPS assembler has different syntax for .set. We set it to
-    .dummy to trap any errors.  */
---- a/gcc/config/mips/linux64.h
-+++ b/gcc/config/mips/linux64.h
-@@ -28,7 +28,7 @@ NO_SHARED_SPECS \
- #undef SUBTARGET_ASM_SPEC
- #define SUBTARGET_ASM_SPEC "\
--%{mabicalls:%{fpic|fPIC|fpie|fPIE:-KPIC;:-mnon-pic-abicalls}} \
-+%{!fno-PIC:%{!fno-pic:-KPIC}} \
- %{fno-PIC:-non_shared} %{fno-pic:-non_shared}"
- #undef LIB_SPEC