* elf-eh-frame.c (ENSURE_NO_RELOCS): Disregard R_*_NONE relocs.
[deliverable/binutils-gdb.git] / bfd / mmo.c
index e9034de3db79c2033957b8f1707dfe3d221998a1..96c654e8ee92dd4726c239ae6820ef56de18653e 100644 (file)
--- a/bfd/mmo.c
+++ b/bfd/mmo.c
@@ -1,5 +1,5 @@
 /* BFD back-end for mmo objects (MMIX-specific object-format).
-   Copyright 2001
+   Copyright 2001, 2002
    Free Software Foundation, Inc.
    Written by Hans-Peter Nilsson (hp@bitrange.com).
    Infrastructure and other bits originally copied from srec.c and
@@ -201,7 +201,6 @@ EXAMPLE
 #include "libiberty.h"
 #include "elf/mmix.h"
 #include "opcode/mmix.h"
-#include <ctype.h>
 
 #define LOP 0x98
 #define LOP_QUOTE 0
@@ -261,7 +260,7 @@ enum mmo_sym_type { mmo_reg_sym, mmo_undef_sym, mmo_data_sym, mmo_abs_sym};
 struct mmo_symbol
   {
     struct mmo_symbol *next;
-    CONST char *name;
+    const char *name;
     bfd_vma value;
     enum mmo_sym_type sym_type;
     unsigned int serno;
@@ -358,7 +357,7 @@ static void mmo_write_section_unless_reg_contents
  PARAMS ((bfd *, asection *, PTR));
 static void mmo_find_sec_w_addr PARAMS ((bfd *, asection *, PTR));
 static void mmo_find_sec_w_addr_grow PARAMS ((bfd *, asection *, PTR));
-static asection *mmo_make_section PARAMS ((bfd *, CONST char *));
+static asection *mmo_make_section PARAMS ((bfd *, const char *));
 static void mmo_get_symbol_info PARAMS ((bfd *, asymbol *, symbol_info *));
 static void mmo_print_symbol
  PARAMS ((bfd *, PTR, asymbol *, bfd_print_symbol_type));
@@ -372,16 +371,15 @@ static INLINE bfd_byte *mmo_get_loc PARAMS ((asection *, bfd_vma, int));
 static void mmo_xore_64 PARAMS ((asection *, bfd_vma vma, bfd_vma value));
 static void mmo_xore_32 PARAMS ((asection *, bfd_vma vma, unsigned int));
 static void mmo_xore_16 PARAMS ((asection *, bfd_vma vma, unsigned int));
-static CONST bfd_target *mmo_object_p PARAMS ((bfd *));
+static const bfd_target *mmo_object_p PARAMS ((bfd *));
 static void mmo_map_set_sizes PARAMS ((bfd *, asection *, PTR));
 static boolean mmo_get_symbols PARAMS ((bfd *));
-static boolean mmo_create_symbol PARAMS ((bfd *, CONST char *, bfd_vma,
+static boolean mmo_create_symbol PARAMS ((bfd *, const char *, bfd_vma,
                                          enum mmo_sym_type, unsigned int));
 static boolean mmo_get_section_contents
   PARAMS ((bfd *, asection *, PTR, file_ptr, bfd_size_type));
 static long mmo_get_symtab_upper_bound PARAMS ((bfd *));
 static long mmo_get_symtab PARAMS ((bfd *, asymbol **));
-static asymbol *mmo_make_empty_symbol PARAMS ((bfd *));
 static void mmo_get_symbol_info PARAMS ((bfd *, asymbol *, symbol_info *));
 static void mmo_print_symbol PARAMS ((bfd *, PTR, asymbol *,
                                      bfd_print_symbol_type));
@@ -393,7 +391,7 @@ static long mmo_get_reloc_upper_bound PARAMS ((bfd *, asection *));
 static boolean mmo_internal_write_header PARAMS ((bfd *));
 static boolean mmo_internal_write_post PARAMS ((bfd *, int, asection *));
 static boolean mmo_internal_add_3_sym
- PARAMS ((bfd *, struct mmo_symbol_trie *, CONST struct mmo_symbol *));
+ PARAMS ((bfd *, struct mmo_symbol_trie *, const struct mmo_symbol *));
 static unsigned int mmo_internal_3_length
  PARAMS ((bfd *, struct mmo_symbol_trie *));
 static void mmo_internal_3_dump
@@ -406,9 +404,10 @@ static void mmo_write_tetra_raw PARAMS ((bfd *, unsigned int));
 static void mmo_write_octa PARAMS ((bfd *, bfd_vma));
 static void mmo_write_octa_raw PARAMS ((bfd *, bfd_vma));
 static boolean mmo_write_chunk
-  PARAMS ((bfd *, CONST bfd_byte *, unsigned int));
+  PARAMS ((bfd *, const bfd_byte *, unsigned int));
+static boolean mmo_flush_chunk PARAMS ((bfd *));
 static boolean mmo_write_loc_chunk
-  PARAMS ((bfd *, bfd_vma, CONST bfd_byte *, unsigned int));
+  PARAMS ((bfd *, bfd_vma, const bfd_byte *, unsigned int, bfd_vma *));
 static boolean mmo_write_chunk_list PARAMS ((bfd *, mmo_data_list_type *));
 static boolean mmo_write_loc_chunk_list
   PARAMS ((bfd *, mmo_data_list_type *));
@@ -418,7 +417,7 @@ static flagword bfd_sec_flags_from_mmo_flags PARAMS ((flagword));
 static bfd_byte mmo_get_byte PARAMS ((bfd *));
 static void mmo_write_byte PARAMS ((bfd *, bfd_byte));
 static boolean mmo_new_section_hook PARAMS ((bfd *, asection *));
-static int mmo_sort_mmo_symbols PARAMS ((CONST PTR, CONST PTR));
+static int mmo_sort_mmo_symbols PARAMS ((const PTR, const PTR));
 static boolean mmo_write_object_contents PARAMS ((bfd *));
 static long mmo_canonicalize_reloc
   PARAMS ((bfd *, sec_ptr, arelent **, asymbol **));
@@ -447,7 +446,7 @@ char valid_mmo_symbol_character_set[/* A-Z a-z (we assume consecutive
 static asection *
 mmo_make_section (abfd, secname)
      bfd *abfd;
-     CONST char *secname;
+     const char *secname;
 {
   asection *sec = bfd_get_section_by_name (abfd, secname);
 
@@ -482,10 +481,9 @@ mmo_init ()
   static const char letters[]
     = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789:_";
 
-  if (inited == false)
-    {
-      inited = true;
-    }
+  if (inited)
+    return;
+  inited = true;
 
   /* Fill in the set of valid symbol characters.  */
   strcpy (valid_mmo_symbol_character_set, letters);
