gas: ensure sections contents is zero for BFD_RELOC_PPC*_TLSM on XCOFF.
[deliverable/binutils-gdb.git] / gas / write.c
index 36fc40488d97f6c986958b502bb6c44dbed8f691..7dc78b7fe19f42c172d386dd62583c6078b5dd28 100644 (file)
@@ -1,5 +1,5 @@
 /* write.c - emit .o file
-   Copyright (C) 1986-2020 Free Software Foundation, Inc.
+   Copyright (C) 1986-2021 Free Software Foundation, Inc.
 
    This file is part of GAS, the GNU Assembler.
 
@@ -235,7 +235,7 @@ fix_new (fragS *frag,                       /* Which frag?  */
         RELOC_ENUM r_type              /* Relocation type.  */)
 {
   return fix_new_internal (frag, where, size, add_symbol,
-                          (symbolS *) NULL, offset, pcrel, r_type, FALSE);
+                          (symbolS *) NULL, offset, pcrel, r_type, false);
 }
 
 /* Create a fixup for an expression.  Currently we only support fixups
@@ -304,7 +304,7 @@ fix_new_exp (fragS *frag,           /* Which frag?  */
     }
 
   return fix_new_internal (frag, where, size, add, sub, off, pcrel,
-                          r_type, FALSE);
+                          r_type, false);
 }
 
 /* Create a fixup at the beginning of FRAG.  The arguments are the same
@@ -315,7 +315,7 @@ fix_at_start (fragS *frag, unsigned long size, symbolS *add_symbol,
              offsetT offset, int pcrel, RELOC_ENUM r_type)
 {
   return fix_new_internal (frag, 0, size, add_symbol,
-                          (symbolS *) NULL, offset, pcrel, r_type, TRUE);
+                          (symbolS *) NULL, offset, pcrel, r_type, true);
 }
 
 /* Generic function to determine whether a fixup requires a relocation.  */
@@ -737,7 +737,9 @@ resolve_reloc_expr_symbols (void)
                 prevent the offset from overflowing the relocated field,
                 unless it has enough bits to cover the whole address
                 space.  */
-             if (S_IS_LOCAL (sym) && !symbol_section_p (sym)
+             if (S_IS_LOCAL (sym)
+                 && S_IS_DEFINED (sym)
+                 && !symbol_section_p (sym)
                  && (sec->use_rela_p
                      || (howto->partial_inplace
                          && (!howto->pc_relative
@@ -771,22 +773,6 @@ resolve_reloc_expr_symbols (void)
     }
 }
 
-static bfd_boolean
-is_dwo_section (asection *sec)
-{
-  const char *name;
-  int len;
-
-  if (sec == NULL || (name = bfd_section_name (sec)) == NULL)
-    return FALSE;
-
-  len = strlen (name);
-  if (len < 5)
-    return FALSE;
-
-  return strncmp (name + len - 4, ".dwo", 4) == 0;
-}
-
 /* This pass over fixups decides whether symbols can be replaced with
    section symbols.  */
 
@@ -864,7 +850,12 @@ adjust_reloc_syms (bfd *abfd ATTRIBUTE_UNUSED,
        /* Since we're reducing to section symbols, don't attempt to reduce
           anything that's already using one.  */
        if (symbol_section_p (sym))
-         continue;
+         {
+           /* Mark the section symbol used in relocation so that it will
+              be included in the symbol table.  */
+           symbol_mark_used_in_reloc (sym);
+           continue;
+         }
 
        symsec = S_GET_SEGMENT (sym);
        if (symsec == NULL)
@@ -889,8 +880,7 @@ adjust_reloc_syms (bfd *abfd ATTRIBUTE_UNUSED,
                    /* The GNU toolchain uses an extension for ELF: a
                       section beginning with the magic string
                       .gnu.linkonce is a linkonce section.  */
-                   && strncmp (segment_name (symsec), ".gnu.linkonce",
-                               sizeof ".gnu.linkonce" - 1) == 0))
+                   && startswith (segment_name (symsec), ".gnu.linkonce")))
              continue;
          }
 
