Add TMS320C4x support
[deliverable/binutils-gdb.git] / gas / config / obj-coff.c
index d242268689f63346579687ae3bd9f66c4c70265d..56b55d7cbc833ec93c0647e5e8754396c658c15e 100644 (file)
@@ -1,5 +1,6 @@
 /* coff object file format
-   Copyright (C) 1989, 90, 91, 92, 93, 94, 95, 96, 97, 98, 1999
+   Copyright 1989, 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
+   1999, 2000, 2001, 2002
    Free Software Foundation, Inc.
 
    This file is part of GAS.
 #define KEEP_RELOC_INFO
 #endif
 
-static void obj_coff_bss PARAMS ((int));
-const char *s_get_name PARAMS ((symbolS * s));
+/* The BFD_ASSEMBLER version of obj_coff_section will use this macro to set
+   a new section's attributes when a directive has no valid flags or the
+   "w" flag is used. This default should be appropriate for most.  */
+#ifndef TC_COFF_SECTION_DEFAULT_ATTRIBUTES
+#define TC_COFF_SECTION_DEFAULT_ATTRIBUTES (SEC_LOAD | SEC_DATA)
+#endif
+
+/* This is used to hold the symbol built by a sequence of pseudo-ops
+   from .def and .endef.  */
 static symbolS *def_symbol_in_progress;
 
-\f
-/* stack stuff */
 typedef struct
   {
     unsigned long chunk_size;
@@ -46,6 +52,33 @@ typedef struct
   }
 stack;
 
+static stack *stack_init PARAMS ((unsigned long, unsigned long));
+static char *stack_push PARAMS ((stack *, char *));
+static char *stack_pop PARAMS ((stack *));
+static void tag_init PARAMS ((void));
+static void tag_insert PARAMS ((const char *, symbolS *));
+static symbolS *tag_find PARAMS ((char *));
+static symbolS *tag_find_or_make PARAMS ((char *));
+static void obj_coff_bss PARAMS ((int));
+static void obj_coff_weak PARAMS ((int));
+const char *s_get_name PARAMS ((symbolS * s));
+static void obj_coff_ln PARAMS ((int));
+static void obj_coff_def PARAMS ((int));
+static void obj_coff_endef PARAMS ((int));
+static void obj_coff_dim PARAMS ((int));
+static void obj_coff_line PARAMS ((int));
+static void obj_coff_size PARAMS ((int));
+static void obj_coff_scl PARAMS ((int));
+static void obj_coff_tag PARAMS ((int));
+static void obj_coff_val PARAMS ((int));
+static void obj_coff_type PARAMS ((int));
+static void obj_coff_ident PARAMS ((int));
+#ifdef BFD_ASSEMBLER
+static void obj_coff_loc PARAMS((int));
+#endif
+\f
+/* stack stuff */
+
 static stack *
 stack_init (chunk_size, element_size)
      unsigned long chunk_size;
@@ -171,7 +204,7 @@ tag_find_or_make (name)
 
 static void
 obj_coff_bss (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (*input_line_pointer == '\n')
     subseg_new (".bss", get_absolute_expression ());
@@ -183,12 +216,12 @@ obj_coff_bss (ignore)
 
 static void
 obj_coff_weak (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *name;
   int c;
   symbolS *symbolP;
+
   do
     {
       name = input_line_pointer;
@@ -197,7 +230,7 @@ obj_coff_weak (ignore)
       *input_line_pointer = c;
       SKIP_WHITESPACE ();
 
-#ifdef BFD_ASSEMLER
+#if defined BFD_ASSEMBLER || defined S_SET_WEAK
       S_SET_WEAK (symbolP);
 #endif
 
@@ -222,10 +255,14 @@ obj_coff_weak (ignore)
 
 #ifdef BFD_ASSEMBLER
 
+static segT fetch_coff_debug_section PARAMS ((void));
 static void SA_SET_SYM_TAGNDX PARAMS ((symbolS *, symbolS *));
+static int S_GET_DATA_TYPE PARAMS ((symbolS *));
+void c_symbol_merge PARAMS ((symbolS *, symbolS *));
+static void add_lineno PARAMS ((fragS *, addressT, int));
 
 #define GET_FILENAME_STRING(X) \
-((char*)(&((X)->sy_symbol.ost_auxent->x_file.x_n.x_offset))[1])
+((char*) (&((X)->sy_symbol.ost_auxent->x_file.x_n.x_offset))[1])
 
 /* @@ Ick.  */
 static segT
@@ -234,7 +271,7 @@ fetch_coff_debug_section ()
   static segT debug_section;
   if (!debug_section)
     {
-      CONST asymbol *s;
+      const asymbol *s;
       s = bfd_make_debug_symbol (stdoutput, (char *) 0, 0);
       assert (s != 0);
       debug_section = s->section;
@@ -300,7 +337,7 @@ S_SET_STORAGE_CLASS (sym, val)
   return val;
 }
 
-/* Merge a debug symbol containing debug information into a normal symbol. */
+/* Merge a debug symbol containing debug information into a normal symbol.  */
 
 void
 c_symbol_merge (debug, normal)
@@ -324,16 +361,18 @@ c_symbol_merge (debug, normal)
               * sizeof (*SYM_AUXINFO (debug))));
     }
 
-  /* Move the debug flags. */
+  /* Move the debug flags.  */
   SF_SET_DEBUG_FIELD (normal, SF_GET_DEBUG_FIELD (debug));
 }
 
 void
 c_dot_file_symbol (filename)
