gas: ensure sections contents is zero for BFD_RELOC_PPC*_TLSM on XCOFF.
[deliverable/binutils-gdb.git] / gas / write.c
index 9b5ae6ff32a3b43453bdf03da62bbb12adda569c..7dc78b7fe19f42c172d386dd62583c6078b5dd28 100644 (file)
@@ -1,5 +1,5 @@
 /* write.c - emit .o file
-   Copyright (C) 1986-2019 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.  */
@@ -443,7 +443,7 @@ cvt_frag_to_fill (segT sec ATTRIBUTE_UNUSED, fragS *fragP)
 #ifdef HANDLE_ALIGN
       HANDLE_ALIGN (fragP);
 #endif
-skip_align:
+    skip_align:
       know (fragP->fr_next != NULL);
       fragP->fr_offset = (fragP->fr_next->fr_address
                          - fragP->fr_address
@@ -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
@@ -848,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)
@@ -873,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;
          }
 
@@ -916,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);
@@ -947,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);
@@ -1100,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,
@@ -1443,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 ();
@@ -1625,7 +1635,7 @@ write_contents (bfd *abfd ATTRIBUTE_UNUSED,
                                "to section %s of %s: '%s'",
                                (long) f->fr_fix),
                      (long) f->fr_fix,
-                     sec->name, stdoutput->filename,
+                     bfd_section_name (sec), bfd_get_filename (stdoutput),
                      bfd_errmsg (bfd_get_error ()));
          offset += f->fr_fix;
        }
@@ -1649,9 +1659,11 @@ write_contents (bfd *abfd ATTRIBUTE_UNUSED,
                                    "in section %s of %s: '%s'",
                                    "can't fill %ld bytes "
                                    "in section %s of %s: '%s'",
-                                   (long) count), (long) count,
-                                   sec->name, stdoutput->filename,
-                                   bfd_errmsg (bfd_get_error ()));
+                                   (long) count),
+                         (long) count,
+                         bfd_section_name (sec),
+                         bfd_get_filename (stdoutput),
+                         bfd_errmsg (bfd_get_error ()));
              offset += count;
              free (buf);
            }
@@ -1678,7 +1690,8 @@ write_contents (bfd *abfd ATTRIBUTE_UNUSED,
                                        "in section %s of %s: '%s'",
                                        (long) fill_size),
                              (long) fill_size,
-                             sec->name, stdoutput->filename,
+                             bfd_section_name (sec),
+                             bfd_get_filename (stdoutput),
                              bfd_errmsg (bfd_get_error ()));
                  offset += fill_size;
                }
@@ -1714,7 +1727,8 @@ write_contents (bfd *abfd ATTRIBUTE_UNUSED,
                                        "in section %s of %s: '%s'",
                                        (long) (n_per_buf * fill_size)),
                              (long) (n_per_buf * fill_size),
-                             sec->name, stdoutput->filename,
+                             bfd_section_name (sec),
+                             bfd_get_filename (stdoutput),
                              bfd_errmsg (bfd_get_error ()));
                  offset += n_per_buf * fill_size;
                }
@@ -1739,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)
     {
@@ -1755,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;
@@ -1893,6 +1917,7 @@ create_note_reloc (segT           sec,
                   symbolS *      sym,
                   bfd_size_type  note_offset,
                   bfd_size_type  desc2_offset,
+                  offsetT        desc2_size,
                   int            reloc_type,
                   bfd_vma        addend,
                   char *         note)
@@ -1927,15 +1952,21 @@ create_note_reloc (segT           sec,
         but still stores the addend in the word being relocated.  */
       || strstr (bfd_get_target (stdoutput), "-sh") != NULL)
     {
+      offsetT i;
+
+      /* Zero out the addend, since it is now stored in the note.  */
+      reloc->u.b.r.addend = 0;
+
       if (target_big_endian)
        {
-         if (bfd_arch_bits_per_address (stdoutput) <= 32)
-           note[desc2_offset + 3] = addend;
-         else
-           note[desc2_offset + 7] = addend;
+         for (i = desc2_size; addend != 0 && i > 0; addend >>= 8, i--)
+           note[desc2_offset + i - 1] = (addend & 0xff);
        }
       else
-       note[desc2_offset] = addend;
+       {
+         for (i = 0; addend != 0 && i < desc2_size; addend >>= 8, i++)
+           note[desc2_offset + i] = (addend & 0xff);
+       }
     }
 }
 
@@ -1958,9 +1989,10 @@ 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);
+  bfd_set_section_flags (sec, (SEC_READONLY | SEC_HAS_CONTENTS | SEC_DATA
+                              | SEC_OCTETS));
   bfd_set_section_alignment (sec, 2);
 
   /* Work out the size of the notes that we will create,
@@ -2010,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 ();
@@ -2020,14 +2052,14 @@ maybe_generate_build_notes (void)
        if (target_big_endian)
          {
            note[3] = 8; /* strlen (name) + 1.  */
