gdb: fix vfork with multiple threads
[deliverable/binutils-gdb.git] / bfd / elf32-rl78.c
index 8cddb58aa8b244da4f15105911c9d25b83855842..163966f1d6988e6e004b191368276b770f87f9f4 100644 (file)
@@ -1,5 +1,5 @@
 /* Renesas RL78 specific support for 32-bit ELF.
-   Copyright (C) 2011-2015 Free Software Foundation, Inc.
+   Copyright (C) 2011-2021 Free Software Foundation, Inc.
 
    This file is part of BFD, the Binary File Descriptor library.
 
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston,
+   MA 02110-1301, USA.  */
 
 #include "sysdep.h"
 #include "bfd.h"
-#include "bfd_stdint.h"
 #include "libbfd.h"
 #include "elf-bfd.h"
 #include "elf/rl78.h"
@@ -29,7 +29,7 @@
 
 #define RL78REL(n,sz,bit,shift,complain,pcrel)                              \
   HOWTO (R_RL78_##n, shift, sz, bit, pcrel, 0, complain_overflow_ ## complain, \
-        bfd_elf_generic_reloc, "R_RL78_" #n, FALSE, 0, ~0, FALSE)
+        bfd_elf_generic_reloc, "R_RL78_" #n, false, 0, ~0, false)
 
 static bfd_reloc_status_type rl78_special_reloc (bfd *, arelent *, asymbol *, void *,
                                                 asection *, bfd *, char **);
@@ -37,7 +37,7 @@ static bfd_reloc_status_type rl78_special_reloc (bfd *, arelent *, asymbol *, vo
 /* FIXME: We could omit the SHIFT parameter, it is always zero.  */
 #define RL78_OP_REL(n,sz,bit,shift,complain,pcrel)                     \
   HOWTO (R_RL78_##n, shift, sz, bit, pcrel, 0, complain_overflow_ ## complain, \
-        rl78_special_reloc, "R_RL78_" #n, FALSE, 0, ~0, FALSE)
+        rl78_special_reloc, "R_RL78_" #n, false, 0, ~0, false)
 
 /* Note that the relocations around 0x7f are internal to this file;
    feel free to move them as needed to avoid conflicts with published
@@ -45,25 +45,25 @@ static bfd_reloc_status_type rl78_special_reloc (bfd *, arelent *, asymbol *, vo
 
 static reloc_howto_type rl78_elf_howto_table [] =
 {
-  RL78REL (NONE,         3,  0, 0, dont,     FALSE),
-  RL78REL (DIR32,        2, 32, 0, signed,   FALSE),
-  RL78REL (DIR24S,       2, 24, 0, signed,   FALSE),
-  RL78REL (DIR16,        1, 16, 0, dont,     FALSE),
-  RL78REL (DIR16U,       1, 16, 0, unsigned, FALSE),
-  RL78REL (DIR16S,       1, 16, 0, signed,   FALSE),
-  RL78REL (DIR8,         0,  8, 0, dont,     FALSE),
-  RL78REL (DIR8U,        0,  8, 0, unsigned, FALSE),
-  RL78REL (DIR8S,        0,  8, 0, signed,   FALSE),
-  RL78REL (DIR24S_PCREL, 2, 24, 0, signed,   TRUE),
-  RL78REL (DIR16S_PCREL, 1, 16, 0, signed,   TRUE),
-  RL78REL (DIR8S_PCREL,  0,  8, 0, signed,   TRUE),
-  RL78REL (DIR16UL,      1, 16, 2, unsigned, FALSE),
-  RL78REL (DIR16UW,      1, 16, 1, unsigned, FALSE),
-  RL78REL (DIR8UL,       0,  8, 2, unsigned, FALSE),
-  RL78REL (DIR8UW,       0,  8, 1, unsigned, FALSE),
-  RL78REL (DIR32_REV,    1, 16, 0, dont,     FALSE),
-  RL78REL (DIR16_REV,    1, 16, 0, dont,     FALSE),
-  RL78REL (DIR3U_PCREL,  0,  3, 0, dont,     TRUE),
+  RL78REL (NONE,        3,  0, 0, dont,     false),
+  RL78REL (DIR32,       2, 32, 0, signed,   false),
+  RL78REL (DIR24S,      2, 24, 0, signed,   false),
+  RL78REL (DIR16,       1, 16, 0, dont,     false),
+  RL78REL (DIR16U,      1, 16, 0, unsigned, false),
+  RL78REL (DIR16S,      1, 16, 0, signed,   false),
+  RL78REL (DIR8,        0,  8, 0, dont,     false),
+  RL78REL (DIR8U,       0,  8, 0, unsigned, false),
+  RL78REL (DIR8S,       0,  8, 0, signed,   false),
+  RL78REL (DIR24S_PCREL, 2, 24, 0, signed,   true),
+  RL78REL (DIR16S_PCREL, 1, 16, 0, signed,   true),
+  RL78REL (DIR8S_PCREL,         0,  8, 0, signed,   true),
+  RL78REL (DIR16UL,     1, 16, 2, unsigned, false),
+  RL78REL (DIR16UW,     1, 16, 1, unsigned, false),
+  RL78REL (DIR8UL,      0,  8, 2, unsigned, false),
+  RL78REL (DIR8UW,      0,  8, 1, unsigned, false),
+  RL78REL (DIR32_REV,   1, 16, 0, dont,     false),
+  RL78REL (DIR16_REV,   1, 16, 0, dont,     false),
+  RL78REL (DIR3U_PCREL,         0,  3, 0, dont,     true),
 
   EMPTY_HOWTO (0x13),
   EMPTY_HOWTO (0x14),
@@ -92,10 +92,10 @@ static reloc_howto_type rl78_elf_howto_table [] =
   EMPTY_HOWTO (0x2a),
   EMPTY_HOWTO (0x2b),
   EMPTY_HOWTO (0x2c),
-  RL78REL (RH_RELAX, 0,  0, 0, dont,     FALSE),
+  RL78REL (RH_RELAX, 0,         0, 0, dont,     false),
 
   EMPTY_HOWTO (0x2e),
-  RL78REL (RH_SADDR, 0,  0, 0, dont,     FALSE),
+  RL78REL (RH_SADDR, 0,         0, 0, dont,     false),
   EMPTY_HOWTO (0x30),
   EMPTY_HOWTO (0x31),
   EMPTY_HOWTO (0x32),
@@ -114,23 +114,23 @@ static reloc_howto_type rl78_elf_howto_table [] =
   EMPTY_HOWTO (0x3f),
   EMPTY_HOWTO (0x40),
 
-  RL78_OP_REL (ABS32,        2, 32, 0, dont,     FALSE),
-  RL78_OP_REL (ABS24S,       2, 24, 0, signed,   FALSE),
-  RL78_OP_REL (ABS16,        1, 16, 0, dont,     FALSE),
-  RL78_OP_REL (ABS16U,       1, 16, 0, unsigned, FALSE),
-  RL78_OP_REL (ABS16S,       1, 16, 0, signed,   FALSE),
-  RL78_OP_REL (ABS8,         0,  8, 0, dont,     FALSE),
-  RL78_OP_REL (ABS8U,        0,  8, 0, unsigned, FALSE),
-  RL78_OP_REL (ABS8S,        0,  8, 0, signed,   FALSE),
-  RL78_OP_REL (ABS24S_PCREL, 2, 24, 0, signed,   TRUE),
-  RL78_OP_REL (ABS16S_PCREL, 1, 16, 0, signed,   TRUE),
-  RL78_OP_REL (ABS8S_PCREL,  0,  8, 0, signed,   TRUE),
-  RL78_OP_REL (ABS16UL,      1, 16, 0, unsigned, FALSE),
-  RL78_OP_REL (ABS16UW,      1, 16, 0, unsigned, FALSE),
-  RL78_OP_REL (ABS8UL,       0,  8, 0, unsigned, FALSE),
-  RL78_OP_REL (ABS8UW,       0,  8, 0, unsigned, FALSE),
-  RL78_OP_REL (ABS32_REV,    2, 32, 0, dont,     FALSE),
-  RL78_OP_REL (ABS16_REV,    1, 16, 0, dont,     FALSE),
+  RL78_OP_REL (ABS32,       2, 32, 0, dont,     false),
+  RL78_OP_REL (ABS24S,      2, 24, 0, signed,   false),
+  RL78_OP_REL (ABS16,       1, 16, 0, dont,     false),
+  RL78_OP_REL (ABS16U,      1, 16, 0, unsigned, false),
+  RL78_OP_REL (ABS16S,      1, 16, 0, signed,   false),
+  RL78_OP_REL (ABS8,        0,  8, 0, dont,     false),
+  RL78_OP_REL (ABS8U,       0,  8, 0, unsigned, false),
+  RL78_OP_REL (ABS8S,       0,  8, 0, signed,   false),
+  RL78_OP_REL (ABS24S_PCREL, 2, 24, 0, signed,  true),
+  RL78_OP_REL (ABS16S_PCREL, 1, 16, 0, signed,  true),
+  RL78_OP_REL (ABS8S_PCREL,  0,         8, 0, signed,   true),
+  RL78_OP_REL (ABS16UL,             1, 16, 0, unsigned, false),
+  RL78_OP_REL (ABS16UW,             1, 16, 0, unsigned, false),
+  RL78_OP_REL (ABS8UL,      0,  8, 0, unsigned, false),
+  RL78_OP_REL (ABS8UW,      0,  8, 0, unsigned, false),
+  RL78_OP_REL (ABS32_REV,    2, 32, 0, dont,    false),
+  RL78_OP_REL (ABS16_REV,    1, 16, 0, dont,    false),
 
 #define STACK_REL_P(x) ((x) <= R_RL78_ABS16_REV && (x) >= R_RL78_ABS32)
 
@@ -182,29 +182,29 @@ static reloc_howto_type rl78_elf_howto_table [] =
   EMPTY_HOWTO (0x7e),
   EMPTY_HOWTO (0x7f),
 
-  RL78_OP_REL (SYM,       2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPneg,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPadd,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPsub,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPmul,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPdiv,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPshla,    2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPshra,    2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPsctsize, 2, 32, 0, dont, FALSE),
+  RL78_OP_REL (SYM,      2, 32, 0, dont, false),
+  RL78_OP_REL (OPneg,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPadd,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPsub,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPmul,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPdiv,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPshla,   2, 32, 0, dont, false),
+  RL78_OP_REL (OPshra,   2, 32, 0, dont, false),
+  RL78_OP_REL (OPsctsize, 2, 32, 0, dont, false),
   EMPTY_HOWTO (0x89),
   EMPTY_HOWTO (0x8a),
   EMPTY_HOWTO (0x8b),
   EMPTY_HOWTO (0x8c),
-  RL78_OP_REL (OPscttop,  2, 32, 0, dont, FALSE),
+  RL78_OP_REL (OPscttop,  2, 32, 0, dont, false),
   EMPTY_HOWTO (0x8e),
   EMPTY_HOWTO (0x8f),
-  RL78_OP_REL (OPand,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPor,      2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPxor,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPnot,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPmod,     2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPromtop,  2, 32, 0, dont, FALSE),
-  RL78_OP_REL (OPramtop,  2, 32, 0, dont, FALSE)
+  RL78_OP_REL (OPand,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPor,     2, 32, 0, dont, false),
+  RL78_OP_REL (OPxor,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPnot,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPmod,    2, 32, 0, dont, false),
+  RL78_OP_REL (OPromtop,  2, 32, 0, dont, false),
+  RL78_OP_REL (OPramtop,  2, 32, 0, dont, false)
 };
 \f
 /* Map BFD reloc types to RL78 ELF reloc types.  */
@@ -212,7 +212,7 @@ static reloc_howto_type rl78_elf_howto_table [] =
 struct rl78_reloc_map
 {
   bfd_reloc_code_real_type  bfd_reloc_val;
-  unsigned int              rl78_reloc_val;
+  unsigned int             rl78_reloc_val;
 };
 
 static const struct rl78_reloc_map rl78_reloc_map [] =
@@ -277,9 +277,9 @@ rl78_reloc_name_lookup (bfd * abfd ATTRIBUTE_UNUSED, const char * r_name)
 
 /* Set the howto pointer for an RL78 ELF reloc.  */
 
-static void
-rl78_info_to_howto_rela (bfd *               abfd ATTRIBUTE_UNUSED,
-                        arelent *           cache_ptr,
+static bool
+rl78_info_to_howto_rela (bfd *              abfd,
+                        arelent *           cache_ptr,
                         Elf_Internal_Rela * dst)
 {
   unsigned int r_type;
@@ -287,18 +287,21 @@ rl78_info_to_howto_rela (bfd *               abfd ATTRIBUTE_UNUSED,
   r_type = ELF32_R_TYPE (dst->r_info);
   if (r_type >= (unsigned int) R_RL78_max)
     {
-      _bfd_error_handler (_("%B: invalid RL78 reloc number: %d"), abfd, r_type);
-      r_type = 0;
+      /* xgettext:c-format */
+      _bfd_error_handler (_("%pB: unsupported relocation type %#x"),
+                         abfd, r_type);
+      bfd_set_error (bfd_error_bad_value);
+      return false;
     }
   cache_ptr->howto = rl78_elf_howto_table + r_type;
+  return true;
 }
 \f
 static bfd_vma
-get_symbol_value (const char *            name,
-                 bfd_reloc_status_type * status,
+get_symbol_value (const char *           name,
                  struct bfd_link_info *  info,
-                 bfd *                   input_bfd,
-                 asection *              input_section,
+                 bfd *                   input_bfd,
+                 asection *              input_section,
                  int                     offset)
 {
   struct bfd_link_hash_entry * h;
@@ -306,18 +309,14 @@ get_symbol_value (const char *            name,
   if (info == NULL)
     return 0;
 
-  h = bfd_link_hash_lookup (info->hash, name, FALSE, FALSE, TRUE);
+  h = bfd_link_hash_lookup (info->hash, name, false, false, true);
 
   if (h == NULL
       || (h->type != bfd_link_hash_defined
          && h->type != bfd_link_hash_defweak))
     {
-      bfd_reloc_status_type res;
-
-      res = info->callbacks->undefined_symbol
-       (info, name, input_bfd, input_section, offset, TRUE);
-      if (status)
-       * status = res;
+      (*info->callbacks->undefined_symbol)
+       (info, name, input_bfd, input_section, offset, true);
       return 0;
     }
 
@@ -327,44 +326,36 @@ get_symbol_value (const char *            name,
 }
 
 static bfd_vma
-get_romstart (bfd_reloc_status_type * status,
-             struct bfd_link_info *  info,
-             bfd *                   abfd,
-             asection *              sec,
+get_romstart (struct bfd_link_info *  info,
+             bfd *                   abfd,
+             asection *              sec,
              int                     offset)
 {
-  static bfd_boolean cached = FALSE;
-  static bfd_vma     cached_value = 0;
-  static bfd_reloc_status_type cached_status;
+  static bool cached = false;
+  static bfd_vma cached_value = 0;
 
   if (!cached)
     {
-      cached_value = get_symbol_value ("_start", & cached_status, info, abfd, sec, offset);
-      cached = TRUE;
+      cached_value = get_symbol_value ("_start", info, abfd, sec, offset);
+      cached = true;
     }
-  if (status)
-    * status = cached_status;
   return cached_value;
 }
 
 static bfd_vma
-get_ramstart (bfd_reloc_status_type * status,
-             struct bfd_link_info *  info,
-             bfd *                   abfd,
-             asection *              sec,
+get_ramstart (struct bfd_link_info *  info,
+             bfd *                   abfd,
+             asection *              sec,
              int                     offset)
 {
-  static bfd_boolean cached = FALSE;
-  static bfd_vma     cached_value = 0;
-  static bfd_reloc_status_type cached_status;
+  static bool cached = false;
+  static bfd_vma cached_value = 0;
 
   if (!cached)
     {
-      cached_value = get_symbol_value ("__datastart", & cached_status, info, abfd, sec, offset);
-      cached = TRUE;
+      cached_value = get_symbol_value ("__datastart", info, abfd, sec, offset);
+      cached = true;
     }
-  if (status)
-    * status = cached_status;
   return cached_value;
 }
 
@@ -376,9 +367,9 @@ static unsigned int rl78_stack_top;
   do                                           \
     {                                          \
       if (rl78_stack_top < NUM_STACK_ENTRIES)  \
-        rl78_stack [rl78_stack_top ++] = (val);        \
+       rl78_stack [rl78_stack_top ++] = (val); \
       else                                     \
-       _bfd_error_handler (_("Internal Error: RL78 reloc stack overflow")); \
+       _bfd_error_handler (_("internal error: RL78 reloc stack overflow")); \
     }                                          \
   while (0)
 
@@ -386,12 +377,12 @@ static unsigned int rl78_stack_top;
   do                                           \
     {                                          \
       if (rl78_stack_top > 0)                  \
-        (dest) = rl78_stack [-- rl78_stack_top];\
+       (dest) = rl78_stack [-- rl78_stack_top];\
       else                                     \
        {                                       \
-          _bfd_error_handler (_("Internal Error: RL78 reloc stack underflow")); \
-          (dest) = 0;                          \
-        }                                      \
+         _bfd_error_handler (_("internal error: RL78 reloc stack underflow")); \
+         (dest) = 0;                           \
+                                             \
     }                                          \
   while (0)
 
@@ -402,8 +393,8 @@ static unsigned int rl78_stack_top;
 
 static bfd_vma
 rl78_compute_complex_reloc (unsigned long  r_type,
-                           bfd_vma        symval,
-                           asection *     input_section)
+                           bfd_vma        symval,
+                           asection *     input_section)
 {
   int32_t tmp1, tmp2;
   bfd_vma relocation;
@@ -557,10 +548,10 @@ rl78_special_reloc (bfd *      input_bfd,
                    bfd *      output_bfd ATTRIBUTE_UNUSED,
                    char **    error_message ATTRIBUTE_UNUSED)
 {
-  bfd_reloc_status_type  r = bfd_reloc_ok;
-  bfd_vma                relocation = 0;
-  unsigned long          r_type = reloc->howto->type;
-  bfd_byte *             contents = data;
+  bfd_reloc_status_type         r = bfd_reloc_ok;
+  bfd_vma               relocation = 0;
+  unsigned long                 r_type = reloc->howto->type;
+  bfd_byte *            contents = data;
 
   /* If necessary, compute the symbolic value of the relocation.  */
   switch (r_type)
@@ -573,12 +564,12 @@ rl78_special_reloc (bfd *      input_bfd,
        break;
 
     case R_RL78_OPromtop:
-      relocation = get_romstart (&r, NULL, input_bfd, input_section,
+      relocation = get_romstart (NULL, input_bfd, input_section,
                                 reloc->address);
       break;
 
     case R_RL78_OPramtop:
-      relocation = get_ramstart (&r, NULL, input_bfd, input_section,
+      relocation = get_ramstart (NULL, input_bfd, input_section,
                                 reloc->address);
       break;
     }
@@ -682,51 +673,47 @@ rl78_special_reloc (bfd *      input_bfd,
    section, which means that the addend must be adjusted
    accordingly.  */
 
-static bfd_boolean
+static int
 rl78_elf_relocate_section
-    (bfd *                   output_bfd,
+    (bfd *                  output_bfd,
      struct bfd_link_info *  info,
-     bfd *                   input_bfd,
-     asection *              input_section,
-     bfd_byte *              contents,
+     bfd *                  input_bfd,
+     asection *                     input_section,
+     bfd_byte *                     contents,
      Elf_Internal_Rela *     relocs,
-     Elf_Internal_Sym *      local_syms,
-     asection **             local_sections)
+     Elf_Internal_Sym *             local_syms,
+     asection **            local_sections)
 {
-  Elf_Internal_Shdr *           symtab_hdr;
+  Elf_Internal_Shdr *          symtab_hdr;
   struct elf_link_hash_entry ** sym_hashes;
-  Elf_Internal_Rela *           rel;
-  Elf_Internal_Rela *           relend;
-  bfd *dynobj;
+  Elf_Internal_Rela *          rel;
+  Elf_Internal_Rela *          relend;
   asection *splt;
 
   symtab_hdr = & elf_tdata (input_bfd)->symtab_hdr;
   sym_hashes = elf_sym_hashes (input_bfd);
   relend     = relocs + input_section->reloc_count;
 
-  dynobj = elf_hash_table (info)->dynobj;
-  splt = NULL;
-  if (dynobj != NULL)
-    splt = bfd_get_linker_section (dynobj, ".plt");
+  splt = elf_hash_table (info)->splt;
 
   for (rel = relocs; rel < relend; rel ++)
     {
-      reloc_howto_type *           howto;
-      unsigned long                r_symndx;
-      Elf_Internal_Sym *           sym;
-      asection *                   sec;
-      struct elf_link_hash_entry * h;
-      bfd_vma                      relocation;
-      bfd_reloc_status_type        r;
-      const char *                 name = NULL;
-      bfd_boolean                  unresolved_reloc = TRUE;
-      int                          r_type;
+      reloc_howto_type *howto;
+      unsigned long r_symndx;
+      Elf_Internal_Sym *sym;
+      asection *sec;
+      struct elf_link_hash_entry *h;
+      bfd_vma relocation;
+      bfd_reloc_status_type r;
+      const char *name = NULL;
+      bool unresolved_reloc = true;
+      int r_type;
 
       r_type = ELF32_R_TYPE (rel->r_info);
       r_symndx = ELF32_R_SYM (rel->r_info);
 
       howto  = rl78_elf_howto_table + ELF32_R_TYPE (rel->r_info);
-      h      = NULL;
+      h             = NULL;
       sym    = NULL;
       sec    = NULL;
       relocation = 0;
@@ -739,12 +726,12 @@ rl78_elf_relocate_section
 
          name = bfd_elf_string_from_elf_section
            (input_bfd, symtab_hdr->sh_link, sym->st_name);
-         name = (sym->st_name == 0) ? bfd_section_name (input_bfd, sec) : name;
+         name = sym->st_name == 0 ? bfd_section_name (sec) : name;
        }
       else
        {
-         bfd_boolean warned ATTRIBUTE_UNUSED;
-         bfd_boolean ignored ATTRIBUTE_UNUSED;
+         bool warned ATTRIBUTE_UNUSED;
+         bool ignored ATTRIBUTE_UNUSED;
 
          RELOC_FOR_GLOBAL_SYMBOL (info, input_bfd, input_section, rel,
                                   r_symndx, symtab_hdr, sym_hashes, h,
@@ -761,9 +748,9 @@ rl78_elf_relocate_section
       if (bfd_link_relocatable (info))
        {
          /* This is a relocatable link.  We don't have to change
-             anything, unless the reloc is against a section symbol,
-             in which case we have to adjust according to where the
-             section symbol winds up in the output section.  */
+            anything, unless the reloc is against a section symbol,
+            in which case we have to adjust according to where the
+            section symbol winds up in the output section.  */
          if (sym != NULL && ELF_ST_TYPE (sym->st_info) == STT_SECTION)
            rel->r_addend += sec->output_offset;
          continue;
@@ -936,7 +923,7 @@ rl78_elf_relocate_section
 
        case R_RL78_RH_SADDR:
          RANGE (0xffe20, 0xfff1f);
-         OP (0) = relocation & 0xff;
+         OP (0) = (relocation - 0x20) & 0xff;
          break;
 
          /* Complex reloc handling:  */
@@ -1062,18 +1049,19 @@ rl78_elf_relocate_section
            {
              relocation = 0;
              if (h->root.type != bfd_link_hash_undefweak)
-               _bfd_error_handler (_("Warning: RL78_SYM reloc with an unknown symbol"));
+               _bfd_error_handler
+                 (_("warning: RL78_SYM reloc with an unknown symbol"));
            }
          (void) rl78_compute_complex_reloc (r_type, relocation, input_section);
          break;
 
        case R_RL78_OPromtop:
-         relocation = get_romstart (&r, info, input_bfd, input_section, rel->r_offset);
+         relocation = get_romstart (info, input_bfd, input_section, rel->r_offset);
          (void) rl78_compute_complex_reloc (r_type, relocation, input_section);
          break;
 
        case R_RL78_OPramtop:
-         relocation = get_ramstart (&r, info, input_bfd, input_section, rel->r_offset);
+         relocation = get_ramstart (info, input_bfd, input_section, rel->r_offset);
          (void) rl78_compute_complex_reloc (r_type, relocation, input_section);
          break;
 
@@ -1092,67 +1080,69 @@ rl78_elf_relocate_section
              /* Catch the case of a missing function declaration
                 and emit a more helpful error message.  */
              if (r_type == R_RL78_DIR24S_PCREL)
-               msg = _("%B(%A): error: call to undefined function '%s'");
+               /* xgettext:c-format */
+               msg = _("%pB(%pA): error: call to undefined function '%s'");
              else
-               r = info->callbacks->reloc_overflow
+               (*info->callbacks->reloc_overflow)
                  (info, (h ? &h->root : NULL), name, howto->name, (bfd_vma) 0,
                   input_bfd, input_section, rel->r_offset);
              break;
 
            case bfd_reloc_undefined:
-             r = info->callbacks->undefined_symbol
-               (info, name, input_bfd, input_section, rel->r_offset,
-                TRUE);
+             (*info->callbacks->undefined_symbol)
+               (info, name, input_bfd, input_section, rel->r_offset, true);
              break;
 
            case bfd_reloc_other:
-             msg = _("%B(%A): warning: unaligned access to symbol '%s' in the small data area");
+             /* xgettext:c-format */
+             msg = _("%pB(%pA): warning: unaligned access to symbol '%s' in the small data area");
              break;
 
            case bfd_reloc_outofrange:
-             msg = _("%B(%A): internal error: out of range error");
+             /* xgettext:c-format */
+             msg = _("%pB(%pA): internal error: out of range error");
              break;
 
            case bfd_reloc_notsupported:
-             msg = _("%B(%A): internal error: unsupported relocation error");
+             /* xgettext:c-format */
+             msg = _("%pB(%pA): internal error: unsupported relocation error");
              break;
 
            case bfd_reloc_dangerous:
-             msg = _("%B(%A): internal error: dangerous relocation");
+             /* xgettext:c-format */
+             msg = _("%pB(%pA): internal error: dangerous relocation");
              break;
 
            default:
-             msg = _("%B(%A): internal error: unknown error");
+             /* xgettext:c-format */
+             msg = _("%pB(%pA): internal error: unknown error");
              break;
            }
 
          if (msg)
            _bfd_error_handler (msg, input_bfd, input_section, name);
-
-         if (! r)
-           return FALSE;
        }
     }
 
-  return TRUE;
+  return true;
 }
 \f
 /* Function to set the ELF flag bits.  */
 
-static bfd_boolean
+static bool
 rl78_elf_set_private_flags (bfd * abfd, flagword flags)
 {
   elf_elfheader (abfd)->e_flags = flags;
-  elf_flags_init (abfd) = TRUE;
-  return TRUE;
+  elf_flags_init (abfd) = true;
+  return true;
 }
 
-static bfd_boolean no_warn_mismatch = FALSE;
+static bool no_warn_mismatch = false;
 
-void bfd_elf32_rl78_set_target_flags (bfd_boolean);
+void bfd_elf32_rl78_set_target_flags (bool);
 
 void
-bfd_elf32_rl78_set_target_flags (bfd_boolean user_no_warn_mismatch)
+bfd_elf32_rl78_set_target_flags (bool user_no_warn_mismatch)
 {
   no_warn_mismatch = user_no_warn_mismatch;
 }
@@ -1172,12 +1162,13 @@ rl78_cpu_name (flagword flags)
 /* Merge backend specific data from an object file to the output
    object file when linking.  */
 
-static bfd_boolean
-rl78_elf_merge_private_bfd_data (bfd * ibfd, bfd * obfd)
+static bool
+rl78_elf_merge_private_bfd_data (bfd *ibfd, struct bfd_link_info *info)
 {
+  bfd *obfd = info->output_bfd;
   flagword new_flags;
   flagword old_flags;
-  bfd_boolean error = FALSE;
+  bool error = false;
 
   new_flags = elf_elfheader (ibfd)->e_flags;
   old_flags = elf_elfheader (obfd)->e_flags;
@@ -1185,7 +1176,7 @@ rl78_elf_merge_private_bfd_data (bfd * ibfd, bfd * obfd)
   if (!elf_flags_init (obfd))
     {
       /* First call, no flags set.  */
-      elf_flags_init (obfd) = TRUE;
+      elf_flags_init (obfd) = true;
       elf_elfheader (obfd)->e_flags = new_flags;
     }
   else if (old_flags != new_flags)
@@ -1207,12 +1198,13 @@ rl78_elf_merge_private_bfd_data (bfd * ibfd, bfd * obfd)
                     If the output is set to "any" this means that it is
                     a G14 file that does not use hardware multiply/divide,
                     but that is still incompatible with the G10 ABI.  */
-                 error = TRUE;
+                 error = true;
 
-                 (*_bfd_error_handler)
-                   (_("RL78 ABI conflict: G10 file %s cannot be linked with %s file %s"),
-                    bfd_get_filename (ibfd),
-                    rl78_cpu_name (out_cpu), bfd_get_filename (obfd));
+                 _bfd_error_handler
+                   /* xgettext:c-format */
+                   (_("RL78 ABI conflict: G10 file %pB cannot be linked"
+                      " with %s file %pB"),
+                    ibfd, rl78_cpu_name (out_cpu), obfd);
                }
              else
                {
@@ -1223,34 +1215,37 @@ rl78_elf_merge_private_bfd_data (bfd * ibfd, bfd * obfd)
            }
          else
            {
-             error = TRUE;
+             error = true;
 
-             (*_bfd_error_handler)
-               (_("RL78 ABI conflict: cannot link %s file %s with %s file %s"),
-                rl78_cpu_name (in_cpu),  bfd_get_filename (ibfd),
-                rl78_cpu_name (out_cpu), bfd_get_filename (obfd));
+             _bfd_error_handler
+               /* xgettext:c-format */
+               (_("RL78 ABI conflict: cannot link %s file %pB with %s file %pB"),
+                rl78_cpu_name (in_cpu),  ibfd,
+                rl78_cpu_name (out_cpu), obfd);
            }
        }
 
       if (changed_flags & E_FLAG_RL78_64BIT_DOUBLES)
        {
-         (*_bfd_error_handler)
+         _bfd_error_handler
            (_("RL78 merge conflict: cannot link 32-bit and 64-bit objects together"));
 
          if (old_flags & E_FLAG_RL78_64BIT_DOUBLES)
-           (*_bfd_error_handler) (_("- %s is 64-bit, %s is not"),
-                                  bfd_get_filename (obfd), bfd_get_filename (ibfd));
+           /* xgettext:c-format */
+           _bfd_error_handler (_("- %pB is 64-bit, %pB is not"),
+                               obfd, ibfd);
          else
-           (*_bfd_error_handler) (_("- %s is 64-bit, %s is not"),
-                                  bfd_get_filename (ibfd), bfd_get_filename (obfd));
-         error = TRUE;
+           /* xgettext:c-format */
+           _bfd_error_handler (_("- %pB is 64-bit, %pB is not"),
+                               ibfd, obfd);
+         error = true;
        }
     }
 
   return !error;
 }
 \f
-static bfd_boolean
+static bool
 rl78_elf_print_private_bfd_data (bfd * abfd, void * ptr)
 {
   FILE * file = (FILE *) ptr;
@@ -1271,7 +1266,7 @@ rl78_elf_print_private_bfd_data (bfd * abfd, void * ptr)
     fprintf (file, _(" [64-bit doubles]"));
 
   fputc ('\n', file);
-  return TRUE;
+  return true;
 }
 
 /* Return the MACH for an e_flags value.  */
@@ -1282,12 +1277,12 @@ elf32_rl78_machine (bfd * abfd ATTRIBUTE_UNUSED)
   return bfd_mach_rl78;
 }
 
-static bfd_boolean
+static bool
 rl78_elf_object_p (bfd * abfd)
 {
   bfd_default_set_arch_mach (abfd, bfd_arch_rl78,
                             elf32_rl78_machine (abfd));
-  return TRUE;
+  return true;
 }
 \f
 /* support PLT for 16-bit references to 24-bit functions.  */
@@ -1295,28 +1290,27 @@ rl78_elf_object_p (bfd * abfd)
 /* We support 16-bit pointers to code above 64k by generating a thunk
    below 64k containing a JMP instruction to the final address.  */
 
-static bfd_boolean
+static bool
 rl78_elf_check_relocs
-    (bfd *                     abfd,
+    (bfd *                    abfd,
      struct bfd_link_info *    info,
-     asection *                sec,
+     asection *                       sec,
      const Elf_Internal_Rela * relocs)
 {
-  Elf_Internal_Shdr *           symtab_hdr;
+  Elf_Internal_Shdr *          symtab_hdr;
   struct elf_link_hash_entry ** sym_hashes;
-  const Elf_Internal_Rela *     rel;
-  const Elf_Internal_Rela *     rel_end;
+  const Elf_Internal_Rela *    rel;
+  const Elf_Internal_Rela *    rel_end;
   bfd_vma *local_plt_offsets;
   asection *splt;
   bfd *dynobj;
 
   if (bfd_link_relocatable (info))
-    return TRUE;
+    return true;
 
   symtab_hdr = &elf_tdata (abfd)->symtab_hdr;
   sym_hashes = elf_sym_hashes (abfd);
   local_plt_offsets = elf_local_got_offsets (abfd);
-  splt = NULL;
   dynobj = elf_hash_table(info)->dynobj;
 
   rel_end = relocs + sec->reloc_count;
@@ -1328,41 +1322,35 @@ rl78_elf_check_relocs
 
       r_symndx = ELF32_R_SYM (rel->r_info);
       if (r_symndx < symtab_hdr->sh_info)
-        h = NULL;
+       h = NULL;
       else
        {
          h = sym_hashes[r_symndx - symtab_hdr->sh_info];
          while (h->root.type == bfd_link_hash_indirect
                 || h->root.type == bfd_link_hash_warning)
            h = (struct elf_link_hash_entry *) h->root.u.i.link;
-
-         /* PR15323, ref flags aren't set for references in the same
-            object.  */
-         h->root.non_ir_ref = 1;
        }
 
       switch (ELF32_R_TYPE (rel->r_info))
-        {
+       {
          /* This relocation describes a 16-bit pointer to a function.
             We may need to allocate a thunk in low memory; reserve memory
             for it now.  */
        case R_RL78_DIR16S:
          if (dynobj == NULL)
            elf_hash_table (info)->dynobj = dynobj = abfd;
+         splt = elf_hash_table (info)->splt;
          if (splt == NULL)
            {
-             splt = bfd_get_linker_section (dynobj, ".plt");
-             if (splt == NULL)
-               {
-                 flagword flags = (SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS
-                                   | SEC_IN_MEMORY | SEC_LINKER_CREATED
-                                   | SEC_READONLY | SEC_CODE);
-                 splt = bfd_make_section_anyway_with_flags (dynobj, ".plt",
-                                                            flags);
-                 if (splt == NULL
-                     || ! bfd_set_section_alignment (dynobj, splt, 1))
-                   return FALSE;
-               }
+             flagword flags = (SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS
+                               | SEC_IN_MEMORY | SEC_LINKER_CREATED
+                               | SEC_READONLY | SEC_CODE);
+             splt = bfd_make_section_anyway_with_flags (dynobj, ".plt",
+                                                        flags);
+             elf_hash_table (info)->splt = splt;
+             if (splt == NULL
+                 || !bfd_set_section_alignment (splt, 1))
+               return false;
            }
 
          if (h != NULL)
@@ -1377,7 +1365,7 @@ rl78_elf_check_relocs
                  size = symtab_hdr->sh_info * sizeof (bfd_vma);
                  local_plt_offsets = (bfd_vma *) bfd_alloc (abfd, size);
                  if (local_plt_offsets == NULL)
-                   return FALSE;
+                   return false;
                  elf_local_got_offsets (abfd) = local_plt_offsets;
 
                  for (i = 0; i < symtab_hdr->sh_info; i++)
@@ -1392,23 +1380,23 @@ rl78_elf_check_relocs
              splt->size += 4;
            }
          break;
-        }
+       }
     }
 
-  return TRUE;
+  return true;
 }
 
 /* This must exist if dynobj is ever set.  */
 
-static bfd_boolean
+static bool
 rl78_elf_finish_dynamic_sections (bfd *abfd ATTRIBUTE_UNUSED,
-                                  struct bfd_link_info *info)
+                                 struct bfd_link_info *info)
 {
   bfd *dynobj;
   asection *splt;
 
   if (!elf_hash_table (info)->dynamic_sections_created)
-    return TRUE;
+    return true;
 
   /* As an extra sanity check, verify that all plt entries have been
      filled in.  However, relaxing might have changed the relocs so
@@ -1417,10 +1405,11 @@ rl78_elf_finish_dynamic_sections (bfd *abfd ATTRIBUTE_UNUSED,
      called before relaxation.  */
 
   if (info->relax_trip > 0)
-    return TRUE;
+    return true;
 
-  if ((dynobj = elf_hash_table (info)->dynobj) != NULL
-      && (splt = bfd_get_linker_section (dynobj, ".plt")) != NULL)
+  dynobj = elf_hash_table (info)->dynobj;
+  splt = elf_hash_table (info)->splt;
+  if (dynobj != NULL && splt != NULL)
     {
       bfd_byte *contents = splt->contents;
       unsigned int i, size = splt->size;
@@ -1432,31 +1421,31 @@ rl78_elf_finish_dynamic_sections (bfd *abfd ATTRIBUTE_UNUSED,
        }
     }
 
-  return TRUE;
+  return true;
 }
 
-static bfd_boolean
+static bool
 rl78_elf_always_size_sections (bfd *output_bfd ATTRIBUTE_UNUSED,
-                               struct bfd_link_info *info)
+                              struct bfd_link_info *info)
 {
   bfd *dynobj;
   asection *splt;
 
   if (bfd_link_relocatable (info))
-    return TRUE;
+    return true;
 
   dynobj = elf_hash_table (info)->dynobj;
   if (dynobj == NULL)
-    return TRUE;
+    return true;
 
-  splt = bfd_get_linker_section (dynobj, ".plt");
+  splt = elf_hash_table (info)->splt;
   BFD_ASSERT (splt != NULL);
 
   splt->contents = (bfd_byte *) bfd_zalloc (dynobj, splt->size);
   if (splt->contents == NULL)
-    return FALSE;
+    return false;
 
-  return TRUE;
+  return true;
 }
 
 \f
@@ -1469,10 +1458,10 @@ rl78_elf_always_size_sections (bfd *output_bfd ATTRIBUTE_UNUSED,
 struct relax_plt_data
 {
   asection *splt;
-  bfd_boolean *again;
+  bool *again;
 };
 
-static bfd_boolean
+static bool
 rl78_relax_plt_check (struct elf_link_hash_entry *h, void * xdata)
 {
   struct relax_plt_data *data = (struct relax_plt_data *) xdata;
@@ -1493,17 +1482,17 @@ rl78_relax_plt_check (struct elf_link_hash_entry *h, void * xdata)
        {
          h->plt.offset = -1;
          data->splt->size -= 4;
-         *data->again = TRUE;
+         *data->again = true;
        }
     }
 
-  return TRUE;
+  return true;
 }
 
 /* A subroutine of rl78_elf_relax_section.  If the global symbol H
    previously had a plt entry, give it a new entry offset.  */
 
-static bfd_boolean
+static bool
 rl78_relax_plt_realloc (struct elf_link_hash_entry *h, void * xdata)
 {
   bfd_vma *entry = (bfd_vma *) xdata;
@@ -1514,32 +1503,32 @@ rl78_relax_plt_realloc (struct elf_link_hash_entry *h, void * xdata)
       *entry += 4;
     }
 
-  return TRUE;
+  return true;
 }
 
-static bfd_boolean
+static bool
 rl78_elf_relax_plt_section (bfd *dynobj,
-                            asection *splt,
-                            struct bfd_link_info *info,
-                            bfd_boolean *again)
+                           asection *splt,
+                           struct bfd_link_info *info,
+                           bool *again)
 {
   struct relax_plt_data relax_plt_data;
   bfd *ibfd;
 
   /* Assume nothing changes.  */
-  *again = FALSE;
+  *again = false;
 
   if (bfd_link_relocatable (info))
-    return TRUE;
+    return true;
 
   /* We only relax the .plt section at the moment.  */
   if (dynobj != elf_hash_table (info)->dynobj
       || strcmp (splt->name, ".plt") != 0)
-    return TRUE;
+    return true;
 
   /* Quick check for an empty plt.  */
   if (splt->size == 0)
-    return TRUE;
+    return true;
 
   /* Map across all global symbols; see which ones happen to
      fall in the low 64k.  */
@@ -1569,7 +1558,7 @@ rl78_elf_relax_plt_section (bfd *dynobj,
                                            symtab_hdr->sh_info, 0,
                                            NULL, NULL, NULL);
          if (isymbuf == NULL)
-           return FALSE;
+           return false;
        }
 
       for (idx = 0; idx < symtab_hdr->sh_info; ++idx)
@@ -1598,7 +1587,7 @@ rl78_elf_relax_plt_section (bfd *dynobj,
            {
              local_plt_offsets[idx] = -1;
              splt->size -= 4;
-             *again = TRUE;
+             *again = true;
            }
        }
 
@@ -1636,30 +1625,30 @@ rl78_elf_relax_plt_section (bfd *dynobj,
          for (idx = 0; idx < nlocals; ++idx)
            if (local_plt_offsets[idx] != (bfd_vma) -1)
              {
-               local_plt_offsets[idx] = entry;
+               local_plt_offsets[idx] = entry;
                entry += 4;
              }
        }
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Delete some bytes from a section while relaxing.  */
 
-static bfd_boolean
+static bool
 elf32_rl78_relax_delete_bytes (bfd *abfd, asection *sec, bfd_vma addr, int count,
                               Elf_Internal_Rela *alignment_rel, int force_snip)
 {
   Elf_Internal_Shdr * symtab_hdr;
-  unsigned int        sec_shndx;
-  bfd_byte *          contents;
+  unsigned int       sec_shndx;
+  bfd_byte *         contents;
   Elf_Internal_Rela * irel;
   Elf_Internal_Rela * irelend;
   Elf_Internal_Sym *  isym;
   Elf_Internal_Sym *  isymend;
-  bfd_vma             toaddr;
-  unsigned int        symcount;
+  bfd_vma            toaddr;
+  unsigned int       symcount;
   struct elf_link_hash_entry ** sym_hashes;
   struct elf_link_hash_entry ** end_hashes;
 
@@ -1679,7 +1668,7 @@ elf32_rl78_relax_delete_bytes (bfd *abfd, asection *sec, bfd_vma addr, int count
   irel = elf_section_data (sec)->relocs;
   if (irel == NULL)
     {
-      _bfd_elf_link_read_relocs (sec->owner, sec, NULL, NULL, TRUE);
+      _bfd_elf_link_read_relocs (sec->owner, sec, NULL, NULL, true);
       irel = elf_section_data (sec)->relocs;
     }
 
@@ -1766,7 +1755,7 @@ elf32_rl78_relax_delete_bytes (bfd *abfd, asection *sec, bfd_vma addr, int count
        }
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Used to sort relocs by address.  If relocs have the same address,
@@ -1777,34 +1766,34 @@ static void
 reloc_bubblesort (Elf_Internal_Rela * r, int count)
 {
   int i;
-  bfd_boolean again;
-  bfd_boolean swappit;
+  bool again;
+  bool swappit;
 
   /* This is almost a classic bubblesort.  It's the slowest sort, but
      we're taking advantage of the fact that the relocations are
      mostly in order already (the assembler emits them that way) and
      we need relocs with the same address to remain in the same
      relative order.  */
-  again = TRUE;
+  again = true;
   while (again)
     {
-      again = FALSE;
+      again = false;
       for (i = 0; i < count - 1; i ++)
        {
          if (r[i].r_offset > r[i + 1].r_offset)
-           swappit = TRUE;
+           swappit = true;
          else if (r[i].r_offset < r[i + 1].r_offset)
-           swappit = FALSE;
+           swappit = false;
          else if (ELF32_R_TYPE (r[i + 1].r_info) == R_RL78_RH_RELAX
                   && (r[i + 1].r_addend & RL78_RELAXA_ALIGN))
-           swappit = TRUE;
+           swappit = true;
          else if (ELF32_R_TYPE (r[i + 1].r_info) == R_RL78_RH_RELAX
                   && (r[i + 1].r_addend & RL78_RELAXA_ELIGN)
                   && !(ELF32_R_TYPE (r[i].r_info) == R_RL78_RH_RELAX
                        && (r[i].r_addend & RL78_RELAXA_ALIGN)))
-           swappit = TRUE;
+           swappit = true;
          else
-           swappit = FALSE;
+           swappit = false;
 
          if (swappit)
            {
@@ -1818,7 +1807,7 @@ reloc_bubblesort (Elf_Internal_Rela * r, int count)
                 most of the O(n^2) behavior for our cases.  */
              if (i > 0)
                i -= 2;
-             again = TRUE;
+             again = true;
            }
        }
     }
@@ -1830,16 +1819,16 @@ reloc_bubblesort (Elf_Internal_Rela * r, int count)
                         lrel, abfd, sec, link_info, scale)
 
 static bfd_vma
-rl78_offset_for_reloc (bfd *                    abfd,
-                      Elf_Internal_Rela *      rel,
-                      Elf_Internal_Shdr *      symtab_hdr,
-                      Elf_External_Sym_Shndx * shndx_buf ATTRIBUTE_UNUSED,
-                      Elf_Internal_Sym *       intsyms,
-                      Elf_Internal_Rela **     lrel,
-                      bfd *                    input_bfd,
-                      asection *               input_section,
-                      struct bfd_link_info *   info,
-                      int *                    scale)
+rl78_offset_for_reloc (bfd *                   abfd,
+                      Elf_Internal_Rela *      rel,
+                      Elf_Internal_Shdr *      symtab_hdr,
+                      bfd_byte *               shndx_buf ATTRIBUTE_UNUSED,
+                      Elf_Internal_Sym *       intsyms,
+                      Elf_Internal_Rela **     lrel,
+                      bfd *                    input_bfd,
+                      asection *               input_section,
+                      struct bfd_link_info *   info,
+                      int *                    scale)
 {
   bfd_vma symval;
 
@@ -1932,12 +1921,12 @@ rl78_offset_for_reloc (bfd *                    abfd,
          break;
 
        case R_RL78_OPromtop:
-         symval = get_romstart (NULL, info, input_bfd, input_section, rel->r_offset);
+         symval = get_romstart (info, input_bfd, input_section, rel->r_offset);
          (void) rl78_compute_complex_reloc (r_type, symval, input_section);
          break;
 
        case R_RL78_OPramtop:
-         symval = get_ramstart (NULL, info, input_bfd, input_section, rel->r_offset);
+         symval = get_ramstart (info, input_bfd, input_section, rel->r_offset);
          (void) rl78_compute_complex_reloc (r_type, symval, input_section);
          break;
 
@@ -1974,7 +1963,16 @@ rl78_offset_for_reloc (bfd *                    abfd,
 
        default:
        reloc_computes_value:
-         symval = rl78_compute_complex_reloc (r_type, 0, input_section);
+         symval = rl78_compute_complex_reloc (r_type, symval, input_section);
+         /* Fall through.  */
+       case R_RL78_DIR32:
+       case R_RL78_DIR24S:
+       case R_RL78_DIR16:
+       case R_RL78_DIR16U:
+       case R_RL78_DIR16S:
+       case R_RL78_DIR24S_PCREL:
+       case R_RL78_DIR16S_PCREL:
+       case R_RL78_DIR8S_PCREL:
          if (lrel)
            *lrel = rel;
          return symval;
@@ -1984,7 +1982,7 @@ rl78_offset_for_reloc (bfd *                    abfd,
     }
 }
 
-struct {
+const struct {
   int prefix;          /* or -1 for "no prefix" */
   int insn;            /* or -1 for "end of list" */
   int insn_for_saddr;  /* or -1 for "no alternative" */
@@ -2051,12 +2049,11 @@ struct {
 
 /* Relax one section.  */
 
-static bfd_boolean
-rl78_elf_relax_section
-    (bfd *                  abfd,
-     asection *             sec,
-     struct bfd_link_info * link_info,
-     bfd_boolean *          again)
+static bool
+rl78_elf_relax_section (bfd *abfd,
+                       asection *sec,
+                       struct bfd_link_info *link_info,
+                       bool *again)
 {
   Elf_Internal_Shdr * symtab_hdr;
   Elf_Internal_Shdr * shndx_hdr;
@@ -2066,11 +2063,11 @@ rl78_elf_relax_section
   Elf_Internal_Rela * srel;
   Elf_Internal_Rela * irelend;
   Elf_Internal_Rela * next_alignment;
-  bfd_byte *          contents = NULL;
-  bfd_byte *          free_contents = NULL;
+  bfd_byte *         contents = NULL;
+  bfd_byte *         free_contents = NULL;
   Elf_Internal_Sym *  intsyms = NULL;
   Elf_Internal_Sym *  free_intsyms = NULL;
-  Elf_External_Sym_Shndx * shndx_buf = NULL;
+  bfd_byte *         shndx_buf = NULL;
   bfd_vma pc;
   bfd_vma symval ATTRIBUTE_UNUSED = 0;
   int pcrel ATTRIBUTE_UNUSED = 0;
@@ -2083,7 +2080,7 @@ rl78_elf_relax_section
     return rl78_elf_relax_plt_section (abfd, sec, link_info, again);
 
   /* Assume nothing changes.  */
-  *again = FALSE;
+  *again = false;
 
   /* We don't have to do anything for a relocatable link, if
      this section does not have relocs, or if this is not a
@@ -2092,10 +2089,13 @@ rl78_elf_relax_section
       || (sec->flags & SEC_RELOC) == 0
       || sec->reloc_count == 0
       || (sec->flags & SEC_CODE) == 0)
-    return TRUE;
+    return true;
 
-  symtab_hdr = &elf_tdata (abfd)->symtab_hdr;
-  shndx_hdr = &elf_tdata (abfd)->symtab_shndx_hdr;
+  symtab_hdr = & elf_symtab_hdr (abfd);
+  if (elf_symtab_shndx_list (abfd))
+    shndx_hdr = & elf_symtab_shndx_list (abfd)->hdr;
+  else
+    shndx_hdr = NULL;
 
   /* Get the section contents.  */
   if (elf_section_data (sec)->this_hdr.contents != NULL)
@@ -2118,19 +2118,22 @@ rl78_elf_relax_section
       symtab_hdr->contents = (bfd_byte *) intsyms;
     }
 
-  if (shndx_hdr->sh_size != 0)
+  if (shndx_hdr && shndx_hdr->sh_size != 0)
     {
-      bfd_size_type amt;
+      size_t amt;
 
-      amt = symtab_hdr->sh_info;
-      amt *= sizeof (Elf_External_Sym_Shndx);
-      shndx_buf = (Elf_External_Sym_Shndx *) bfd_malloc (amt);
-      if (shndx_buf == NULL)
+      if (_bfd_mul_overflow (symtab_hdr->sh_info,
+                            sizeof (Elf_External_Sym_Shndx), &amt))
+       {
+         bfd_set_error (bfd_error_no_memory);
+         goto error_return;
+       }
+      if (bfd_seek (abfd, shndx_hdr->sh_offset, SEEK_SET) != 0)
        goto error_return;
-      if (bfd_seek (abfd, shndx_hdr->sh_offset, SEEK_SET) != 0
-         || bfd_bread (shndx_buf, amt, abfd) != amt)
+      shndx_buf = _bfd_malloc_and_read (abfd, amt, amt);
+      if (shndx_buf == NULL)
        goto error_return;
-      shndx_hdr->contents = (bfd_byte *) shndx_buf;
+      shndx_hdr->contents = shndx_buf;
     }
 
   /* Get a copy of the native relocations.  */
@@ -2228,7 +2231,7 @@ rl78_elf_relax_section
 
          elf32_rl78_relax_delete_bytes (abfd, sec, erel->r_offset - nbytes, nbytes,
                                         next_alignment, erel->r_offset == sec->size);
-         *again = TRUE;
+         *again = true;
 
          continue;
        }
@@ -2246,9 +2249,9 @@ rl78_elf_relax_section
         reloc entry (such as gp-rel symbols).  */
 
       /* Get the value of the symbol referred to by the reloc.  Just
-         in case this is the last reloc in the list, use the RL's
-         addend to choose between this reloc (no addend) or the next
-         (yes addend, which means at least one following reloc).  */
+        in case this is the last reloc in the list, use the RL's
+        addend to choose between this reloc (no addend) or the next
+        (yes addend, which means at least one following reloc).  */
 
       /* srel points to the "current" reloction for this insn -
         actually the last reloc for a given operand, which is the one
@@ -2270,7 +2273,7 @@ rl78_elf_relax_section
        elf32_rl78_relax_delete_bytes (abfd, sec, (insn - contents) + offset, nbytes, next_alignment, 0);
 
 #define SNIP(offset, nbytes, newtype)                                  \
-        SNIPNR (offset, nbytes);                                       \
+       SNIPNR (offset, nbytes);                                        \
        srel->r_info = ELF32_R_INFO (ELF32_R_SYM (srel->r_info), newtype)
 
       /* The order of these bit tests must match the order that the
@@ -2313,6 +2316,27 @@ rl78_elf_relax_section
 
          switch (insn[0])
            {
+           case 0xdc: /* BC */
+           case 0xdd: /* BZ */
+           case 0xde: /* BNC */
+           case 0xdf: /* BNZ */
+             if (insn[1] == 0x03 && insn[2] == 0xee /* BR */
+                 && (srel->r_offset - irel->r_offset) > 1) /* a B<c> without its own reloc */
+               {
+                 /* This is a "long" conditional as generated by gas:
+                    DC 03 EE ad.dr  */
+                 if (pcrel < 127
+                     && pcrel > -127)
+                   {
+                     insn[0] ^= 0x02; /* invert conditional */
+                     SNIPNR (4, 1);
+                     SNIP (1, 2, R_RL78_DIR8S_PCREL);
+                     insn[1] = pcrel;
+                     *again = true;
+                   }
+               }
+             break;
+
            case 0xec: /* BR !!abs20 */
 
              if (pcrel < 127
@@ -2321,15 +2345,15 @@ rl78_elf_relax_section
                  insn[0] = 0xef;
                  insn[1] = pcrel;
                  SNIP (2, 2, R_RL78_DIR8S_PCREL);
-                 *again = TRUE;
+                 *again = true;
                }
              else if (symval < 65536)
                {
                  insn[0] = 0xed;
                  insn[1] = symval & 0xff;
                  insn[2] = symval >> 8;
-                 SNIP (2, 1, R_RL78_DIR16S);
-                 *again = TRUE;
+                 SNIP (2, 1, R_RL78_DIR16U);
+                 *again = true;
                }
              else if (pcrel < 32767
                       && pcrel > -32767)
@@ -2338,7 +2362,7 @@ rl78_elf_relax_section
                  insn[1] = pcrel & 0xff;
                  insn[2] = pcrel >> 8;
                  SNIP (2, 1, R_RL78_DIR16S_PCREL);
-                 *again = TRUE;
+                 *again = true;
                }
              break;
 
@@ -2350,7 +2374,7 @@ rl78_elf_relax_section
                  insn[0] = 0xef;
                  insn[1] = pcrel;
                  SNIP (2, 1, R_RL78_DIR8S_PCREL);
-                 *again = TRUE;
+                 *again = true;
                }
              break;
 
@@ -2360,8 +2384,8 @@ rl78_elf_relax_section
                  insn[0] = 0xfd;
                  insn[1] = symval & 0xff;
                  insn[2] = symval >> 8;
-                 SNIP (2, 1, R_RL78_DIR16S);
-                 *again = TRUE;
+                 SNIP (2, 1, R_RL78_DIR16U);
+                 *again = true;
                }
              else if (pcrel < 32767
                       && pcrel > -32767)
@@ -2370,7 +2394,7 @@ rl78_elf_relax_section
                  insn[1] = pcrel & 0xff;
                  insn[2] = pcrel >> 8;
                  SNIP (2, 1, R_RL78_DIR16S_PCREL);
-                 *again = TRUE;
+                 *again = true;
                }
              break;
 
@@ -2383,6 +2407,25 @@ rl78_elf_relax_section
                 here anyway. */
              switch (insn[1])
                {
+               case 0xd3: /* BNH */
+               case 0xc3: /* BH */
+                 if (insn[2] == 0x03 && insn[3] == 0xee
+                     && (srel->r_offset - irel->r_offset) > 2) /* a B<c> without its own reloc */
+                   {
+                     /* Another long branch by gas:
+                        61 D3 03 EE ad.dr  */
+                     if (pcrel < 127
+                         && pcrel > -127)
+                       {
+                         insn[1] ^= 0x10; /* invert conditional */
+                         SNIPNR (5, 1);
+                         SNIP (2, 2, R_RL78_DIR8S_PCREL);
+                         insn[2] = pcrel;
+                         *again = true;
+                       }
+                   }
+                 break;
+
                case 0xc8: /* SKC */
                  if (insn[2] == 0xef)
                    {
@@ -2436,7 +2479,7 @@ rl78_elf_relax_section
        }
 
       if ((irel->r_addend &  RL78_RELAXA_MASK) == RL78_RELAXA_ADDR16
-          && nrelocs > 0)
+         && nrelocs > 0)
        {
          /*----------------------------------------------------------------------*/
          /* Some insns have both a 16-bit address operand and an 8-bit
@@ -2444,7 +2487,7 @@ rl78_elf_relax_section
 
             Address            16-bit operand  SADDR range     SFR range
             FFF00-FFFFF        0xff00-0xffff   0x00-0xff
-            FFE20-FFF1F        0xfe20-0xff1f                   0x00-0xff
+            FFE20-FFF1F        0xfe20-0xff1f                   0x00-0xff
 
             The RELAX_ADDR16[] array has the insn encodings for the
             16-bit operand version, as well as the SFR and SADDR
@@ -2505,14 +2548,11 @@ rl78_elf_relax_section
       /*----------------------------------------------------------------------*/
     }
 
-  return TRUE;
+  return true;
 
  error_return:
-  if (free_relocs != NULL)
-    free (free_relocs);
-
-  if (free_contents != NULL)
-    free (free_contents);
+  free (free_relocs);
+  free (free_contents);
 
   if (shndx_buf != NULL)
     {
@@ -2520,10 +2560,9 @@ rl78_elf_relax_section
       free (shndx_buf);
     }
 
-  if (free_intsyms != NULL)
-    free (free_intsyms);
+  free (free_intsyms);
 
-  return TRUE;
+  return true;
 }
 
 \f
@@ -2539,7 +2578,7 @@ rl78_elf_relax_section
 #define elf_info_to_howto                      rl78_info_to_howto_rela
 #define elf_backend_object_p                   rl78_elf_object_p
 #define elf_backend_relocate_section           rl78_elf_relocate_section
-#define elf_symbol_leading_char                 ('_')
+#define elf_symbol_leading_char                        ('_')
 #define elf_backend_can_gc_sections            1
 
 #define bfd_elf32_bfd_reloc_type_lookup                rl78_reloc_type_lookup
@@ -2549,7 +2588,7 @@ rl78_elf_relax_section
 #define bfd_elf32_bfd_print_private_bfd_data   rl78_elf_print_private_bfd_data
 
 #define bfd_elf32_bfd_relax_section            rl78_elf_relax_section
-#define elf_backend_check_relocs                rl78_elf_check_relocs
+#define elf_backend_check_relocs               rl78_elf_check_relocs
 #define elf_backend_always_size_sections \
   rl78_elf_always_size_sections
 #define elf_backend_finish_dynamic_sections \
This page took 0.048613 seconds and 4 git commands to generate.