-     char *filename;
+     const char *filename;
 {
   symbolS *symbolP;
 
+  /* BFD converts filename to a .file symbol with an aux entry.  It
+     also handles chaining.  */
   symbolP = symbol_new (filename, bfd_abs_section_ptr, 0, &zero_address_frag);
 
   S_SET_STORAGE_CLASS (symbolP, C_FILE);
@@ -383,7 +422,7 @@ coff_obj_symbol_new_hook (symbolP)
 {
   long   sz = (OBJ_COFF_MAX_AUXENTRIES + 1) * sizeof (combined_entry_type);
   char * s  = (char *) xmalloc (sz);
-  
+
   memset (s, 0, sz);
   coffsymbol (symbol_get_bfdsym (symbolP))->native = (combined_entry_type *) s;
 
@@ -393,7 +432,7 @@ coff_obj_symbol_new_hook (symbolP)
 
   if (S_IS_STRING (symbolP))
     SF_SET_STRING (symbolP);
-  
+
   if (S_IS_LOCAL (symbolP))
     SF_SET_LOCAL (symbolP);
 }
@@ -411,7 +450,7 @@ int coff_n_line_nos;
 static void
 add_lineno (frag, offset, num)
      fragS *frag;
-     int offset;
+     addressT offset;
      int num;
 {
   struct line_no *new_line =
@@ -420,6 +459,17 @@ add_lineno (frag, offset, num)
     {
       abort ();
     }
+
+#ifndef OBJ_XCOFF
+  /* The native aix assembler accepts negative line number */
+
+  if (num <= 0)
+    {
+      /* Zero is used as an end marker in the file.  */
+      as_warn (_("Line numbers must be positive integers\n"));
+      num = 1;
+    }
+#endif /* OBJ_XCOFF */
   new_line->next = line_nos;
   new_line->frag = frag;
   new_line->l.line_number = num;
@@ -456,13 +506,13 @@ obj_coff_ln (appline)
     }
 
   l = get_absolute_expression ();
-  if (!appline)
-    {
-      add_lineno (frag_now, frag_now_fix (), l);
-    }
 
-  if (appline)
+  /* If there is no lineno symbol, treat a .ln
+     directive as if it were a .appline directive.  */
+  if (appline || current_lineno_sym == NULL)
     new_logical_line ((char *) NULL, l - 1);
+  else
+    add_lineno (frag_now, frag_now_fix (), l);
 
 #ifndef NO_LISTING
   {
@@ -480,6 +530,84 @@ obj_coff_ln (appline)
   demand_empty_rest_of_line ();
 }
 
+/* .loc is essentially the same as .ln; parse it for assembler
+   compatibility.  */
+
+static void
+obj_coff_loc (ignore)
+     int ignore ATTRIBUTE_UNUSED;
+{
+  int lineno;
+
+  /* FIXME: Why do we need this check?  We need it for ECOFF, but why
+     do we need it for COFF?  */
+  if (now_seg != text_section)
+    {
+      as_warn (_(".loc outside of .text"));
+      demand_empty_rest_of_line ();
+      return;
+    }
+
+  if (def_symbol_in_progress != NULL)
+    {
+      as_warn (_(".loc pseudo-op inside .def/.endef: ignored."));
+      demand_empty_rest_of_line ();
+      return;
+    }
+
+  /* Skip the file number.  */
+  SKIP_WHITESPACE ();
+  get_absolute_expression ();
+  SKIP_WHITESPACE ();
+
+  lineno = get_absolute_expression ();
+
+#ifndef NO_LISTING
+  {
+    extern int listing;
+
+    if (listing)
+      {
+       lineno += coff_line_base - 1;
+       listing_source_line (lineno);
+      }
+  }
+#endif
+
+  demand_empty_rest_of_line ();
+
+  add_lineno (frag_now, frag_now_fix (), lineno);
+}
+
+/* Handle the .ident pseudo-op.  */
+
+static void
+obj_coff_ident (ignore)
+     int ignore ATTRIBUTE_UNUSED;
+{
+  segT current_seg = now_seg;
+  subsegT current_subseg = now_subseg;
+
+#ifdef TE_PE
+  {
+    segT sec;
+
+    /* We could put it in .comment, but that creates an extra section
+       that shouldn't be loaded into memory, which requires linker
+       changes...  For now, until proven otherwise, use .rdata.  */
+    sec = subseg_new (".rdata$zzz", 0);
+    bfd_set_section_flags (stdoutput, sec,
+                          ((SEC_ALLOC | SEC_LOAD | SEC_READONLY | SEC_DATA)
+                           & bfd_applicable_section_flags (stdoutput)));
+  }
+#else
+  subseg_new (".comment", 0);
+#endif
+
+  stringer (1);
+  subseg_set (current_seg, current_subseg);
+}
+
 /*
  *                     def()
  *
@@ -501,7 +629,7 @@ obj_coff_ln (appline)
 
 static void
 obj_coff_def (what)
-     int what;
+     int what ATTRIBUTE_UNUSED;
 {
   char name_end;               /* Char after the end of name */
   char *symbol_name;           /* Name of the debug symbol */
@@ -548,9 +676,9 @@ unsigned int dim_index;
 
 static void
 obj_coff_endef (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
-  symbolS *symbolP;
+  symbolS *symbolP = NULL;
 
   /* DIM BUG FIX sac@cygnus.com */
   dim_index = 0;
@@ -561,7 +689,7 @@ obj_coff_endef (ignore)
       return;
     }                          /* if not inside .def/.endef */
 
-  /* Set the section number according to storage class. */
+  /* Set the section number according to storage class.  */
   switch (S_GET_STORAGE_CLASS (def_symbol_in_progress))
     {
     case C_STRTAG:
@@ -576,25 +704,41 @@ obj_coff_endef (ignore)
       break;
 
     case C_EFCN:
-      SF_SET_LOCAL (def_symbol_in_progress);   /* Do not emit this symbol. */
+      SF_SET_LOCAL (def_symbol_in_progress);   /* Do not emit this symbol.  */
       /* intentional fallthrough */
     case C_BLOCK:
       SF_SET_PROCESS (def_symbol_in_progress); /* Will need processing before writing */
       /* intentional fallthrough */
     case C_FCN:
       {
-       CONST char *name;
+       const char *name;
        S_SET_SEGMENT (def_symbol_in_progress, text_section);
 
        name = S_GET_NAME (def_symbol_in_progress);
-       if (name[1] == 'b' && name[2] == 'f')
+       if (name[0] == '.' && name[2] == 'f' && name[3] == '\0')
          {
-           if (! in_function ())
-             as_warn (_("`%s' symbol without preceding function"), name);
-/*         SA_SET_SYM_LNNO (def_symbol_in_progress, 12345);*/
-           /* Will need relocating */
-           SF_SET_PROCESS (def_symbol_in_progress);
-           clear_function ();
+           switch (name[1])
+             {
+             case 'b':
+               /* .bf */
+               if (! in_function ())
+                 as_warn (_("`%s' symbol without preceding function"), name);
+               /* Will need relocating.  */
+               SF_SET_PROCESS (def_symbol_in_progress);
+               clear_function ();
+               break;
+#ifdef TE_PE
+             case 'e':
+               /* .ef */
+               /* The MS compilers output the actual endline, not the
+                  function-relative one... we want to match without
+                  changing the assembler input.  */
+               SA_SET_SYM_LNNO (def_symbol_in_progress,
+                                (SA_GET_SYM_LNNO (def_symbol_in_progress)
+                                 + coff_line_base));
+               break;
+#endif
+             }
          }
       }
       break;
@@ -607,16 +751,49 @@ obj_coff_endef (ignore)
     case C_ARG:
     case C_REGPARM:
     case C_FIELD:
+
+    /* According to the COFF documentation:
+
+       http://osr5doc.sco.com:1996/topics/COFF_SectNumFld.html
+
+       A special section number (-2) marks symbolic debugging symbols,
+       including structure/union/enumeration tag names, typedefs, and
+       the name of the file. A section number of -1 indicates that the
+       symbol has a value but is not relocatable. Examples of
+       absolute-valued symbols include automatic and register variables,
+       function arguments, and .eos symbols.
+
+       But from Ian Lance Taylor:
+
+       http://sources.redhat.com/ml/binutils/2000-08/msg00202.html
+
+       the actual tools all marked them as section -1. So the GNU COFF
+       assembler follows historical COFF assemblers.
+
+       However, it causes problems for djgpp
+
+       http://sources.redhat.com/ml/binutils/2000-08/msg00210.html
+
+       By defining STRICTCOFF, a COFF port can make the assembler to
+       follow the documented behavior.  */
+#ifdef STRICTCOFF
+    case C_MOS:
+    case C_MOE:
+    case C_MOU:
+    case C_EOS:
+#endif
       SF_SET_DEBUG (def_symbol_in_progress);
       S_SET_SEGMENT (def_symbol_in_progress, absolute_section);
       break;
 
+#ifndef STRICTCOFF
     case C_MOS:
     case C_MOE:
     case C_MOU:
     case C_EOS:
       S_SET_SEGMENT (def_symbol_in_progress, absolute_section);
       break;
+#endif
 
     case C_EXT:
     case C_WEAKEXT:
@@ -639,7 +816,10 @@ obj_coff_endef (ignore)
 
   /* Now that we have built a debug symbol, try to find if we should
      merge with an existing symbol or not.  If a symbol is C_EFCN or
-     SEG_ABSOLUTE or untagged SEG_DEBUG it never merges. */
+     absolute_section or untagged SEG_DEBUG it never merges.  We also
+     don't merge labels, which are in a different namespace, nor
+     symbols which have not yet been defined since they are typically
+     unique, nor do we merge tags with non-tags.  */
 
   /* Two cases for functions.  Either debug followed by definition or
      definition followed by debug.  For definition first, we will
@@ -647,23 +827,31 @@ obj_coff_endef (ignore)
      lineno entry MUST point to the definition function or else it
      will point off into space when obj_crawl_symbol_chain() merges
      the debug symbol into the real symbol.  Therefor, let's presume
-     the debug symbol is a real function reference. */
+     the debug symbol is a real function reference.  */
 
   /* FIXME-SOON If for some reason the definition label/symbol is
      never seen, this will probably leave an undefined symbol at link
-     time. */
+     time.  */
 
   if (S_GET_STORAGE_CLASS (def_symbol_in_progress) == C_EFCN
+      || S_GET_STORAGE_CLASS (def_symbol_in_progress) == C_LABEL
       || (!strcmp (bfd_get_section_name (stdoutput,
                                         S_GET_SEGMENT (def_symbol_in_progress)),
                   "*DEBUG*")
          && !SF_GET_TAG (def_symbol_in_progress))
       || S_GET_SEGMENT (def_symbol_in_progress) == absolute_section
-      || (symbolP = symbol_find_base (S_GET_NAME (def_symbol_in_progress), DO_NOT_STRIP)) == NULL)
+      || ! symbol_constant_p (def_symbol_in_progress)
+      || (symbolP = symbol_find_base (S_GET_NAME (def_symbol_in_progress),
+                                     DO_NOT_STRIP)) == NULL
+      || SF_GET_TAG (def_symbol_in_progress) != SF_GET_TAG (symbolP))
     {
+      /* If it already is at the end of the symbol list, do nothing */
       if (def_symbol_in_progress != symbol_lastP)
-       symbol_append (def_symbol_in_progress, symbol_lastP, &symbol_rootP,
-                      &symbol_lastP);
+       {
+         symbol_remove (def_symbol_in_progress, &symbol_rootP, &symbol_lastP);
+         symbol_append (def_symbol_in_progress, symbol_lastP, &symbol_rootP,
+                        &symbol_lastP);
+       }
     }
   else
     {
@@ -674,7 +862,7 @@ obj_coff_endef (ignore)
         symbols. [loic] */
 
       /* The debug entry (def_symbol_in_progress) is merged into the
-        previous definition. */
+        previous definition.  */
 
       c_symbol_merge (def_symbol_in_progress, symbolP);
       symbol_remove (def_symbol_in_progress, &symbol_rootP, &symbol_lastP);
@@ -687,7 +875,7 @@ obj_coff_endef (ignore)
        {
          /* For functions, and tags, and static symbols, the symbol
             *must* be where the debug symbol appears.  Move the
-            existing symbol to the current place. */
+            existing symbol to the current place.  */
          /* If it already is at the end of the symbol list, do nothing */
          if (def_symbol_in_progress != symbol_lastP)
            {
@@ -717,7 +905,7 @@ obj_coff_endef (ignore)
       if (symbolP == NULL)
        {
          /* That is, if this is the first time we've seen the
-            function... */
+            function...  */
          symbol_table_insert (def_symbol_in_progress);
        } /* definition follows debug */
     } /* Create the line number entry pointing to the function being defined */
@@ -728,7 +916,7 @@ obj_coff_endef (ignore)
 
 static void
 obj_coff_dim (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int dim_index;
 
@@ -768,7 +956,7 @@ obj_coff_dim (ignore)
 
 static void
 obj_coff_line (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int this_base;
 
@@ -784,7 +972,7 @@ obj_coff_line (ignore)
     coff_line_base = this_base;
 
   S_SET_NUMBER_AUXILIARY (def_symbol_in_progress, 1);
-  SA_SET_SYM_LNNO (def_symbol_in_progress, coff_line_base);
+  SA_SET_SYM_LNNO (def_symbol_in_progress, this_base);
 
   demand_empty_rest_of_line ();
 
@@ -794,14 +982,14 @@ obj_coff_line (ignore)
       extern int listing;
 
       if (listing)
-       listing_source_line ((unsigned int) coff_line_base);
+       listing_source_line ((unsigned int) this_base);
     }
 #endif
 }
 
 static void
 obj_coff_size (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
@@ -817,7 +1005,7 @@ obj_coff_size (ignore)
 
 static void
 obj_coff_scl (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
@@ -832,7 +1020,7 @@ obj_coff_scl (ignore)
 
 static void
 obj_coff_tag (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *symbol_name;
   char name_end;
@@ -853,7 +1041,7 @@ obj_coff_tag (ignore)
 #endif
 
   /* Assume that the symbol referred to by .tag is always defined.
-     This was a bad assumption.  I've added find_or_make. xoxorich. */
+     This was a bad assumption.  I've added find_or_make. xoxorich.  */
   SA_SET_SYM_TAGNDX (def_symbol_in_progress,
                     tag_find_or_make (symbol_name));
   if (SA_GET_SYM_TAGNDX (def_symbol_in_progress) == 0L)
@@ -869,7 +1057,7 @@ obj_coff_tag (ignore)
 
 static void
 obj_coff_type (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
@@ -891,7 +1079,7 @@ obj_coff_type (ignore)
 
 static void
 obj_coff_val (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
@@ -928,8 +1116,15 @@ obj_coff_val (ignore)
             resolved, then copy the segment id from the forward
             symbol.  */
          SF_SET_GET_SEGMENT (def_symbol_in_progress);
+
+         /* FIXME: gcc can generate address expressions here in
+            unusual cases (search for "obscure" in sdbout.c).  We
+            just ignore the offset here, thus generating incorrect
+            debugging information.  We ignore the rest of the line
+            just below.  */
        }
-      /* Otherwise, it is the name of a non debug symbol and its value will be calculated later. */
+      /* Otherwise, it is the name of a non debug symbol and its value
+         will be calculated later.  */
       *input_line_pointer = name_end;
     }
   else
@@ -943,7 +1138,7 @@ obj_coff_val (ignore)
 void
 coff_obj_read_begin_hook ()
 {
-  /* These had better be the same.  Usually 18 bytes. */
+  /* These had better be the same.  Usually 18 bytes.  */
 #ifndef BFD_HEADERS
   know (sizeof (SYMENT) == sizeof (AUXENT));
   know (SYMESZ == AUXESZ);
@@ -951,7 +1146,6 @@ coff_obj_read_begin_hook ()
   tag_init ();
 }
 
-
 symbolS *coff_last_function;
 static symbolS *coff_last_bf;
 
@@ -993,15 +1187,21 @@ coff_frob_symbol (symp, punt)
 
   if (!SF_GET_DEBUG (symp))
     {
-      symbolS *real;
+      symbolS * real;
+
       if (!SF_GET_LOCAL (symp)
          && !SF_GET_STATICS (symp)
+         && S_GET_STORAGE_CLASS (symp) != C_LABEL
+         && symbol_constant_p(symp)
          && (real = symbol_find_base (S_GET_NAME (symp), DO_NOT_STRIP))
+         && S_GET_STORAGE_CLASS (real) == C_NULL
          && real != symp)
        {
          c_symbol_merge (symp, real);
          *punt = 1;
+         return;
        }
+
       if (!S_IS_DEFINED (symp) && !SF_GET_LOCAL (symp))
        {
          assert (S_GET_VALUE (symp) == 0);
@@ -1015,6 +1215,7 @@ coff_frob_symbol (symp, punt)
          else
            S_SET_STORAGE_CLASS (symp, C_STAT);
        }
+
       if (SF_GET_PROCESS (symp))
        {
          if (S_GET_STORAGE_CLASS (symp) == C_BLOCK)
@@ -1024,6 +1225,7 @@ coff_frob_symbol (symp, punt)
              else
                {
                  symbolS *begin;
+
                  begin = *(symbolS **) stack_pop (block_stack);
                  if (begin == 0)
                    as_warn (_("mismatched .eb"));
@@ -1031,17 +1233,19 @@ coff_frob_symbol (symp, punt)
                    next_set_end = begin;
                }
            }
+
          if (coff_last_function == 0 && SF_GET_FUNCTION (symp))
            {
              union internal_auxent *auxp;
+
              coff_last_function = symp;
              if (S_GET_NUMBER_AUXILIARY (symp) < 1)
                S_SET_NUMBER_AUXILIARY (symp, 1);
-             auxp =
-               &coffsymbol (symbol_get_bfdsym (symp))->native[1].u.auxent;
+             auxp = SYM_AUXENT (symp);
              memset (auxp->x_sym.x_fcnary.x_ary.x_dimen, 0,
                      sizeof (auxp->x_sym.x_fcnary.x_ary.x_dimen));
            }
+
          if (S_GET_STORAGE_CLASS (symp) == C_EFCN)
            {
              if (coff_last_function == 0)
@@ -1053,6 +1257,7 @@ coff_frob_symbol (symp, punt)
              coff_last_function = 0;
            }
        }
+
       if (S_IS_EXTERNAL (symp))
        S_SET_STORAGE_CLASS (symp, C_EXT);
       else if (SF_GET_LOCAL (symp))
@@ -1061,9 +1266,14 @@ coff_frob_symbol (symp, punt)
       if (SF_GET_FUNCTION (symp))
        symbol_get_bfdsym (symp)->flags |= BSF_FUNCTION;
 
-      /* more ... */
+      /* more ...  */
     }
 
+  /* Double check weak symbols.  */
+  if (S_IS_WEAK (symp) && S_IS_COMMON (symp))
+    as_bad (_("Symbol `%s' can not be both weak and common"),
+           S_GET_NAME (symp));
+
   if (SF_GET_TAG (symp))
     last_tagP = symp;
   else if (S_GET_STORAGE_CLASS (symp) == C_EOS)
@@ -1091,10 +1301,15 @@ coff_frob_symbol (symp, punt)
       set_end = NULL;
     }
 
-  if (next_set_end != NULL
-      && ! *punt)
-    set_end = next_set_end;
+  if (next_set_end != NULL)
+    {
+      if (set_end != NULL)
+       as_warn ("Warning: internal error: forgetting to set endndx of %s",
+                S_GET_NAME (set_end));
+      set_end = next_set_end;
+    }
 
+#ifndef OBJ_XCOFF
   if (! *punt
       && S_GET_STORAGE_CLASS (symp) == C_FCN
       && strcmp (S_GET_NAME (symp), ".bf") == 0)
@@ -1103,7 +1318,7 @@ coff_frob_symbol (symp, punt)
        SA_SET_SYM_ENDNDX (coff_last_bf, symp);
       coff_last_bf = symp;
     }
-
+#endif
   if (coffsymbol (symbol_get_bfdsym (symp))->lineno)
     {
       int i;
@@ -1125,7 +1340,7 @@ coff_frob_symbol (symp, punt)
       for (; i > 0; i--)
        {
          if (lptr->frag)
-           lptr->l.u.offset += lptr->frag->fr_address;
+           lptr->l.u.offset += lptr->frag->fr_address / OCTETS_PER_BYTE;
          l[i] = lptr->l;
          lptr = lptr->next;
        }
@@ -1134,9 +1349,9 @@ coff_frob_symbol (symp, punt)
 
 void
 coff_adjust_section_syms (abfd, sec, x)
-     bfd *abfd;
+     bfd *abfd ATTRIBUTE_UNUSED;
      asection *sec;
-     PTR x;
+     PTR x ATTRIBUTE_UNUSED;
 {
   symbolS *secsym;
   segment_info_type *seginfo = seg_info (sec);
@@ -1170,6 +1385,8 @@ coff_adjust_section_syms (abfd, sec, x)
       && sec != bss_section)
     return;
   secsym = section_symbol (sec);
+  /* This is an estimate; we'll plug in the real value using
+     SET_SECTION_RELOCS later */
   SA_SET_SCN_NRELOC (secsym, nrelocs);
   SA_SET_SCN_NLINNO (secsym, nlnno);
 }
@@ -1180,33 +1397,35 @@ coff_frob_file_after_relocs ()
   bfd_map_over_sections (stdoutput, coff_adjust_section_syms, (char*) 0);
 }
 
-/*
- * implement the .section pseudo op:
- *     .section name {, "flags"}
- *                ^         ^
- *                |         +--- optional flags: 'b' for bss
- *                |                              'i' for info
- *                +-- section name               'l' for lib
- *                                               'n' for noload
- *                                               'o' for over
- *                                               'w' for data
- *                                              'd' (apparently m88k for data)
- *                                               'x' for text
- *                                              'r' for read-only data
- * But if the argument is not a quoted string, treat it as a
- * subsegment number.
- */
+/* Implement the .section pseudo op:
+       .section name {, "flags"}
+                  ^         ^
+                  |         +--- optional flags: 'b' for bss
+                  |                              'i' for info
+                  +-- section name               'l' for lib
+                                                 'n' for noload
+                                                 'o' for over
+                                                 'w' for data
+                                                'd' (apparently m88k for data)
+                                                 'x' for text
+                                                'r' for read-only data
+                                                's' for shared data (PE)
+   But if the argument is not a quoted string, treat it as a
+   subsegment number.
+
+   Note the 'a' flag is silently ignored.  This allows the same
+   .section directive to be parsed in both ELF and COFF formats.  */
 
 void
 obj_coff_section (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   /* Strip out the section name */
   char *section_name;
   char c;
   char *name;
   unsigned int exp;
-  flagword flags;
+  flagword flags, oldflags;
   asection *sec;
 
   if (flag_mri)
@@ -1245,11 +1464,13 @@ obj_coff_section (ignore)
              switch (*input_line_pointer)
                {
                case 'b': flags |= SEC_ALLOC; flags &=~ SEC_LOAD; break;
-               case 'n': flags &=~ SEC_LOAD; break;
-               case 'd':
+               case 'n': flags &=~ SEC_LOAD; flags |= SEC_NEVER_LOAD; break;
+               case 'd': flags |= SEC_DATA | SEC_LOAD; /* fall through */
                case 'w': flags &=~ SEC_READONLY; break;
-               case 'x': flags |= SEC_CODE; break;
+               case 'a': break; /* For compatability with ELF.  */
+               case 'x': flags |= SEC_CODE | SEC_LOAD; break;
                case 'r': flags |= SEC_READONLY; break;
+               case 's': flags |= SEC_SHARED; break;
 
                case 'i': /* STYP_INFO */
                case 'l': /* STYP_LIB */
@@ -1272,19 +1493,36 @@ obj_coff_section (ignore)
 
   sec = subseg_new (name, (subsegT) exp);
 
-  if (flags != SEC_NO_FLAGS)
+  oldflags = bfd_get_section_flags (stdoutput, sec);
+  if (oldflags == SEC_NO_FLAGS)
     {
-      flagword oldflags;
+      /* Set section flags for a new section just created by subseg_new.
+         Provide a default if no flags were parsed.  */
+      if (flags == SEC_NO_FLAGS)
+       flags = TC_COFF_SECTION_DEFAULT_ATTRIBUTES;
 
-      oldflags = bfd_get_section_flags (stdoutput, sec);
-      oldflags &= SEC_LINK_ONCE | SEC_LINK_DUPLICATES;
-      flags |= oldflags;
+#ifdef COFF_LONG_SECTION_NAMES
+      /* Add SEC_LINK_ONCE and SEC_LINK_DUPLICATES_DISCARD to .gnu.linkonce
+         sections so adjust_reloc_syms in write.c will correctly handle
+         relocs which refer to non-local symbols in these sections.  */
+      if (strncmp (name, ".gnu.linkonce", sizeof (".gnu.linkonce") - 1) == 0)
+       flags |= SEC_LINK_ONCE | SEC_LINK_DUPLICATES_DISCARD;
+#endif
 
       if (! bfd_set_section_flags (stdoutput, sec, flags))
        as_warn (_("error setting flags for \"%s\": %s"),
                 bfd_section_name (stdoutput, sec),
                 bfd_errmsg (bfd_get_error ()));
     }
+  else if (flags != SEC_NO_FLAGS)
+    {
+      /* This section's attributes have already been set. Warn if the
+         attributes don't match.  */
+      flagword matchflags = (SEC_ALLOC | SEC_LOAD | SEC_READONLY | SEC_CODE
+                            | SEC_DATA | SEC_SHARED | SEC_NEVER_LOAD);
+      if ((flags ^ oldflags) & matchflags)
+       as_warn (_("Ignoring changed section attributes for %s"), name);
+    }
 
   demand_empty_rest_of_line ();
 }
@@ -1305,25 +1543,44 @@ coff_frob_section (sec)
   char *p;
   fragS *fragp;
   bfd_vma size, n_entries, mask;
+  bfd_vma align_power = (bfd_vma)sec->alignment_power + OCTETS_PER_BYTE_POWER;
 
   /* The COFF back end in BFD requires that all section sizes be
-     rounded up to multiples of the corresponding section alignments.
-     Seems kinda silly to me, but that's the way it is.  */
+     rounded up to multiples of the corresponding section alignments,
+     supposedly because standard COFF has no other way of encoding alignment
+     for sections.  If your COFF flavor has a different way of encoding
+     section alignment, then skip this step, as TICOFF does.  */
   size = bfd_get_section_size_before_reloc (sec);
-  mask = ((bfd_vma) 1 << (bfd_vma) sec->alignment_power) - 1;
+  mask = ((bfd_vma) 1 << align_power) - 1;
+#if !defined(TICOFF)
   if (size & mask)
     {
-      size = (size + mask) & ~mask;
-      bfd_set_section_size (stdoutput, sec, size);
+      bfd_vma new_size;
+      fragS *last;
+
+      new_size = (size + mask) & ~mask;
+      bfd_set_section_size (stdoutput, sec, new_size);
+
+      /* If the size had to be rounded up, add some padding in
+         the last non-empty frag.  */
+      fragp = seg_info (sec)->frchainP->frch_root;
+      last = seg_info (sec)->frchainP->frch_last;
+      while (fragp->fr_next != last)
+       fragp = fragp->fr_next;
+      last->fr_address = size;
+      fragp->fr_offset += new_size - size;
     }
+#endif
 
   /* If the section size is non-zero, the section symbol needs an aux
      entry associated with it, indicating the size.  We don't know
      all the values yet; coff_frob_symbol will fill them in later.  */
+#ifndef TICOFF
   if (size != 0
       || sec == text_section
       || sec == data_section
       || sec == bss_section)
+#endif
     {
       symbolS *secsym = section_symbol (sec);
 
@@ -1371,12 +1628,12 @@ obj_coff_init_stab_section (seg)
   char *stabstr_name;
   unsigned int stroff;
 
-  /* Make space for this first symbol. */
+  /* Make space for this first symbol.  */
   p = frag_more (12);
-  /* Zero it out. */
+  /* Zero it out.  */
   memset (p, 0, 12);
   as_where (&file, (unsigned int *) NULL);
-  stabstr_name = (char *) alloca (strlen (seg->name) + 4);
+  stabstr_name = (char *) xmalloc (strlen (seg->name) + 4);
   strcpy (stabstr_name, seg->name);
   strcat (stabstr_name, "str");
   stroff = get_stab_string_offset (file, stabstr_name);
@@ -1400,7 +1657,7 @@ symbol_dump ()
 
   for (symbolP = symbol_rootP; symbolP; symbolP = symbol_next (symbolP))
     {
-      printf(_("0x%lx: \"%s\" type = %ld, class = %d, segment = %d\n"),
+      printf (_("0x%lx: \"%s\" type = %ld, class = %d, segment = %d\n"),
             (unsigned long) symbolP,
             S_GET_NAME(symbolP),
             (long) S_GET_DATA_TYPE(symbolP),
@@ -1414,7 +1671,7 @@ symbol_dump ()
 #else /* not BFD_ASSEMBLER */
 
 #include "frags.h"
-/* This is needed because we include internal bfd things. */
+/* This is needed because we include internal bfd things.  */
 #include <time.h>
 
 #include "libbfd.h"
@@ -1434,9 +1691,10 @@ symbol_dump ()
 #define S_SET_ZEROES(s,v)              ((s)->sy_symbol.ost_entry.n_zeroes = (v))
 
 #define MIN(a,b) ((a) < (b)? (a) : (b))
-/* This vector is used to turn an internal segment into a section #
-   suitable for insertion into a coff symbol table
- */
+
+/* This vector is used to turn a gas internal segment number into a
+   section number suitable for insertion into a coff symbol table.
+   This must correspond to seg_info_off_by_4.  */
 
 const short seg_N_TYPE[] =
 {                              /* in: segT   out: N_TYPE bits */
@@ -1457,10 +1715,10 @@ const short seg_N_TYPE[] =
 int function_lineoff = -1;     /* Offset in line#s where the last function
                                   started (the odd entry for line #0) */
 
-/* structure used to keep the filenames which
+/* Structure used to keep the filenames which
    are too long around so that we can stick them
-   into the string table */
-struct filename_list 
+   into the string table */
+struct filename_list
 {
   char *filename;
   struct filename_list *next;
@@ -1473,91 +1731,78 @@ static symbolS *last_line_symbol;
 
 /* Add 4 to the real value to get the index and compensate the
    negatives. This vector is used by S_GET_SEGMENT to turn a coff
-   section number into a segment number
-*/
-static symbolS *previous_file_symbol;
-void c_symbol_merge ();
-static int line_base;
+   section number into a segment number.  */
 
-symbolS *c_section_symbol ();
 bfd *abfd;
+static symbolS *previous_file_symbol;
+static int line_base;
 
-static void fixup_segment PARAMS ((segment_info_type *segP,
-                                  segT this_segment_type));
-
-
-static void fixup_mdeps PARAMS ((fragS *,
-                                object_headers *,
-                                segT));
-
-
-static void fill_section PARAMS ((bfd * abfd,
-                                 object_headers *,
-                                 unsigned long *));
-
-
-static int c_line_new PARAMS ((symbolS * symbol, long paddr,
-                              int line_number,
-                              fragS * frag));
-
-
-static void w_symbols PARAMS ((bfd * abfd, char *where,
-                              symbolS * symbol_rootP));
-
-static void adjust_stab_section PARAMS ((bfd *abfd, segT seg));
-
+void c_symbol_merge PARAMS ((symbolS *, symbolS *));
+symbolS *c_section_symbol PARAMS ((char *, int));
+void obj_coff_section PARAMS ((int));
+void do_relocs_for PARAMS ((bfd *, object_headers *, unsigned long *));
+char * symbol_to_chars PARAMS ((bfd *, char *, symbolS *));
+void w_strings PARAMS ((char *));
+
+static void fixup_segment PARAMS ((segment_info_type *, segT));
+static void fixup_mdeps PARAMS ((fragS *, object_headers *, segT));
+static void fill_section PARAMS ((bfd *,  object_headers *, unsigned long *));
+static int c_line_new PARAMS ((symbolS *, long, int, fragS *));
+static void w_symbols PARAMS ((bfd *, char *, symbolS *));
+static void adjust_stab_section PARAMS ((bfd *, segT));
 static void obj_coff_lcomm PARAMS ((int));
 static void obj_coff_text PARAMS ((int));
 static void obj_coff_data PARAMS ((int));
-static void obj_coff_ident PARAMS ((int));
-void obj_coff_section PARAMS ((int));
-
-/* Section stuff
+static unsigned int count_entries_in_chain PARAMS ((unsigned int));
+static void coff_header_append PARAMS ((bfd *, object_headers *));
+static unsigned int yank_symbols PARAMS ((void));
+static unsigned int glue_symbols PARAMS ((symbolS **, symbolS **));
+static unsigned int tie_tags PARAMS ((void));
+static void crawl_symbols PARAMS ((object_headers *, bfd *));
+static void do_linenos_for PARAMS ((bfd *, object_headers *, unsigned long *));
+static void remove_subsegs PARAMS ((void));
 
-   We allow more than just the standard 3 sections, infact, we allow
-   40 sections, (though the usual three have to be there).
 
-   This structure performs the mappings for us:
-*/
 
+/* When not using BFD_ASSEMBLER, we permit up to 40 sections.
 
-typedef struct
-{
-  segT seg_t;
-  int i;
-} seg_info_type;
+   This array maps a COFF section number into a gas section number.
+   Because COFF uses negative section numbers, you must add 4 to the
+   COFF section number when indexing into this array; this is done via
+   the SEG_INFO_FROM_SECTION_NUMBER macro.  This must correspond to
+   seg_N_TYPE.  */
 
-static const seg_info_type seg_info_off_by_4[] =
+static const segT seg_info_off_by_4[] =
 {
{SEG_PTV,  },
{SEG_NTV,  },
{SEG_DEBUG, },
{SEG_ABSOLUTE,  },
{SEG_UNKNOWN,  },
{SEG_E0}, {SEG_E1}, {SEG_E2}, {SEG_E3}, {SEG_E4},
{SEG_E5}, {SEG_E6}, {SEG_E7}, {SEG_E8}, {SEG_E9},
{SEG_E10},{SEG_E11},{SEG_E12},{SEG_E13},{SEG_E14},
{SEG_E15},{SEG_E16},{SEG_E17},{SEG_E18},{SEG_E19},
{SEG_E20},{SEG_E21},{SEG_E22},{SEG_E23},{SEG_E24},
{SEG_E25},{SEG_E26},{SEG_E27},{SEG_E28},{SEG_E29},
{SEG_E30},{SEG_E31},{SEG_E32},{SEG_E33},{SEG_E34},
{SEG_E35},{SEG_E36},{SEG_E37},{SEG_E38},{SEG_E39},
{(segT)40},
{(segT)41},
{(segT)42},
{(segT)43},
{(segT)44},
{(segT)45},
{(segT)0},
{(segT)0},
{(segT)0},
- {SEG_REGISTER}
SEG_PTV,
SEG_NTV,
SEG_DEBUG,
SEG_ABSOLUTE,
SEG_UNKNOWN,
SEG_E0,  SEG_E1,  SEG_E2,  SEG_E3,  SEG_E4,
SEG_E5,  SEG_E6,  SEG_E7,  SEG_E8,  SEG_E9,
SEG_E10, SEG_E11, SEG_E12, SEG_E13, SEG_E14,
SEG_E15, SEG_E16, SEG_E17, SEG_E18, SEG_E19,
SEG_E20, SEG_E21, SEG_E22, SEG_E23, SEG_E24,
SEG_E25, SEG_E26, SEG_E27, SEG_E28, SEG_E29,
SEG_E30, SEG_E31, SEG_E32, SEG_E33, SEG_E34,
SEG_E35, SEG_E36, SEG_E37, SEG_E38, SEG_E39,
(segT) 40,
(segT) 41,
(segT) 42,
(segT) 43,
(segT) 44,
(segT) 45,
(segT) 0,
(segT) 0,
(segT) 0,
+ SEG_REGISTER
 };
 
-
-
 #define SEG_INFO_FROM_SECTION_NUMBER(x) (seg_info_off_by_4[(x)+4])
 
+static relax_addressT relax_align PARAMS ((relax_addressT, long));
+
 static relax_addressT
 relax_align (address, alignment)
      relax_addressT address;
@@ -1571,24 +1816,27 @@ relax_align (address, alignment)
   return (new_address - address);
 }
 
-
 segT
 s_get_segment (x)
      symbolS * x;
 {
-  return SEG_INFO_FROM_SECTION_NUMBER (x->sy_symbol.ost_entry.n_scnum).seg_t;
+  return SEG_INFO_FROM_SECTION_NUMBER (x->sy_symbol.ost_entry.n_scnum);
 }
 
-/* calculate the size of the frag chain and fill in the section header
-   to contain all of it, also fill in the addr of the sections */
+static unsigned int size_section PARAMS ((bfd *, unsigned int));
+
+/* Calculate the size of the frag chain and fill in the section header
+   to contain all of it, also fill in the addr of the sections.  */
+
 static unsigned int
 size_section (abfd, idx)
-     bfd * abfd;
+     bfd *abfd ATTRIBUTE_UNUSED;
      unsigned int idx;
 {
 
   unsigned int size = 0;
   fragS *frag = segment_info[idx].frchainP->frch_root;
+
   while (frag)
     {
       size = frag->fr_address;
@@ -1606,7 +1854,6 @@ size_section (abfd, idx)
          break;
 #endif
        case rs_space:
-         assert (frag->fr_symbol == 0);
        case rs_fill:
        case rs_org:
          size += frag->fr_fix;
@@ -1614,6 +1861,7 @@ size_section (abfd, idx)
          break;
        case rs_align:
        case rs_align_code:
+       case rs_align_test:
          {
            addressT off;
 
@@ -1634,7 +1882,6 @@ size_section (abfd, idx)
   return size;
 }
 
-
 static unsigned int
 count_entries_in_chain (idx)
      unsigned int idx;
@@ -1642,14 +1889,14 @@ count_entries_in_chain (idx)
   unsigned int nrelocs;
   fixS *fixup_ptr;
 
-  /* Count the relocations */
+  /* Count the relocations */
   fixup_ptr = segment_info[idx].fix_root;
   nrelocs = 0;
   while (fixup_ptr != (fixS *) NULL)
     {
       if (fixup_ptr->fx_done == 0 && TC_COUNT_RELOC (fixup_ptr))
        {
-#ifdef TC_A29K
+#if defined(TC_A29K) || defined(TC_OR32)
          if (fixup_ptr->fx_r_type == RELOC_CONSTH)
            nrelocs += 2;
          else
@@ -1668,7 +1915,8 @@ count_entries_in_chain (idx)
 
 static int compare_external_relocs PARAMS ((const PTR, const PTR));
 
-/* AUX's ld expects relocations to be sorted */
+/* AUX's ld expects relocations to be sorted.  */
+
 static int
 compare_external_relocs (x, y)
      const PTR x;
@@ -1683,7 +1931,8 @@ compare_external_relocs (x, y)
 
 #endif
 
-/* output all the relocations for a section */
+/* Output all the relocations for a section.  */
+
 void
 do_relocs_for (abfd, h, file_cursor)
      bfd * abfd;
@@ -1721,13 +1970,12 @@ do_relocs_for (abfd, h, file_cursor)
                {
                  struct internal_reloc intr;
 
-                 /* Only output some of the relocations */
+                 /* Only output some of the relocations */
                  if (fix_ptr->fx_done == 0 && TC_COUNT_RELOC (fix_ptr))
                    {
 #ifdef TC_RELOC_MANGLE
                      TC_RELOC_MANGLE (&segment_info[idx], fix_ptr, &intr,
                                       base);
-
 #else
                      symbolS *dot;
                      symbolS *symbol_ptr = fix_ptr->fx_addsy;
@@ -1759,7 +2007,7 @@ do_relocs_for (abfd, h, file_cursor)
                      /* Turn the segment of the symbol into an offset.  */
                      if (symbol_ptr)
                        {
-                         resolve_symbol_value (symbol_ptr, 1);
+                         resolve_symbol_value (symbol_ptr);
                          if (! symbol_ptr->sy_resolved)
                            {
                              char *file;
@@ -1772,56 +2020,58 @@ do_relocs_for (abfd, h, file_cursor)
                                as_bad (_("bad relocation: symbol `%s' not in symbol table"),
                                        S_GET_NAME (symbol_ptr));
                            }
+
                          dot = segment_info[S_GET_SEGMENT (symbol_ptr)].dot;
                          if (dot)
-                           {
-                             intr.r_symndx = dot->sy_number;
-                           }
+                           intr.r_symndx = dot->sy_number;
                          else
-                           {
-                             intr.r_symndx = symbol_ptr->sy_number;
-                           }
-
+                           intr.r_symndx = symbol_ptr->sy_number;
                        }
                      else
-                       {
-                         intr.r_symndx = -1;
-                       }
+                       intr.r_symndx = -1;
 #endif
-
                      (void) bfd_coff_swap_reloc_out (abfd, &intr, ext_ptr);
                      ext_ptr++;
-
 #if defined(TC_A29K)
-
                      /* The 29k has a special kludge for the high 16 bit
                         reloc.  Two relocations are emited, R_IHIHALF,
                         and R_IHCONST. The second one doesn't contain a
                         symbol, but uses the value for offset.  */
-
                      if (intr.r_type == R_IHIHALF)
                        {
-                         /* now emit the second bit */
+                         /* Now emit the second bit.  */
                          intr.r_type = R_IHCONST;
                          intr.r_symndx = fix_ptr->fx_addnumber;
                          (void) bfd_coff_swap_reloc_out (abfd, &intr, ext_ptr);
                          ext_ptr++;
                        }
+#endif
+#if defined(TC_OR32)
+                     /* The or32 has a special kludge for the high 16 bit
+                        reloc.  Two relocations are emited, R_IHIHALF,
+                        and R_IHCONST. The second one doesn't contain a
+                        symbol, but uses the value for offset.  */
+                     if (intr.r_type == R_IHIHALF)
+                       {
+                         /* Now emit the second bit.  */
+                         intr.r_type = R_IHCONST;
+                         intr.r_symndx = fix_ptr->fx_addnumber;
+                         (void) bfd_coff_swap_reloc_out (abfd, & intr, ext_ptr);
+                         ext_ptr ++;
+                       }
 #endif
                    }
 
                  fix_ptr = fix_ptr->fx_next;
                }
-
 #ifdef TE_AUX
-             /* Sort the reloc table */
+             /* Sort the reloc table */
              qsort ((PTR) external_reloc_vec, nrelocs,
                     sizeof (struct external_reloc), compare_external_relocs);
 #endif
-
-             /* Write out the reloc table */
-             bfd_write ((PTR) external_reloc_vec, 1, external_reloc_size,
-                        abfd);
+             /* Write out the reloc table.  */
+             bfd_bwrite ((PTR) external_reloc_vec,
+                         (bfd_size_type) external_reloc_size, abfd);
              free (external_reloc_vec);
 
              /* Fill in section header info.  */
@@ -1831,26 +2081,25 @@ do_relocs_for (abfd, h, file_cursor)
            }
          else
            {
-             /* No relocs */
+             /* No relocs */
              segment_info[idx].scnhdr.s_relptr = 0;
            }
        }
     }
-  /* Set relocation_size field in file headers */
+
+  /* Set relocation_size field in file headers.  */
   H_SET_RELOCATION_SIZE (h, *file_cursor - reloc_start, 0);
 }
 
+/* Run through a frag chain and write out the data to go with it, fill
+   in the scnhdrs with the info on the file postions.  */
 
-/* run through a frag chain and write out the data to go with it, fill
-   in the scnhdrs with the info on the file postions
-*/
 static void
 fill_section (abfd, h, file_cursor)
      bfd * abfd;
-     object_headers *h;
+     object_headers *h ATTRIBUTE_UNUSED;
      unsigned long *file_cursor;
 {
-
   unsigned int i;
   unsigned int paddr = 0;
 
@@ -1864,7 +2113,7 @@ fill_section (abfd, h, file_cursor)
       if (s->s_name[0])
        {
          fragS *frag = segment_info[i].frchainP->frch_root;
-         char *buffer;
+         char *buffer = NULL;
 
          if (s->s_size == 0)
            s->s_scnptr = 0;
@@ -1888,12 +2137,14 @@ fill_section (abfd, h, file_cursor)
                 COFF_NOLOAD_PROBLEM, and have only one test here.  */
 #ifndef TC_I386
 #ifndef TC_A29K
+#ifndef TC_OR32
 #ifndef COFF_NOLOAD_PROBLEM
              /* Apparently the SVR3 linker (and exec syscall) and UDI
                 mondfe progrem are confused by noload sections.  */
              s->s_flags |= STYP_NOLOAD;
 #endif
 #endif
+#endif
 #endif
            }
          else if (strcmp (s->s_name, ".lit") == 0)
@@ -1921,10 +2172,10 @@ fill_section (abfd, h, file_cursor)
 
                  break;
                case rs_space:
-                 assert (frag->fr_symbol == 0);
                case rs_fill:
                case rs_align:
                case rs_align_code:
+               case rs_align_test:
                case rs_org:
                  if (frag->fr_fix)
                    {
@@ -1964,7 +2215,7 @@ fill_section (abfd, h, file_cursor)
            {
              if (s->s_scnptr != 0)
                {
-                 bfd_write (buffer, s->s_size, 1, abfd);
+                 bfd_bwrite (buffer, s->s_size, abfd);
                  *file_cursor += s->s_size;
                }
              free (buffer);
@@ -1974,7 +2225,7 @@ fill_section (abfd, h, file_cursor)
     }
 }
 
-/* Coff file generation & utilities */
+/* Coff file generation & utilities */
 
 static void
 coff_header_append (abfd, h)
@@ -1988,7 +2239,7 @@ coff_header_append (abfd, h)
   unsigned long string_size = 4;
 #endif
 
-  bfd_seek (abfd, 0, 0);
+  bfd_seek (abfd, (file_ptr) 0, 0);
 
 #ifndef OBJ_COFF_OMIT_OPTIONAL_HEADER
   H_SET_MAGIC_NUMBER (h, COFF_MAGIC);
@@ -2004,8 +2255,8 @@ coff_header_append (abfd, h)
 
   i = bfd_coff_swap_filehdr_out (abfd, &h->filehdr, buffer);
 
-  bfd_write (buffer, i, 1, abfd);
-  bfd_write (buffero, H_GET_SIZEOF_OPTIONAL_HEADER (h), 1, abfd);
+  bfd_bwrite (buffer, (bfd_size_type) i, abfd);
+  bfd_bwrite (buffero, (bfd_size_type) H_GET_SIZEOF_OPTIONAL_HEADER (h), abfd);
 
   for (i = SEG_E0; i < SEG_LAST; i++)
     {
@@ -2024,18 +2275,16 @@ coff_header_append (abfd, h)
              string_size += strlen (segment_info[i].name) + 1;
            }
 #endif
-
          size = bfd_coff_swap_scnhdr_out (abfd,
                                           &(segment_info[i].scnhdr),
                                           buffer);
          if (size == 0)
            as_bad (_("bfd_coff_swap_scnhdr_out failed"));
-         bfd_write (buffer, size, 1, abfd);
+         bfd_bwrite (buffer, (bfd_size_type) size, abfd);
        }
     }
 }
 
-
 char *
 symbol_to_chars (abfd, where, symbolP)
      bfd * abfd;
@@ -2046,13 +2295,11 @@ symbol_to_chars (abfd, where, symbolP)
   unsigned int i;
   valueT val;
 
-  /* Turn any symbols with register attributes into abs symbols */
+  /* Turn any symbols with register attributes into abs symbols */
   if (S_GET_SEGMENT (symbolP) == reg_section)
-    {
-      S_SET_SEGMENT (symbolP, absolute_section);
-    }
-  /* At the same time, relocate all symbols to their output value */
+    S_SET_SEGMENT (symbolP, absolute_section);
 
+  /* At the same time, relocate all symbols to their output value.  */
 #ifndef TE_PE
   val = (segment_info[S_GET_SEGMENT (symbolP)].scnhdr.s_paddr
         + S_GET_VALUE (symbolP));
@@ -2075,25 +2322,25 @@ symbol_to_chars (abfd, where, symbolP)
                                      S_GET_STORAGE_CLASS (symbolP),
                                      i, numaux, where);
     }
-  return where;
 
+  return where;
 }
 
 void
 coff_obj_symbol_new_hook (symbolP)
      symbolS *symbolP;
 {
-  char underscore = 0;         /* Symbol has leading _ */
+  char underscore = 0;         /* Symbol has leading _  */
 
-  /* Effective symbol */
-  /* Store the pointer in the offset. */
+  /* Effective symbol */
+  /* Store the pointer in the offset.  */
   S_SET_ZEROES (symbolP, 0L);
   S_SET_DATA_TYPE (symbolP, T_NULL);
   S_SET_STORAGE_CLASS (symbolP, 0);
   S_SET_NUMBER_AUXILIARY (symbolP, 0);
-  /* Additional information */
+  /* Additional information */
   symbolP->sy_symbol.ost_flags = 0;
-  /* Auxiliary entries */
+  /* Auxiliary entries */
   memset ((char *) &symbolP->sy_symbol.ost_auxent[0], 0, AUXESZ);
 
   if (S_IS_STRING (symbolP))
@@ -2102,9 +2349,7 @@ coff_obj_symbol_new_hook (symbolP)
     SF_SET_LOCAL (symbolP);
 }
 
-/*
- * Handle .ln directives.
- */
+/* Handle .ln directives.  */
 
 static void
 obj_coff_ln (appline)
@@ -2114,10 +2359,11 @@ obj_coff_ln (appline)
 
   if (! appline && def_symbol_in_progress != NULL)
     {
+      /* Wrong context.  */
       as_warn (_(".ln pseudo-op inside .def/.endef: ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* wrong context */
+    }
 
   l = get_absolute_expression ();
   c_line_new (0, frag_now_fix (), l, frag_now);
@@ -2141,19 +2387,14 @@ obj_coff_ln (appline)
   demand_empty_rest_of_line ();
 }
 
-/*
- *                     def()
- *
- * Handle .def directives.
- *
- * One might ask : why can't we symbol_new if the symbol does not
- * already exist and fill it with debug information.  Because of
- * the C_EFCN special symbol. It would clobber the value of the
- * function symbol before we have a chance to notice that it is
- * a C_EFCN. And a second reason is that the code is more clear this
- * way. (at least I think it is :-).
- *
- */
+/* Handle .def directives.
+  One might ask : why can't we symbol_new if the symbol does not
+  already exist and fill it with debug information.  Because of
+  the C_EFCN special symbol. It would clobber the value of the
+  function symbol before we have a chance to notice that it is
+  a C_EFCN. And a second reason is that the code is more clear this
+  way. (at least I think it is :-).  */
 
 #define SKIP_SEMI_COLON()      while (*input_line_pointer++ != ';')
 #define SKIP_WHITESPACES()     while (*input_line_pointer == ' ' || \
@@ -2162,11 +2403,11 @@ obj_coff_ln (appline)
 
 static void
 obj_coff_def (what)
-     int what;
+     int what ATTRIBUTE_UNUSED;
 {
-  char name_end;               /* Char after the end of name */
-  char *symbol_name;           /* Name of the debug symbol */
-  char *symbol_name_copy;      /* Temporary copy of the name */
+  char name_end;               /* Char after the end of name */
+  char *symbol_name;           /* Name of the debug symbol */
+  char *symbol_name_copy;      /* Temporary copy of the name */
   unsigned int symbol_name_length;
 
   if (def_symbol_in_progress != NULL)
@@ -2174,7 +2415,7 @@ obj_coff_def (what)
       as_warn (_(".def pseudo-op used inside of .def/.endef: ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
   SKIP_WHITESPACES ();
 
@@ -2190,7 +2431,7 @@ obj_coff_def (what)
   symbol_name_copy = tc_canonicalize_symbol_name (symbol_name_copy);
 #endif
 
-  /* Initialize the new symbol */
+  /* Initialize the new symbol */
 #ifdef STRIP_UNDERSCORE
   S_SET_NAME (def_symbol_in_progress, (*symbol_name_copy == '_'
                                       ? symbol_name_copy + 1
@@ -2214,10 +2455,9 @@ obj_coff_def (what)
 
 unsigned int dim_index;
 
-
 static void
 obj_coff_endef (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   symbolS *symbolP = 0;
   /* DIM BUG FIX sac@cygnus.com */
@@ -2227,16 +2467,17 @@ obj_coff_endef (ignore)
       as_warn (_(".endef pseudo-op used outside of .def/.endef: ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
-  /* Set the section number according to storage class. */
+  /* Set the section number according to storage class.  */
   switch (S_GET_STORAGE_CLASS (def_symbol_in_progress))
     {
     case C_STRTAG:
     case C_ENTAG:
     case C_UNTAG:
       SF_SET_TAG (def_symbol_in_progress);
-      /* intentional fallthrough */
+      /* Intentional fallthrough.  */
+
     case C_FILE:
     case C_TPDEF:
       SF_SET_DEBUG (def_symbol_in_progress);
@@ -2244,20 +2485,23 @@ obj_coff_endef (ignore)
       break;
 
     case C_EFCN:
-      SF_SET_LOCAL (def_symbol_in_progress);   /* Do not emit this symbol. */
-      /* intentional fallthrough */
+      /* Do not emit this symbol.  */
+      SF_SET_LOCAL (def_symbol_in_progress);
+      /* Intentional fallthrough. */
+      
     case C_BLOCK:
-      SF_SET_PROCESS (def_symbol_in_progress); /* Will need processing before writing */
-      /* intentional fallthrough */
+      /* Will need processing before writing.  */
+      SF_SET_PROCESS (def_symbol_in_progress);
+      /* Intentional fallthrough.  */
+
     case C_FCN:
       S_SET_SEGMENT (def_symbol_in_progress, SEG_E0);
 
       if (strcmp (S_GET_NAME (def_symbol_in_progress), ".bf") == 0)
        {                       /* .bf */
          if (function_lineoff < 0)
-           {
-             fprintf (stderr, _("`.bf' symbol without preceding function\n"));
-           }                   /* missing function symbol */
+           fprintf (stderr, _("`.bf' symbol without preceding function\n"));
+
          SA_GET_SYM_LNNOPTR (last_line_symbol) = function_lineoff;
 
          SF_SET_PROCESS (last_line_symbol);
@@ -2265,7 +2509,8 @@ obj_coff_endef (ignore)
          SF_SET_PROCESS (def_symbol_in_progress);
          function_lineoff = -1;
        }
-      /* Value is always set to . */
+
+      /* Value is always set to .  */
       def_symbol_in_progress->sy_frag = frag_now;
       S_SET_VALUE (def_symbol_in_progress, (valueT) frag_now_fix ());
       break;
@@ -2293,7 +2538,7 @@ obj_coff_endef (ignore)
 #endif
     case C_STAT:
     case C_LABEL:
-      /* Valid but set somewhere else (s_comm, s_lcomm, colon) */
+      /* Valid but set somewhere else (s_comm, s_lcomm, colon) */
       break;
 
     case C_USTATIC:
@@ -2301,7 +2546,7 @@ obj_coff_endef (ignore)
     case C_ULABEL:
       as_warn (_("unexpected storage class %d"), S_GET_STORAGE_CLASS (def_symbol_in_progress));
       break;
-    }                          /* switch on storage class */
+    }
 
   /* Now that we have built a debug symbol, try to find if we should
      merge with an existing symbol or not.  If a symbol is C_EFCN or
@@ -2316,11 +2561,11 @@ obj_coff_endef (ignore)
      lineno entry MUST point to the definition function or else it
      will point off into space when crawl_symbols() merges the debug
      symbol into the real symbol.  Therefor, let's presume the debug
-     symbol is a real function reference. */
+     symbol is a real function reference.  */
 
   /* FIXME-SOON If for some reason the definition label/symbol is
      never seen, this will probably leave an undefined symbol at link
-     time. */
+     time.  */
 
   if (S_GET_STORAGE_CLASS (def_symbol_in_progress) == C_EFCN
       || S_GET_STORAGE_CLASS (def_symbol_in_progress) == C_LABEL
@@ -2346,7 +2591,7 @@ obj_coff_endef (ignore)
         previous definition.  */
 
       c_symbol_merge (def_symbol_in_progress, symbolP);
-      /* FIXME-SOON Should *def_symbol_in_progress be free'd? xoxorich. */
+      /* FIXME-SOON Should *def_symbol_in_progress be free'd? xoxorich.  */
       def_symbol_in_progress = symbolP;
 
       if (SF_GET_FUNCTION (def_symbol_in_progress)
@@ -2355,17 +2600,17 @@ obj_coff_endef (ignore)
        {
          /* For functions, and tags, and static symbols, the symbol
             *must* be where the debug symbol appears.  Move the
-            existing symbol to the current place. */
-         /* If it already is at the end of the symbol list, do nothing */
+            existing symbol to the current place.  */
+         /* If it already is at the end of the symbol list, do nothing */
          if (def_symbol_in_progress != symbol_lastP)
            {
              symbol_remove (def_symbol_in_progress, &symbol_rootP,
                             &symbol_lastP);
              symbol_append (def_symbol_in_progress, symbol_lastP,
                             &symbol_rootP, &symbol_lastP);
-           }                   /* if not already in place */
-       }                       /* if function */
-    }                          /* normal or mergable */
+           }
+       }
+    }
 
   if (SF_GET_TAG (def_symbol_in_progress))
     {
@@ -2389,10 +2634,10 @@ obj_coff_endef (ignore)
       if (symbolP == NULL)
        {
          /* That is, if this is the first time we've seen the
-            function... */
+            function...  */
          symbol_table_insert (def_symbol_in_progress);
-       }                       /* definition follows debug */
-    }                          /* Create the line number entry pointing to the function being defined */
+       }
+    }
 
   def_symbol_in_progress = NULL;
   demand_empty_rest_of_line ();
@@ -2400,7 +2645,7 @@ obj_coff_endef (ignore)
 
 static void
 obj_coff_dim (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int dim_index;
 
@@ -2409,7 +2654,7 @@ obj_coff_dim (ignore)
       as_warn (_(".dim pseudo-op used outside of .def/.endef: ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
   S_SET_NUMBER_AUXILIARY (def_symbol_in_progress, 1);
 
@@ -2427,7 +2672,8 @@ obj_coff_dim (ignore)
 
        default:
          as_warn (_("badly formed .dim directive ignored"));
-         /* intentional fallthrough */
+         /* Intentional fallthrough.  */
+
        case '\n':
        case ';':
          dim_index = DIMNUM;
@@ -2440,7 +2686,7 @@ obj_coff_dim (ignore)
 
 static void
 obj_coff_line (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int this_base;
   const char *name;
@@ -2462,17 +2708,13 @@ obj_coff_line (ignore)
 #if 0 /* XXX Can we ever have line numbers going backwards?  */
       if (this_base > line_base)
 #endif
-       {
-         line_base = this_base;
-       }
+       line_base = this_base;
 
 #ifndef NO_LISTING
       {
        extern int listing;
        if (listing)
-         {
-           listing_source_line ((unsigned int) line_base);
-         }
+         listing_source_line ((unsigned int) line_base);
       }
 #endif
     }
@@ -2485,14 +2727,14 @@ obj_coff_line (ignore)
 
 static void
 obj_coff_size (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
       as_warn (_(".size pseudo-op used outside of .def/.endef ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
   S_SET_NUMBER_AUXILIARY (def_symbol_in_progress, 1);
   SA_SET_SYM_SIZE (def_symbol_in_progress, get_absolute_expression ());
@@ -2501,14 +2743,14 @@ obj_coff_size (ignore)
 
 static void
 obj_coff_scl (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
       as_warn (_(".scl pseudo-op used outside of .def/.endef ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
   S_SET_STORAGE_CLASS (def_symbol_in_progress, get_absolute_expression ());
   demand_empty_rest_of_line ();
@@ -2516,7 +2758,7 @@ obj_coff_scl (ignore)
 
 static void
 obj_coff_tag (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *symbol_name;
   char name_end;
@@ -2536,13 +2778,11 @@ obj_coff_tag (ignore)
 #endif
 
   /* Assume that the symbol referred to by .tag is always defined.
-     This was a bad assumption.  I've added find_or_make. xoxorich. */
+     This was a bad assumption.  I've added find_or_make. xoxorich.  */
   SA_SET_SYM_TAGNDX (def_symbol_in_progress,
                     (long) tag_find_or_make (symbol_name));
   if (SA_GET_SYM_TAGNDX (def_symbol_in_progress) == 0L)
-    {
-      as_warn (_("tag not found for .tag %s"), symbol_name);
-    }                          /* not defined */
+    as_warn (_("tag not found for .tag %s"), symbol_name);
 
   SF_SET_TAGGED (def_symbol_in_progress);
   *input_line_pointer = name_end;
@@ -2552,36 +2792,34 @@ obj_coff_tag (ignore)
 
 static void
 obj_coff_type (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
       as_warn (_(".type pseudo-op used outside of .def/.endef ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
   S_SET_DATA_TYPE (def_symbol_in_progress, get_absolute_expression ());
 
   if (ISFCN (S_GET_DATA_TYPE (def_symbol_in_progress)) &&
       S_GET_STORAGE_CLASS (def_symbol_in_progress) != C_TPDEF)
-    {
-      SF_SET_FUNCTION (def_symbol_in_progress);
-    }                          /* is a function */
+    SF_SET_FUNCTION (def_symbol_in_progress);
 
   demand_empty_rest_of_line ();
 }
 
 static void
 obj_coff_val (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (def_symbol_in_progress == NULL)
     {
       as_warn (_(".val pseudo-op used outside of .def/.endef ignored."));
       demand_empty_rest_of_line ();
       return;
-    }                          /* if not inside .def/.endef */
+    }
 
   if (is_name_beginner (*input_line_pointer))
     {
@@ -2596,7 +2834,7 @@ obj_coff_val (ignore)
        {
          def_symbol_in_progress->sy_frag = frag_now;
          S_SET_VALUE (def_symbol_in_progress, (valueT) frag_now_fix ());
-         /* If the .val is != from the .def (e.g. statics) */
+         /* If the .val is != from the .def (e.g. statics) */
        }
       else if (strcmp (S_GET_NAME (def_symbol_in_progress), symbol_name))
        {
@@ -2611,15 +2849,14 @@ obj_coff_val (ignore)
             symbol.  */
          SF_SET_GET_SEGMENT (def_symbol_in_progress);
 
-         /* FIXME: gcc can generate address expressions
-            here in unusual cases (search for "obscure"
-            in sdbout.c).  We just ignore the offset
-            here, thus generating incorrect debugging
-            information.  We ignore the rest of the
-            line just below.  */
+         /* FIXME: gcc can generate address expressions here in
+            unusual cases (search for "obscure" in sdbout.c).  We
+            just ignore the offset here, thus generating incorrect
+            debugging information.  We ignore the rest of the line
+            just below.  */
        }
       /* Otherwise, it is the name of a non debug symbol and
-        its value will be calculated later. */
+        its value will be calculated later.  */
       *input_line_pointer = name_end;
 
       /* FIXME: this is to avoid an error message in the
@@ -2657,7 +2894,7 @@ obj_coff_pe_handle_link_once (type)
 void
 coff_obj_read_begin_hook ()
 {
-  /* These had better be the same.  Usually 18 bytes. */
+  /* These had better be the same.  Usually 18 bytes.  */
 #ifndef BFD_HEADERS
   know (sizeof (SYMENT) == sizeof (AUXENT));
   know (SYMESZ == AUXESZ);
@@ -2666,13 +2903,13 @@ coff_obj_read_begin_hook ()
 }
 
 /* This function runs through the symbol table and puts all the
-   externals onto another chain */
+   externals onto another chain */
 
 /* The chain of globals.  */
 symbolS *symbol_globalP;
 symbolS *symbol_global_lastP;
 
-/* The chain of externals */
+/* The chain of externals */
 symbolS *symbol_externP;
 symbolS *symbol_extern_lastP;
 
@@ -2709,10 +2946,10 @@ yank_symbols ()
 
       if (!SF_GET_DEBUG (symbolP))
        {
-         /* Debug symbols do not need all this rubbish */
+         /* Debug symbols do not need all this rubbish */
          symbolS *real_symbolP;
 
-         /* L* and C_EFCN symbols never merge. */
+         /* L* and C_EFCN symbols never merge.  */
          if (!SF_GET_LOCAL (symbolP)
              && !SF_GET_STATICS (symbolP)
              && S_GET_STORAGE_CLASS (symbolP) != C_LABEL
@@ -2721,29 +2958,27 @@ yank_symbols ()
              && real_symbolP != symbolP)
            {
              /* FIXME-SOON: where do dups come from?
-                Maybe tag references before definitions? xoxorich. */
+                Maybe tag references before definitions? xoxorich.  */
              /* Move the debug data from the debug symbol to the
                 real symbol. Do NOT do the oposite (i.e. move from
                 real symbol to debug symbol and remove real symbol from the
                 list.) Because some pointers refer to the real symbol
-                whereas no pointers refer to the debug symbol. */
+                whereas no pointers refer to the debug symbol.  */
              c_symbol_merge (symbolP, real_symbolP);
-             /* Replace the current symbol by the real one */
+             /* Replace the current symbol by the real one */
              /* The symbols will never be the last or the first
                 because : 1st symbol is .file and 3 last symbols are
-                .text, .data, .bss */
+                .text, .data, .bss */
              symbol_remove (real_symbolP, &symbol_rootP, &symbol_lastP);
              symbol_insert (real_symbolP, symbolP, &symbol_rootP, &symbol_lastP);
              symbol_remove (symbolP, &symbol_rootP, &symbol_lastP);
              symbolP = real_symbolP;
-           }                   /* if not local but dup'd */
+           }
 
          if (flag_readonly_data_in_text && (S_GET_SEGMENT (symbolP) == SEG_E1))
-           {
-             S_SET_SEGMENT (symbolP, SEG_E0);
-           }                   /* push data into text */
+           S_SET_SEGMENT (symbolP, SEG_E0);
 
-         resolve_symbol_value (symbolP, 1);
+         resolve_symbol_value (symbolP);
 
          if (S_GET_STORAGE_CLASS (symbolP) == C_NULL)
            {
@@ -2751,27 +2986,27 @@ yank_symbols ()
                {
                  S_SET_EXTERNAL (symbolP);
                }
+
              else if (S_GET_SEGMENT (symbolP) == SEG_E0)
-               {
-                 S_SET_STORAGE_CLASS (symbolP, C_LABEL);
-               }
+               S_SET_STORAGE_CLASS (symbolP, C_LABEL);
+
              else
-               {
-                 S_SET_STORAGE_CLASS (symbolP, C_STAT);
-               }
+               S_SET_STORAGE_CLASS (symbolP, C_STAT);
            }
 
-         /* Mainly to speed up if not -g */
+         /* Mainly to speed up if not -g */
          if (SF_GET_PROCESS (symbolP))
            {
-             /* Handle the nested blocks auxiliary info. */
+             /* Handle the nested blocks auxiliary info.  */
              if (S_GET_STORAGE_CLASS (symbolP) == C_BLOCK)
                {
                  if (!strcmp (S_GET_NAME (symbolP), ".bb"))
                    stack_push (block_stack, (char *) &symbolP);
                  else
-                   {           /* .eb */
-                     register symbolS *begin_symbolP;
+                   {
+                     /* .eb */
+                     symbolS *begin_symbolP;
+
                      begin_symbolP = *(symbolS **) stack_pop (block_stack);
                      if (begin_symbolP == (symbolS *) 0)
                        as_warn (_("mismatched .eb"));
@@ -2782,20 +3017,18 @@ yank_symbols ()
              /* If we are able to identify the type of a function, and we
               are out of a function (last_functionP == 0) then, the
               function symbol will be associated with an auxiliary
-              entry. */
+              entry.  */
              if (last_functionP == (symbolS *) 0 &&
                  SF_GET_FUNCTION (symbolP))
                {
                  last_functionP = symbolP;
 
                  if (S_GET_NUMBER_AUXILIARY (symbolP) < 1)
-                   {
-                     S_SET_NUMBER_AUXILIARY (symbolP, 1);
-                   }           /* make it at least 1 */
+                   S_SET_NUMBER_AUXILIARY (symbolP, 1);
 
-                 /* Clobber possible stale .dim information. */
+                 /* Clobber possible stale .dim information.  */
 #if 0
-                 /* Iffed out by steve - this fries the lnnoptr info too */
+                 /* Iffed out by steve - this fries the lnnoptr info too */
                  bzero (symbolP->sy_symbol.ost_auxent[0].x_sym.x_fcnary.x_ary.x_dimen,
                         sizeof (symbolP->sy_symbol.ost_auxent[0].x_sym.x_fcnary.x_ary.x_dimen));
 #endif
@@ -2826,19 +3059,19 @@ yank_symbols ()
       else if (SF_GET_TAG (symbolP))
        {
          /* First descriptor of a structure must point to
-              the first slot after the structure description. */
+            the first slot after the structure description.  */
          last_tagP = symbolP;
 
        }
       else if (S_GET_STORAGE_CLASS (symbolP) == C_EOS)
        {
-         /* +2 take in account the current symbol */
+         /* +2 take in account the current symbol */
          SA_SET_SYM_ENDNDX (last_tagP, symbol_number + 2);
        }
       else if (S_GET_STORAGE_CLASS (symbolP) == C_FILE)
        {
          /* If the filename was too long to fit in the
-            auxent, put it in the string table */
+            auxent, put it in the string table */
          if (SA_GET_FILE_FNAME_ZEROS (symbolP) == 0
              && SA_GET_FILE_FNAME_OFFSET (symbolP) != 0)
            {
@@ -2850,8 +3083,8 @@ yank_symbols ()
            {
              S_SET_VALUE (symbolP, last_file_symno);
              last_file_symno = symbol_number;
-           }                   /* no one points at the first .file symbol */
-       }                       /* if debug or tag or eos or file */
+           }
+       }
 
 #ifdef tc_frob_coff_symbol
       tc_frob_coff_symbol (symbolP);
@@ -2866,13 +3099,12 @@ yank_symbols ()
         [21] foo external
         [22] .be
         ld will move the symbol 21 to the end of the list but
-        endndx will still be 22 instead of 21. */
-
+        endndx will still be 22 instead of 21.  */
 
       if (SF_GET_LOCAL (symbolP))
        {
-         /* remove C_EFCN and LOCAL (L...) symbols */
-         /* next pointer remains valid */
+         /* Remove C_EFCN and LOCAL (L...) symbols.  */
+         /* Next pointer remains valid.  */
          symbol_remove (symbolP, &symbol_rootP, &symbol_lastP);
 
        }
@@ -2892,7 +3124,7 @@ yank_symbols ()
 #endif
                   || S_GET_STORAGE_CLASS (symbolP) == C_WEAKEXT))
        {
-         /* if external, Remove from the list */
+         /* If external, Remove from the list.  */
          symbolS *hold = symbol_previous (symbolP);
 
          symbol_remove (symbolP, &symbol_rootP, &symbol_lastP);
@@ -2914,7 +3146,6 @@ yank_symbols ()
          /* The O'Reilly COFF book says that defined global symbols
              come at the end of the symbol table, just before
              undefined global symbols.  */
-
          symbol_remove (symbolP, &symbol_rootP, &symbol_lastP);
          symbol_clear_list_pointers (symbolP);
          symbol_append (symbolP, symbol_global_lastP, &symbol_globalP,
@@ -2931,17 +3162,16 @@ yank_symbols ()
          else
            {
              symbolP->sy_name_offset = 0;
-           }                   /* fix "long" names */
+           }
 
          symbolP->sy_number = symbol_number;
          symbol_number += 1 + S_GET_NUMBER_AUXILIARY (symbolP);
-       }                       /* if local symbol */
-    }                          /* traverse the symbol list */
-  return symbol_number;
+       }
+    }
 
+  return symbol_number;
 }
 
-
 static unsigned int
 glue_symbols (head, tail)
      symbolS **head;
@@ -2953,11 +3183,11 @@ glue_symbols (head, tail)
     {
       symbolS *tmp = *head;
 
-      /* append */
+      /* Append.  */
       symbol_remove (tmp, head, tail);
       symbol_append (tmp, symbol_lastP, &symbol_rootP, &symbol_lastP);
 
-      /* and process */
+      /* Process.  */
       if (SF_GET_STRING (tmp))
        {
          tmp->sy_name_offset = string_byte_count;
@@ -2965,12 +3195,13 @@ glue_symbols (head, tail)
        }
       else
        {
+         /* Fix "long" names.  */
          tmp->sy_name_offset = 0;
-       }                       /* fix "long" names */
+       }
 
       tmp->sy_number = symbol_number;
       symbol_number += 1 + S_GET_NUMBER_AUXILIARY (tmp);
-    }                          /* append the entire extern chain */
+    }
 
   return symbol_number;
 }
@@ -2998,48 +3229,43 @@ tie_tags ()
   return symbol_number;
 }
 
+
 static void
 crawl_symbols (h, abfd)
      object_headers *h;
-     bfd * abfd;
+     bfd *abfd ATTRIBUTE_UNUSED;
 {
   unsigned int i;
 
-  /* Initialize the stack used to keep track of the matching .bb .be */
+  /* Initialize the stack used to keep track of the matching .bb .be */
 
   block_stack = stack_init (512, sizeof (symbolS *));
 
   /* The symbol list should be ordered according to the following sequence
-   * order :
-   * . .file symbol
-   * . debug entries for functions
-   * . fake symbols for the sections, including .text .data and .bss
-   * . defined symbols
-   * . undefined symbols
-   * But this is not mandatory. The only important point is to put the
-   * undefined symbols at the end of the list.
-   */
-
-  /* Is there a .file symbol ? If not insert one at the beginning. */
+     order :
+     . .file symbol
+     . debug entries for functions
+     . fake symbols for the sections, including .text .data and .bss
+     . defined symbols
+     . undefined symbols
+     But this is not mandatory. The only important point is to put the
+     undefined symbols at the end of the list.  */
+
+  /* Is there a .file symbol ? If not insert one at the beginning.  */
   if (symbol_rootP == NULL
       || S_GET_STORAGE_CLASS (symbol_rootP) != C_FILE)
-    {
-      c_dot_file_symbol ("fake");
-    }
-
-  /*
-   * Build up static symbols for the sections, they are filled in later
-   */
+    c_dot_file_symbol ("fake");
 
+  /* Build up static symbols for the sections, they are filled in later.  */
 
   for (i = SEG_E0; i < SEG_LAST; i++)
     if (segment_info[i].scnhdr.s_name[0])
-      segment_info[i].dot = c_section_symbol (segment_info[i].name,
+      segment_info[i].dot = c_section_symbol ((char *) segment_info[i].name,
                                              i - SEG_E0 + 1);
 
-  /* Take all the externals out and put them into another chain */
+  /* Take all the externals out and put them into another chain */
   H_SET_SYMBOL_TABLE_SIZE (h, yank_symbols ());
-  /* Take the externals and glue them onto the end.*/
+  /* Take the externals and glue them onto the end.  */
   H_SET_SYMBOL_TABLE_SIZE (h,
                           (H_GET_SYMBOL_COUNT (h)
                            + glue_symbols (&symbol_globalP,
@@ -3054,9 +3280,7 @@ crawl_symbols (h, abfd)
   know (symbol_extern_lastP == NULL);
 }
 
-/*
- * Find strings by crawling along symbol table chain.
- */
+/* Find strings by crawling along symbol table chain.  */
 
 void
 w_strings (where)
@@ -3065,7 +3289,7 @@ w_strings (where)
   symbolS *symbolP;
   struct filename_list *filename_list_scan = filename_list_head;
 
-  /* Gotta do md_ byte-ordering stuff for string_byte_count first - KWK */
+  /* Gotta do md_ byte-ordering stuff for string_byte_count first - KWK */
   md_number_to_chars (where, (valueT) string_byte_count, 4);
   where += 4;
 
@@ -3127,7 +3351,6 @@ do_linenos_for (abfd, h, file_cursor)
     {
       segment_info_type *s = segment_info + idx;
 
-
       if (s->scnhdr.s_nlnno != 0)
        {
          struct lineno_list *line_ptr;
@@ -3138,45 +3361,42 @@ do_linenos_for (abfd, h, file_cursor)
          struct external_lineno *dst = buffer;
 
          /* Run through the table we've built and turn it into its external
-        form, take this chance to remove duplicates */
+            form, take this chance to remove duplicates.  */
 
          for (line_ptr = s->lineno_list_head;
               line_ptr != (struct lineno_list *) NULL;
               line_ptr = line_ptr->next)
            {
-
              if (line_ptr->line.l_lnno == 0)
                {
-                 /* Turn a pointer to a symbol into the symbols' index */
-                 line_ptr->line.l_addr.l_symndx =
-                   ((symbolS *) line_ptr->line.l_addr.l_symndx)->sy_number;
+                 /* Turn a pointer to a symbol into the symbols' index,
+                    provided that it has been initialised.  */
+                 if (line_ptr->line.l_addr.l_symndx)
+                   line_ptr->line.l_addr.l_symndx =
+                     ((symbolS *) line_ptr->line.l_addr.l_symndx)->sy_number;
                }
              else
-               {
-                 line_ptr->line.l_addr.l_paddr += ((struct frag *) (line_ptr->frag))->fr_address;
-               }
-
+               line_ptr->line.l_addr.l_paddr += ((struct frag *) (line_ptr->frag))->fr_address;
 
              (void) bfd_coff_swap_lineno_out (abfd, &(line_ptr->line), dst);
              dst++;
-
            }
 
          s->scnhdr.s_lnnoptr = *file_cursor;
 
-         bfd_write (buffer, 1, s->scnhdr.s_nlnno * LINESZ, abfd);
+         bfd_bwrite (buffer, (bfd_size_type) s->scnhdr.s_nlnno * LINESZ, abfd);
          free (buffer);
 
          *file_cursor += s->scnhdr.s_nlnno * LINESZ;
        }
     }
+
   H_SET_LINENO_SIZE (h, *file_cursor - start);
 }
 
-
 /* Now we run through the list of frag chains in a segment and
    make all the subsegment frags appear at the end of the
-   list, as if the seg 0 was extra long */
+   list, as if the seg 0 was extra long */
 
 static void
 remove_subsegs ()
@@ -3201,6 +3421,20 @@ remove_subsegs ()
 
 unsigned long machine;
 int coff_flags;
+
+#ifndef SUB_SEGMENT_ALIGN
+#ifdef HANDLE_ALIGN
+/* The last subsegment gets an aligment corresponding to the alignment
+   of the section.  This allows proper nop-filling at the end of
+   code-bearing sections.  */
+#define SUB_SEGMENT_ALIGN(SEG, FRCHAIN)                                        \
+  (!(FRCHAIN)->frch_next || (FRCHAIN)->frch_next->frch_seg != (SEG)    \
+   ? get_recorded_alignment (SEG) : 0)
+#else
+#define SUB_SEGMENT_ALIGN(SEG, FRCHAIN) 1
+#endif
+#endif
+
 extern void
 write_object_file ()
 {
@@ -3214,7 +3448,6 @@ write_object_file ()
   unsigned int addr;
   abfd = bfd_openw (out_file_name, TARGET_FORMAT);
 
-
   if (abfd == 0)
     {
       as_perror (_("FATAL: Can't create %s"), out_file_name);
@@ -3225,43 +3458,44 @@ write_object_file ()
 
   string_byte_count = 4;
 
+  /* Run through all the sub-segments and align them up.  Also
+     close any open frags.  We tack a .fill onto the end of the
+     frag chain so that any .align's size can be worked by looking
+     at the next frag.  */
   for (frchain_ptr = frchain_root;
        frchain_ptr != (struct frchain *) NULL;
        frchain_ptr = frchain_ptr->frch_next)
     {
-      /* Run through all the sub-segments and align them up.  Also
-        close any open frags.  We tack a .fill onto the end of the
-        frag chain so that any .align's size can be worked by looking
-        at the next frag.  */
+      int alignment;
 
       subseg_set (frchain_ptr->frch_seg, frchain_ptr->frch_subseg);
 
-#ifndef SUB_SEGMENT_ALIGN
-#define SUB_SEGMENT_ALIGN(SEG) 1
-#endif
+      alignment = SUB_SEGMENT_ALIGN (now_seg, frchain_ptr);
+
 #ifdef md_do_align
-      md_do_align (SUB_SEGMENT_ALIGN (now_seg), (char *) NULL, 0, 0,
-                  alignment_done);
+      md_do_align (alignment, (char *) NULL, 0, 0, alignment_done);
 #endif
-      frag_align (SUB_SEGMENT_ALIGN (now_seg),
-                 subseg_text_p (now_seg) ? NOP_OPCODE : 0,
-                 0);
+      if (subseg_text_p (now_seg))
+       frag_align_code (alignment, 0);
+      else
+       frag_align (alignment, 0, 0);
+
 #ifdef md_do_align
     alignment_done:
 #endif
+
       frag_wane (frag_now);
       frag_now->fr_fix = 0;
       know (frag_now->fr_next == NULL);
     }
 
-
   remove_subsegs ();
 
-
   for (i = SEG_E0; i < SEG_UNKNOWN; i++)
-    {
-      relax_segment (segment_info[i].frchainP->frch_root, i);
-    }
+    relax_segment (segment_info[i].frchainP->frch_root, i);
+
+  /* Relaxation has completed.  Freeze all syms.  */
+  finalize_syms = 1;
 
   H_SET_NUMBER_OF_SECTIONS (&headers, 0);
 
@@ -3312,7 +3546,7 @@ write_object_file ()
        H_SET_BSS_SIZE (&headers, size);
     }
 
-  /* Turn the gas native symbol table shape into a coff symbol table */
+  /* Turn the gas native symbol table shape into a coff symbol table */
   crawl_symbols (&headers, abfd);
 
   if (string_byte_count == 4)
@@ -3331,7 +3565,7 @@ write_object_file ()
     }
 
   /* Look for ".stab" segments and fill in their initial symbols
-     correctly. */
+     correctly.  */
   for (i = SEG_E0; i < SEG_UNKNOWN; i++)
     {
       name = segment_info[i].name;
@@ -3346,8 +3580,7 @@ write_object_file ()
 
   bfd_seek (abfd, (file_ptr) file_cursor, 0);
 
-  /* Plant the data */
-
+  /* Plant the data.  */
   fill_section (abfd, &headers, &file_cursor);
 
   do_relocs_for (abfd, &headers, &file_cursor);
@@ -3385,14 +3618,15 @@ write_object_file ()
     w_symbols (abfd, buffer1, symbol_rootP);
     if (string_byte_count > 0)
       w_strings (buffer1 + symtable_size);
-    bfd_write (buffer1, 1, symtable_size + string_byte_count, abfd);
+    bfd_bwrite (buffer1, (bfd_size_type) symtable_size + string_byte_count,
+               abfd);
     free (buffer1);
   }
 
   coff_header_append (abfd, &headers);
 #if 0
   /* Recent changes to write need this, but where it should
-     go is up to Ken.. */
+     go is up to Ken..  */
   if (bfd_close_all_done (abfd) == false)
     as_fatal (_("Can't close %s: %s"), out_file_name,
              bfd_errmsg (bfd_get_error ()));
@@ -3441,28 +3675,26 @@ obj_coff_add_segment (name)
   return (segT) i;
 }
 
-/*
- * implement the .section pseudo op:
- *     .section name {, "flags"}
- *                ^         ^
- *                |         +--- optional flags: 'b' for bss
- *                |                              'i' for info
- *                +-- section name               'l' for lib
- *                                               'n' for noload
- *                                               'o' for over
- *                                               'w' for data
- *                                              'd' (apparently m88k for data)
- *                                               'x' for text
- *                                              'r' for read-only data
- * But if the argument is not a quoted string, treat it as a
- * subsegment number.
- */
+/* Implement the .section pseudo op:
+       .section name {, "flags"}
+                  ^         ^
+                  |         +--- optional flags: 'b' for bss
+                  |                              'i' for info
+                  +-- section name               'l' for lib
+                                                 'n' for noload
+                                                 'o' for over
+                                                 'w' for data
+                                                'd' (apparently m88k for data)
+                                                 'x' for text
+                                                'r' for read-only data
+   But if the argument is not a quoted string, treat it as a
+   subsegment number.  */
 
 void
 obj_coff_section (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
-  /* Strip out the section name */
+  /* Strip out the section name */
   char *section_name, *name;
   char c;
   unsigned int exp;
@@ -3538,18 +3770,16 @@ obj_coff_section (ignore)
   demand_empty_rest_of_line ();
 }
 
-
 static void
 obj_coff_text (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   subseg_new (".text", get_absolute_expression ());
 }
 
-
 static void
 obj_coff_data (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (flag_readonly_data_in_text)
     subseg_new (".text", get_absolute_expression () + 1000);
@@ -3559,13 +3789,14 @@ obj_coff_data (ignore)
 
 static void
 obj_coff_ident (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
-  segT current_seg = now_seg;          /* save current seg     */
+  segT current_seg = now_seg;          /* Save current seg.  */
   subsegT current_subseg = now_subseg;
-  subseg_new (".comment", 0);          /* .comment seg         */
-  stringer (1);                                /* read string          */
-  subseg_set (current_seg, current_subseg);    /* restore current seg  */
+
+  subseg_new (".comment", 0);          /* .comment seg.  */
+  stringer (1);                                /* Read string.  */
+  subseg_set (current_seg, current_subseg);    /* Restore current seg.  */
 }
 
 void
@@ -3577,20 +3808,16 @@ c_symbol_merge (debug, normal)
   S_SET_STORAGE_CLASS (normal, S_GET_STORAGE_CLASS (debug));
 
   if (S_GET_NUMBER_AUXILIARY (debug) > S_GET_NUMBER_AUXILIARY (normal))
-    {
-      S_SET_NUMBER_AUXILIARY (normal, S_GET_NUMBER_AUXILIARY (debug));
-    }                          /* take the most we have */
+    S_SET_NUMBER_AUXILIARY (normal, S_GET_NUMBER_AUXILIARY (debug));
 
   if (S_GET_NUMBER_AUXILIARY (debug) > 0)
-    {
-      memcpy ((char *) &normal->sy_symbol.ost_auxent[0],
-             (char *) &debug->sy_symbol.ost_auxent[0],
-             (unsigned int) (S_GET_NUMBER_AUXILIARY (debug) * AUXESZ));
-    }                          /* Move all the auxiliary information */
+    memcpy ((char *) &normal->sy_symbol.ost_auxent[0],
+           (char *) &debug->sy_symbol.ost_auxent[0],
+           (unsigned int) (S_GET_NUMBER_AUXILIARY (debug) * AUXESZ));
 
-  /* Move the debug flags. */
+  /* Move the debug flags.  */
   SF_SET_DEBUG_FIELD (normal, SF_GET_DEBUG_FIELD (debug));
-}                              /* c_symbol_merge() */
+}
 
 static int
 c_line_new (symbol, paddr, line_number, frag)
@@ -3618,15 +3845,11 @@ c_line_new (symbol, paddr, line_number, frag)
   new_line->frag = (char *) frag;
   new_line->next = (struct lineno_list *) NULL;
 
-
   if (s->lineno_list_head == (struct lineno_list *) NULL)
-    {
-      s->lineno_list_head = new_line;
-    }
+    s->lineno_list_head = new_line;
   else
-    {
-      s->lineno_list_tail->next = new_line;
-    }
+    s->lineno_list_tail->next = new_line;
+
   s->lineno_list_tail = new_line;
   return LINESZ * s->scnhdr.s_nlnno++;
 }
@@ -3650,7 +3873,7 @@ c_dot_file_symbol (filename)
       /* Filename is too long to fit into an auxent,
         we stick it into the string table instead.  We keep
         a linked list of the filenames we find so we can emit
-        them later.*/
+        them later.  */
       struct filename_list *f = ((struct filename_list *)
                                 xmalloc (sizeof (struct filename_list)));
 
@@ -3660,13 +3883,13 @@ c_dot_file_symbol (filename)
       SA_SET_FILE_FNAME_ZEROS (symbolP, 0);
       SA_SET_FILE_FNAME_OFFSET (symbolP, 1);
 
-      if (filename_list_tail) 
+      if (filename_list_tail)
        filename_list_tail->next = f;
       else
        filename_list_head = f;
-      filename_list_tail = f;      
+      filename_list_tail = f;
     }
-  else 
+  else
     {
       SA_SET_FILE_FNAME (symbolP, filename);
     }
@@ -3674,29 +3897,23 @@ c_dot_file_symbol (filename)
   {
     extern int listing;
     if (listing)
-      {
-       listing_source_file (filename);
-      }
-
+      listing_source_file (filename);
   }
-
 #endif
   SF_SET_DEBUG (symbolP);
   S_SET_VALUE (symbolP, (valueT) previous_file_symbol);
 
   previous_file_symbol = symbolP;
 
-  /* Make sure that the symbol is first on the symbol chain */
+  /* Make sure that the symbol is first on the symbol chain */
   if (symbol_rootP != symbolP)
     {
       symbol_remove (symbolP, &symbol_rootP, &symbol_lastP);
       symbol_insert (symbolP, symbol_rootP, &symbol_rootP, &symbol_lastP);
     }
-}                              /* c_dot_file_symbol() */
+}
 
-/*
- * Build a 'section static' symbol.
- */
+/* Build a 'section static' symbol.  */
 
 symbolS *
 c_section_symbol (name, idx)
@@ -3756,7 +3973,7 @@ c_section_symbol (name, idx)
 #endif /* TE_PE */
 
   return symbolP;
-}                              /* c_section_symbol() */
+}
 
 static void
 w_symbols (abfd, where, symbol_rootP)
@@ -3767,7 +3984,7 @@ w_symbols (abfd, where, symbol_rootP)
   symbolS *symbolP;
   unsigned int i;
 
-  /* First fill in those values we have only just worked out */
+  /* First fill in those values we have only just worked out */
   for (i = SEG_E0; i < SEG_LAST; i++)
     {
       symbolP = segment_info[i].dot;
@@ -3779,23 +3996,19 @@ w_symbols (abfd, where, symbol_rootP)
        }
     }
 
-  /*
-     * Emit all symbols left in the symbol chain.
-     */
+  /* Emit all symbols left in the symbol chain.  */
   for (symbolP = symbol_rootP; symbolP; symbolP = symbol_next (symbolP))
     {
       /* Used to save the offset of the name. It is used to point
-              to the string in memory but must be a file offset. */
-      register char *temp;
+        to the string in memory but must be a file offset.  */
+      char *temp;
 
       /* We can't fix the lnnoptr field in yank_symbols with the other
          adjustments, because we have to wait until we know where they
          go in the file.  */
       if (SF_GET_ADJ_LNNOPTR (symbolP))
-       {
-         SA_GET_SYM_LNNOPTR (symbolP) +=
-           segment_info[S_GET_SEGMENT (symbolP)].scnhdr.s_lnnoptr;
-       }
+       SA_GET_SYM_LNNOPTR (symbolP) +=
+         segment_info[S_GET_SEGMENT (symbolP)].scnhdr.s_lnnoptr;
 
       tc_coff_symbol_emit_hook (symbolP);
 
@@ -3813,12 +4026,11 @@ w_symbols (abfd, where, symbol_rootP)
       where = symbol_to_chars (abfd, where, symbolP);
       S_SET_NAME (symbolP, temp);
     }
-
-}                              /* w_symbols() */
+}
 
 static void
 obj_coff_lcomm (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   s_lcomm(0);
   return;
@@ -3856,15 +4068,15 @@ obj_coff_lcomm (ignore)
     }
   *p = 0;
 
-  symbolP = symbol_find_or_make(name);
+  symbolP = symbol_find_or_make (name);
 
-  if (S_GET_SEGMENT(symbolP) == SEG_UNKNOWN &&
-      S_GET_VALUE(symbolP) == 0)
+  if (S_GET_SEGMENT (symbolP) == SEG_UNKNOWN &&
+      S_GET_VALUE (symbolP) == 0)
     {
       if (! need_pass_2)
        {
          char *p;
-         segT current_seg = now_seg;   /* save current seg     */
+         segT current_seg = now_seg;   /* Save current seg.  */
          subsegT current_subseg = now_subseg;
 
          subseg_set (SEG_E2, 1);
@@ -3872,15 +4084,15 @@ obj_coff_lcomm (ignore)
          p = frag_var(rs_org, 1, 1, (relax_substateT)0, symbolP,
                       (offsetT) temp, (char *) 0);
          *p = 0;
-         subseg_set (current_seg, current_subseg); /* restore current seg */
-         S_SET_SEGMENT(symbolP, SEG_E2);
-         S_SET_STORAGE_CLASS(symbolP, C_STAT);
+         subseg_set (current_seg, current_subseg); /* Restore current seg.  */
+         S_SET_SEGMENT (symbolP, SEG_E2);
+         S_SET_STORAGE_CLASS (symbolP, C_STAT);
        }
     }
   else
-    as_bad(_("Symbol %s already defined"), name);
+    as_bad (_("Symbol %s already defined"), name);
 
-  demand_empty_rest_of_line();
+  demand_empty_rest_of_line ();
 #endif
 }
 
@@ -3891,12 +4103,14 @@ fixup_mdeps (frags, h, this_segment)
      segT this_segment;
 {
   subseg_change (this_segment, 0);
+
   while (frags)
     {
       switch (frags->fr_type)
        {
        case rs_align:
        case rs_align_code:
+       case rs_align_test:
        case rs_org:
 #ifdef HANDLE_ALIGN
          HANDLE_ALIGN (frags);
@@ -3928,16 +4142,16 @@ fixup_segment (segP, this_segment_type)
      segment_info_type * segP;
      segT this_segment_type;
 {
-  register fixS * fixP;
-  register symbolS *add_symbolP;
-  register symbolS *sub_symbolP;
+  fixS * fixP;
+  symbolS *add_symbolP;
+  symbolS *sub_symbolP;
   long add_number;
-  register int size;
-  register char *place;
-  register long where;
-  register char pcrel;
-  register fragS *fragP;
-  register segT add_symbol_segment = absolute_section;
+  int size;
+  char *place;
+  long where;
+  char pcrel;
+  fragS *fragP;
+  segT add_symbol_segment = absolute_section;
 
   for (fixP = segP->fix_root; fixP; fixP = fixP->fx_next)
     {
@@ -3967,7 +4181,7 @@ fixup_segment (segP, this_segment_type)
       if (fixP->fx_tcbit && SF_GET_CALLNAME (add_symbolP))
        {
          /* Relocation should be done via the associated 'bal' entry
-            point symbol. */
+            point symbol.  */
 
          if (!SF_GET_BALNAME (tc_get_bal_of_call (add_symbolP)))
            {
@@ -3983,7 +4197,7 @@ fixup_segment (segP, this_segment_type)
       /* Make sure the symbols have been resolved; this may not have
          happened if these are expression symbols.  */
       if (add_symbolP != NULL && ! add_symbolP->sy_resolved)
-       resolve_symbol_value (add_symbolP, 1);
+       resolve_symbol_value (add_symbolP);
 
       if (add_symbolP != NULL)
        {
@@ -4013,7 +4227,7 @@ fixup_segment (segP, this_segment_type)
        }
 
       if (sub_symbolP != NULL && ! sub_symbolP->sy_resolved)
-       resolve_symbol_value (sub_symbolP, 1);
+       resolve_symbol_value (sub_symbolP);
 
       if (add_symbolP != NULL
          && add_symbolP->sy_mri_common)
@@ -4025,9 +4239,7 @@ fixup_segment (segP, this_segment_type)
        }
 
       if (add_symbolP)
-       {
-         add_symbol_segment = S_GET_SEGMENT (add_symbolP);
-       }                       /* if there is an addend */
+       add_symbol_segment = S_GET_SEGMENT (add_symbolP);
 
       if (sub_symbolP)
        {
@@ -4058,22 +4270,20 @@ fixup_segment (segP, this_segment_type)
                }               /* not absolute */
 
              /* if sub_symbol is in the same segment that add_symbol
-                and add_symbol is either in DATA, TEXT, BSS or ABSOLUTE */
+                and add_symbol is either in DATA, TEXT, BSS or ABSOLUTE */
            }
          else if (S_GET_SEGMENT (sub_symbolP) == add_symbol_segment
                   && SEG_NORMAL (add_symbol_segment))
            {
              /* Difference of 2 symbols from same segment.  Can't
                 make difference of 2 undefineds: 'value' means
-                something different for N_UNDF. */
+                something different for N_UNDF.  */
 #ifdef TC_I960
              /* Makes no sense to use the difference of 2 arbitrary symbols
                 as the target of a call instruction.  */
              if (fixP->fx_tcbit)
-               {
-                 as_bad_where (fixP->fx_file, fixP->fx_line,
-                               _("callj to difference of 2 symbols"));
-               }
+               as_bad_where (fixP->fx_file, fixP->fx_line,
+                             _("callj to difference of 2 symbols"));
 #endif /* TC_I960 */
              add_number += S_GET_VALUE (add_symbolP) -
                S_GET_VALUE (sub_symbolP);
@@ -4092,16 +4302,15 @@ fixup_segment (segP, this_segment_type)
            }
          else
            {
-             /* Different segments in subtraction. */
+             /* Different segments in subtraction.  */
              know (!(S_IS_EXTERNAL (sub_symbolP) && (S_GET_SEGMENT (sub_symbolP) == absolute_section)));
 
              if ((S_GET_SEGMENT (sub_symbolP) == absolute_section))
-               {
-                 add_number -= S_GET_VALUE (sub_symbolP);
-               }
+               add_number -= S_GET_VALUE (sub_symbolP);
+
 #ifdef DIFF_EXPR_OK
              else if (S_GET_SEGMENT (sub_symbolP) == this_segment_type
-#if 0 /* Okay for 68k, at least... */
+#if 0 /* Okay for 68k, at least...  */
                       && !pcrel
 #endif
                       )
@@ -4122,25 +4331,22 @@ fixup_segment (segP, this_segment_type)
                                segment_name (S_GET_SEGMENT (sub_symbolP)),
                                S_GET_NAME (sub_symbolP),
                                (long) (fragP->fr_address + where));
-               }               /* if absolute */
+               }
            }
-       }                       /* if sub_symbolP */
+       }
 
       if (add_symbolP)
        {
          if (add_symbol_segment == this_segment_type && pcrel)
            {
-             /*
-              * This fixup was made when the symbol's segment was
-              * SEG_UNKNOWN, but it is now in the local segment.
-              * So we know how to do the address without relocation.
-              */
+             /* This fixup was made when the symbol's segment was
+                SEG_UNKNOWN, but it is now in the local segment.
+                So we know how to do the address without relocation.  */
 #ifdef TC_I960
              /* reloc_callj() may replace a 'call' with a 'calls' or a 'bal',
-              * in which cases it modifies *fixP as appropriate.  In the case
-              * of a 'calls', no further work is required, and *fixP has been
-              * set up to make the rest of the code below a no-op.
-              */
+                in which cases it modifies *fixP as appropriate.  In the case
+                of a 'calls', no further work is required, and *fixP has been
+                set up to make the rest of the code below a no-op.  */
              reloc_callj (fixP);
 #endif /* TC_I960 */
 
@@ -4161,7 +4367,7 @@ fixup_segment (segP, this_segment_type)
                 the i386 are relaxed, this could only arise with a
                 call instruction.  */
 
-             pcrel = 0;        /* Lie. Don't want further pcrel processing. */
+             pcrel = 0;        /* Lie. Don't want further pcrel processing.  */
              if (!TC_FORCE_RELOCATION (fixP))
                {
                  fixP->fx_addsy = NULL;
@@ -4174,7 +4380,8 @@ fixup_segment (segP, this_segment_type)
                {
                case absolute_section:
 #ifdef TC_I960
-                 reloc_callj (fixP);   /* See comment about reloc_callj() above*/
+                 /* See comment about reloc_callj() above.  */
+                 reloc_callj (fixP);
 #endif /* TC_I960 */
                  add_number += S_GET_VALUE (add_symbolP);
                  add_symbolP = NULL;
@@ -4187,8 +4394,7 @@ fixup_segment (segP, this_segment_type)
                  break;
                default:
 
-
-#if defined(TC_A29K) || (defined(TE_PE) && defined(TC_I386)) || defined(TC_M88K)
+#if defined(TC_A29K) || (defined(TE_PE) && defined(TC_I386)) || defined(TC_M88K) || defined(TC_OR32)
                  /* This really should be handled in the linker, but
                     backward compatibility forbids.  */
                  add_number += S_GET_VALUE (add_symbolP);
@@ -4203,16 +4409,15 @@ fixup_segment (segP, this_segment_type)
                  if ((int) fixP->fx_bit_fixP == 13)
                    {
                      /* This is a COBR instruction.  They have only a
-                      * 13-bit displacement and are only to be used
-                      * for local branches: flag as error, don't generate
-                      * relocation.
-                      */
+                        13-bit displacement and are only to be used
+                        for local branches: flag as error, don't generate
+                        relocation.  */
                      as_bad_where (fixP->fx_file, fixP->fx_line,
                                    _("can't use COBR format with external label"));
                      fixP->fx_addsy = NULL;
                      fixP->fx_done = 1;
                      continue;
-                   }           /* COBR */
+                   }
 #endif /* TC_I960 */
 #if ((defined (TC_I386) || defined (TE_LYNX) || defined (TE_AUX)) && !defined(TE_PE)) || defined (COFF_COMMON_ADDEND)
                  /* 386 COFF uses a peculiar format in which the
@@ -4222,28 +4427,25 @@ fixup_segment (segP, this_segment_type)
                  /* This is also true for 68k COFF on sysv machines
                     (Checked on Motorola sysv68 R3V6 and R3V7.1, and also on
                     UNIX System V/M68000, Release 1.0 from ATT/Bell Labs)
-                    Philippe De Muyter <phdm@info.ucl.ac.be>. */
+                    Philippe De Muyter <phdm@info.ucl.ac.be>.  */
                  if (S_IS_COMMON (add_symbolP))
                    add_number += S_GET_VALUE (add_symbolP);
 #endif
                  break;
 
-
-               }               /* switch on symbol seg */
-           }                   /* if not in local seg */
-       }                       /* if there was a + symbol */
+               }
+           }
+       }
 
       if (pcrel)
        {
-#if !defined(TC_M88K) && !(defined(TE_PE) && defined(TC_I386)) && !defined(TC_A29K)
+#if !defined(TC_M88K) && !(defined(TE_PE) && defined(TC_I386)) && !defined(TC_A29K) && !defined(TC_OR32)
          /* This adjustment is not correct on the m88k, for which the
             linker does all the computation.  */
          add_number -= md_pcrel_from (fixP);
 #endif
          if (add_symbolP == 0)
-           {
-             fixP->fx_addsy = &abs_symbol;
-           }                   /* if there's an add_symbol */
+           fixP->fx_addsy = &abs_symbol;
 #if defined (TC_I386) || defined (TE_LYNX) || defined (TC_I960) || defined (TC_M68K)
          /* On the 386 we must adjust by the segment vaddr as well.
             Ian Taylor.
@@ -4263,13 +4465,9 @@ fixup_segment (segP, this_segment_type)
 
          add_number -= segP->scnhdr.s_vaddr;
 #endif
-       }                       /* if pcrel */
+       }
 
-#ifdef MD_APPLY_FIX3
-      md_apply_fix3 (fixP, (valueT *) &add_number, this_segment_type);
-#else
-      md_apply_fix (fixP, add_number);
-#endif
+      md_apply_fix3 (fixP, (valueT *) & add_number, this_segment_type);
 
       if (!fixP->fx_bit_fixP && ! fixP->fx_no_overflow)
        {
@@ -4306,9 +4504,9 @@ fixup_segment (segP, this_segment_type)
                          (long) add_number,
                          (unsigned long) (fragP->fr_address + where));
 #endif
-       }                       /* not a bit fix */
-    }                          /* For each fixS in this segment. */
-}                              /* fixup_segment() */
+       }
+    }
+}
 
 #endif
 
