2004-12-05 Randolph Chung <tausq@debian.org>
[deliverable/binutils-gdb.git] / bfd / elf-eh-frame.c
index be4e3072345b97645bc572fcaa4a6a65e924919b..0c0b32c36d5cb3e0096f0a1a08b8cc5e871a197d 100644 (file)
@@ -200,6 +200,60 @@ int cie_compare (struct cie *c1, struct cie *c2)
   return 1;
 }
 
+/* Return the number of extra bytes that we'll be inserting into
+   ENTRY's augmentation string.  */
+
+static INLINE unsigned int
+extra_augmentation_string_bytes (struct eh_cie_fde *entry)
+{
+  unsigned int size = 0;
+  if (entry->cie)
+    {
+      if (entry->add_augmentation_size)
+       size++;
+      if (entry->add_fde_encoding)
+       size++;
+    }
+  return size;
+}
+
+/* Likewise ENTRY's augmentation data.  */
+
+static INLINE unsigned int
+extra_augmentation_data_bytes (struct eh_cie_fde *entry)
+{
+  unsigned int size = 0;
+  if (entry->cie)
+    {
+      if (entry->add_augmentation_size)
+       size++;
+      if (entry->add_fde_encoding)
+       size++;
+    }
+  else
+    {
+      if (entry->cie_inf->add_augmentation_size)
+       size++;
+    }
+  return size;
+}
+
+/* Return the size that ENTRY will have in the output.  ALIGNMENT is the
+   required alignment of ENTRY in bytes.  */
+
+static unsigned int
+size_of_output_cie_fde (struct eh_cie_fde *entry, unsigned int alignment)
+{
+  if (entry->removed)
+    return 0;
+  if (entry->size == 4)
+    return 4;
+  return (entry->size
+         + extra_augmentation_string_bytes (entry)
+         + extra_augmentation_data_bytes (entry)
+         + alignment - 1) & -alignment;
+}
+
 /* This function is called for each input file before the .eh_frame
    section is relocated.  It discards duplicate CIEs and FDEs for discarded
    functions.  The function returns TRUE iff any entries have been
@@ -213,15 +267,14 @@ _bfd_elf_discard_section_eh_frame
 {
   bfd_byte *ehbuf = NULL, *buf;
   bfd_byte *last_cie, *last_fde;
+  struct eh_cie_fde *ent, *last_cie_inf, *this_inf;
   struct cie_header hdr;
   struct cie cie;
   struct elf_link_hash_table *htab;
   struct eh_frame_hdr_info *hdr_info;
   struct eh_frame_sec_info *sec_info = NULL;
   unsigned int leb128_tmp;
-  unsigned int cie_usage_count, last_cie_ndx, i, offset;
-  unsigned int make_relative, make_lsda_relative;
-  bfd_size_type new_size;
+  unsigned int cie_usage_count, offset;
   unsigned int ptr_size;
 
   if (sec->size == 0)
@@ -234,7 +287,7 @@ _bfd_elf_discard_section_eh_frame
        && bfd_is_abs_section (sec->output_section)))
     {
       /* At least one of the sections is being discarded from the
-         link, so we should just ignore them.  */
+        link, so we should just ignore them.  */
       return FALSE;
     }
 
@@ -264,12 +317,9 @@ _bfd_elf_discard_section_eh_frame
              == ELFCLASS64) ? 8 : 4;
   buf = ehbuf;
   last_cie = NULL;
-  last_cie_ndx = 0;
+  last_cie_inf = NULL;
   memset (&cie, 0, sizeof (cie));
   cie_usage_count = 0;
-  new_size = sec->size;
-  make_relative = hdr_info->last_cie.make_relative;
-  make_lsda_relative = hdr_info->last_cie.make_lsda_relative;
   sec_info = bfd_zmalloc (sizeof (struct eh_frame_sec_info)
                          + 99 * sizeof (struct eh_cie_fde));
   if (sec_info == NULL)