@@ -915,14 +905,6 @@ adjust_reloc_syms (bfd *abfd ATTRIBUTE_UNUSED,
 #endif
       }
 
-  /* PR 26841: DWO sections are not supposed to have relocations.  */
-  if (is_dwo_section (sec) && seginfo->fix_root != NULL)
-    {
-      as_bad (_("DWO section '%s' contains unresolved expressions - this is not allowed"),
-             bfd_section_name (sec));
-      seginfo->fix_root = NULL; /* FIXME: Memory leak ?  */
-    }
-
   dump_section_relocs (abfd, sec, stderr);
 }
 
@@ -940,7 +922,6 @@ fixup_segment (fixS *fixP, segT this_segment)
 {
   valueT add_number;
   fragS *fragP;
-  segT add_symbol_segment = absolute_section;
 
   if (fixP != NULL && abs_section_sym == NULL)
     abs_section_sym = section_symbol (absolute_section);
@@ -971,6 +952,8 @@ fixup_segment (fixS *fixP, segT this_segment)
 
   for (; fixP; fixP = fixP->fx_next)
     {
+      segT add_symbol_segment = absolute_section;
+
 #ifdef DEBUG5
       fprintf (stderr, "\nprocessing fixup:\n");
       print_fixup (fixP);
@@ -1124,12 +1107,15 @@ fixup_segment (fixS *fixP, segT this_segment)
              mask = 0;
              mask--;           /* Set all bits to one.  */
              mask <<= fixP->fx_size * 8 - (fixP->fx_signed ? 1 : 0);
-             if ((add_number & mask) != 0 && (add_number & mask) != mask)
+             if ((add_number & mask) != 0
+                 && (fixP->fx_signed
+                     ? (add_number & mask) != mask
+                     : (-add_number & mask) != 0))
                {
                  char buf[50], buf2[50];
-                 sprint_value (buf, fragP->fr_address + fixP->fx_where);
+                 bfd_sprintf_vma (stdoutput, buf, fragP->fr_address + fixP->fx_where);
                  if (add_number > 1000)
-                   sprint_value (buf2, add_number);
+                   bfd_sprintf_vma (stdoutput, buf2, add_number);
                  else
                    sprintf (buf2, "%ld", (long) add_number);
                  as_bad_where (fixP->fx_file, fixP->fx_line,
@@ -1467,7 +1453,7 @@ compress_debug (bfd *abfd, asection *sec, void *xxx ATTRIBUTE_UNUSED)
     return;
 
   section_name = bfd_section_name (sec);
-  if (strncmp (section_name, ".debug_", 7) != 0)
+  if (!startswith (section_name, ".debug_"))
     return;
 
   strm = compress_init ();
@@ -1767,14 +1753,17 @@ set_symtab (void)
   int nsyms;
   asymbol **asympp;
   symbolS *symp;
-  bfd_boolean result;
+  bool result;
 
   /* Count symbols.  We can't rely on a count made by the loop in
      write_object_file, because *_frob_file may add a new symbol or
-     two.  */
+     two.  Generate unused section symbols only if needed.  */
   nsyms = 0;
   for (symp = symbol_rootP; symp; symp = symbol_next (symp))
-    nsyms++;
+    if (bfd_keep_unused_section_symbols (stdoutput)
+       || !symbol_section_p (symp)
+       || symbol_used_in_reloc_p (symp))
+      nsyms++;
 
   if (nsyms)
     {
@@ -1783,15 +1772,22 @@ set_symtab (void)
 
       asympp = (asymbol **) bfd_alloc (stdoutput, amt);
       symp = symbol_rootP;
-      for (i = 0; i < nsyms; i++, symp = symbol_next (symp))
-       {
-         asympp[i] = symbol_get_bfdsym (symp);
-         if (asympp[i]->flags != BSF_SECTION_SYM
-             || !(bfd_is_const_section (asympp[i]->section)
-                  && asympp[i]->section->symbol == asympp[i]))
-           asympp[i]->flags |= BSF_KEEP;
-         symbol_mark_written (symp);
-       }
+      for (i = 0; i < nsyms; symp = symbol_next (symp))
+       if (bfd_keep_unused_section_symbols (stdoutput)
+           || !symbol_section_p (symp)
+           || symbol_used_in_reloc_p (symp))
+         {
+           asympp[i] = symbol_get_bfdsym (symp);
+           if (asympp[i]->flags != BSF_SECTION_SYM
+               || !(bfd_is_const_section (asympp[i]->section)
+                    && asympp[i]->section->symbol == asympp[i]))
+             asympp[i]->flags |= BSF_KEEP;
+           symbol_mark_written (symp);
+           /* Include this section symbol in the symbol table.  */
+           if (symbol_section_p (symp))
+             asympp[i]->flags |= BSF_SECTION_SYM_USED;
+           i++;
+         }
     }
   else
     asympp = 0;
@@ -1993,7 +1989,7 @@ maybe_generate_build_notes (void)
     return;
 
   /* Create a GNU Build Attribute section.  */
-  sec = subseg_new (GNU_BUILD_ATTRS_SECTION_NAME, FALSE);
+  sec = subseg_new (GNU_BUILD_ATTRS_SECTION_NAME, false);
   elf_section_type (sec) = SHT_NOTE;
   bfd_set_section_flags (sec, (SEC_READONLY | SEC_HAS_CONTENTS | SEC_DATA
                               | SEC_OCTETS));
@@ -2046,7 +2042,7 @@ maybe_generate_build_notes (void)
        /* Skip linkonce sections - we cannot use these section symbols as they may disappear.  */
        && (bsym->section->flags & (SEC_CODE | SEC_LINK_ONCE)) == SEC_CODE
        /* Not all linkonce sections are flagged...  */
-       && strncmp (S_GET_NAME (sym), ".gnu.linkonce", sizeof ".gnu.linkonce" - 1) != 0)
+       && !startswith (S_GET_NAME (sym), ".gnu.linkonce"))
       {
        /* Create a version note.  */
        frag_now_fix ();
@@ -2082,6 +2078,10 @@ maybe_generate_build_notes (void)
                           bfd_section_size (bsym->section),
                           note);
 
+       /* Mark the section symbol used in relocation so that it will be
+          included in the symbol table.  */
+       symbol_mark_used_in_reloc (sym);
+
        total_size += note_size;
        /* FIXME: Maybe add a note recording the assembler command line and version ?  */
       }
@@ -2334,7 +2334,7 @@ write_object_file (void)
   if (symbol_rootP)
     {
       symbolS *symp;
-      bfd_boolean skip_next_symbol = FALSE;
+      bool skip_next_symbol = false;
 
       for (symp = symbol_rootP; symp; symp = symbol_next (symp))
        {
@@ -2346,7 +2346,7 @@ write_object_file (void)
              /* Don't do anything besides moving the value of the
                 symbol from the GAS value-field to the BFD value-field.  */
              symbol_get_bfdsym (symp)->value = S_GET_VALUE (symp);
-             skip_next_symbol = FALSE;
+             skip_next_symbol = false;
              continue;
            }
 
@@ -2450,7 +2450,7 @@ write_object_file (void)
             symbol warned about.  Don't let anything object-format or
             target-specific muck with it; it's ready for output.  */
          if (symbol_get_bfdsym (symp)->flags & BSF_WARNING)
-           skip_next_symbol = TRUE;
+           skip_next_symbol = true;
        }
     }
 
@@ -2869,7 +2869,9 @@ relax_segment (struct frag *segment_frag_root, segT segment, int pass)
                          if (flag_warn_displacement)
                            {
                              char buf[50];
-                             sprint_value (buf, (addressT) lie->addnum);
+
+                             bfd_sprintf_vma (stdoutput, buf,
+                                              (addressT) lie->addnum);
                              as_warn_where (fragP->fr_file, fragP->fr_line,
                                             _(".word %s-%s+%s didn't fit"),
                                             S_GET_NAME (lie->add),
This page took 0.029904 seconds and 4 git commands to generate.