@@ -4323,9 +4521,9 @@ obj_coff_init_stab_section (seg)
   char *stabstr_name;
   unsigned int stroff;
 
-  /* Make space for this first symbol. */
+  /* Make space for this first symbol.  */
   p = frag_more (12);
-  /* Zero it out. */
+  /* Zero it out.  */
   memset (p, 0, 12);
   as_where (&file, (unsigned int *) NULL);
   stabstr_name = (char *) alloca (strlen (segment_info[seg].name) + 4);
@@ -4350,7 +4548,7 @@ adjust_stab_section(abfd, seg)
   int i, strsz = 0, nsyms;
   fragS *frag = segment_info[seg].frchainP->frch_root;
 
-  /* Look for the associated string table section. */
+  /* Look for the associated string table section.  */
 
   secname = segment_info[seg].name;
   name = (char *) alloca (strlen (secname) + 4);
@@ -4367,14 +4565,14 @@ adjust_stab_section(abfd, seg)
        }
     }
 
-  /* If we found the section, get its size. */
+  /* If we found the section, get its size.  */
   if (stabstrseg != SEG_UNKNOWN)
     strsz = size_section (abfd, stabstrseg);
 
   nsyms = size_section (abfd, seg) / 12 - 1;
 
   /* Look for the first frag of sufficient size for the initial stab
-     symbol, and collect a pointer to it. */
+     symbol, and collect a pointer to it.  */
   while (frag && frag->fr_fix < 12)
     frag = frag->fr_next;
   assert (frag != 0);