@@ -286,14 +336,14 @@ _bfd_elf_discard_section_eh_frame
 
 #define SKIP_RELOCS(buf)                               \
   while (cookie->rel < cookie->relend                  \
-         && (cookie->rel->r_offset                     \
+        && (cookie->rel->r_offset                      \
             < (bfd_size_type) ((buf) - ehbuf)))        \
     cookie->rel++
 
 #define GET_RELOC(buf)                                 \
   ((cookie->rel < cookie->relend                       \
     && (cookie->rel->r_offset                          \
-        == (bfd_size_type) ((buf) - ehbuf)))           \
+       == (bfd_size_type) ((buf) - ehbuf)))            \
    ? cookie->rel : NULL)
 
   for (;;)
@@ -302,18 +352,25 @@ _bfd_elf_discard_section_eh_frame
 
       if (sec_info->count == sec_info->alloced)
        {
+         struct eh_cie_fde *old_entry = sec_info->entry;
          sec_info = bfd_realloc (sec_info,
                                  sizeof (struct eh_frame_sec_info)
-                                 + (sec_info->alloced + 99)
-                                    * sizeof (struct eh_cie_fde));
+                                 + ((sec_info->alloced + 99)
+                                    * sizeof (struct eh_cie_fde)));
          if (sec_info == NULL)
            goto free_no_table;
 
          memset (&sec_info->entry[sec_info->alloced], 0,
                  100 * sizeof (struct eh_cie_fde));
          sec_info->alloced += 100;
+
+         /* Now fix any pointers into the array.  */
+         if (last_cie_inf >= old_entry
+             && last_cie_inf < old_entry + sec_info->count)
+           last_cie_inf = sec_info->entry + (last_cie_inf - old_entry);
        }
 
+      this_inf = sec_info->entry + sec_info->count;
       last_fde = buf;
       /* If we are at the end of the section, we still need to decide
         on whether to output or discard last encountered CIE (if any).  */
@@ -334,8 +391,8 @@ _bfd_elf_discard_section_eh_frame
            /* CIE/FDE not contained fully in this .eh_frame input section.  */
            goto free_no_table;
 
-         sec_info->entry[sec_info->count].offset = last_fde - ehbuf;
-         sec_info->entry[sec_info->count].size = 4 + hdr.length;
+         this_inf->offset = last_fde - ehbuf;
+         this_inf->size = 4 + hdr.length;
 
          if (hdr.length == 0)
            {
@@ -374,23 +431,14 @@ _bfd_elf_discard_section_eh_frame
                       == hdr_info->last_cie_sec->output_section)
                   && cie_compare (&cie, &hdr_info->last_cie) == 0)
                  || cie_usage_count == 0)
-               {
-                 new_size -= cie.hdr.length + 4;
-                 sec_info->entry[last_cie_ndx].removed = 1;
-                 sec_info->entry[last_cie_ndx].sec = hdr_info->last_cie_sec;
-                 sec_info->entry[last_cie_ndx].new_offset
-                   = hdr_info->last_cie_offset;
-               }
+               last_cie_inf->removed = 1;
              else
                {
                  hdr_info->last_cie = cie;
                  hdr_info->last_cie_sec = sec;
-                 hdr_info->last_cie_offset = last_cie - ehbuf;
-                 sec_info->entry[last_cie_ndx].make_relative
-                   = cie.make_relative;
-                 sec_info->entry[last_cie_ndx].make_lsda_relative
-                   = cie.make_lsda_relative;
-                 sec_info->entry[last_cie_ndx].per_encoding_relative
+                 last_cie_inf->make_relative = cie.make_relative;
+                 last_cie_inf->make_lsda_relative = cie.make_lsda_relative;
+                 last_cie_inf->per_encoding_relative
                    = (cie.per_encoding & 0x70) == DW_EH_PE_pcrel;
                }
            }
@@ -398,8 +446,8 @@ _bfd_elf_discard_section_eh_frame
          if (hdr.id == (unsigned int) -1)
            break;
 
-         last_cie_ndx = sec_info->count;
-         sec_info->entry[sec_info->count].cie = 1;
+         last_cie_inf = this_inf;
+         this_inf->cie = 1;
 
          cie_usage_count = 0;
          memset (&cie, 0, sizeof (cie));
@@ -499,7 +547,10 @@ _bfd_elf_discard_section_eh_frame
 
                              cie.personality = h;
                            }