@@ -497,7 +495,7 @@ mmo_init ()
 
 /* Check whether an existing file is an mmo file.  */
 
-static CONST bfd_target *
+static const bfd_target *
 mmo_object_p (abfd)
      bfd *abfd;
 {
@@ -815,26 +813,42 @@ mmo_write_octa_raw (abfd, value)
   mmo_write_tetra_raw (abfd, (unsigned int) value);
 }
 
-/* Write quoted contents, zero filled.  */
+/* Write quoted contents.  Intended to be called multiple times in
+   sequence, followed by a call to mmo_flush_chunk.  */
 
 static INLINE boolean
 mmo_write_chunk (abfd, loc, len)
      bfd *abfd;
-     CONST bfd_byte *loc;
+     const bfd_byte *loc;
      unsigned int len;
 {
   boolean retval = true;
-  bfd_byte buf[4] = {0, 0, 0, 0};
+
+  /* Fill up a tetra from bytes remaining from a previous chunk.  */
+  if (abfd->tdata.mmo_data->byte_no != 0)
+    {
+      while (abfd->tdata.mmo_data->byte_no < 4 && len != 0)
+       {
+         abfd->tdata.mmo_data->buf[abfd->tdata.mmo_data->byte_no++] = *loc++;
+         len--;
+       }
+
+      if (abfd->tdata.mmo_data->byte_no == 4)
+       {
+         mmo_write_tetra (abfd,
+                          bfd_get_32 (abfd, abfd->tdata.mmo_data->buf));
+         abfd->tdata.mmo_data->byte_no = 0;
+       }
+    }
 
   while (len >= 4)
     {
       if (loc[0] == LOP)
        mmo_write_tetra_raw (abfd, LOP_QUOTE_NEXT);
 
-      retval
-       = (retval == true
-          && abfd->tdata.mmo_data->have_error == false
-          && 4 == bfd_bwrite ((PTR) loc, 4, abfd));
+      retval = (retval
+               && ! abfd->tdata.mmo_data->have_error
+               && 4 == bfd_bwrite ((PTR) loc, 4, abfd));
 
       loc += 4;
       len -= 4;
@@ -842,19 +856,34 @@ mmo_write_chunk (abfd, loc, len)
 
   if (len)
     {
-      memcpy (buf, loc, len);
-      if (buf[0] == LOP)
-       mmo_write_tetra_raw (abfd, LOP_QUOTE_NEXT);
-
-      retval
-       = (retval == true
-          && abfd->tdata.mmo_data->have_error == false
-          && 4 == bfd_bwrite ((PTR) buf, 4, abfd));
+      memcpy (abfd->tdata.mmo_data->buf, loc, len);
+      abfd->tdata.mmo_data->byte_no = len;
     }
 
+  if (! retval)
+    abfd->tdata.mmo_data->have_error = true;
   return retval;
 }
 
+/* Flush remaining bytes, from a previous mmo_write_chunk, zero-padded to
+   4 bytes.  */
+
+static INLINE boolean
+mmo_flush_chunk (abfd)
+     bfd *abfd;
+{
+  if (abfd->tdata.mmo_data->byte_no != 0)
+    {
+      memset (abfd->tdata.mmo_data->buf + abfd->tdata.mmo_data->byte_no,
+             0, 4 - abfd->tdata.mmo_data->byte_no);
+      mmo_write_tetra (abfd,
+                      bfd_get_32 (abfd, abfd->tdata.mmo_data->buf));
+      abfd->tdata.mmo_data->byte_no = 0;
+    }
+
+  return ! abfd->tdata.mmo_data->have_error;
+}
+
 /* Same, but from a list.  */
 
 static INLINE boolean
@@ -866,39 +895,60 @@ mmo_write_chunk_list (abfd, datap)
     if (! mmo_write_chunk (abfd, datap->data, datap->size))
       return false;
 
-  return true;
+  return mmo_flush_chunk (abfd);
 }
 