@@ -4382,20 +4580,23 @@ adjust_stab_section(abfd, seg)
   assert (p != 0);
 
   /* Write in the number of stab symbols and the size of the string
-     table. */
+     table.  */
   bfd_h_put_16 (abfd, (bfd_vma) nsyms, (bfd_byte *) p + 6);
   bfd_h_put_32 (abfd, (bfd_vma) strsz, (bfd_byte *) p + 8);
 }
 
 #endif /* not BFD_ASSEMBLER */
 
-const pseudo_typeS obj_pseudo_table[] =
+const pseudo_typeS coff_pseudo_table[] =
 {
   {"def", obj_coff_def, 0},
   {"dim", obj_coff_dim, 0},
   {"endef", obj_coff_endef, 0},
   {"line", obj_coff_line, 0},
   {"ln", obj_coff_ln, 0},
+#ifdef BFD_ASSEMBLER
+  {"loc", obj_coff_loc, 0},
+#endif
   {"appline", obj_coff_ln, 1},
   {"scl", obj_coff_scl, 0},
   {"size", obj_coff_size, 0},
@@ -4411,78 +4612,74 @@ const pseudo_typeS obj_pseudo_table[] =
      earlier versions of gas.  */
   {"bss", obj_coff_bss, 0},
   {"weak", obj_coff_weak, 0},
+  {"ident", obj_coff_ident, 0},
 #ifndef BFD_ASSEMBLER
   {"use", obj_coff_section, 0},
   {"text", obj_coff_text, 0},
   {"data", obj_coff_data, 0},
   {"lcomm", obj_coff_lcomm, 0},
-  {"ident", obj_coff_ident, 0},
 #else
   {"optim", s_ignore, 0},      /* For sun386i cc (?) */
-  {"ident", s_ignore, 0},      /* we don't yet handle this. */
 #endif
   {"version", s_ignore, 0},
   {"ABORT", s_abort, 0},
-#ifdef TC_M88K
-  /* The m88k uses sdef instead of def.  */
+#if defined( TC_M88K ) || defined ( TC_TIC4X )
+  /* The m88k and tic4x uses sdef instead of def.  */
   {"sdef", obj_coff_def, 0},
 #endif
-  {NULL}                       /* end sentinel */
-};                             /* obj_pseudo_table */
+  {NULL, NULL, 0}              /* end sentinel */
+};                             /* coff_pseudo_table */
 \f
 #ifdef BFD_ASSEMBLER
 
 /* Support for a COFF emulation.  */
 