-                         cookie->rel++;
+                         /* Cope with MIPS-style composite relocations.  */
+                         do
+                           cookie->rel++;
+                         while (GET_RELOC (buf) != NULL);
                        }
                      buf += per_width;
                    }
@@ -512,12 +563,25 @@ _bfd_elf_discard_section_eh_frame
 
          /* For shared libraries, try to get rid of as many RELATIVE relocs
             as possible.  */
-          if (info->shared
+         if (info->shared
              && (get_elf_backend_data (abfd)
                  ->elf_backend_can_make_relative_eh_frame
-                 (abfd, info, sec))
-             && (cie.fde_encoding & 0xf0) == DW_EH_PE_absptr)
-           cie.make_relative = 1;
+                 (abfd, info, sec)))
+           {
+             if ((cie.fde_encoding & 0xf0) == DW_EH_PE_absptr)
+               cie.make_relative = 1;
+             /* If the CIE doesn't already have an 'R' entry, it's fairly
+                easy to add one, provided that there's no aligned data
+                after the augmentation string.  */
+             else if (cie.fde_encoding == DW_EH_PE_omit
+                      && (cie.per_encoding & 0xf0) != DW_EH_PE_aligned)
+               {
+                 if (*cie.augmentation == 0)
+                   this_inf->add_augmentation_size = 1;
+                 this_inf->add_fde_encoding = 1;
+                 cie.make_relative = 1;
+               }
+           }
 
          if (info->shared
              && (get_elf_backend_data (abfd)
@@ -552,13 +616,11 @@ _bfd_elf_discard_section_eh_frame
          if (GET_RELOC (buf) == NULL)
            /* This should not happen.  */
            goto free_no_table;
+
          if ((*reloc_symbol_deleted_p) (buf - ehbuf, cookie))
-           {
-             /* This is a FDE against a discarded section.  It should
-                be deleted.  */
-             new_size -= hdr.length + 4;
-             sec_info->entry[sec_info->count].removed = 1;
-           }
+           /* This is a FDE against a discarded section.  It should
+              be deleted.  */
+           this_inf->removed = 1;
          else
            {
              if (info->shared
@@ -585,14 +647,14 @@ _bfd_elf_discard_section_eh_frame
                read_uleb128 (dummy, buf);
              /* If some new augmentation data is added before LSDA
                 in FDE augmentation area, this need to be adjusted.  */
-             sec_info->entry[sec_info->count].lsda_offset = (buf - aug);
+             this_inf->lsda_offset = (buf - aug);
            }
          buf = last_fde + 4 + hdr.length;
          SKIP_RELOCS (buf);
        }
 
-      sec_info->entry[sec_info->count].fde_encoding = cie.fde_encoding;
-      sec_info->entry[sec_info->count].lsda_encoding = cie.lsda_encoding;
+      this_inf->fde_encoding = cie.fde_encoding;
+      this_inf->lsda_encoding = cie.lsda_encoding;
       sec_info->count++;
     }
 
@@ -601,50 +663,27 @@ _bfd_elf_discard_section_eh_frame
 
   /* Ok, now we can assign new offsets.  */
   offset = 0;
-  last_cie_ndx = 0;
-  for (i = 0; i < sec_info->count; i++)
-    {
-      if (! sec_info->entry[i].removed)
-       {
-         sec_info->entry[i].new_offset = offset;
-         offset += sec_info->entry[i].size;
-         if (sec_info->entry[i].cie)
-           {
-             last_cie_ndx = i;
-             make_relative = sec_info->entry[i].make_relative;
-             make_lsda_relative = sec_info->entry[i].make_lsda_relative;
-           }
-         else
-           {
-             sec_info->entry[i].make_relative = make_relative;
-             sec_info->entry[i].make_lsda_relative = make_lsda_relative;
-             sec_info->entry[i].per_encoding_relative = 0;
-           }
-       }
-      else if (sec_info->entry[i].cie && sec_info->entry[i].sec == sec)
-       {
-         /* Need to adjust new_offset too.  */
-         BFD_ASSERT (sec_info->entry[last_cie_ndx].offset
-                     == sec_info->entry[i].new_offset);
-         sec_info->entry[i].new_offset
-           = sec_info->entry[last_cie_ndx].new_offset;
-       }
-    }
-  if (hdr_info->last_cie_sec == sec)
-    {
-      BFD_ASSERT (sec_info->entry[last_cie_ndx].offset
-                 == hdr_info->last_cie_offset);
-      hdr_info->last_cie_offset = sec_info->entry[last_cie_ndx].new_offset;
-    }
+  last_cie_inf = hdr_info->last_cie_inf;
+  for (ent = sec_info->entry; ent < sec_info->entry + sec_info->count; ++ent)
+    if (!ent->removed)
+      {
+       if (ent->cie)
+         last_cie_inf = ent;
+       else
+         ent->cie_inf = last_cie_inf;
+       ent->new_offset = offset;
+       offset += size_of_output_cie_fde (ent, ptr_size);
+      }
+  hdr_info->last_cie_inf = last_cie_inf;
 
-  /* Shrink the sec as needed.  */
+  /* Resize the sec as needed.  */
   sec->rawsize = sec->size;
-  sec->size = new_size;
+  sec->size = offset;
   if (sec->size == 0)
     sec->flags |= SEC_EXCLUDE;
 
   free (ehbuf);
-  return new_size != sec->rawsize;
+  return offset != sec->rawsize;
 
 free_no_table:
   if (ehbuf)
@@ -736,10 +775,13 @@ _bfd_elf_maybe_strip_eh_frame_hdr (struct bfd_link_info *info)
 
 bfd_vma
 _bfd_elf_eh_frame_section_offset (bfd *output_bfd ATTRIBUTE_UNUSED,
+                                 struct bfd_link_info *info,
                                  asection *sec,
                                  bfd_vma offset)
 {
   struct eh_frame_sec_info *sec_info;
+  struct elf_link_hash_table *htab;
+  struct eh_frame_hdr_info *hdr_info;
   unsigned int lo, hi, mid;
 
   if (sec->sec_info_type != ELF_INFO_TYPE_EH_FRAME)
@@ -749,6 +791,11 @@ _bfd_elf_eh_frame_section_offset (bfd *output_bfd ATTRIBUTE_UNUSED,
   if (offset >= sec->rawsize)
     return offset - sec->rawsize + sec->size;
 
+  htab = elf_hash_table (info);
+  hdr_info = &htab->eh_info;
+  if (hdr_info->offsets_adjusted)
+    offset += sec->output_offset;
+
   lo = 0;
   hi = sec_info->count;
   mid = 0;
@@ -772,21 +819,31 @@ _bfd_elf_eh_frame_section_offset (bfd *output_bfd ATTRIBUTE_UNUSED,
 
   /* If converting to DW_EH_PE_pcrel, there will be no need for run-time
      relocation against FDE's initial_location field.  */
-  if (sec_info->entry[mid].make_relative
-      && ! sec_info->entry[mid].cie
+  if (!sec_info->entry[mid].cie
+      && sec_info->entry[mid].cie_inf->make_relative
       && offset == sec_info->entry[mid].offset + 8)
     return (bfd_vma) -2;
 
   /* If converting LSDA pointers to DW_EH_PE_pcrel, there will be no need
      for run-time relocation against LSDA field.  */
-  if (sec_info->entry[mid].make_lsda_relative
-      && ! sec_info->entry[mid].cie
+  if (!sec_info->entry[mid].cie
+      && sec_info->entry[mid].cie_inf->make_lsda_relative
       && (offset == (sec_info->entry[mid].offset + 8
-                    + sec_info->entry[mid].lsda_offset)))
-    return (bfd_vma) -2;
+                    + sec_info->entry[mid].lsda_offset))
+      && (sec_info->entry[mid].cie_inf->need_lsda_relative
+         || !hdr_info->offsets_adjusted))
+    {
+      sec_info->entry[mid].cie_inf->need_lsda_relative = 1;
+      return (bfd_vma) -2;
+    }
 
+  if (hdr_info->offsets_adjusted)
+    offset -= sec->output_offset;
+  /* Any new augmentation bytes go before the first relocation.  */
   return (offset + sec_info->entry[mid].new_offset
-         - sec_info->entry[mid].offset);
+         - sec_info->entry[mid].offset
+         + extra_augmentation_string_bytes (sec_info->entry + mid)
+         + extra_augmentation_data_bytes (sec_info->entry + mid));
 }
 
 /* Write out .eh_frame section.  This is called with the relocated
@@ -801,11 +858,9 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
   struct eh_frame_sec_info *sec_info;
   struct elf_link_hash_table *htab;
   struct eh_frame_hdr_info *hdr_info;
-  unsigned int i;
-  bfd_byte *p, *buf;
   unsigned int leb128_tmp;
-  unsigned int cie_offset = 0;
   unsigned int ptr_size;
+  struct eh_cie_fde *ent;
 
   ptr_size = (elf_elfheader (sec->owner)->e_ident[EI_CLASS]
              == ELFCLASS64) ? 8 : 4;
@@ -816,54 +871,105 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
   sec_info = elf_section_data (sec)->sec_info;
   htab = elf_hash_table (info);
   hdr_info = &htab->eh_info;
+
+  /* First convert all offsets to output section offsets, so that a
+     CIE offset is valid if the CIE is used by a FDE from some other
+     section.  This can happen when duplicate CIEs are deleted in
+     _bfd_elf_discard_section_eh_frame.  We do all sections here because
+     this function might not be called on sections in the same order as
+     _bfd_elf_discard_section_eh_frame.  */
+  if (!hdr_info->offsets_adjusted)
+    {
+      bfd *ibfd;
+      asection *eh;
+      struct eh_frame_sec_info *eh_inf;
+
+      for (ibfd = info->input_bfds; ibfd != NULL; ibfd = ibfd->link_next)
+       {
+         if (bfd_get_flavour (ibfd) != bfd_target_elf_flavour
+             || (ibfd->flags & DYNAMIC) != 0)
+           continue;
+
+         eh = bfd_get_section_by_name (ibfd, ".eh_frame");
+         if (eh == NULL || eh->sec_info_type != ELF_INFO_TYPE_EH_FRAME)
+           continue;
+
+         eh_inf = elf_section_data (eh)->sec_info;
+         for (ent = eh_inf->entry; ent < eh_inf->entry + eh_inf->count; ++ent)
+           {
+             ent->offset += eh->output_offset;
+             ent->new_offset += eh->output_offset;
+           }
+       }
+      hdr_info->offsets_adjusted = TRUE;
+    }
+
   if (hdr_info->table && hdr_info->array == NULL)
     hdr_info->array
       = bfd_malloc (hdr_info->fde_count * sizeof(*hdr_info->array));
   if (hdr_info->array == NULL)
     hdr_info = NULL;
 
-  p = contents;
-  for (i = 0; i < sec_info->count; ++i)
+  /* The new offsets can be bigger or smaller than the original offsets.
+     We therefore need to make two passes over the section: one backward
+     pass to move entries up and one forward pass to move entries down.
+     The two passes won't interfere with each other because entries are
+     not reordered  */
+  for (ent = sec_info->entry + sec_info->count; ent-- != sec_info->entry;)
+    if (!ent->removed && ent->new_offset > ent->offset)
+      memmove (contents + ent->new_offset - sec->output_offset,
+              contents + ent->offset - sec->output_offset, ent->size);
+
+  for (ent = sec_info->entry; ent < sec_info->entry + sec_info->count; ++ent)
+    if (!ent->removed && ent->new_offset < ent->offset)
+      memmove (contents + ent->new_offset - sec->output_offset,
+              contents + ent->offset - sec->output_offset, ent->size);
+
+  for (ent = sec_info->entry; ent < sec_info->entry + sec_info->count; ++ent)
     {
-      if (sec_info->entry[i].removed)
+      unsigned char *buf, *end;
+      unsigned int new_size;
+
+      if (ent->removed)
+       continue;
+
+      if (ent->size == 4)
        {
-         if (sec_info->entry[i].cie)
-           {
-             /* If CIE is removed due to no remaining FDEs referencing it
-                and there were no CIEs kept before it, sec_info->entry[i].sec
-                will be zero.  */
-             if (sec_info->entry[i].sec == NULL)
-               cie_offset = 0;
-             else
-               {
-                 cie_offset = sec_info->entry[i].new_offset;
-                 cie_offset += (sec_info->entry[i].sec->output_section->vma
-                                + sec_info->entry[i].sec->output_offset
-                                - sec->output_section->vma
-                                - sec->output_offset);
-               }
-           }
+         /* Any terminating FDE must be at the end of the section.  */
+         BFD_ASSERT (ent == sec_info->entry + sec_info->count - 1);
          continue;
        }
 
-      if (sec_info->entry[i].cie)
+      buf = contents + ent->new_offset - sec->output_offset;
+      end = buf + ent->size;
+      new_size = size_of_output_cie_fde (ent, ptr_size);
+
+      /* Install the new size, filling the extra bytes with DW_CFA_nops.  */
+      if (new_size != ent->size)
+       {
+         memset (end, 0, new_size - ent->size);
+         bfd_put_32 (abfd, new_size - 4, buf);
+       }
+
+      if (ent->cie)
        {
          /* CIE */
-         cie_offset = sec_info->entry[i].new_offset;
-         if (sec_info->entry[i].make_relative
-             || sec_info->entry[i].make_lsda_relative
-             || sec_info->entry[i].per_encoding_relative)
+         if (ent->make_relative
+             || ent->need_lsda_relative
+             || ent->per_encoding_relative)
            {
              unsigned char *aug;
-             unsigned int action;
+             unsigned int action, extra_string, extra_data;
              unsigned int dummy, per_width, per_encoding;
 
              /* Need to find 'R' or 'L' augmentation's argument and modify
                 DW_EH_PE_* value.  */
-             action = (sec_info->entry[i].make_relative ? 1 : 0)
-                      | (sec_info->entry[i].make_lsda_relative ? 2 : 0)
-                      | (sec_info->entry[i].per_encoding_relative ? 4 : 0);
-             buf = contents + sec_info->entry[i].offset;
+             action = ((ent->make_relative ? 1 : 0)
+                       | (ent->need_lsda_relative ? 2 : 0)
+                       | (ent->per_encoding_relative ? 4 : 0));
+             extra_string = extra_augmentation_string_bytes (ent);
+             extra_data = extra_augmentation_data_bytes (ent);
+
              /* Skip length, id and version.  */
              buf += 9;
              aug = buf;
@@ -873,17 +979,37 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
              read_uleb128 (dummy, buf);
              if (*aug == 'z')
                {
-                 read_uleb128 (dummy, buf);
+                 /* The uleb128 will always be a single byte for the kind
+                    of augmentation strings that we're prepared to handle.  */
+                 *buf++ += extra_data;
                  aug++;
                }
 
+             /* Make room for the new augmentation string and data bytes.  */
+             memmove (buf + extra_string + extra_data, buf, end - buf);
+             memmove (aug + extra_string, aug, buf - aug);
+             buf += extra_string;
+
+             if (ent->add_augmentation_size)
+               {
+                 *aug++ = 'z';
+                 *buf++ = extra_data - 1;
+               }
+             if (ent->add_fde_encoding)
+               {
+                 BFD_ASSERT (action & 1);
+                 *aug++ = 'R';
+                 *buf++ = DW_EH_PE_pcrel;
+                 action &= ~1;
+               }
+
              while (action)
                switch (*aug++)
                  {
                  case 'L':
                    if (action & 2)
                      {
-                       BFD_ASSERT (*buf == sec_info->entry[i].lsda_encoding);
+                       BFD_ASSERT (*buf == ent->lsda_encoding);
                        *buf |= DW_EH_PE_pcrel;
                        action &= ~2;
                      }
@@ -891,25 +1017,23 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
                    break;
                  case 'P':
                    per_encoding = *buf++;
-                    per_width = get_DW_EH_PE_width (per_encoding,
-                                                   ptr_size);
+                   per_width = get_DW_EH_PE_width (per_encoding, ptr_size);
                    BFD_ASSERT (per_width != 0);
                    BFD_ASSERT (((per_encoding & 0x70) == DW_EH_PE_pcrel)
-                               == sec_info->entry[i].per_encoding_relative);
+                               == ent->per_encoding_relative);
                    if ((per_encoding & 0xf0) == DW_EH_PE_aligned)
                      buf = (contents
                             + ((buf - contents + per_width - 1)
                                & ~((bfd_size_type) per_width - 1)));
                    if (action & 4)
                      {
-                       bfd_vma value;
-
-                       value = read_value (abfd, buf, per_width,
-                                           get_DW_EH_PE_signed
-                                           (per_encoding));
-                       value += (sec_info->entry[i].offset
-                                 - sec_info->entry[i].new_offset);
-                       write_value (abfd, buf, value, per_width);
+                       bfd_vma val;
+
+                       val = read_value (abfd, buf, per_width,
+                                         get_DW_EH_PE_signed (per_encoding));
+                       val += ent->offset - ent->new_offset;
+                       val -= extra_string + extra_data;
+                       write_value (abfd, buf, val, per_width);
                        action &= ~4;
                      }
                    buf += per_width;
@@ -917,7 +1041,7 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
                  case 'R':
                    if (action & 1)
                      {
-                       BFD_ASSERT (*buf == sec_info->entry[i].fde_encoding);
+                       BFD_ASSERT (*buf == ent->fde_encoding);
                        *buf |= DW_EH_PE_pcrel;
                        action &= ~1;
                      }
@@ -928,26 +1052,24 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
                  }
            }
        }
-      else if (sec_info->entry[i].size > 4)
+      else
        {
          /* FDE */
-         bfd_vma value = 0, address;
+         bfd_vma value, address;
          unsigned int width;
 
-         buf = contents + sec_info->entry[i].offset;
          /* Skip length.  */
          buf += 4;
-         bfd_put_32 (abfd,
-                     sec_info->entry[i].new_offset + 4 - cie_offset, buf);
+         value = ent->new_offset + 4 - ent->cie_inf->new_offset;
+         bfd_put_32 (abfd, value, buf);
          buf += 4;
-         width = get_DW_EH_PE_width (sec_info->entry[i].fde_encoding,
-                                     ptr_size);
-         address = value = read_value (abfd, buf, width,
-                                       get_DW_EH_PE_signed
-                                       (sec_info->entry[i].fde_encoding));
+         width = get_DW_EH_PE_width (ent->fde_encoding, ptr_size);
+         value = read_value (abfd, buf, width,
+                             get_DW_EH_PE_signed (ent->fde_encoding));
+         address = value;
          if (value)
            {
-             switch (sec_info->entry[i].fde_encoding & 0xf0)
+             switch (ent->fde_encoding & 0xf0)
                {
                case DW_EH_PE_indirect:
                case DW_EH_PE_textrel:
@@ -962,15 +1084,12 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
                  }
                  break;
                case DW_EH_PE_pcrel:
-                 value += (sec_info->entry[i].offset
-                           - sec_info->entry[i].new_offset);
-                 address += (sec->output_section->vma + sec->output_offset
-                             + sec_info->entry[i].offset + 8);
+                 value += ent->offset - ent->new_offset;
+                 address += sec->output_section->vma + ent->offset + 8;
                  break;
                }
-             if (sec_info->entry[i].make_relative)
-               value -= (sec->output_section->vma + sec->output_offset
-                         + sec_info->entry[i].new_offset + 8);
+             if (ent->cie_inf->make_relative)
+               value -= sec->output_section->vma + ent->new_offset + 8;
              write_value (abfd, buf, value, width);
            }
 
@@ -978,41 +1097,35 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
            {
              hdr_info->array[hdr_info->array_count].initial_loc = address;
              hdr_info->array[hdr_info->array_count++].fde
-               = (sec->output_section->vma + sec->output_offset
-                  + sec_info->entry[i].new_offset);
+               = sec->output_section->vma + ent->new_offset;
            }
 
-         if ((sec_info->entry[i].lsda_encoding & 0xf0) == DW_EH_PE_pcrel
-             || sec_info->entry[i].make_lsda_relative)
+         if ((ent->lsda_encoding & 0xf0) == DW_EH_PE_pcrel
+             || ent->cie_inf->need_lsda_relative)
            {
-             buf += sec_info->entry[i].lsda_offset;
-             width = get_DW_EH_PE_width (sec_info->entry[i].lsda_encoding,
-                                         ptr_size);
+             buf += ent->lsda_offset;
+             width = get_DW_EH_PE_width (ent->lsda_encoding, ptr_size);
              value = read_value (abfd, buf, width,
-                                 get_DW_EH_PE_signed
-                                 (sec_info->entry[i].lsda_encoding));
+                                 get_DW_EH_PE_signed (ent->lsda_encoding));
              if (value)
                {
-                 if ((sec_info->entry[i].lsda_encoding & 0xf0)
-                     == DW_EH_PE_pcrel)
-                   value += (sec_info->entry[i].offset
-                             - sec_info->entry[i].new_offset);
-                 else if (sec_info->entry[i].make_lsda_relative)
-                   value -= (sec->output_section->vma + sec->output_offset
-                             + sec_info->entry[i].new_offset + 8
-                             + sec_info->entry[i].lsda_offset);
+                 if ((ent->lsda_encoding & 0xf0) == DW_EH_PE_pcrel)
+                   value += ent->offset - ent->new_offset;
+                 else if (ent->cie_inf->need_lsda_relative)
+                   value -= (sec->output_section->vma + ent->new_offset + 8
+                             + ent->lsda_offset);
                  write_value (abfd, buf, value, width);
                }
            }