-           note[7] = desc_size; /* Two 8-byte offsets.  */
+           note[7] = desc_size; /* Two N-byte offsets.  */
            note[10] = NT_GNU_BUILD_ATTRIBUTE_OPEN >> 8;
            note[11] = NT_GNU_BUILD_ATTRIBUTE_OPEN & 0xff;
          }
        else
          {
            note[0] = 8; /* strlen (name) + 1.  */
-           note[4] = desc_size; /* Two 8-byte offsets.  */
+           note[4] = desc_size; /* Two N-byte offsets.  */
            note[8] = NT_GNU_BUILD_ATTRIBUTE_OPEN & 0xff;
            note[9] = NT_GNU_BUILD_ATTRIBUTE_OPEN >> 8;
          }
@@ -2037,13 +2069,19 @@ maybe_generate_build_notes (void)
        memcpy (note + 12, "GA$\ 13a1", 8);
 
        /* Create a relocation to install the start address of the note...  */
-       create_note_reloc (sec, sym, total_size, 20, desc_reloc, 0, note);
+       create_note_reloc (sec, sym, total_size, 20, desc_size / 2, desc_reloc, 0, note);
 
        /* ...and another one to install the end address.  */
-       create_note_reloc (sec, sym, total_size, desc2_offset, desc_reloc,
+       create_note_reloc (sec, sym, total_size, desc2_offset,
+                          desc_size / 2,
+                          desc_reloc,
                           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 ?  */
       }
@@ -2296,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))
        {
@@ -2308,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;
            }
 
@@ -2412,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;
        }
     }
 
@@ -2428,7 +2466,7 @@ write_object_file (void)
 #endif
 
   /* Stop if there is an error.  */
-  if (had_errors ())
+  if (!flag_always_generate_output && had_errors ())
     return;
 
   /* Now that all the sizes are known, and contents correct, we can
@@ -2480,6 +2518,10 @@ write_object_file (void)
 }
 
 #ifdef TC_GENERIC_RELAX_TABLE
+#ifndef md_generic_table_relax_frag
+#define md_generic_table_relax_frag relax_frag
+#endif
+
 /* Relax a fragment by scanning TC_GENERIC_RELAX_TABLE.  */
 
 long
@@ -2497,7 +2539,7 @@ relax_frag (segT segment, fragS *fragP, long stretch)
   const relax_typeS *table;
 
   target = fragP->fr_offset;
-  address = fragP->fr_address;
+  address = fragP->fr_address + fragP->fr_fix;
   table = TC_GENERIC_RELAX_TABLE;
   this_state = fragP->fr_subtype;
   start_type = this_type = table + this_state;
@@ -2537,13 +2579,13 @@ relax_frag (segT segment, fragS *fragP, long stretch)
             negative.  Don't allow this in case the negative reach is
             large enough to require a larger branch instruction.  */
          else if (target < address)
-           target = fragP->fr_next->fr_address + stretch;
+           return 0;
        }
     }
 
-  aim = target - address - fragP->fr_fix;
+  aim = target - address;
 #ifdef TC_PCREL_ADJUST
-  /* Currently only the ns32k family needs this.  */
+  /* Currently only the ns32k and arc needs this.  */
   aim += TC_PCREL_ADJUST (fragP);
 #endif
 
@@ -2827,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),
@@ -2931,7 +2975,7 @@ relax_segment (struct frag *segment_frag_root, segT segment, int pass)
 
              case rs_org:
                {
-                 addressT target = offset;
+                 offsetT target = offset;
                  addressT after;
 
                  if (symbolP)
@@ -2951,7 +2995,7 @@ relax_segment (struct frag *segment_frag_root, segT segment, int pass)
                  /* Growth may be negative, but variable part of frag
                     cannot have fewer than 0 chars.  That is, we can't
                     .org backwards.  */
-                 if (address + fragP->fr_fix > target)
+                 if ((offsetT) (address + fragP->fr_fix) > target)
                    {
                      growth = 0;
 
@@ -2999,7 +3043,7 @@ relax_segment (struct frag *segment_frag_root, segT segment, int pass)
                        || ! S_IS_DEFINED (symbolP))
                      {
                        as_bad_where (fragP->fr_file, fragP->fr_line,
-                                     _(".space specifies non-absolute value"));
+                                     _(".space, .nops or .fill specifies non-absolute value"));
                        /* Prevent repeat of this error message.  */
                        fragP->fr_symbol = 0;
                      }
@@ -3030,7 +3074,8 @@ relax_segment (struct frag *segment_frag_root, segT segment, int pass)
 #ifdef TC_GENERIC_RELAX_TABLE
                /* The default way to relax a frag is to look through
                   TC_GENERIC_RELAX_TABLE.  */
-               growth = relax_frag (segment, fragP, stretch);
+               growth = md_generic_table_relax_frag (segment, fragP,
+                                                     stretch);
 #endif /* TC_GENERIC_RELAX_TABLE  */
 #endif
                break;
This page took 0.038613 seconds and 4 git commands to generate.