+static void coff_pop_insert PARAMS ((void));
+static int coff_separate_stab_sections PARAMS ((void));
+
 static void
 coff_pop_insert ()
 {
-  pop_insert (obj_pseudo_table);
+  pop_insert (coff_pseudo_table);
 }
 
 static int
-coff_sec_sym_ok_for_reloc (sec)
-     asection *sec;
-{
-  return 0;
-}
-
-static void
-no_func ()
+coff_separate_stab_sections ()
 {
-  abort ();
+  return 1;
 }
 
 const struct format_ops coff_format_ops =
 {
   bfd_target_coff_flavour,
-  0,
-  1,
+  0,   /* dfl_leading_underscore */
+  1,   /* emit_section_symbols */
+  0,    /* begin */
+  c_dot_file_symbol,
   coff_frob_symbol,
-  no_func,
+  0,   /* frob_file */
+  0,   /* frob_file_before_adjust */
   coff_frob_file_after_relocs,
-  0, 0,
-  0, 0,
-  0,
-#if 0
-  obj_generate_asm_lineno,
-#else
-  no_func,
-#endif
-#if 0
-  obj_stab,
-#else
-  no_func,
-#endif
-  coff_sec_sym_ok_for_reloc,
+  0,   /* s_get_size */
+  0,   /* s_set_size */
+  0,   /* s_get_align */
+  0,   /* s_set_align */
+  0,   /* s_get_other */
+  0,   /* s_set_other */
+  0,   /* s_get_desc */
+  0,   /* s_set_desc */
+  0,   /* s_get_type */
+  0,   /* s_set_type */
+  0,   /* copy_symbol_attributes */
+  0,   /* generate_asm_lineno */
+  0,   /* process_stab */
+  coff_separate_stab_sections,
+  obj_coff_init_stab_section,
+  0,   /* sec_sym_ok_for_reloc */
   coff_pop_insert,
-#if 0
-  obj_set_ext,
-#else
-  no_func,
-#endif
+  0,   /* ecoff_set_ext */
   coff_obj_read_begin_hook,
-  coff_obj_symbol_new_hook,
+  coff_obj_symbol_new_hook
 };
 
 #endif
This page took 0.065596 seconds and 4 git commands to generate.