-/* Write a lop_loc and some contents.  */
+/* Write a lop_loc and some contents.  A caller needs to call
+   mmo_flush_chunk after calling this function.  The location is only
+   output if different than *LAST_VMAP, which is updated after this call.  */
 
 static boolean
-mmo_write_loc_chunk (abfd, vma, loc, len)
+mmo_write_loc_chunk (abfd, vma, loc, len, last_vmap)
      bfd *abfd;
      bfd_vma vma;
-     CONST bfd_byte *loc;
+     const bfd_byte *loc;
      unsigned int len;
+     bfd_vma *last_vmap;
 {
-  /* We always write the location as 64 bits; no use saving bytes here.  */
-  mmo_write_tetra_raw (abfd, (LOP << 24) | (LOP_LOC << 16) | 2);
-
   /* Find an initial and trailing section of zero tetras; we don't need to
      write out zeros.  FIXME: When we do this, we should emit section size
-     and address specifiers, else objcopy can't perform a unity
-     translation.  */
-  while (len >= 4 && bfd_get_32 (abfd, loc) == 0)
+     and address specifiers, else objcopy can't always perform an identity
+     translation.  Only do this if we *don't* have left-over data from a
+     previous write or the vma of this chunk is *not* the next address,
+     because then data isn't tetrabyte-aligned and we're concatenating to
+     that left-over data.  */
+
+  if (abfd->tdata.mmo_data->byte_no == 0 || vma != *last_vmap)
     {
-      vma += 4;
-      len -= 4;
-      loc += 4;
+      while (len >= 4 && bfd_get_32 (abfd, loc) == 0)
+       {
+         vma += 4;
+         len -= 4;
+         loc += 4;
+       }
+
+      while (len >= 4 && bfd_get_32 (abfd, loc + len - 4) == 0)
+       len -= 4;
+    }
+
+  /* Only write out the location if it's different than the one the caller
+     (supposedly) previously handled, accounting for omitted leading zeros.  */
+  if (vma != *last_vmap)
+    {
+      /* We might be in the middle of a sequence.  */
+      mmo_flush_chunk (abfd);
+
+      /* We always write the location as 64 bits; no use saving bytes
+         here.  */
+      mmo_write_tetra_raw (abfd, (LOP << 24) | (LOP_LOC << 16) | 2);
+      mmo_write_octa_raw (abfd, vma);
     }
 
-  while (len >= 4 && bfd_get_32 (abfd, loc + len - 4) == 0)
-    len -= 4;
+  /* Update to reflect end of this chunk, with trailing zeros omitted.  */
+  *last_vmap = vma + len;
 
-  mmo_write_octa_raw (abfd, vma);
-  return
-    abfd->tdata.mmo_data->have_error == false
-    && mmo_write_chunk (abfd, loc, len);
+  return (! abfd->tdata.mmo_data->have_error
+         && mmo_write_chunk (abfd, loc, len));
 }
 
 /* Same, but from a list.  */