+         else if (ent->cie_inf->add_augmentation_size)
+           {
+             /* Skip the PC and length and insert a zero byte for the
+                augmentation size.  */
+             buf += width * 2;
+             memmove (buf + 1, buf, end - buf);
+             *buf = 0;
+           }
        }
-      else
-       /* Terminating FDE must be at the end of .eh_frame section only.  */
-       BFD_ASSERT (i == sec_info->count - 1);
-
-      BFD_ASSERT (p == contents + sec_info->entry[i].new_offset);
-      memmove (p, contents + sec_info->entry[i].offset,
-              sec_info->entry[i].size);
-      p += sec_info->entry[i].size;
     }
 
     {
@@ -1025,37 +1138,34 @@ _bfd_elf_write_section_eh_frame (bfd *abfd,
          && ((sec->output_offset + sec->size + pad)
              <= sec->output_section->size))
        {
+         bfd_byte *buf;
+         unsigned int new_size;
+
          /* Find the last CIE/FDE.  */
-         for (i = sec_info->count - 1; i > 0; i--)
-           if (! sec_info->entry[i].removed)
+         ent = sec_info->entry + sec_info->count;
+         while (--ent != sec_info->entry)
+           if (!ent->removed)
              break;
 
          /* The size of the last CIE/FDE must be at least 4.  */
-         if (sec_info->entry[i].removed
-             || sec_info->entry[i].size < 4)
+         if (ent->removed || ent->size < 4)
            abort ();
 
          pad = alignment - pad;
-
-         buf = contents + sec_info->entry[i].new_offset;
-
-         /* Update length.  */
-         sec_info->entry[i].size += pad;
-         bfd_put_32 (abfd, sec_info->entry[i].size - 4, buf);
+         buf = contents + ent->new_offset - sec->output_offset;
+         new_size = size_of_output_cie_fde (ent, ptr_size);
 
          /* Pad it with DW_CFA_nop  */
-         memset (p, 0, pad);
-         p += pad;
+         memset (buf + new_size, 0, pad);
+         bfd_put_32 (abfd, new_size + pad - 4, buf);
 
          sec->size += pad;
        }
     }
 
-  BFD_ASSERT ((bfd_size_type) (p - contents) == sec->size);
-
   return bfd_set_section_contents (abfd, sec->output_section,
-                                   contents, (file_ptr) sec->output_offset,
-                                   sec->size);
+                                  contents, (file_ptr) sec->output_offset,
+                                  sec->size);
 }
 
 /* Helper function used to sort .eh_frame_hdr search table by increasing
This page took 0.034454 seconds and 4 git commands to generate.