@@ -908,11 +958,15 @@ mmo_write_loc_chunk_list (abfd, datap)
      bfd *abfd;
      mmo_data_list_type *datap;
 {
+  /* Get an address different than the address of the first chunk.  */
+  bfd_vma last_vma = datap ? datap->where - 1 : 0;
+
   for (; datap != NULL; datap = datap->next)
-    if (! mmo_write_loc_chunk (abfd, datap->where, datap->data, datap->size))
+    if (! mmo_write_loc_chunk (abfd, datap->where, datap->data, datap->size,
+                              &last_vma))
       return false;
 
-  return true;
+  return mmo_flush_chunk (abfd);
 }
 
 /* Make a .MMIX.spec_data.N section.  */
@@ -1112,7 +1166,7 @@ mmo_get_byte (abfd)
 
   if (abfd->tdata.mmo_data->byte_no == 0)
     {
-      if (abfd->tdata.mmo_data->have_error == false
+      if (! abfd->tdata.mmo_data->have_error
          && bfd_bread (abfd->tdata.mmo_data->buf, 4, abfd) != 4)
        {
          abfd->tdata.mmo_data->have_error = true;
@@ -1139,7 +1193,7 @@ mmo_write_byte (abfd, value)
   abfd->tdata.mmo_data->buf[(abfd->tdata.mmo_data->byte_no++ % 4)] = value;
   if ((abfd->tdata.mmo_data->byte_no % 4) == 0)
     {
-      if (abfd->tdata.mmo_data->have_error == false
+      if (! abfd->tdata.mmo_data->have_error
          && bfd_bwrite (abfd->tdata.mmo_data->buf, 4, abfd) != 4)
        abfd->tdata.mmo_data->have_error = true;
     }
@@ -1150,7 +1204,7 @@ mmo_write_byte (abfd, value)
 static boolean
 mmo_create_symbol (abfd, symname, addr, sym_type, serno)
      bfd *abfd;
-     CONST char *symname;
+     const char *symname;
      bfd_vma addr;
      enum mmo_sym_type sym_type;
      unsigned int serno;
@@ -1305,7 +1359,7 @@ SUBSECTION
   bfd_byte m = mmo_get_byte (abfd);
 
   /* Check first if we have a bad hair day.  */
-  if (abfd->tdata.mmo_data->have_error == true)
+  if (abfd->tdata.mmo_data->have_error)
     return false;
 
   if (m & MMO3_LEFT)
@@ -1389,7 +1443,7 @@ SUBSECTION
          serno -= 128;
 
          /* Got it.  Now enter it.  Skip a leading ":".  */
-         if (abfd->tdata.mmo_data->have_error == false
+         if (! abfd->tdata.mmo_data->have_error
              && ! mmo_create_symbol (abfd,
                                      abfd->tdata.mmo_data->lop_stab_symbol
                                      + 1,
@@ -1408,7 +1462,7 @@ SUBSECTION
     /* Traverse right trie.  */
     mmo_get_symbols (abfd);
 
-  return abfd->tdata.mmo_data->have_error == false;
+  return ! abfd->tdata.mmo_data->have_error;
 }
 
 /* Get the location of memory area [VMA..VMA + SIZE - 1], which we think
@@ -1428,6 +1482,8 @@ mmo_get_loc (sec, vma, size)
   struct mmo_data_list_struct *datap = sdatap->head;
   struct mmo_data_list_struct *entry;
 
+  /* First search the list to see if we have the requested chunk in one
+     piece, or perhaps if we have a suitable chunk with room to fit.  */
   for (; datap != NULL; datap = datap->next)
     {
       if (datap->where <= vma
@@ -1443,7 +1499,11 @@ mmo_get_loc (sec, vma, size)
             it.  Do that now.  */
          datap->size += (vma + size) - (datap->where + datap->size);
 
-         /* Update the section size.  */
+         /* Update the section size.  This happens only if we update the
+            32-bit-aligned chunk size.  Callers that have
+            non-32-bit-aligned sections should do all allocation and
+            size-setting by themselves or at least set the section size
+            after the last allocating call to this function.  */
          if (vma + size > sec->vma + sec->_raw_size)
            sec->_raw_size += (vma + size) - (sec->vma + sec->_raw_size);
 
@@ -1468,7 +1528,7 @@ mmo_get_loc (sec, vma, size)
   entry = (mmo_data_list_type *)
     bfd_zalloc (sec->owner, sizeof (mmo_data_list_type) + allocated_size);
   if (entry == NULL)
-    return false;
+    return NULL;
   entry->where = vma;
   entry->size = size;
   entry->allocated_size = allocated_size;
@@ -1505,7 +1565,8 @@ mmo_get_loc (sec, vma, size)
        }
     }
 
-  /* Update the section size.  */
+  /* Update the section size.  This happens only when we add contents and
+     re-size as we go.  The section size will then be aligned to 32 bits.  */
   if (vma + size > sec->vma + sec->_raw_size)
     sec->_raw_size += (vma + size) - (sec->vma + sec->_raw_size);
   return entry->data;
@@ -2004,7 +2065,7 @@ mmo_scan (abfd)
   for (i = 0; i < sizeof (file_names) / sizeof (file_names[0]); i++)
     if (file_names[i])
       free (file_names[i]);
-  return error ? false : true;
+  return ! error;
 }
 
 /* A hook to set up object file dependent section information.  For mmo,
@@ -2079,11 +2140,11 @@ mmo_get_symtab_upper_bound (abfd)
 
 static int
 mmo_sort_mmo_symbols (arg1, arg2)
-     CONST PTR arg1;
-     CONST PTR arg2;
+     const PTR arg1;
+     const PTR arg2;
 {
-  CONST struct mmo_symbol *sym1 = *(CONST struct mmo_symbol **) arg1;
-  CONST struct mmo_symbol *sym2 = *(CONST struct mmo_symbol **) arg2;
+  const struct mmo_symbol *sym1 = *(const struct mmo_symbol **) arg1;
+  const struct mmo_symbol *sym2 = *(const struct mmo_symbol **) arg2;
 
   /* Sort by serial number first.  */
   if (sym1->serno < sym2->serno)
@@ -2092,7 +2153,7 @@ mmo_sort_mmo_symbols (arg1, arg2)
     return 1;
 
   /* Then sort by address of the table entries.  */
-  return ((CONST char *) arg1 - (CONST char *) arg2);
+  return ((const char *) arg1 - (const char *) arg2);
 }
 
 /* Translate the symbol table.  */
@@ -2188,19 +2249,6 @@ mmo_get_symtab (abfd, alocation)
   return symcount;
 }
 
-/* Make an empty symbol.  */
-
-static asymbol *
-mmo_make_empty_symbol (abfd)
-     bfd *abfd;
-{
-  asymbol *new = (asymbol *) bfd_zalloc (abfd, sizeof (asymbol));
-
-  if (new)
-    new->the_bfd = abfd;
-  return new;
-}
-
 /* Get information about a symbol.  */
 
 static void
@@ -2252,7 +2300,7 @@ static boolean
 mmo_internal_write_header (abfd)
      bfd *abfd;
 {
-  CONST char lop_pre_bfd[] = { LOP, LOP_PRE, 1, 1};
+  const char lop_pre_bfd[] = { LOP, LOP_PRE, 1, 1};
 
   if (bfd_bwrite (lop_pre_bfd, 4, abfd) != 4)
     return false;
@@ -2292,9 +2340,7 @@ mmo_internal_write_post (abfd, z, sec)
      Z == 255, don't assume DATA is valid.  */
   bfd_put_64 (abfd, bfd_get_start_address (abfd), buf);
 
-  return
-    abfd->tdata.mmo_data->have_error == false
-    && bfd_bwrite (buf, 8, abfd) == 8;
+  return ! abfd->tdata.mmo_data->have_error && bfd_bwrite (buf, 8, abfd) == 8;
 }
 
 /* Translate to and from BFD flags.  This is to make sure that we don't
@@ -2399,11 +2445,10 @@ mmo_internal_write_section (abfd, sec)
     {
       int n = atoi (sec->name + strlen (MMIX_OTHER_SPEC_SECTION_PREFIX));
       mmo_write_tetra_raw (abfd, (LOP << 24) | (LOP_SPEC << 16) | n);
-      return
-       abfd->tdata.mmo_data->have_error == false
-       && mmo_write_chunk_list (abfd,
-                                ((struct mmo_section_data_struct *)
-                                 (sec->used_by_bfd))->head);
+      return (! abfd->tdata.mmo_data->have_error
+             && mmo_write_chunk_list (abfd,
+                                      ((struct mmo_section_data_struct *)
+                                       (sec->used_by_bfd))->head));
     }
   /* Ignore sections that are just allocated or empty; we write out
      _contents_ here.  */
@@ -2465,8 +2510,8 @@ EXAMPLE
 | 0x00000000 - high 32 bits of section address
 | 0x00000004 - section address is 4
 | 0x98010002 - 64 bits with address of following data
-| 0x00000000 - high 64 bits of address
-| 0x00000004 - data starts at address 4
+| 0x00000000 - high 32 bits of address
+| 0x00000004 - low 32 bits: data starts at address 4
 | 0x00000001 - 1
 | 0x00000002 - 2
 | 0x00000003 - 3
@@ -2492,13 +2537,13 @@ EXAMPLE
 | 0x00000010 - flag READONLY
 | 0x00000000 - high 32 bits of section length
 | 0x0000000c - section length is 12 bytes; 2 * 4 + 2 + alignment to 32 bits
-| 0x20000000 - high 64 bits of address
-| 0x0000001c - low 64 bits of address 0x200000000000001c
+| 0x20000000 - high 32 bits of address
+| 0x0000001c - low 32 bits of address 0x200000000000001c
 | 0x00030d41 - 200001
 | 0x000186a2 - 100002
 | 0x26280000 - 38, 40 as bytes, padded with zeros
 
-       For the latter example, the section contents must not to appear
+       For the latter example, the section contents must not be
        loaded in memory, and is therefore specified as part of the
        special data.  The address is usually unimportant but might
        provide information for e.g.@: the DWARF 2 debugging format.  */
@@ -2506,6 +2551,7 @@ EXAMPLE
       mmo_write_tetra_raw (abfd, LOP_SPEC_SECTION);
       mmo_write_tetra (abfd, (strlen (sec->name) + 3) / 4);
       mmo_write_chunk (abfd, sec->name, strlen (sec->name));
+      mmo_flush_chunk (abfd);
       /* FIXME: We can get debug sections (.debug_line & Co.) with a
         section flag still having SEC_RELOC set.  Investigate.  This
         might be true for all alien sections; perhaps mmo.em should clear
@@ -2519,13 +2565,13 @@ EXAMPLE
       /* Writing a LOP_LOC ends the LOP_SPEC data, and makes data actually
         loaded.  */
       if (bfd_get_section_flags (abfd, sec) & SEC_LOAD)
-         return
-           abfd->tdata.mmo_data->have_error == false
+         return 
+           ! abfd->tdata.mmo_data->have_error
            && mmo_write_loc_chunk_list (abfd,
                                         ((struct mmo_section_data_struct *)
                                          (sec->used_by_bfd))->head);
       return
-       abfd->tdata.mmo_data->have_error == false
+       ! abfd->tdata.mmo_data->have_error
        && mmo_write_chunk_list (abfd,
                                 ((struct mmo_section_data_struct *)
                                  (sec->used_by_bfd))->head);
@@ -2575,9 +2621,9 @@ static boolean
 mmo_internal_add_3_sym (abfd, rootp, symp)
      bfd *abfd;
      struct mmo_symbol_trie *rootp;
-     CONST struct mmo_symbol *symp;
+     const struct mmo_symbol *symp;
 {
-  CONST char *name = symp->name;
+  const char *name = symp->name;
   struct mmo_symbol_trie *trie = rootp;
   struct mmo_symbol_trie **triep = NULL;
 
@@ -2801,48 +2847,95 @@ mmo_internal_3_dump (abfd, trie)
   mmo_internal_3_dump (abfd, trie->right);
 }
 
-/* Write symbols, either in mmo format or hidden in a lop_spec 80 section.
-   Write the lop_end terminator also.  */
+/* Write symbols in mmo format.  Also write the lop_end terminator.  */
 
 static boolean
 mmo_write_symbols_and_terminator (abfd)
      bfd *abfd;
 {
   int count = bfd_get_symcount (abfd);
-  asymbol *fakemain[2];
+  asymbol *maintable[2];
   asymbol **table;
-  int serno = 2;
+  asymbol **orig_table = bfd_get_outsymbols (abfd);
+  int serno;
   struct mmo_symbol_trie root;
   int trie_len;
   int i;
   bfd_byte buf[4];
 
   /* Create a symbol for "Main".  */
-  asymbol *mainsym = bfd_make_empty_symbol (abfd);
+  asymbol *fakemain = bfd_make_empty_symbol (abfd);
 
-  mainsym->flags = BSF_GLOBAL;
-  mainsym->value = bfd_get_start_address (abfd);
-  mainsym->name = MMIX_START_SYMBOL_NAME;
-  mainsym->section = bfd_abs_section_ptr;
-  fakemain[0] = mainsym;
-  fakemain[1] = NULL;
+  fakemain->flags = BSF_GLOBAL;
+  fakemain->value = bfd_get_start_address (abfd);
+  fakemain->name = MMIX_START_SYMBOL_NAME;
+  fakemain->section = bfd_abs_section_ptr;
+  maintable[0] = fakemain;
+  maintable[1] = NULL;
 
   memset (&root, 0, sizeof (root));
 
   /* Make all symbols take a left turn.  */
   root.symchar = 0xff;
 
-  /* There must always be a ":Main", so we'll add one
-     if there are no symbols.  */
-  if (count == 0)
+  /* There must always be a ":Main", so we'll add one if there are no
+     symbols.  Make sure we have room for it.  */
+  table = bfd_alloc (abfd, (count + 1) * sizeof (asymbol *));
+  if (table == NULL)
+    return false;
+
+  memcpy (table, orig_table, count * sizeof (asymbol *));
+
+  /* Move :Main (if there is one) to the first position.  This is
+     necessary to get the same layout of the trie-tree when linking as
+     when objcopying the result as in the objcopy.exp test "simple objcopy
+     of executable".  It also automatically takes care of assigning serial
+     number 1 to :Main (as is mandatory).  */
+  for (i = 0; i < count; i++)
+    if (table[i] != NULL
+       && strcmp (table[i]->name, MMIX_START_SYMBOL_NAME) == 0
+       && (table[i]->flags & (BSF_DEBUGGING|BSF_GLOBAL)) == BSF_GLOBAL)
+      {
+       asymbol *mainsym = table[i];
+       memcpy (table + 1, orig_table, i * sizeof (asymbol *));
+       table[0] = mainsym;
+
+       /* Check that the value assigned to :Main is the same as the entry
+          address.  The default linker script asserts this.  This is as
+          good a place as any to check this consistency. */
+       if ((mainsym->value
+            + mainsym->section->output_section->vma
+            + mainsym->section->output_offset)
+           != bfd_get_start_address (abfd))
+         {
+           /* Arbitrary buffer to hold the printable representation of a
+              vma.  */
+           char vmas_main[40];
+           char vmas_start[40];
+           bfd_vma vma_start = bfd_get_start_address (abfd);
+
+           sprintf_vma (vmas_main, mainsym->value);
+           sprintf_vma (vmas_start, vma_start);
+
+           (*_bfd_error_handler)
+             (_("%s: Bad symbol definition: `Main' set to %s rather\
+ than the start address %s\n"),
+              bfd_get_filename (abfd), vmas_main, vmas_start);
+           bfd_set_error (bfd_error_bad_value);
+           return false;
+         }
+       break;
+      }
+  if (i == count && count != 0)
     {
-      table = fakemain;
-      count = 1;
+      /* When there are symbols, there must be a :Main.  There was no
+        :Main, so we need to add it manually.  */
+      memcpy (table + 1, orig_table, count * sizeof (asymbol *));
+      table[0] = fakemain;
+      count++;
     }
-  else
-    table = bfd_get_outsymbols (abfd);
 
-  for (i = 0; i < count && table[i] != NULL; i++)
+  for (i = 0, serno = 1; i < count && table[i] != NULL; i++)
     {
       asymbol *s = table[i];
 
@@ -2887,37 +2980,9 @@ mmo_write_symbols_and_terminator (abfd)
          /* FIXME: We assume the order of the received symbols is an
             ordered mapping of the serial numbers.  This is not
             necessarily true if we e.g. objcopy a mmo file to another and
-            there are gaps in the numbering.  Note sure if this can
+            there are gaps in the numbering.  Not sure if this can
             happen.  Not sure what to do.  */
-         /* Make sure Main has serial number 1; others start at 2.  */
-         if (strcmp (s->name, MMIX_START_SYMBOL_NAME) == 0)
-           {
-             sym.serno = 1;
-
-             /* Check that the value assigned to :Main is the same as the
-                entry address.  The default linker script asserts this.
-                This is as good a place as any to check this consistency. */
-             if (sym.value != bfd_get_start_address (abfd))
-               {
-                 /* Arbitrary buffer to hold the printable representation
-                    of a vma.  */
-                 char vmas_main[40];
-                 char vmas_start[40];
-                 bfd_vma vma_start = bfd_get_start_address (abfd);
-
-                 sprintf_vma (vmas_main, s->value);
-                 sprintf_vma (vmas_start, vma_start);
-
-                 (*_bfd_error_handler)
-                   (_("%s: Bad symbol definition: `Main' set to %s rather\
- than the start address %s\n"),
-                    bfd_get_filename (abfd), vmas_main, vmas_start);
-                 bfd_set_error (bfd_error_bad_value);
-                 return false;
-               }
-           }
-         else
-           sym.serno = serno++;
+         sym.serno = serno++;
 
          if (! mmo_internal_add_3_sym (abfd, &root, &sym))
            return false;
@@ -3025,7 +3090,7 @@ mmo_write_section_unless_reg_contents (abfd, sec, p)
 {
   struct mmo_write_sec_info *infop = (struct mmo_write_sec_info *) p;
 
-  if (infop->retval == false)
+  if (! infop->retval)
     return;
 
   if (strcmp (sec->name, MMIX_REG_CONTENTS_SECTION_NAME) == 0)
@@ -3076,7 +3141,7 @@ mmo_write_object_contents (abfd)
   bfd_map_over_sections (abfd, mmo_write_section_unless_reg_contents,
                         (PTR) &wsecinfo);
 
-  if (wsecinfo.retval == false)
+  if (! wsecinfo.retval)
     return false;
 
   if (wsecinfo.reg_section != NULL)
@@ -3171,6 +3236,7 @@ mmo_canonicalize_reloc (abfd, section, relptr, symbols)
 /* FIXME: We can do better on this one, if we have a dwarf2 .debug_line
    section or if MMO line numbers are implemented.  */
 #define mmo_find_nearest_line _bfd_nosymbols_find_nearest_line
+#define mmo_make_empty_symbol _bfd_generic_make_empty_symbol
 #define mmo_bfd_make_debug_symbol _bfd_nosymbols_bfd_make_debug_symbol
 #define mmo_read_minisymbols _bfd_generic_read_minisymbols
 #define mmo_minisymbol_to_symbol _bfd_generic_minisymbol_to_symbol
@@ -3181,7 +3247,9 @@ mmo_canonicalize_reloc (abfd, section, relptr, symbols)
   bfd_generic_get_relocated_section_contents
 #define mmo_bfd_gc_sections bfd_generic_gc_sections
 #define mmo_bfd_link_hash_table_create _bfd_generic_link_hash_table_create
+#define mmo_bfd_link_hash_table_free _bfd_generic_link_hash_table_free
 #define mmo_bfd_link_add_symbols _bfd_generic_link_add_symbols
+#define mmo_bfd_link_just_syms _bfd_generic_link_just_syms
 #define mmo_bfd_final_link _bfd_generic_final_link
 #define mmo_bfd_link_split_section _bfd_generic_link_split_section
 
@@ -3190,6 +3258,7 @@ mmo_canonicalize_reloc (abfd, section, relptr, symbols)
 #define mmo_set_arch_mach bfd_default_set_arch_mach
 #define mmo_bfd_relax_section bfd_generic_relax_section
 #define mmo_bfd_merge_sections bfd_generic_merge_sections
+#define mmo_bfd_discard_group bfd_generic_discard_group
 
 /* objcopy will be upset if we return -1 from bfd_get_reloc_upper_bound by
    using BFD_JUMP_TABLE_RELOCS (_bfd_norelocs) rather than 0.  FIXME: Most
@@ -3211,7 +3280,7 @@ mmo_canonicalize_reloc (abfd, section, relptr, symbols)
 #define mmo_bfd_set_private_flags _bfd_generic_bfd_set_private_flags
 #define mmo_bfd_print_private_bfd_data _bfd_generic_bfd_print_private_bfd_data
 
-CONST bfd_target bfd_mmo_vec =
+const bfd_target bfd_mmo_vec =
 {
   "mmo",                       /* name */
   bfd_target_mmo_flavour,
This page took 0.033791 seconds and 4 git commands to generate.