* deffilep.y (def_import): Use default extension of "dll"
[deliverable/binutils-gdb.git] / gas / read.c
index c9e4461b6c5548224b7ac93421fccd76a5ee211e..f8d5d7e0066d4964e3e43fede782d38aa21ad01c 100644 (file)
@@ -1,6 +1,6 @@
 /* read.c - read a source file -
-   Copyright (C) 1986, 87, 90, 91, 92, 93, 94, 95, 96, 97, 1998
-   Free Software Foundation, Inc.
+   Copyright 1986, 1987, 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997,
+   1998, 1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
 
 This file is part of GAS, the GNU Assembler.
 
@@ -17,31 +17,26 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with GAS; see the file COPYING.  If not, write to the Free
 Software Foundation, 59 Temple Place - Suite 330, Boston, MA
-02111-1307, USA. */
+02111-1307, USA.  */
 
 #if 0
-#define MASK_CHAR (0xFF)       /* If your chars aren't 8 bits, you will
-                                  change this a bit.  But then, GNU isn't
-                                  spozed to run on your machine anyway.
-                                  (RMS is so shortsighted sometimes.)
-                                  */
+/* If your chars aren't 8 bits, you will change this a bit.
+   But then, GNU isn't spozed to run on your machine anyway.
+   (RMS is so shortsighted sometimes.)  */
+#define MASK_CHAR (0xFF)
 #else
-#define MASK_CHAR ((int)(unsigned char)-1)
+#define MASK_CHAR ((int)(unsigned char) -1)
 #endif
 
-
 /* This is the largest known floating point format (for now). It will
-   grow when we do 4361 style flonums. */
-
+   grow when we do 4361 style flonums.  */
 #define MAXIMUM_NUMBER_OF_CHARS_FOR_FLOAT (16)
 
 /* Routines that read assembler source text to build spagetti in memory.
    Another group of these functions is in the expr.c module.  */
 
-/* for isdigit() */
-#include <ctype.h>
-
 #include "as.h"
+#include "safe-ctype.h"
 #include "subsegs.h"
 #include "sb.h"
 #include "macro.h"
@@ -50,18 +45,27 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 #include "ecoff.h"
 
 #ifndef TC_START_LABEL
-#define TC_START_LABEL(x,y) (x==':')
+#define TC_START_LABEL(x,y) (x == ':')
 #endif
 
-/* The NOP_OPCODE is for the alignment fill value.
- * fill it a nop instruction so that the disassembler does not choke
- * on it
- */
-#ifndef NOP_OPCODE
-#define NOP_OPCODE 0x00
+/* Set by the object-format or the target.  */
+#ifndef TC_IMPLICIT_LCOMM_ALIGNMENT
+#define TC_IMPLICIT_LCOMM_ALIGNMENT(SIZE, P2VAR)               \
+  do                                                           \
+    {                                                          \
+      if ((SIZE) >= 8)                                         \
+       (P2VAR) = 3;                                            \
+      else if ((SIZE) >= 4)                                    \
+       (P2VAR) = 2;                                            \
+      else if ((SIZE) >= 2)                                    \
+       (P2VAR) = 1;                                            \
+      else                                                     \
+       (P2VAR) = 0;                                            \
+    }                                                          \
+  while (0)
 #endif
 
-char *input_line_pointer;      /*->next char of source file to parse. */
+char *input_line_pointer;      /*->next char of source file to parse.  */
 
 #if BITS_PER_CHAR != 8
 /*  The following table is indexed by[(char)] and will break if
@@ -89,6 +93,12 @@ die horribly;
 #define LEX_QM 0
 #endif
 
+#ifndef LEX_HASH
+/* The IA-64 assembler uses # as a suffix designating a symbol.  We include
+   it in the symbol and strip it out in tc_canonicalize_symbol_name.  */
+#define LEX_HASH 0
+#endif
+
 #ifndef LEX_DOLLAR
 /* The a29k assembler does not permits labels to start with $.  */
 #define LEX_DOLLAR 3
@@ -99,78 +109,79 @@ die horribly;
 #define LEX_TILDE 0
 #endif
 
-/* used by is_... macros. our ctype[] */
-char lex_type[256] =
-{
+/* Used by is_... macros. our ctype[].  */
+char lex_type[256] = {
   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* @ABCDEFGHIJKLMNO */
   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* PQRSTUVWXYZ[\]^_ */
-  0, 0, 0, 0, LEX_DOLLAR, LEX_PCT, 0, 0, 0, 0, 0, 0, 0, 0, 3, 0, /* _!"#$%&'()*+,-./ */
+  0, 0, 0, LEX_HASH, LEX_DOLLAR, LEX_PCT, 0, 0, 0, 0, 0, 0, 0, 0, 3, 0, /* _!"#$%&'()*+,-./ */
   1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, LEX_QM, /* 0123456789:;<=>? */
   LEX_AT, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, /* @ABCDEFGHIJKLMNO */
   3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, LEX_BR, 0, LEX_BR, 0, 3, /* PQRSTUVWXYZ[\]^_ */
   0, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,      /* `abcdefghijklmno */
-  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, LEX_BR, 0, LEX_BR, LEX_TILDE, 0, /* pqrstuvwxyz{|}~. */
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, LEX_BR, 0, LEX_BR, LEX_TILDE, 0, /* pqrstuvwxyz{|}~.  */
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
+  3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3
 };
 
-
-/*
- * In: a character.
- * Out: 1 if this character ends a line.
- */
-#define Z_ (0)
-char is_end_of_line[256] =
-{
+/* In: a character.
+   Out: 1 if this character ends a line.  */
+char is_end_of_line[256] = {
 #ifdef CR_EOL
-  99, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, 99, Z_, Z_, 99, Z_, Z_,      /* @abcdefghijklmno */
+  1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 1, 0, 0,      /* @abcdefghijklmno */
 #else
-  99, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, 99, Z_, Z_, Z_, Z_, Z_,      /* @abcdefghijklmno */
+  1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0,      /* @abcdefghijklmno */
 #endif
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-#ifdef TC_HPPA
-  Z_,99, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,       /* _!"#$%&'()*+,-./ */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* 0123456789:;<=>? */
-#else
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, 99, Z_, Z_, Z_, Z_,      /* 0123456789:;<=>? */
-#endif
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
-  Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_, Z_,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* _!"#$%&'()*+,-./ */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* 0123456789:;<=>? */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,      /* */
+  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0       /* */
 };
-#undef Z_
 
-/* Functions private to this file. */
+#ifdef  IGNORE_OPCODE_CASE
+char original_case_string[128];
+#endif
+
+/* Functions private to this file.  */
 
-static char *buffer;   /* 1st char of each buffer of lines is here. */
-static char *buffer_limit;     /*->1 + last char in buffer. */
+static char *buffer;   /* 1st char of each buffer of lines is here.  */
+static char *buffer_limit;     /*->1 + last char in buffer.  */
 
-/* TARGET_BYTES_BIG_ENDIAN is required to be defined to either 0 or 1 in the
-   tc-<CPU>.h file.  See the "Porting GAS" section of the internals manual. */
+/* TARGET_BYTES_BIG_ENDIAN is required to be defined to either 0 or 1
+   in the tc-<CPU>.h file.  See the "Porting GAS" section of the
+   internals manual.  */
 int target_big_endian = TARGET_BYTES_BIG_ENDIAN;
 
-static char *old_buffer;       /* JF a hack */
+static char *old_buffer;       /* JF a hack */
 static char *old_input;
 static char *old_limit;
 
-/* Variables for handling include file directory table. */
+/* Variables for handling include file directory table.  */
 
-char **include_dirs;   /* Table of pointers to directories to
-                          search for .include's */
-int include_dir_count; /* How many are in the table */
-int include_dir_maxlen = 1;/* Length of longest in table */
+/* Table of pointers to directories to search for .include's.  */
+char **include_dirs;
+
+/* How many are in the table.  */
+int include_dir_count;
+
+/* Length of longest in table.  */
+int include_dir_maxlen = 1;
 
 #ifndef WORKING_DOT_WORD
 struct broken_word *broken_words;
@@ -207,10 +218,18 @@ static int dwarf_file_string;
 #endif
 
 static void cons_worker PARAMS ((int, int));
-static int scrub_from_string PARAMS ((char **));
+static int scrub_from_string PARAMS ((char *, int));
 static void do_align PARAMS ((int, char *, int, int));
 static void s_align PARAMS ((int, int));
+static void s_lcomm_internal PARAMS ((int, int));
 static int hex_float PARAMS ((int, char *));
+static inline int sizeof_sleb128 PARAMS ((offsetT));
+static inline int sizeof_uleb128 PARAMS ((valueT));
+static inline int output_sleb128 PARAMS ((char *, offsetT));
+static inline int output_uleb128 PARAMS ((char *, valueT));
+static inline int output_big_sleb128 PARAMS ((char *, LITTLENUM_TYPE *, int));
+static inline int output_big_uleb128 PARAMS ((char *, LITTLENUM_TYPE *, int));
+static int output_big_leb128 PARAMS ((char *, LITTLENUM_TYPE *, int, int));
 static void do_org PARAMS ((segT, expressionS *, int));
 char *demand_copy_string PARAMS ((int *lenP));
 static segT get_segmented_expression PARAMS ((expressionS *expP));
@@ -218,9 +237,7 @@ static segT get_known_segmented_expression PARAMS ((expressionS * expP));
 static void pobegin PARAMS ((void));
 static int get_line_sb PARAMS ((sb *));
 static void generate_file_debug PARAMS ((void));
-static void generate_lineno_debug PARAMS ((void));
 \f
-
 void
 read_begin ()
 {
@@ -234,21 +251,20 @@ read_begin ()
   obstack_begin (&notes, chunksize);
   obstack_begin (&cond_obstack, chunksize);
 
-  /* Use machine dependent syntax */
+  /* Use machine dependent syntax */
   for (p = line_separator_chars; *p; p++)
     is_end_of_line[(unsigned char) *p] = 1;
-  /* Use more.  FIXME-SOMEDAY. */
+  /* Use more.  FIXME-SOMEDAY.  */
 
   if (flag_mri)
     lex_type['?'] = 3;
 }
 \f
-/* set up pseudo-op tables */
+/* Set up pseudo-op tables.  */
 
 static struct hash_control *po_hash;
 
-static const pseudo_typeS potable[] =
-{
+static const pseudo_typeS potable[] = {
   {"abort", s_abort, 0},
   {"align", s_align_ptwo, 0},
   {"ascii", stringer, 0},
@@ -256,7 +272,7 @@ static const pseudo_typeS potable[] =
   {"balign", s_align_bytes, 0},
   {"balignw", s_align_bytes, -2},
   {"balignl", s_align_bytes, -4},
-/* block */
+/* block  */
   {"byte", cons, 1},
   {"comm", s_comm, 0},
   {"common", s_mri_common, 0},
@@ -288,22 +304,25 @@ static const pseudo_typeS potable[] =
 #ifdef S_SET_DESC
   {"desc", s_desc, 0},
 #endif
-/* dim */
+/* dim  */
   {"double", float_cons, 'd'},
-/* dsect */
-  {"eject", listing_eject, 0}, /* Formfeed listing */
+/* dsect  */
+  {"eject", listing_eject, 0}, /* Formfeed listing */
   {"else", s_else, 0},
   {"elsec", s_else, 0},
+  {"elseif", s_elseif, (int) O_ne},
   {"end", s_end, 0},
   {"endc", s_endif, 0},
+  {"endfunc", s_func, 1},
   {"endif", s_endif, 0},
-/* endef */
+  {"endr", s_bad_endr, 0},
+/* endef  */
   {"equ", s_set, 0},
   {"equiv", s_set, 1},
   {"err", s_err, 0},
   {"exitm", s_mexit, 0},
-/* extend */
-  {"extern", s_ignore, 0},     /* We treat all undef as ext */
+/* extend  */
+  {"extern", s_ignore, 0},     /* We treat all undef as ext */
   {"appfile", s_app_file, 1},
   {"appline", s_app_line, 0},
   {"fail", s_fail, 0},
@@ -311,6 +330,7 @@ static const pseudo_typeS potable[] =
   {"fill", s_fill, 0},
   {"float", float_cons, 'f'},
   {"format", s_ignore, 0},
+  {"func", s_func, 0},
   {"global", s_globl, 0},
   {"globl", s_globl, 0},
   {"hword", cons, 2},
@@ -328,6 +348,7 @@ static const pseudo_typeS potable[] =
   {"ifne", s_if, (int) O_ne},
   {"ifnes", s_ifeqs, 1},
   {"ifnotdef", s_ifdef, 1},
+  {"incbin", s_incbin, 0},
   {"include", s_include, 0},
   {"int", cons, 4},
   {"irp", s_irp, 0},
@@ -335,9 +356,9 @@ static const pseudo_typeS potable[] =
   {"irpc", s_irp, 1},
   {"irepc", s_irp, 1},
   {"lcomm", s_lcomm, 0},
-  {"lflags", listing_flags, 0},        /* Listing flags */
+  {"lflags", listing_flags, 0},        /* Listing flags */
   {"linkonce", s_linkonce, 0},
-  {"list", listing_list, 1},   /* Turn listing on */
+  {"list", listing_list, 1},   /* Turn listing on */
   {"llen", listing_psize, 1},
   {"long", cons, 4},
   {"lsym", s_lsym, 0},
@@ -347,7 +368,7 @@ static const pseudo_typeS potable[] =
   {".mri", s_mri, 0},  /* Special case so .mri works in MRI mode.  */
   {"name", s_ignore, 0},
   {"noformat", s_ignore, 0},
-  {"nolist", listing_list, 0}, /* Turn listing off */
+  {"nolist", listing_list, 0}, /* Turn listing off */
   {"nopage", listing_nopage, 0},
   {"octa", cons, 16},
   {"offset", s_struct, 0},
@@ -358,19 +379,19 @@ static const pseudo_typeS potable[] =
   {"page", listing_eject, 0},
   {"plen", listing_psize, 0},
   {"print", s_print, 0},
-  {"psize", listing_psize, 0}, /* set paper size */
+  {"psize", listing_psize, 0}, /* Set paper size.  */
   {"purgem", s_purgem, 0},
   {"quad", cons, 8},
   {"rep", s_rept, 0},
   {"rept", s_rept, 0},
   {"rva", s_rva, 4},
-  {"sbttl", listing_title, 1}, /* Subtitle of listing */
-/* scl */
-/* sect */
+  {"sbttl", listing_title, 1}, /* Subtitle of listing */
+/* scl  */
+/* sect  */
   {"set", s_set, 0},
   {"short", cons, 2},
   {"single", float_cons, 'f'},
-/* size */
+/* size  */
   {"space", s_space, 0},
   {"skip", s_space, 0},
   {"sleb128", s_leb128, 1},
@@ -380,7 +401,7 @@ static const pseudo_typeS potable[] =
   {"stabs", s_stab, 's'},
   {"string", stringer, 1},
   {"struct", s_struct, 0},
-/* tag */
+/* tag  */
   {"text", s_text, 0},
 
   /* This is for gcc to use.  It's only just been added (2/94), so gcc
@@ -393,19 +414,19 @@ static const pseudo_typeS potable[] =
      this one.  Match it either way...  */
   {"this_gcc_requires_the_gnu_assembler", s_ignore, 0},
 
-  {"title", listing_title, 0}, /* Listing title */
+  {"title", listing_title, 0}, /* Listing title */
   {"ttl", listing_title, 0},
-/* type */
+/* type  */
   {"uleb128", s_leb128, 0},
-/* use */
-/* val */
+/* use  */
+/* val  */
   {"xcom", s_comm, 0},
   {"xdef", s_globl, 0},
   {"xref", s_ignore, 0},
   {"xstabs", s_xstab, 's'},
   {"word", cons, 2},
   {"zero", s_space, 0},
-  {NULL}                       /* end sentinel */
+  {NULL, NULL, 0}                      /* End sentinel.  */
 };
 
 static int pop_override_ok = 0;
@@ -434,21 +455,21 @@ pop_insert (table)
 #define obj_pop_insert()       pop_insert(obj_pseudo_table)
 #endif
 
-static void 
+static void
 pobegin ()
 {
   po_hash = hash_new ();
 
-  /* Do the target-specific pseudo ops. */
+  /* Do the target-specific pseudo ops.  */
   pop_table_name = "md";
   md_pop_insert ();
 
-  /* Now object specific.  Skip any that were in the target table. */
+  /* Now object specific.  Skip any that were in the target table.  */
   pop_table_name = "obj";
   pop_override_ok = 1;
   obj_pop_insert ();
 
-  /* Now portable ones.  Skip any that we've seen already. */
+  /* Now portable ones.  Skip any that we've seen already.  */
   pop_table_name = "standard";
   pop_insert (potable);
 }
@@ -456,13 +477,12 @@ pobegin ()
 #define HANDLE_CONDITIONAL_ASSEMBLY()                                  \
   if (ignore_input ())                                                 \
     {                                                                  \
-      while (! is_end_of_line[(unsigned char) *input_line_pointer++])  \
+      while (!is_end_of_line[(unsigned char) *input_line_pointer++])   \
        if (input_line_pointer == buffer_limit)                         \
          break;                                                        \
       continue;                                                                \
     }
 
-
 /* This function is used when scrubbing the characters between #APP
    and #NO_APP.  */
 
@@ -470,31 +490,35 @@ static char *scrub_string;
 static char *scrub_string_end;
 
 static int
-scrub_from_string (from)
-     char **from;
+scrub_from_string (buf, buflen)
+     char *buf;
+     int buflen;
 {
-  int size;
-
-  *from = scrub_string;
-  size = scrub_string_end - scrub_string;
-  scrub_string = scrub_string_end;
-  return size;
+  int copy;
+
+  copy = scrub_string_end - scrub_string;
+  if (copy > buflen)
+    copy = buflen;
+  memcpy (buf, scrub_string, copy);
+  scrub_string += copy;
+  return copy;
 }
 
-/*     read_a_source_file()
- *
- * We read the file, putting things into a web that
- * represents what we have been reading.
- */
-void 
+/* We read the file, putting things into a web that represents what we
+   have been reading.  */
+void
 read_a_source_file (name)
      char *name;
 {
   register char c;
-  register char *s;            /* string of symbol, '\0' appended */
+  register char *s;            /* String of symbol, '\0' appended.  */
   register int temp;
   pseudo_typeS *pop;
 
+#ifdef WARN_COMMENTS
+  found_comment = 0;
+#endif
+
   buffer = input_scrub_new_file (name);
 
   listing_file (name);
@@ -507,36 +531,29 @@ read_a_source_file (name)
   generate_file_debug ();
 
   while ((buffer_limit = input_scrub_next_buffer (&input_line_pointer)) != 0)
-    {                          /* We have another line to parse. */
-      know (buffer_limit[-1] == '\n'); /* Must have a sentinel. */
+    {                          /* We have another line to parse.  */
+      know (buffer_limit[-1] == '\n'); /* Must have a sentinel.  */
     contin:                    /* JF this goto is my fault I admit it.
                                   Someone brave please re-write the whole
                                   input section here?  Pleeze???  */
       while (input_line_pointer < buffer_limit)
        {
-         /* We have more of this buffer to parse. */
+         /* We have more of this buffer to parse.  */
 
-         /*
-          * We now have input_line_pointer->1st char of next line.
-          * If input_line_pointer [-1] == '\n' then we just
-          * scanned another line: so bump line counters.
-          */
+         /* We now have input_line_pointer->1st char of next line.
+            If input_line_pointer [-1] == '\n' then we just
+            scanned another line: so bump line counters.  */
          if (is_end_of_line[(unsigned char) input_line_pointer[-1]])
            {
 #ifdef md_start_line_hook
              md_start_line_hook ();
 #endif
-
              if (input_line_pointer[-1] == '\n')
                bump_line_counters ();
 
              line_label = NULL;
 
-             if (flag_m68k_mri
-#ifdef LABELS_WITHOUT_COLONS
-                 || 1
-#endif
-                 )
+             if (LABELS_WITHOUT_COLONS || flag_m68k_mri)
                {
                  /* Text at the start of a line must be a label, we
                     run down and stick a colon in.  */
@@ -581,7 +598,12 @@ read_a_source_file (name)
                      /* In MRI mode, we need to handle the MACRO
                          pseudo-op specially: we don't want to put the
                          symbol in the symbol table.  */
-                     if (! mri_line_macro)
+                     if (!mri_line_macro
+#ifdef TC_START_LABEL_WITHOUT_COLON
+                         && TC_START_LABEL_WITHOUT_COLON(c,
+                                                         input_line_pointer)
+#endif
+                         )
                        line_label = colon (line_start);
                      else
                        line_label = symbol_create (line_start,
@@ -596,28 +618,25 @@ read_a_source_file (name)
                }
            }
 
-         /*
-          * We are at the begining of a line, or similar place.
-          * We expect a well-formed assembler statement.
-          * A "symbol-name:" is a statement.
-          *
-          * Depending on what compiler is used, the order of these tests
-          * may vary to catch most common case 1st.
-          * Each test is independent of all other tests at the (top) level.
-          * PLEASE make a compiler that doesn't use this assembler.
-          * It is crufty to waste a compiler's time encoding things for this
-          * assembler, which then wastes more time decoding it.
-          * (And communicating via (linear) files is silly!
-          * If you must pass stuff, please pass a tree!)
-          */
+         /* We are at the begining of a line, or similar place.
+            We expect a well-formed assembler statement.
+            A "symbol-name:" is a statement.
+
+            Depending on what compiler is used, the order of these tests
+            may vary to catch most common case 1st.
+            Each test is independent of all other tests at the (top) level.
+            PLEASE make a compiler that doesn't use this assembler.
+            It is crufty to waste a compiler's time encoding things for this
+            assembler, which then wastes more time decoding it.
+            (And communicating via (linear) files is silly!
+            If you must pass stuff, please pass a tree!)  */
          if ((c = *input_line_pointer++) == '\t'
              || c == ' '
              || c == '\f'
              || c == 0)
-           {
-             c = *input_line_pointer++;
-           }
-         know (c != ' ');      /* No further leading whitespace. */
+           c = *input_line_pointer++;
+
+         know (c != ' ');      /* No further leading whitespace.  */
 
 #ifndef NO_LISTING
          /* If listing is on, and we are expanding a macro, then give
@@ -630,18 +649,18 @@ read_a_source_file (name)
                  int len;
 
                  /* Find the end of the current expanded macro line.  */
-                 for (s = input_line_pointer-1; *s ; ++s)
+                 for (s = input_line_pointer - 1; *s; ++s)
                    if (is_end_of_line[(unsigned char) *s])
                      break;
 
                  /* Copy it for safe keeping.  Also give an indication of
                     how much macro nesting is involved at this point.  */
-                 len = s - (input_line_pointer-1);
+                 len = s - (input_line_pointer - 1);
                  copy = (char *) xmalloc (len + macro_nest + 2);
                  memset (copy, '>', macro_nest);
                  copy[macro_nest] = ' ';
-                 memcpy (copy + macro_nest + 1, input_line_pointer-1, len);
-                 copy[macro_nest+1+len] = '\0';
+                 memcpy (copy + macro_nest + 1, input_line_pointer - 1, len);
+                 copy[macro_nest + 1 + len] = '\0';
 
                  /* Install the line with the listing facility.  */
                  listing_newline (copy);
@@ -650,37 +669,34 @@ read_a_source_file (name)
                listing_newline (NULL);
            }
 #endif
-
-         /*
-          * C is the 1st significant character.
-          * Input_line_pointer points after that character.
-          */
+         /* C is the 1st significant character.
+            Input_line_pointer points after that character.  */
          if (is_name_beginner (c))
            {
-             /* want user-defined label or pseudo/opcode */
+             /* Want user-defined label or pseudo/opcode.  */
              HANDLE_CONDITIONAL_ASSEMBLY ();
 
              s = --input_line_pointer;
-             c = get_symbol_end ();    /* name's delimiter */
-             /*
-              * C is character after symbol.
-              * That character's place in the input line is now '\0'.
-              * S points to the beginning of the symbol.
-              *   [In case of pseudo-op, s->'.'.]
-              * Input_line_pointer->'\0' where c was.
-              */
-             if (TC_START_LABEL(c, input_line_pointer))
+             c = get_symbol_end ();    /* name's delimiter.  */
+
+             /* C is character after symbol.
+                That character's place in the input line is now '\0'.
+                S points to the beginning of the symbol.
+                  [In case of pseudo-op, s->'.'.]
+                Input_line_pointer->'\0' where c was.  */
+             if (TC_START_LABEL (c, input_line_pointer))
                {
                  if (flag_m68k_mri)
                    {
                      char *rest = input_line_pointer + 1;
 
                      /* In MRI mode, \tsym: set 0 is permitted.  */
-
                      if (*rest == ':')
                        ++rest;
+
                      if (*rest == ' ' || *rest == '\t')
                        ++rest;
+
                      if ((strncasecmp (rest, "EQU", 3) == 0
                           || strncasecmp (rest, "SET", 3) == 0)
                          && (rest[3] == ' ' || rest[3] == '\t'))
@@ -691,18 +707,17 @@ read_a_source_file (name)
                        }
                    }
 
-                 line_label = colon (s);       /* user-defined label */
-                 *input_line_pointer++ = ':';  /* Put ':' back for error messages' sake. */
-                 /* Input_line_pointer->after ':'. */
+                 line_label = colon (s);       /* User-defined label.  */
+                 /* Put ':' back for error messages' sake.  */
+                 *input_line_pointer++ = ':';
+                 /* Input_line_pointer->after ':'.  */
                  SKIP_WHITESPACE ();
-
-
                }
              else if (c == '='
                       || ((c == ' ' || c == '\t')
                           && input_line_pointer[1] == '='
 #ifdef TC_EQUAL_IN_INSN
-                          && ! TC_EQUAL_IN_INSN (c, input_line_pointer)
+                          && !TC_EQUAL_IN_INSN (c, input_line_pointer)
 #endif
                           ))
                {
@@ -710,27 +725,25 @@ read_a_source_file (name)
                  demand_empty_rest_of_line ();
                }
              else
-               {               /* expect pseudo-op or machine instruction */
+               {
+                 /* Expect pseudo-op or machine instruction.  */
                  pop = NULL;
 
-#define IGNORE_OPCODE_CASE
 #ifdef IGNORE_OPCODE_CASE
                  {
                    char *s2 = s;
+
+                   strncpy (original_case_string, s2, sizeof (original_case_string));
+                   original_case_string[sizeof (original_case_string) - 1] = 0;
+
                    while (*s2)
                      {
-                       if (isupper ((unsigned char) *s2))
-                         *s2 = tolower (*s2);
+                       *s2 = TOLOWER (*s2);
                        s2++;
                      }
                  }
 #endif
-
-                 if (flag_m68k_mri
-#ifdef NO_PSEUDO_DOT
-                     || 1
-#endif
-                     )
+                 if (NO_PSEUDO_DOT || flag_m68k_mri)
                    {
                      /* The MRI assembler and the m88k use pseudo-ops
                          without a period.  */
@@ -740,15 +753,13 @@ read_a_source_file (name)
                    }
 
                  if (pop != NULL
-                     || (! flag_m68k_mri && *s == '.'))
+                     || (!flag_m68k_mri && *s == '.'))
                    {
-                     /*
-                      * PSEUDO - OP.
-                      *
-                      * WARNING: c has next char, which may be end-of-line.
-                      * We lookup the pseudo-op table with s+1 because we
-                      * already know that the pseudo-op begins with a '.'.
-                      */
+                     /* PSEUDO - OP.
+
+                        WARNING: c has next char, which may be end-of-line.
+                        We lookup the pseudo-op table with s+1 because we
+                        already know that the pseudo-op begins with a '.'.  */
 
                      if (pop == NULL)
                        pop = (pseudo_typeS *) hash_find (po_hash, s + 1);
@@ -758,52 +769,52 @@ read_a_source_file (name)
                          this is.  */
                      if (mri_pending_align
                          && (pop == NULL
-                             || ! ((pop->poc_handler == cons
-                                    && pop->poc_val == 1)
-                                   || (pop->poc_handler == s_space
-                                       && pop->poc_val == 1)
+                             || !((pop->poc_handler == cons
+                                   && pop->poc_val == 1)
+                                  || (pop->poc_handler == s_space
+                                      && pop->poc_val == 1)
 #ifdef tc_conditional_pseudoop
-                                   || tc_conditional_pseudoop (pop)
+                                  || tc_conditional_pseudoop (pop)
 #endif
-                                   || pop->poc_handler == s_if
-                                   || pop->poc_handler == s_ifdef
-                                   || pop->poc_handler == s_ifc
-                                   || pop->poc_handler == s_ifeqs
-                                   || pop->poc_handler == s_else
-                                   || pop->poc_handler == s_endif
-                                   || pop->poc_handler == s_globl
-                                   || pop->poc_handler == s_ignore)))
+                                  || pop->poc_handler == s_if
+                                  || pop->poc_handler == s_ifdef
+                                  || pop->poc_handler == s_ifc
+                                  || pop->poc_handler == s_ifeqs
+                                  || pop->poc_handler == s_else
+                                  || pop->poc_handler == s_endif
+                                  || pop->poc_handler == s_globl
+                                  || pop->poc_handler == s_ignore)))
                        {
                          do_align (1, (char *) NULL, 0, 0);
                          mri_pending_align = 0;
+
                          if (line_label != NULL)
                            {
-                             line_label->sy_frag = frag_now;
+                             symbol_set_frag (line_label, frag_now);
                              S_SET_VALUE (line_label, frag_now_fix ());
                            }
                        }
 
-                     /* Print the error msg now, while we still can */
+                     /* Print the error msg now, while we still can */
                      if (pop == NULL)
                        {
-                         as_bad (_("Unknown pseudo-op:  `%s'"), s);
+                         as_bad (_("unknown pseudo-op: `%s'"), s);
                          *input_line_pointer = c;
                          s_ignore (0);
                          continue;
                        }
 
-                     /* Put it back for error messages etc. */
+                     /* Put it back for error messages etc.  */
                      *input_line_pointer = c;
                      /* The following skip of whitespace is compulsory.
                         A well shaped space is sometimes all that separates
-                        keyword from operands. */
+                        keyword from operands.  */
                      if (c == ' ' || c == '\t')
                        input_line_pointer++;
-                     /*
-                      * Input_line is restored.
-                      * Input_line_pointer->1st non-blank char
-                      * after pseudo-operation.
-                      */
+
+                     /* Input_line is restored.
+                        Input_line_pointer->1st non-blank char
+                        after pseudo-operation.  */
                      (*pop->poc_handler) (pop->poc_val);
 
                      /* If that was .end, just get out now.  */
@@ -813,9 +824,12 @@ read_a_source_file (name)
                  else
                    {
                      int inquote = 0;
+#ifdef QUOTES_IN_INSN
+                     int inescape = 0;
+#endif
 
-                     /* WARNING: c has char, which may be end-of-line. */
-                     /* Also: input_line_pointer->`\0` where c was. */
+                     /* WARNING: c has char, which may be end-of-line.  */
+                     /* Also: input_line_pointer->`\0` where c was.  */
                      *input_line_pointer = c;
                      while (!is_end_of_line[(unsigned char) *input_line_pointer]
                             || inquote
@@ -825,7 +839,15 @@ read_a_source_file (name)
                             )
                        {
                          if (flag_m68k_mri && *input_line_pointer == '\'')
-                           inquote = ! inquote;
+                           inquote = !inquote;
+#ifdef QUOTES_IN_INSN
+                         if (inescape)
+                           inescape = 0;
+                         else if (*input_line_pointer == '"')
+                           inquote = !inquote;
+                         else if (*input_line_pointer == '\\')
+                           inescape = 1;
+#endif
                          input_line_pointer++;
                        }
 
@@ -838,17 +860,21 @@ read_a_source_file (name)
                        {
                          sb out;
                          const char *err;
+                         macro_entry *macro;
 
-                         if (check_macro (s, &out, '\0', &err))
+                         if (check_macro (s, &out, &err, &macro))
                            {
                              if (err != NULL)
-                               as_bad (err);
+                               as_bad ("%s", err);
                              *input_line_pointer++ = c;
                              input_scrub_include_sb (&out,
-                                                     input_line_pointer);
+                                                     input_line_pointer, 1);
                              sb_kill (&out);
                              buffer_limit =
                                input_scrub_next_buffer (&input_line_pointer);
+#ifdef md_macro_info
+                             md_macro_info (macro);
+#endif
                              continue;
                            }
                        }
@@ -859,42 +885,41 @@ read_a_source_file (name)
                          mri_pending_align = 0;
                          if (line_label != NULL)
                            {
-                             line_label->sy_frag = frag_now;
+                             symbol_set_frag (line_label, frag_now);
                              S_SET_VALUE (line_label, frag_now_fix ());
                            }
                        }
 
-                     md_assemble (s);  /* Assemble 1 instruction. */
+                     md_assemble (s);  /* Assemble 1 instruction.  */
 
                      *input_line_pointer++ = c;
 
                      /* We resume loop AFTER the end-of-line from
-                        this instruction. */
-                   }           /* if (*s=='.') */
-               }               /* if c==':' */
+                        this instruction.  */
+                   }
+               }
              continue;
-           }                   /* if (is_name_beginner(c) */
-
+           }
 
          /* Empty statement?  */
          if (is_end_of_line[(unsigned char) c])
            continue;
 
-         if ((LOCAL_LABELS_DOLLAR || LOCAL_LABELS_FB)
-             && isdigit ((unsigned char) c))
+         if ((LOCAL_LABELS_DOLLAR || LOCAL_LABELS_FB) && ISDIGIT (c))
            {
-             /* local label  ("4:") */
+             /* local label  ("4:")  */
              char *backup = input_line_pointer;
 
              HANDLE_CONDITIONAL_ASSEMBLY ();
 
              temp = c - '0';
 
-             while (isdigit ((unsigned char) *input_line_pointer))
+             /* Read the whole number.  */
+             while (ISDIGIT (*input_line_pointer))
                {
                  temp = (temp * 10) + *input_line_pointer - '0';
                  ++input_line_pointer;
-               }               /* read the whole number */
+               }
 
              if (LOCAL_LABELS_DOLLAR
                  && *input_line_pointer == '$'
@@ -924,7 +949,7 @@ read_a_source_file (name)
            }                   /* local label  ("4:") */
 
          if (c && strchr (line_comment_chars, c))
-           {                   /* Its a comment.  Better say APP or NO_APP */
+           {                   /* Its a comment.  Better say APP or NO_APP */
              char *ends;
              char *new_buf;
              char *new_tmp;
@@ -947,7 +972,7 @@ read_a_source_file (name)
                  /* The end of the #APP wasn't in this buffer.  We
                     keep reading in buffers until we find the #NO_APP
                     that goes with this #APP  There is one.  The specs
-                    guarentee it. . . */
+                    guarentee it...  */
                  tmp_len = buffer_limit - s;
                  tmp_buf = xmalloc (tmp_len + 1);
                  memcpy (tmp_buf, s, tmp_len);
@@ -1015,6 +1040,7 @@ read_a_source_file (name)
              buffer = new_buf;
              input_line_pointer = new_buf;
              buffer_limit = new_tmp;
+
              continue;
            }
 
@@ -1024,11 +1050,10 @@ read_a_source_file (name)
          if (tc_unrecognized_line (c))
            continue;
 #endif
-
-         /* as_warn("Junk character %d.",c);  Now done by ignore_rest */
-         input_line_pointer--; /* Report unknown char as ignored. */
+         input_line_pointer--;
+         /* Report unknown char as ignored.  */
          ignore_rest_of_line ();
-       }                       /* while (input_line_pointer<buffer_limit) */
+       }
 
 #ifdef md_after_pass_hook
       md_after_pass_hook ();
@@ -1047,14 +1072,22 @@ read_a_source_file (name)
              goto contin;
            }
        }
-    }                          /* while (more buffers to scan) */
+    }
 
  quit:
 
 #ifdef md_cleanup
-  md_cleanup();
+  md_cleanup ();
+#endif
+  /* Close the input file.  */
+  input_scrub_close ();
+#ifdef WARN_COMMENTS
+  {
+    if (warn_comment && found_comment)
+      as_warn_where (found_comment_file, found_comment,
+                    "first comment found here");
+  }
 #endif
-  input_scrub_close ();                /* Close the input file */
 }
 
 /* For most MRI pseudo-ops, the line actually ends at the first
@@ -1069,37 +1102,30 @@ char *
 mri_comment_field (stopcp)
      char *stopcp;
 {
-#ifdef TC_M68K
-
   char *s;
+#ifdef TC_M68K
   int inquote = 0;
 
   know (flag_m68k_mri);
 
   for (s = input_line_pointer;
-       ((! is_end_of_line[(unsigned char) *s] && *s != ' ' && *s != '\t')
+       ((!is_end_of_line[(unsigned char) *s] && *s != ' ' && *s != '\t')
        || inquote);
        s++)
     {
       if (*s == '\'')
-       inquote = ! inquote;
+       inquote = !inquote;
     }
-  *stopcp = *s;
-  *s = '\0';
-  return s;
-
 #else
-
-  char *s;
-
-  for (s = input_line_pointer; ! is_end_of_line[(unsigned char) *s]; s++)
+  for (s = input_line_pointer;
+       !is_end_of_line[(unsigned char) *s];
+       s++)
     ;
+#endif
   *stopcp = *s;
   *s = '\0';
-  return s;
-
-#endif
 
+  return s;
 }
 
 /* Skip to the end of an MRI comment field.  */
@@ -1113,13 +1139,13 @@ mri_comment_end (stop, stopc)
 
   input_line_pointer = stop;
   *stop = stopc;
-  while (! is_end_of_line[(unsigned char) *input_line_pointer])
+  while (!is_end_of_line[(unsigned char) *input_line_pointer])
     ++input_line_pointer;
 }
 
-void 
+void
 s_abort (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   as_fatal (_(".abort detected.  Abandoning ship."));
 }
@@ -1130,57 +1156,51 @@ s_abort (ignore)
    the maximum number of characters to skip when doing the alignment,
    or 0 if there is no maximum.  */
 
-static void 
+static void
 do_align (n, fill, len, max)
      int n;
      char *fill;
      int len;
      int max;
 {
-  char default_fill;
+  if (now_seg == absolute_section)
+    {
+      if (fill != NULL)
+       while (len-- > 0)
+         if (*fill++ != '\0')
+           {
+             as_warn (_("ignoring fill value in absolute section"));
+             break;
+           }
+      fill = NULL;
+      len = 0;
+    }
 
 #ifdef md_do_align
   md_do_align (n, fill, len, max, just_record_alignment);
 #endif
 
-  if (fill == NULL)
-    {
-      int maybe_text;
-
-#ifdef BFD_ASSEMBLER
-      if ((bfd_get_section_flags (stdoutput, now_seg) & SEC_CODE) != 0)
-       maybe_text = 1;
-      else
-       maybe_text = 0;
-#else
-      if (now_seg != data_section && now_seg != bss_section)
-       maybe_text = 1;
-      else
-       maybe_text = 0;
-#endif
-
-      if (maybe_text)
-       default_fill = NOP_OPCODE;
-      else
-       default_fill = 0;
-      fill = &default_fill;
-      len = 1;
-    }
-
-  /* Only make a frag if we HAVE to. . . */
+  /* Only make a frag if we HAVE to...  */
   if (n != 0 && !need_pass_2)
     {
-      if (len <= 1)
+      if (fill == NULL)
+       {
+         if (subseg_text_p (now_seg))
+           frag_align_code (n, max);
+         else
+           frag_align (n, 0, max);
+       }
+      else if (len <= 1)
        frag_align (n, *fill, max);
       else
        frag_align_pattern (n, fill, len, max);
     }
 
 #ifdef md_do_align
- just_record_alignment:
+ just_record_alignment: ATTRIBUTE_UNUSED_LABEL
 #endif
 
-  record_alignment (now_seg, n);
+  record_alignment (now_seg, n - OCTETS_PER_BYTE_POWER);
 }
 
 /* Handle the .align pseudo-op.  A positive ARG is a default alignment
@@ -1208,7 +1228,7 @@ s_align (arg, bytes_p)
       if (arg < 0)
        align = 0;
       else
-       align = arg;    /* Default value from pseudo-op table */
+       align = arg;    /* Default value from pseudo-op table */
     }
   else
     {
@@ -1226,7 +1246,8 @@ s_align (arg, bytes_p)
          for (i = 0; (align & 1) == 0; align >>= 1, ++i)
            ;
          if (align != 1)
-           as_bad (_("Alignment not a power of 2"));
+           as_bad (_("alignment not a power of 2"));
+
          align = i;
        }
     }
@@ -1234,7 +1255,7 @@ s_align (arg, bytes_p)
   if (align > 15)
     {
       align = 15;
-      as_bad (_("Alignment too large: %u assumed"), align);
+      as_warn (_("alignment too large: %u assumed"), align);
     }
 
   if (*input_line_pointer != ',')
@@ -1263,7 +1284,7 @@ s_align (arg, bytes_p)
        }
     }
 
-  if (! fill_p)
+  if (!fill_p)
     {
       if (arg < 0)
        as_warn (_("expected fill pattern missing"));
@@ -1276,7 +1297,7 @@ s_align (arg, bytes_p)
       if (arg >= 0)
        fill_len = 1;
       else
-       fill_len = - arg;
+       fill_len = -arg;
       if (fill_len <= 1)
        {
          char fill_char;
@@ -1295,16 +1316,16 @@ s_align (arg, bytes_p)
        }
     }
 
+  demand_empty_rest_of_line ();
+
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }
 
 /* Handle the .align pseudo-op on machines where ".align 4" means
    align to a 4 byte boundary.  */
 
-void 
+void
 s_align_bytes (arg)
      int arg;
 {
@@ -1314,16 +1335,16 @@ s_align_bytes (arg)
 /* Handle the .align pseudo-op on machines where ".align 4" means align
    to a 2**4 boundary.  */
 
-void 
+void
 s_align_ptwo (arg)
      int arg;
 {
   s_align (arg, 0);
 }
 
-void 
+void
 s_comm (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   register char *name;
   register char c;
@@ -1338,43 +1359,59 @@ s_comm (ignore)
 
   name = input_line_pointer;
   c = get_symbol_end ();
-  /* just after name is now '\0' */
+  /* Just after name is now '\0'.  */
   p = input_line_pointer;
   *p = c;
+
+  if (name == p)
+    {
+      as_bad (_("expected symbol name"));
+      discard_rest_of_line ();
+      return;
+    }
+
   SKIP_WHITESPACE ();
+
   if (*input_line_pointer != ',')
     {
-      as_bad (_("Expected comma after symbol-name: rest of line ignored."));
+      *p = 0;
+      as_bad (_("expected comma after \"%s\""), name);
+      *p = c;
+      ignore_rest_of_line ();
       if (flag_mri)
        mri_comment_end (stop, stopc);
-      ignore_rest_of_line ();
       return;
     }
+
   input_line_pointer++;                /* skip ',' */
+
   if ((temp = get_absolute_expression ()) < 0)
     {
-      as_warn (_(".COMMon length (%ld.) <0! Ignored."), (long) temp);
+      as_warn (_(".COMMon length (%ld) < 0 ignored"), (long) temp);
+      ignore_rest_of_line ();
       if (flag_mri)
        mri_comment_end (stop, stopc);
-      ignore_rest_of_line ();
       return;
     }
+
   *p = 0;
   symbolP = symbol_find_or_make (name);
   *p = c;
-  if (S_IS_DEFINED (symbolP) && ! S_IS_COMMON (symbolP))
+
+  if (S_IS_DEFINED (symbolP) && !S_IS_COMMON (symbolP))
     {
-      as_bad (_("Ignoring attempt to re-define symbol `%s'."),
+      as_bad (_("symbol `%s' is already defined"),
              S_GET_NAME (symbolP));
+      ignore_rest_of_line ();
       if (flag_mri)
        mri_comment_end (stop, stopc);
-      ignore_rest_of_line ();
       return;
     }
+
   if (S_GET_VALUE (symbolP))
     {
       if (S_GET_VALUE (symbolP) != (valueT) temp)
-       as_bad (_("Length of .comm \"%s\" is already %ld. Not changed to %ld."),
+       as_bad (_("length of .comm \"%s\" is already %ld; not changing to %ld"),
                S_GET_NAME (symbolP),
                (long) S_GET_VALUE (symbolP),
                (long) temp);
@@ -1387,16 +1424,16 @@ s_comm (ignore)
 #ifdef OBJ_VMS
   {
     extern int flag_one;
-    if ( (!temp) || !flag_one)
+    if (!temp || !flag_one)
       S_GET_OTHER(symbolP) = const_flag;
   }
 #endif /* not OBJ_VMS */
   know (symbolP->sy_frag == &zero_address_frag);
 
+  demand_empty_rest_of_line ();
+
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }                              /* s_comm() */
 
 /* The MRI COMMON pseudo-op.  We handle this by creating a common
@@ -1405,7 +1442,7 @@ s_comm (ignore)
 
 void
 s_mri_common (small)
-     int small;
+     int small ATTRIBUTE_UNUSED;
 {
   char *name;
   char c;
@@ -1415,7 +1452,7 @@ s_mri_common (small)
   char *stop = NULL;
   char stopc;
 
-  if (! flag_mri)
+  if (!flag_mri)
     {
       s_comm (0);
       return;
@@ -1426,7 +1463,7 @@ s_mri_common (small)
   SKIP_WHITESPACE ();
 
   name = input_line_pointer;
-  if (! isdigit ((unsigned char) *name))
+  if (!ISDIGIT (*name))
     c = get_symbol_end ();
   else
     {
@@ -1434,7 +1471,8 @@ s_mri_common (small)
        {
          ++input_line_pointer;
        }
-      while (isdigit ((unsigned char) *input_line_pointer));
+      while (ISDIGIT (*input_line_pointer));
+
       c = *input_line_pointer;
       *input_line_pointer = '\0';
 
@@ -1461,11 +1499,11 @@ s_mri_common (small)
       align = get_absolute_expression ();
     }
 
-  if (S_IS_DEFINED (sym) && ! S_IS_COMMON (sym))
+  if (S_IS_DEFINED (sym) && !S_IS_COMMON (sym))
     {
-      as_bad (_("attempt to re-define symbol `%s'"), S_GET_NAME (sym));
-      mri_comment_end (stop, stopc);
+      as_bad (_("symbol `%s' is already defined"), S_GET_NAME (sym));
       ignore_rest_of_line ();
+      mri_comment_end (stop, stopc);
       return;
     }
 
@@ -1479,10 +1517,12 @@ s_mri_common (small)
 
   if (line_label != NULL)
     {
-      line_label->sy_value.X_op = O_symbol;
-      line_label->sy_value.X_add_symbol = sym;
-      line_label->sy_value.X_add_number = S_GET_VALUE (sym);
-      line_label->sy_frag = &zero_address_frag;
+      expressionS exp;
+      exp.X_op = O_symbol;
+      exp.X_add_symbol = sym;
+      exp.X_add_number = 0;
+      symbol_set_value_expression (line_label, &exp);
+      symbol_set_frag (line_label, &zero_address_frag);
       S_SET_SEGMENT (line_label, expr_section);
     }
 
@@ -1495,14 +1535,14 @@ s_mri_common (small)
   if (*input_line_pointer == ',')
     input_line_pointer += 2;
 
-  mri_comment_end (stop, stopc);
-
   demand_empty_rest_of_line ();
+
+  mri_comment_end (stop, stopc);
 }
 
 void
 s_data (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   segT section;
   register int temp;
@@ -1531,21 +1571,35 @@ s_data (ignore)
    definition for .file; the APPFILE argument is 1 for .appfile, 0 for
    .file.  */
 
-void 
+void
+s_app_file_string (file)
+     char *file;
+{
+#ifdef LISTING
+  if (listing)
+    listing_source_file (file);
+#endif
+  register_dependency (file);
+#ifdef obj_app_file
+  obj_app_file (file);
+#endif
+}
+
+void
 s_app_file (appfile)
      int appfile;
 {
   register char *s;
   int length;
 
-  /* Some assemblers tolerate immediately following '"' */
+  /* Some assemblers tolerate immediately following '"' */
   if ((s = demand_copy_string (&length)) != 0)
     {
       /* If this is a fake .appfile, a fake newline was inserted into
         the buffer.  Passing -2 to new_logical_line tells it to
         account for it.  */
       int may_omit
-       = (! new_logical_line (s, appfile ? -2 : -1) && appfile);
+       = (!new_logical_line (s, appfile ? -2 : -1) && appfile);
 
       /* In MRI mode, the preprocessor may have inserted an extraneous
          backquote.  */
@@ -1555,17 +1609,8 @@ s_app_file (appfile)
        ++input_line_pointer;
 
       demand_empty_rest_of_line ();
-      if (! may_omit)
-       {
-#ifdef LISTING
-         if (listing)
-           listing_source_file (s);
-#endif
-         register_dependency (s);
-#ifdef obj_app_file
-         obj_app_file (s);
-#endif
-       }
+      if (!may_omit)
+       s_app_file_string (s);
     }
 }
 
@@ -1576,7 +1621,7 @@ s_app_file (appfile)
 
 void
 s_app_line (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int l;
 
@@ -1585,7 +1630,8 @@ s_app_line (ignore)
   if (l < 0)
     /* Some of the back ends can't deal with non-positive line numbers.
        Besides, it's silly.  */
-    as_warn (_("Line numbers must be positive; line number %d rejected."), l+1);
+    as_warn (_("line numbers must be positive; line number %d rejected"),
+            l + 1);
   else
     {
       new_logical_line ((char *) NULL, l);
@@ -1602,14 +1648,14 @@ s_app_line (ignore)
 
 void
 s_end (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (flag_mri)
     {
       /* The MRI assembler permits the start symbol to follow .end,
          but we don't support that.  */
       SKIP_WHITESPACE ();
-      if (! is_end_of_line[(unsigned char) *input_line_pointer]
+      if (!is_end_of_line[(unsigned char) *input_line_pointer]
          && *input_line_pointer != '*'
          && *input_line_pointer != '!')
        as_warn (_("start address not supported"));
@@ -1620,7 +1666,7 @@ s_end (ignore)
 
 void
 s_err (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   as_bad (_(".err encountered"));
   demand_empty_rest_of_line ();
@@ -1630,7 +1676,7 @@ s_err (ignore)
 
 void
 s_fail (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   offsetT temp;
   char *stop = NULL;
@@ -1645,15 +1691,15 @@ s_fail (ignore)
   else
     as_bad (_(".fail %ld encountered"), (long) temp);
 
+  demand_empty_rest_of_line ();
+
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }
 
-void 
+void
 s_fill (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   expressionS rep_exp;
   long size = 1;
@@ -1680,18 +1726,18 @@ s_fill (ignore)
 #define BSD_FILL_SIZE_CROCK_8 (8)
   if (size > BSD_FILL_SIZE_CROCK_8)
     {
-      as_warn (_(".fill size clamped to %d."), BSD_FILL_SIZE_CROCK_8);
+      as_warn (_(".fill size clamped to %d"), BSD_FILL_SIZE_CROCK_8);
       size = BSD_FILL_SIZE_CROCK_8;
     }
   if (size < 0)
     {
-      as_warn (_("Size negative: .fill ignored."));
+      as_warn (_("size negative; .fill ignored"));
       size = 0;
     }
   else if (rep_exp.X_op == O_constant && rep_exp.X_add_number <= 0)
     {
       if (rep_exp.X_add_number < 0)
-       as_warn (_("Repeat < 0, .fill ignored"));
+       as_warn (_("repeat < 0; .fill ignored"));
       size = 0;
     }
 
@@ -1729,28 +1775,30 @@ s_fill (ignore)
          p = frag_var (rs_space, (int) size, (int) size,
                        (relax_substateT) 0, rep_sym, (offsetT) 0, (char *) 0);
        }
+
       memset (p, 0, (unsigned int) size);
+
       /* The magic number BSD_FILL_SIZE_CROCK_4 is from BSD 4.2 VAX
-       * flavoured AS.  The following bizzare behaviour is to be
-       * compatible with above.  I guess they tried to take up to 8
-       * bytes from a 4-byte expression and they forgot to sign
-       * extend. Un*x Sux. */
+         flavoured AS.  The following bizarre behaviour is to be
+         compatible with above.  I guess they tried to take up to 8
+         bytes from a 4-byte expression and they forgot to sign
+         extend.  */
 #define BSD_FILL_SIZE_CROCK_4 (4)
       md_number_to_chars (p, (valueT) fill,
                          (size > BSD_FILL_SIZE_CROCK_4
                           ? BSD_FILL_SIZE_CROCK_4
                           : (int) size));
       /* Note: .fill (),0 emits no frag (since we are asked to .fill 0 bytes)
-       * but emits no error message because it seems a legal thing to do.
-       * It is a degenerate case of .fill but could be emitted by a compiler.
-       */
+         but emits no error message because it seems a legal thing to do.
+         It is a degenerate case of .fill but could be emitted by a
+        compiler.  */
     }
   demand_empty_rest_of_line ();
 }
 
-void 
+void
 s_globl (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *name;
   int c;
@@ -1766,23 +1814,25 @@ s_globl (ignore)
       name = input_line_pointer;
       c = get_symbol_end ();
       symbolP = symbol_find_or_make (name);
+      S_SET_EXTERNAL (symbolP);
+
       *input_line_pointer = c;
       SKIP_WHITESPACE ();
-      S_SET_EXTERNAL (symbolP);
+      c = *input_line_pointer;
       if (c == ',')
        {
          input_line_pointer++;
          SKIP_WHITESPACE ();
-         if (*input_line_pointer == '\n')
+         if (is_end_of_line[(unsigned char) *input_line_pointer])
            c = '\n';
        }
     }
   while (c == ',');
 
+  demand_empty_rest_of_line ();
+
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }
 
 /* Handle the MRI IRP and IRPC pseudo-ops.  */
@@ -1800,18 +1850,18 @@ s_irp (irpc)
   as_where (&file, &line);
 
   sb_new (&s);
-  while (! is_end_of_line[(unsigned char) *input_line_pointer])
+  while (!is_end_of_line[(unsigned char) *input_line_pointer])
     sb_add_char (&s, *input_line_pointer++);
 
   sb_new (&out);
 
-  err = expand_irp (irpc, 0, &s, &out, get_line_sb, '\0');
+  err = expand_irp (irpc, 0, &s, &out, get_line_sb);
   if (err != NULL)
     as_bad_where (file, line, "%s", err);
 
   sb_kill (&s);
 
-  input_scrub_include_sb (&out, input_line_pointer);
+  input_scrub_include_sb (&out, input_line_pointer, 1);
   sb_kill (&out);
   buffer_limit = input_scrub_next_buffer (&input_line_pointer);
 }
@@ -1823,7 +1873,7 @@ s_irp (irpc)
 
 void
 s_linkonce (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   enum linkonce_type type;
 
@@ -1831,7 +1881,7 @@ s_linkonce (ignore)
 
   type = LINKONCE_DISCARD;
 
-  if (! is_end_of_line[(unsigned char) *input_line_pointer])
+  if (!is_end_of_line[(unsigned char) *input_line_pointer])
     {
       char *s;
       char c;
@@ -1881,7 +1931,7 @@ s_linkonce (ignore)
        flags |= SEC_LINK_DUPLICATES_SAME_CONTENTS;
        break;
       }
-    if (! bfd_set_section_flags (stdoutput, now_seg, flags))
+    if (!bfd_set_section_flags (stdoutput, now_seg, flags))
       as_bad (_("bfd_set_section_flags: %s"),
              bfd_errmsg (bfd_get_error ()));
   }
@@ -1893,13 +1943,13 @@ s_linkonce (ignore)
   demand_empty_rest_of_line ();
 }
 
-static void 
+static void
 s_lcomm_internal (needs_align, bytes_p)
      /* 1 if this was a ".bss" directive, which may require a 3rd argument
-       (alignment); 0 if it was an ".lcomm" (2 args only)  */
+       (alignment); 0 if it was an ".lcomm" (2 args only).  */
      int needs_align;
      /* 1 if the alignment value should be interpreted as the byte boundary,
-       rather than the power of 2. */
+       rather than the power of 2.  */
      int bytes_p;
 {
   register char *name;
@@ -1917,6 +1967,14 @@ s_lcomm_internal (needs_align, bytes_p)
   c = get_symbol_end ();
   p = input_line_pointer;
   *p = c;
+
+  if (name == p)
+    {
+      as_bad (_("expected symbol name"));
+      discard_rest_of_line ();
+      return;
+    }
+
   SKIP_WHITESPACE ();
 
   /* Accept an optional comma after the name.  The comma used to be
@@ -1927,15 +1985,15 @@ s_lcomm_internal (needs_align, bytes_p)
       SKIP_WHITESPACE ();
     }
 
-  if (*input_line_pointer == '\n')
+  if (is_end_of_line[(unsigned char) *input_line_pointer])
     {
-      as_bad (_("Missing size expression"));
+      as_bad (_("missing size expression"));
       return;
     }
 
   if ((temp = get_absolute_expression ()) < 0)
     {
-      as_warn (_("BSS length (%d.) <0! Ignored."), temp);
+      as_warn (_("BSS length (%d) < 0 ignored"), temp);
       ignore_rest_of_line ();
       return;
     }
@@ -1945,56 +2003,51 @@ s_lcomm_internal (needs_align, bytes_p)
       || OUTPUT_FLAVOR == bfd_target_elf_flavour)
     {
       /* For MIPS and Alpha ECOFF or ELF, small objects are put in .sbss.  */
-      if (temp <= bfd_get_gp_size (stdoutput))
+      if ((unsigned) temp <= bfd_get_gp_size (stdoutput))
        {
          bss_seg = subseg_new (".sbss", 1);
          seg_info (bss_seg)->bss = 1;
 #ifdef BFD_ASSEMBLER
-         if (! bfd_set_section_flags (stdoutput, bss_seg, SEC_ALLOC))
+         if (!bfd_set_section_flags (stdoutput, bss_seg, SEC_ALLOC))
            as_warn (_("error setting flags for \".sbss\": %s"),
                     bfd_errmsg (bfd_get_error ()));
 #endif
        }
     }
 #endif
-   if (!needs_align)
-     {
-       /* FIXME. This needs to be machine independent. */
-       if (temp >= 8)
-        align = 3;
-       else if (temp >= 4)
-        align = 2;
-       else if (temp >= 2)
-        align = 1;
-       else
-        align = 0;
-
-#ifdef OBJ_EVAX
-       /* FIXME: This needs to be done in a more general fashion.  */
-       align = 3;
-#endif
 
-       record_alignment(bss_seg, align);
-     }
+  if (!needs_align)
+    {
+      TC_IMPLICIT_LCOMM_ALIGNMENT (temp, align);
+
+      /* Still zero unless TC_IMPLICIT_LCOMM_ALIGNMENT set it.  */
+      if (align)
+       record_alignment (bss_seg, align);
+    }
 
   if (needs_align)
     {
       align = 0;
       SKIP_WHITESPACE ();
+
       if (*input_line_pointer != ',')
        {
-         as_bad (_("Expected comma after size"));
+         as_bad (_("expected comma after size"));
          ignore_rest_of_line ();
          return;
        }
+
       input_line_pointer++;
       SKIP_WHITESPACE ();
-      if (*input_line_pointer == '\n')
+
+      if (is_end_of_line[(unsigned char) *input_line_pointer])
        {
-         as_bad (_("Missing alignment"));
+         as_bad (_("missing alignment"));
          return;
        }
+
       align = get_absolute_expression ();
+
       if (bytes_p)
        {
          /* Convert to a power of 2.  */
@@ -2005,22 +2058,24 @@ s_lcomm_internal (needs_align, bytes_p)
              for (i = 0; (align & 1) == 0; align >>= 1, ++i)
                ;
              if (align != 1)
-               as_bad (_("Alignment not a power of 2"));
+               as_bad (_("alignment not a power of 2"));
              align = i;
            }
        }
+
       if (align > max_alignment)
        {
          align = max_alignment;
-         as_warn (_("Alignment too large: %d. assumed."), align);
+         as_warn (_("alignment too large; %d assumed"), align);
        }
       else if (align < 0)
        {
          align = 0;
-         as_warn (_("Alignment negative. 0 assumed."));
+         as_warn (_("alignment negative; 0 assumed"));
        }
+
       record_alignment (bss_seg, align);
-    }                          /* if needs align */
+    }
   else
     {
       /* Assume some objects may require alignment on some systems.  */
@@ -2039,12 +2094,17 @@ s_lcomm_internal (needs_align, bytes_p)
   *p = c;
 
   if (
-#if defined(OBJ_AOUT) | defined(OBJ_BOUT)
-       S_GET_OTHER (symbolP) == 0 &&
-       S_GET_DESC (symbolP) == 0 &&
-#endif /* OBJ_AOUT or OBJ_BOUT */
-       (S_GET_SEGMENT (symbolP) == bss_seg
-       || (!S_IS_DEFINED (symbolP) && S_GET_VALUE (symbolP) == 0)))
+#if (defined (OBJ_AOUT) || defined (OBJ_MAYBE_AOUT) \
+     || defined (OBJ_BOUT) || defined (OBJ_MAYBE_BOUT))
+#ifdef BFD_ASSEMBLER
+      (OUTPUT_FLAVOR != bfd_target_aout_flavour
+       || (S_GET_OTHER (symbolP) == 0 && S_GET_DESC (symbolP) == 0)) &&
+#else
+      (S_GET_OTHER (symbolP) == 0 && S_GET_DESC (symbolP) == 0) &&
+#endif
+#endif
+      (S_GET_SEGMENT (symbolP) == bss_seg
+       || (!S_IS_DEFINED (symbolP) && S_GET_VALUE (symbolP) == 0)))
     {
       char *pfrag;
 
@@ -2052,12 +2112,13 @@ s_lcomm_internal (needs_align, bytes_p)
 
       if (align)
        frag_align (align, 0, 0);
-                                       /* detach from old frag */
+
+      /* Detach from old frag.  */
       if (S_GET_SEGMENT (symbolP) == bss_seg)
-       symbolP->sy_frag->fr_symbol = NULL;
+       symbol_get_frag (symbolP)->fr_symbol = NULL;
 
-      symbolP->sy_frag = frag_now;
-      pfrag = frag_var (rs_org, 1, 1, (relax_substateT)0, symbolP,
+      symbol_set_frag (symbolP, frag_now);
+      pfrag = frag_var (rs_org, 1, 1, (relax_substateT) 0, symbolP,
                        (offsetT) temp, (char *) 0);
       *pfrag = 0;
 
@@ -2066,7 +2127,7 @@ s_lcomm_internal (needs_align, bytes_p)
 #ifdef OBJ_COFF
       /* The symbol may already have been created with a preceding
          ".globl" directive -- be careful not to step on storage class
-         in that case.  Otherwise, set it to static. */
+         in that case.  Otherwise, set it to static.  */
       if (S_GET_STORAGE_CLASS (symbolP) != C_EXT)
        {
          S_SET_STORAGE_CLASS (symbolP, C_STAT);
@@ -2078,13 +2139,12 @@ s_lcomm_internal (needs_align, bytes_p)
 #endif
     }
   else
-    as_bad (_("Ignoring attempt to re-define symbol `%s'."),
-           S_GET_NAME (symbolP));
+    as_bad (_("symbol `%s' is already defined"), S_GET_NAME (symbolP));
 
   subseg_set (current_seg, current_subseg);
 
   demand_empty_rest_of_line ();
-}                              /* s_lcomm_internal() */
+}
 
 void
 s_lcomm (needs_align)
@@ -2093,15 +2153,16 @@ s_lcomm (needs_align)
   s_lcomm_internal (needs_align, 0);
 }
 
-void s_lcomm_bytes (needs_align)
+void
+s_lcomm_bytes (needs_align)
      int needs_align;
 {
   s_lcomm_internal (needs_align, 1);
 }
 
-void 
+void
 s_lsym (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   register char *name;
   register char c;
@@ -2109,22 +2170,33 @@ s_lsym (ignore)
   expressionS exp;
   register symbolS *symbolP;
 
-  /* we permit ANY defined expression: BSD4.2 demands constants */
+  /* We permit ANY defined expression: BSD4.2 demands constants.  */
   name = input_line_pointer;
   c = get_symbol_end ();
   p = input_line_pointer;
   *p = c;
+
+  if (name == p)
+    {
+      as_bad (_("expected symbol name"));
+      discard_rest_of_line ();
+      return;
+    }
+
   SKIP_WHITESPACE ();
+
   if (*input_line_pointer != ',')
     {
       *p = 0;
-      as_bad (_("Expected comma after name \"%s\""), name);
+      as_bad (_("expected comma after \"%s\""), name);
       *p = c;
       ignore_rest_of_line ();
       return;
     }
+
   input_line_pointer++;
   expression (&exp);
+
   if (exp.X_op != O_constant
       && exp.X_op != O_register)
     {
@@ -2132,6 +2204,7 @@ s_lsym (ignore)
       ignore_rest_of_line ();
       return;
     }
+
   *p = 0;
   symbolP = symbol_find_or_make (name);
 
@@ -2140,13 +2213,13 @@ s_lsym (ignore)
      those fields, and I can't see when they'd ever be tripped.  I
      don't think I understand why they were here so I may have
      introduced a bug. As recently as 1.37 didn't have this test
-     anyway.  xoxorich. */
+     anyway.  xoxorich.  */
 
   if (S_GET_SEGMENT (symbolP) == undefined_section
       && S_GET_VALUE (symbolP) == 0)
     {
       /* The name might be an undefined .global symbol; be sure to
-        keep the "external" bit. */
+        keep the "external" bit.  */
       S_SET_SEGMENT (symbolP,
                     (exp.X_op == O_constant
                      ? absolute_section
@@ -2155,11 +2228,12 @@ s_lsym (ignore)
     }
   else
     {
-      as_bad (_("Symbol %s already defined"), name);
+      as_bad (_("symbol `%s' is already defined"), name);
     }
+
   *p = c;
   demand_empty_rest_of_line ();
-}                              /* s_lsym() */
+}
 
 /* Read a line into an sb.  */
 
@@ -2181,7 +2255,7 @@ get_line_sb (line)
 
   /* If app.c sets any other characters to LEX_IS_STRINGQUOTE, this
      code needs to be changed.  */
-  if (! flag_m68k_mri)
+  if (!flag_m68k_mri)
     quote1 = '"';
   else
     quote1 = '\0';
@@ -2194,7 +2268,8 @@ get_line_sb (line)
 #endif
 
   inquote = '\0';
-  while (! is_end_of_line[(unsigned char) *input_line_pointer]
+
+  while (!is_end_of_line[(unsigned char) *input_line_pointer]
         || (inquote != '\0' && *input_line_pointer != '\n'))
     {
       if (inquote == *input_line_pointer)
@@ -2206,8 +2281,10 @@ get_line_sb (line)
          else if (*input_line_pointer == quote2)
            inquote = quote2;
        }
+
       sb_add_char (line, *input_line_pointer++);
     }
+
   while (input_line_pointer < buffer_limit
         && is_end_of_line[(unsigned char) *input_line_pointer])
     {
@@ -2215,15 +2292,15 @@ get_line_sb (line)
        bump_line_counters ();
       ++input_line_pointer;
     }
+
   return 1;
 }
 
-/* Define a macro.  This is an interface to macro.c, which is shared
-   between gas and gasp.  */
+/* Define a macro.  This is an interface to macro.c.  */
 
 void
 s_macro (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *file;
   unsigned int line;
@@ -2235,7 +2312,7 @@ s_macro (ignore)
   as_where (&file, &line);
 
   sb_new (&s);
-  while (! is_end_of_line[(unsigned char) *input_line_pointer])
+  while (!is_end_of_line[(unsigned char) *input_line_pointer])
     sb_add_char (&s, *input_line_pointer++);
 
   sb_new (&label);
@@ -2251,16 +2328,12 @@ s_macro (ignore)
        {
          S_SET_SEGMENT (line_label, undefined_section);
          S_SET_VALUE (line_label, 0);
-         line_label->sy_frag = &zero_address_frag;
+         symbol_set_frag (line_label, &zero_address_frag);
        }
 
-      if (((flag_m68k_mri
-#ifdef NO_PSEUDO_DOT
-           || 1
-#endif
-           )
+      if (((NO_PSEUDO_DOT || flag_m68k_mri)
           && hash_find (po_hash, name) != NULL)
-         || (! flag_m68k_mri
+         || (!flag_m68k_mri
              && *name == '.'
              && hash_find (po_hash, name + 1) != NULL))
        as_warn (_("attempt to redefine pseudo-op `%s' ignored"),
@@ -2275,7 +2348,7 @@ s_macro (ignore)
 
 void
 s_mexit (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   cond_exit_macro (macro_nest);
   buffer_limit = input_scrub_next_buffer (&input_line_pointer);
@@ -2285,7 +2358,7 @@ s_mexit (ignore)
 
 void
 s_mri (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int on, old_flag;
 
@@ -2297,13 +2370,21 @@ s_mri (ignore)
 #ifdef TC_M68K
       flag_m68k_mri = 1;
 #endif
+      macro_mri_mode (1);
     }
   else
     {
       flag_mri = 0;
+#ifdef TC_M68K
       flag_m68k_mri = 0;
+#endif
+      macro_mri_mode (0);
     }
 
+  /* Operator precedence changes in m68k MRI mode, so we need to
+     update the operator rankings.  */
+  expr_set_precedence ();
+
 #ifdef MRI_MODE_CHANGE
   if (on != old_flag)
     MRI_MODE_CHANGE (on);
@@ -2321,8 +2402,7 @@ do_org (segment, exp, fill)
      int fill;
 {
   if (segment != now_seg && segment != absolute_section)
-    as_bad (_("invalid segment \"%s\"; segment \"%s\" assumed"),
-           segment_name (segment), segment_name (now_seg));
+    as_bad (_("invalid segment \"%s\""), segment_name (segment));
 
   if (now_seg == absolute_section)
     {
@@ -2338,21 +2418,33 @@ do_org (segment, exp, fill)
   else
     {
       char *p;
+      symbolS *sym = exp->X_add_symbol;
+      offsetT off = exp->X_add_number * OCTETS_PER_BYTE;
+
+      if (exp->X_op != O_constant && exp->X_op != O_symbol)
+       {
+         /* Handle complex expressions.  */
+         sym = make_expr_symbol (exp);
+         off = 0;
+       }
 
-      p = frag_var (rs_org, 1, 1, (relax_substateT) 0, exp->X_add_symbol,
-                   exp->X_add_number, (char *) NULL);
+      p = frag_var (rs_org, 1, 1, (relax_substateT) 0, sym, off, (char *) 0);
       *p = fill;
     }
 }
 
-void 
+void
 s_org (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   register segT segment;
   expressionS exp;
   register long temp_fill;
 
+#ifdef md_flush_pending_output
+  md_flush_pending_output ();
+#endif
+
   /* The m68k MRI assembler has a different meaning for .org.  It
      means to create an absolute section at a given address.  We can't
      support that--use a linker script instead.  */
@@ -2388,7 +2480,7 @@ s_org (ignore)
     do_org (segment, &exp, temp_fill);
 
   demand_empty_rest_of_line ();
-}                              /* s_org() */
+}
 
 /* Handle parsing for the MRI SECT/SECTION pseudo-op.  This should be
    called by the obj-format routine which handles section changing
@@ -2399,7 +2491,7 @@ s_org (ignore)
 
 void
 s_mri_sect (type)
-     char *type;
+     char *type ATTRIBUTE_UNUSED;
 {
 #ifdef TC_M68K
 
@@ -2408,9 +2500,9 @@ s_mri_sect (type)
   segT seg;
 
   SKIP_WHITESPACE ();
-  
+
   name = input_line_pointer;
-  if (! isdigit ((unsigned char) *name))
+  if (!ISDIGIT (*name))
     c = get_symbol_end ();
   else
     {
@@ -2418,7 +2510,8 @@ s_mri_sect (type)
        {
          ++input_line_pointer;
        }
-      while (isdigit ((unsigned char) *input_line_pointer));
+      while (ISDIGIT (*input_line_pointer));
+
       c = *input_line_pointer;
       *input_line_pointer = '\0';
     }
@@ -2442,7 +2535,7 @@ s_mri_sect (type)
   if (*input_line_pointer == ',')
     {
       c = *++input_line_pointer;
-      c = toupper ((unsigned char) c);
+      c = TOUPPER (c);
       if (c == 'C' || c == 'D' || c == 'M' || c == 'R')
        *type = c;
       else
@@ -2462,7 +2555,7 @@ s_mri_sect (type)
          flags = SEC_ALLOC | SEC_LOAD | SEC_DATA | SEC_READONLY | SEC_ROM;
        if (flags != SEC_NO_FLAGS)
          {
-           if (! bfd_set_section_flags (stdoutput, seg, flags))
+           if (!bfd_set_section_flags (stdoutput, seg, flags))
              as_warn (_("error setting flags for \"%s\": %s"),
                       bfd_section_name (stdoutput, seg),
                       bfd_errmsg (bfd_get_error ()));
@@ -2548,7 +2641,7 @@ s_mri_sect (type)
        }
     }
 
-  demand_empty_rest_of_line ();          
+  demand_empty_rest_of_line ();
 
 #else /* ! TC_I960 */
   /* The MRI assembler seems to use different forms of .sect for
@@ -2563,7 +2656,7 @@ s_mri_sect (type)
 
 void
 s_print (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *s;
   int len;
@@ -2577,7 +2670,7 @@ s_print (ignore)
 
 void
 s_purgem (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   if (is_it_end_of_statement ())
     {
@@ -2605,20 +2698,43 @@ s_purgem (ignore)
 
 /* Handle the .rept pseudo-op.  */
 
+void
+s_bad_endr (ignore)
+     int ignore ATTRIBUTE_UNUSED;
+{
+  as_warn (_(".endr encountered without preceeding .rept, .irc, or .irp"));
+  demand_empty_rest_of_line ();
+}
+
+/* Handle the .rept pseudo-op.  */
+
 void
 s_rept (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   int count;
-  sb one;
-  sb many;
 
   count = get_absolute_expression ();
 
+  do_repeat (count, "REPT", "ENDR");
+}
+
+/* This function provides a generic repeat block implementation.   It allows
+   different directives to be used as the start/end keys.  */
+
+void
+do_repeat (count, start, end)
+     int count;
+     const char *start;
+     const char *end;
+{
+  sb one;
+  sb many;
+
   sb_new (&one);
-  if (! buffer_and_nest ("REPT", "ENDR", &one, get_line_sb))
+  if (!buffer_and_nest (start, end, &one, get_line_sb))
     {
-      as_bad (_("rept without endr"));
+      as_bad (_("%s without %s"), start, end);
       return;
     }
 
@@ -2628,17 +2744,34 @@ s_rept (ignore)
 
   sb_kill (&one);
 
-  input_scrub_include_sb (&many, input_line_pointer);
+  input_scrub_include_sb (&many, input_line_pointer, 1);
   sb_kill (&many);
   buffer_limit = input_scrub_next_buffer (&input_line_pointer);
 }
 
-/* Handle the .equ, .equiv and .set directives.  If EQUIV is 1, then
-   this is .equiv, and it is an error if the symbol is already
-   defined.  */
+/* Skip to end of current repeat loop; EXTRA indicates how many additional
+   input buffers to skip.  Assumes that conditionals preceding the loop end
+   are properly nested.
 
-void 
-s_set (equiv)
+   This function makes it easier to implement a premature "break" out of the
+   loop.  The EXTRA arg accounts for other buffers we might have inserted,
+   such as line substitutions.  */
+
+void
+end_repeat (extra)
+     int extra;
+{
+  cond_exit_macro (macro_nest);
+  while (extra-- >= 0)
+    buffer_limit = input_scrub_next_buffer (&input_line_pointer);
+}
+
+/* Handle the .equ, .equiv and .set directives.  If EQUIV is 1, then
+   this is .equiv, and it is an error if the symbol is already
+   defined.  */
+
+void
+s_set (equiv)
      int equiv;
 {
   register char *name;
@@ -2646,21 +2779,27 @@ s_set (equiv)
   register char *end_name;
   register symbolS *symbolP;
 
-  /*
-   * Especial apologies for the random logic:
-   * this just grew, and could be parsed much more simply!
-   * Dean in haste.
-   */
+  /* Especial apologies for the random logic:
+     this just grew, and could be parsed much more simply!
+     Dean in haste.  */
   name = input_line_pointer;
   delim = get_symbol_end ();
   end_name = input_line_pointer;
   *end_name = delim;
+
+  if (name == end_name)
+    {
+      as_bad (_("expected symbol name"));
+      discard_rest_of_line ();
+      return;
+    }
+
   SKIP_WHITESPACE ();
 
   if (*input_line_pointer != ',')
     {
       *end_name = 0;
-      as_bad (_("Expected comma after name \"%s\""), name);
+      as_bad (_("expected comma after \"%s\""), name);
       *end_name = delim;
       ignore_rest_of_line ();
       return;
@@ -2671,7 +2810,7 @@ s_set (equiv)
 
   if (name[0] == '.' && name[1] == '\0')
     {
-      /* Turn '. = mumble' into a .org mumble */
+      /* Turn '. = mumble' into a .org mumble */
       register segT segment;
       expressionS exp;
 
@@ -2694,8 +2833,8 @@ s_set (equiv)
       if (listing & LISTING_SYMBOLS)
        {
          extern struct list_info_struct *listing_tail;
-         fragS *dummy_frag = (fragS *) xmalloc (sizeof(fragS));
-         memset (dummy_frag, 0, sizeof(fragS));
+         fragS *dummy_frag = (fragS *) xmalloc (sizeof (fragS));
+         memset (dummy_frag, 0, sizeof (fragS));
          dummy_frag->fr_type = rs_fill;
          dummy_frag->line = listing_tail;
          symbolP = symbol_new (name, undefined_section, 0, dummy_frag);
@@ -2703,14 +2842,13 @@ s_set (equiv)
        }
       else
 #endif
-        symbolP = symbol_new (name, undefined_section, 0, &zero_address_frag);
-                           
+       symbolP = symbol_new (name, undefined_section, 0, &zero_address_frag);
+
 #ifdef OBJ_COFF
-      /* "set" symbols are local unless otherwise specified. */
+      /* "set" symbols are local unless otherwise specified.  */
       SF_SET_LOCAL (symbolP);
 #endif /* OBJ_COFF */
-
-    }                          /* make a new symbol */
+    }
 
   symbol_table_insert (symbolP);
 
@@ -2719,13 +2857,13 @@ s_set (equiv)
   if (equiv
       && S_IS_DEFINED (symbolP)
       && S_GET_SEGMENT (symbolP) != reg_section)
-    as_bad (_("symbol `%s' already defined"), S_GET_NAME (symbolP));
+    as_bad (_("symbol `%s' is already defined"), S_GET_NAME (symbolP));
 
   pseudo_set (symbolP);
   demand_empty_rest_of_line ();
-}                              /* s_set() */
+}
 
-void 
+void
 s_space (mult)
      int mult;
 {
@@ -2763,9 +2901,12 @@ s_space (mult)
              S_SET_VALUE (mri_common_symbol, val + 1);
              if (line_label != NULL)
                {
-                 know (line_label->sy_value.X_op == O_symbol);
-                 know (line_label->sy_value.X_add_symbol == mri_common_symbol);
-                 line_label->sy_value.X_add_number += 1;
+                 expressionS *symexp;
+
+                 symexp = symbol_get_value_expression (line_label);
+                 know (symexp->X_op == O_symbol);
+                 know (symexp->X_add_symbol == mri_common_symbol);
+                 symexp->X_add_number += 1;
                }
            }
        }
@@ -2774,7 +2915,7 @@ s_space (mult)
          do_align (1, (char *) NULL, 0, 0);
          if (line_label != NULL)
            {
-             line_label->sy_frag = frag_now;
+             symbol_set_frag (line_label, frag_now);
              S_SET_VALUE (line_label, frag_now_fix ());
            }
        }
@@ -2802,7 +2943,7 @@ s_space (mult)
       || (mult != 0 && mult != 1 && val.X_add_number != 0))
     {
       if (exp.X_op != O_constant)
-       as_bad (_("Unsupported variable size or fill value"));
+       as_bad (_("unsupported variable size or fill value"));
       else
        {
          offsetT i;
@@ -2826,7 +2967,7 @@ s_space (mult)
          bytes = repeat;
          if (repeat <= 0)
            {
-             if (! flag_mri)
+             if (!flag_mri)
                as_warn (_(".space repeat count is zero, ignored"));
              else if (repeat < 0)
                as_warn (_(".space repeat count is negative, ignored"));
@@ -2861,11 +3002,13 @@ s_space (mult)
              as_bad (_("space allocation too complex in absolute section"));
              subseg_set (text_section, 0);
            }
+
          if (mri_common_symbol != NULL)
            {
              as_bad (_("space allocation too complex in common section"));
              mri_common_symbol = NULL;
            }
+
          if (!need_pass_2)
            p = frag_var (rs_space, 1, 1, (relax_substateT) 0,
                          make_expr_symbol (&exp), (offsetT) 0, (char *) 0);
@@ -2883,10 +3026,10 @@ s_space (mult)
   if (flag_mri && (bytes & 1) != 0)
     mri_pending_align = 1;
 
+  demand_empty_rest_of_line ();
+
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }
 
 /* This is like s_space, but the value is a floating point number with
@@ -2912,9 +3055,9 @@ s_float_space (float_type)
   if (*input_line_pointer != ',')
     {
       as_bad (_("missing value"));
+      ignore_rest_of_line ();
       if (flag_mri)
        mri_comment_end (stop, stopc);
-      ignore_rest_of_line ();
       return;
     }
 
@@ -2925,7 +3068,7 @@ s_float_space (float_type)
   /* Skip any 0{letter} that may be present.  Don't even check if the
    * letter is legal.  */
   if (input_line_pointer[0] == '0'
-      && isalpha ((unsigned char) input_line_pointer[1]))
+      && ISALPHA (input_line_pointer[1]))
     input_line_pointer += 2;
 
   /* Accept :xxxx, where the x's are hex digits, for a floating point
@@ -2935,9 +3078,9 @@ s_float_space (float_type)
       flen = hex_float (float_type, temp);
       if (flen < 0)
        {
+         ignore_rest_of_line ();
          if (flag_mri)
            mri_comment_end (stop, stopc);
-         ignore_rest_of_line ();
          return;
        }
     }
@@ -2950,10 +3093,10 @@ s_float_space (float_type)
       know (flen > 0);
       if (err)
        {
-         as_bad (_("Bad floating literal: %s"), err);
+         as_bad (_("bad floating literal: %s"), err);
+         ignore_rest_of_line ();
          if (flag_mri)
            mri_comment_end (stop, stopc);
-         ignore_rest_of_line ();
          return;
        }
     }
@@ -2966,17 +3109,17 @@ s_float_space (float_type)
       memcpy (p, temp, (unsigned int) flen);
     }
 
+  demand_empty_rest_of_line ();
+
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }
 
 /* Handle the .struct pseudo-op, as found in MIPS assemblers.  */
 
 void
 s_struct (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   char *stop = NULL;
   char stopc;
@@ -2985,14 +3128,14 @@ s_struct (ignore)
     stop = mri_comment_field (&stopc);
   abs_section_offset = get_absolute_expression ();
   subseg_set (absolute_section, 0);
+  demand_empty_rest_of_line ();
   if (flag_mri)
     mri_comment_end (stop, stopc);
-  demand_empty_rest_of_line ();
 }
 
 void
 s_text (ignore)
-     int ignore;
+     int ignore ATTRIBUTE_UNUSED;
 {
   register int temp;
 
@@ -3002,55 +3145,64 @@ s_text (ignore)
 #ifdef OBJ_VMS
   const_flag &= ~IN_DEFAULT_SECTION;
 #endif
-}                              /* s_text() */
+}
 \f
-
-void 
+void
 demand_empty_rest_of_line ()
 {
   SKIP_WHITESPACE ();
   if (is_end_of_line[(unsigned char) *input_line_pointer])
-    {
-      input_line_pointer++;
-    }
+    input_line_pointer++;
   else
-    {
-      ignore_rest_of_line ();
-    }
-  /* Return having already swallowed end-of-line. */
-}                              /* Return pointing just after end-of-line. */
+    ignore_rest_of_line ();
+
+  /* Return having already swallowed end-of-line.  */
+}
 
 void
-ignore_rest_of_line ()         /* For suspect lines: gives warning. */
+ignore_rest_of_line ()
 {
+  /* For suspect lines: gives warning.  */
   if (!is_end_of_line[(unsigned char) *input_line_pointer])
     {
-      if (isprint ((unsigned char) *input_line_pointer))
-       as_bad (_("Rest of line ignored. First ignored character is `%c'."),
-               *input_line_pointer);
+      if (ISPRINT (*input_line_pointer))
+       as_warn (_("rest of line ignored; first ignored character is `%c'"),
+                *input_line_pointer);
       else
-       as_bad (_("Rest of line ignored. First ignored character valued 0x%x."),
-               *input_line_pointer);
+       as_warn (_("rest of line ignored; first ignored character valued 0x%x"),
+                *input_line_pointer);
+
       while (input_line_pointer < buffer_limit
             && !is_end_of_line[(unsigned char) *input_line_pointer])
-       {
-         input_line_pointer++;
-       }
+       input_line_pointer++;
     }
-  input_line_pointer++;                /* Return pointing just after end-of-line. */
+
+  input_line_pointer++;
+
+  /* Return pointing just after end-of-line.  */
   know (is_end_of_line[(unsigned char) input_line_pointer[-1]]);
 }
 
-/*
- *                     pseudo_set()
- *
- * In: Pointer to a symbol.
- *     Input_line_pointer->expression.
- *
- * Out:        Input_line_pointer->just after any whitespace after expression.
- *     Tried to set symbol to value of expression.
- *     Will change symbols type, value, and frag;
- */
+void
+discard_rest_of_line ()
+{
+  while (input_line_pointer < buffer_limit
+        && !is_end_of_line[(unsigned char) *input_line_pointer])
+    input_line_pointer++;
+
+  input_line_pointer++;
+
+  /* Return pointing just after end-of-line.  */
+  know (is_end_of_line[(unsigned char) input_line_pointer[-1]]);
+}
+
+/* In: Pointer to a symbol.
+       Input_line_pointer->expression.
+
+   Out:        Input_line_pointer->just after any whitespace after expression.
+       Tried to set symbol to value of expression.
+       Will change symbols type, value, and frag;  */
+
 void
 pseudo_set (symbolP)
      symbolS *symbolP;
@@ -3060,7 +3212,7 @@ pseudo_set (symbolP)
   int ext;
 #endif /* OBJ_AOUT or OBJ_BOUT */
 
-  know (symbolP);              /* NULL pointer is logic error. */
+  know (symbolP);              /* NULL pointer is logic error.  */
 #if (defined (OBJ_AOUT) || defined (OBJ_BOUT)) && ! defined (BFD_ASSEMBLER)
   ext = S_IS_EXTERNAL (symbolP);
 #endif /* OBJ_AOUT or OBJ_BOUT */
@@ -3068,21 +3220,20 @@ pseudo_set (symbolP)
   (void) expression (&exp);
 
   if (exp.X_op == O_illegal)
-    as_bad (_("illegal expression; zero assumed"));
+    as_bad (_("illegal expression"));
   else if (exp.X_op == O_absent)
-    as_bad (_("missing expression; zero assumed"));
+    as_bad (_("missing expression"));
   else if (exp.X_op == O_big)
     {
       if (exp.X_add_number > 0)
-       as_bad (_("bignum invalid; zero assumed"));
+       as_bad (_("bignum invalid"));
       else
-       as_bad (_("floating point number invalid; zero assumed"));
+       as_bad (_("floating point number invalid"));
     }
   else if (exp.X_op == O_subtract
-          && (S_GET_SEGMENT (exp.X_add_symbol)
-              == S_GET_SEGMENT (exp.X_op_symbol))
           && SEG_NORMAL (S_GET_SEGMENT (exp.X_add_symbol))
-          && exp.X_add_symbol->sy_frag == exp.X_op_symbol->sy_frag)
+          && (symbol_get_frag (exp.X_add_symbol)
+              == symbol_get_frag (exp.X_op_symbol)))
     {
       exp.X_op = O_constant;
       exp.X_add_number = (S_GET_VALUE (exp.X_add_symbol)
@@ -3106,19 +3257,21 @@ pseudo_set (symbolP)
 #endif /* OBJ_AOUT or OBJ_BOUT */
       S_SET_VALUE (symbolP, (valueT) exp.X_add_number);
       if (exp.X_op != O_constant)
-        symbolP->sy_frag = &zero_address_frag;
+       symbol_set_frag (symbolP, &zero_address_frag);
       break;
 
     case O_register:
       S_SET_SEGMENT (symbolP, reg_section);
       S_SET_VALUE (symbolP, (valueT) exp.X_add_number);
-      symbolP->sy_frag = &zero_address_frag;
+      symbol_set_frag (symbolP, &zero_address_frag);
       break;
 
     case O_symbol:
       if (S_GET_SEGMENT (exp.X_add_symbol) == undefined_section
          || exp.X_add_number != 0)
-       symbolP->sy_value = exp;
+       symbol_set_value_expression (symbolP, &exp);
+      else if (symbol_section_p (symbolP))
+       as_bad ("attempt to set value of section symbol");
       else
        {
          symbolS *s = exp.X_add_symbol;
@@ -3132,7 +3285,7 @@ pseudo_set (symbolP)
 #endif /* OBJ_AOUT or OBJ_BOUT */
          S_SET_VALUE (symbolP,
                       exp.X_add_number + S_GET_VALUE (s));
-         symbolP->sy_frag = s->sy_frag;
+         symbol_set_frag (symbolP, symbol_get_frag (s));
          copy_symbol_attributes (symbolP, s);
        }
       break;
@@ -3140,25 +3293,23 @@ pseudo_set (symbolP)
     default:
       /* The value is some complex expression.
         FIXME: Should we set the segment to anything?  */
-      symbolP->sy_value = exp;
+      symbol_set_value_expression (symbolP, &exp);
       break;
     }
 }
 \f
-/*
- *                     cons()
- *
- * CONStruct more frag of .bytes, or .words etc.
- * Should need_pass_2 be 1 then emit no frag(s).
- * This understands EXPRESSIONS.
- *
- * Bug (?)
- *
- * This has a split personality. We use expression() to read the
- * value. We can detect if the value won't fit in a byte or word.
- * But we can't detect if expression() discarded significant digits
- * in the case of a long. Not worth the crocks required to fix it.
- */
+/*                     cons()
+
+   CONStruct more frag of .bytes, or .words etc.
+   Should need_pass_2 be 1 then emit no frag(s).
+   This understands EXPRESSIONS.
+
+   Bug (?)
+
+   This has a split personality. We use expression() to read the
+   value. We can detect if the value won't fit in a byte or word.
+   But we can't detect if expression() discarded significant digits
+   in the case of a long. Not worth the crocks required to fix it.  */
 
 /* Select a parser for cons expressions.  */
 
@@ -3170,13 +3321,15 @@ pseudo_set (symbolP)
    are defined, which is the normal case, then only simple expressions
    are permitted.  */
 
+#ifdef TC_M68K
 static void
 parse_mri_cons PARAMS ((expressionS *exp, unsigned int nbytes));
+#endif
 
 #ifndef TC_PARSE_CONS_EXPRESSION
 #ifdef BITFIELD_CONS_EXPRESSIONS
 #define TC_PARSE_CONS_EXPRESSION(EXP, NBYTES) parse_bitfield_cons (EXP, NBYTES)
-static void 
+static void
 parse_bitfield_cons PARAMS ((expressionS *exp, unsigned int nbytes));
 #endif
 #ifdef REPEAT_CONS_EXPRESSIONS
@@ -3191,12 +3344,12 @@ parse_repeat_cons PARAMS ((expressionS *exp, unsigned int nbytes));
 #endif
 #endif
 
-/* worker to do .byte etc statements */
-/* clobbers input_line_pointer, checks */
-/* end-of-line. */
-static void 
+/* Worker to do .byte etc statements.
+   Clobbers input_line_pointer and checks end-of-line.  */
+
+static void
 cons_worker (nbytes, rva)
-     register int nbytes;      /* 1=.byte, 2=.word, 4=.long */
+     register int nbytes;      /* 1=.byte, 2=.word, 4=.long */
      int rva;
 {
   int c;
@@ -3213,9 +3366,9 @@ cons_worker (nbytes, rva)
 
   if (is_it_end_of_statement ())
     {
+      demand_empty_rest_of_line ();
       if (flag_mri)
        mri_comment_end (stop, stopc);
-      demand_empty_rest_of_line ();
       return;
     }
 
@@ -3226,9 +3379,11 @@ cons_worker (nbytes, rva)
   c = 0;
   do
     {
+#ifdef TC_M68K
       if (flag_m68k_mri)
        parse_mri_cons (&exp, (unsigned int) nbytes);
       else
+#endif
        TC_PARSE_CONS_EXPRESSION (&exp, (unsigned int) nbytes);
 
       if (rva)
@@ -3249,15 +3404,14 @@ cons_worker (nbytes, rva)
   if (flag_mri && nbytes == 1 && (c & 1) != 0)
     mri_pending_align = 1;
 
-  input_line_pointer--;                /* Put terminator back into stream. */
+  input_line_pointer--;                /* Put terminator back into stream.  */
+
+  demand_empty_rest_of_line ();
 
   if (flag_mri)
     mri_comment_end (stop, stopc);
-
-  demand_empty_rest_of_line ();
 }
 
-
 void
 cons (size)
      int size;
@@ -3265,7 +3419,7 @@ cons (size)
   cons_worker (size, 0);
 }
 
-void 
+void
 s_rva (size)
      int size;
 {
@@ -3369,13 +3523,13 @@ emit_expr (exp, nbytes)
   /* Handle a negative bignum.  */
   if (op == O_uminus
       && exp->X_add_number == 0
-      && exp->X_add_symbol->sy_value.X_op == O_big
-      && exp->X_add_symbol->sy_value.X_add_number > 0)
+      && symbol_get_value_expression (exp->X_add_symbol)->X_op == O_big
+      && symbol_get_value_expression (exp->X_add_symbol)->X_add_number > 0)
     {
       int i;
       unsigned long carry;
 
-      exp = &exp->X_add_symbol->sy_value;
+      exp = symbol_get_value_expression (exp->X_add_symbol);
 
       /* Negate the bignum: one's complement each digit and add 1.  */
       carry = 1;
@@ -3383,7 +3537,7 @@ emit_expr (exp, nbytes)
        {
          unsigned long next;
 
-         next = (((~ (generic_bignum[i] & LITTLENUM_MASK))
+         next = (((~(generic_bignum[i] & LITTLENUM_MASK))
                   & LITTLENUM_MASK)
                  + carry);
          generic_bignum[i] = next & LITTLENUM_MASK;
@@ -3405,7 +3559,7 @@ emit_expr (exp, nbytes)
     }
   else if (op == O_big && exp->X_add_number <= 0)
     {
-      as_bad (_("floating point number invalid; zero assumed"));
+      as_bad (_("floating point number invalid"));
       exp->X_add_number = 0;
       op = O_constant;
     }
@@ -3427,6 +3581,8 @@ emit_expr (exp, nbytes)
       x = (struct broken_word *) xmalloc (sizeof (struct broken_word));
       x->next_broken_word = broken_words;
       broken_words = x;
+      x->seg = now_seg;
+      x->subseg = now_subseg;
       x->frag = frag_now;
       x->word_goes_here = p;
       x->dispfrag = 0;
@@ -3434,6 +3590,7 @@ emit_expr (exp, nbytes)
       x->sub = exp->X_op_symbol;
       x->addnum = exp->X_add_number;
       x->added = 0;
+      x->use_jump = 0;
       new_broken_words++;
       return;
     }
@@ -3446,7 +3603,7 @@ emit_expr (exp, nbytes)
       valueT val;
       int gencnt;
 
-      if (! exp->X_unsigned && exp->X_add_number < 0)
+      if (!exp->X_unsigned && exp->X_add_number < 0)
        extra_digit = (valueT) -1;
       val = (valueT) exp->X_add_number;
       gencnt = 0;
@@ -3470,7 +3627,7 @@ emit_expr (exp, nbytes)
       register valueT unmask;
 
       /* JF << of >= number of bits in the object is undefined.  In
-        particular SPARC (Sun 4) has problems */
+        particular SPARC (Sun 4) has problems */
       if (nbytes >= sizeof (valueT))
        {
          mask = 0;
@@ -3481,16 +3638,16 @@ emit_expr (exp, nbytes)
        }
       else
        {
-         /* Don't store these bits. */
+         /* Don't store these bits.  */
          mask = ~(valueT) 0 << (BITS_PER_CHAR * nbytes);
          hibit = (valueT) 1 << (nbytes * BITS_PER_CHAR - 1);
        }
 
-      unmask = ~mask;          /* Do store these bits. */
+      unmask = ~mask;          /* Do store these bits.  */
 
 #ifdef NEVER
       "Do this mod if you want every overflow check to assume SIGNED 2's complement data.";
-      mask = ~(unmask >> 1);   /* Includes sign bit now. */
+      mask = ~(unmask >> 1);   /* Includes sign bit now.  */
 #endif
 
       get = exp->X_add_number;
@@ -3498,11 +3655,11 @@ emit_expr (exp, nbytes)
       if ((get & mask) != 0
          && ((get & mask) != mask
              || (get & hibit) == 0))
-       {               /* Leading bits contain both 0s & 1s. */
-         as_warn (_("Value 0x%lx truncated to 0x%lx."),
+       {               /* Leading bits contain both 0s & 1s.  */
+         as_warn (_("value 0x%lx truncated to 0x%lx"),
                   (unsigned long) get, (unsigned long) use);
        }
-      /* put bytes in right order. */
+      /* Put bytes in right order.  */
       md_number_to_chars (p, use, (int) nbytes);
     }
   else if (op == O_big)
@@ -3515,7 +3672,7 @@ emit_expr (exp, nbytes)
       size = exp->X_add_number * CHARS_PER_LITTLENUM;
       if (nbytes < size)
        {
-         as_warn (_("Bignum truncated to %d bytes"), nbytes);
+         as_warn (_("bignum truncated to %d bytes"), nbytes);
          size = nbytes;
        }
 
@@ -3529,7 +3686,7 @@ emit_expr (exp, nbytes)
            }
 
          nums = generic_bignum + size / CHARS_PER_LITTLENUM;
-         while (size > 0)
+         while (size >= CHARS_PER_LITTLENUM)
            {
              --nums;
              md_number_to_chars (p, (valueT) *nums, CHARS_PER_LITTLENUM);
@@ -3540,7 +3697,7 @@ emit_expr (exp, nbytes)
       else
        {
          nums = generic_bignum;
-         while (size > 0)
+         while (size >= CHARS_PER_LITTLENUM)
            {
              md_number_to_chars (p, (valueT) *nums, CHARS_PER_LITTLENUM);
              ++nums;
@@ -3549,7 +3706,7 @@ emit_expr (exp, nbytes)
              nbytes -= CHARS_PER_LITTLENUM;
            }
 
-         while (nbytes > 0)
+         while (nbytes >= CHARS_PER_LITTLENUM)
            {
              md_number_to_chars (p, extra_digit, CHARS_PER_LITTLENUM);
              nbytes -= CHARS_PER_LITTLENUM;
@@ -3637,7 +3794,7 @@ emit_expr (exp, nbytes)
    To use this function the tc-XXX.h file should define
    BITFIELD_CONS_EXPRESSIONS.  */
 
-static void 
+static void
 parse_bitfield_cons (exp, nbytes)
      expressionS *exp;
      unsigned int nbytes;
@@ -3648,7 +3805,8 @@ parse_bitfield_cons (exp, nbytes)
   (void) expression (exp);
 
   if (*input_line_pointer == ':')
-    {                  /* bitfields */
+    {
+      /* Bitfields.  */
       long value = 0;
 
       for (;;)
@@ -3659,7 +3817,7 @@ parse_bitfield_cons (exp, nbytes)
            {
              input_line_pointer = hold;
              break;
-           }                   /* next piece is not a bitfield */
+           }                   /* Next piece is not a bitfield.  */
 
          /* In the general case, we can't allow
             full expressions with symbol
@@ -3677,14 +3835,14 @@ parse_bitfield_cons (exp, nbytes)
             backends.  I'm lazy. I'll take any
             SEG_ABSOLUTE. I think that means that
             you can use a previous .set or
-            .equ type symbol.  xoxorich. */
+            .equ type symbol.  xoxorich.  */
 
          if (exp->X_op == O_absent)
            {
              as_warn (_("using a bit field width of zero"));
              exp->X_add_number = 0;
              exp->X_op = O_constant;
-           }                   /* implied zero width bitfield */
+           }                   /* Implied zero width bitfield.  */
 
          if (exp->X_op != O_constant)
            {
@@ -3693,14 +3851,14 @@ parse_bitfield_cons (exp, nbytes)
              *input_line_pointer = ':';
              demand_empty_rest_of_line ();
              return;
-           }                   /* too complex */
+           }                   /* Too complex.  */
 
          if ((width = exp->X_add_number) > (BITS_PER_CHAR * nbytes))
            {
              as_warn (_("field width %lu too big to fit in %d bytes: truncated to %d bits"),
                       width, nbytes, (BITS_PER_CHAR * nbytes));
              width = BITS_PER_CHAR * nbytes;
-           }                   /* too big */
+           }                   /* Too big.  */
 
          if (width > bits_available)
            {
@@ -3708,9 +3866,10 @@ parse_bitfield_cons (exp, nbytes)
              input_line_pointer = hold;
              exp->X_add_number = value;
              break;
-           }                   /* won't fit */
+           }                   /* Won't fit.  */
 
-         hold = ++input_line_pointer; /* skip ':' */
+         /* Skip ':'.  */
+         hold = ++input_line_pointer;
 
          (void) expression (exp);
          if (exp->X_op != O_constant)
@@ -3722,7 +3881,7 @@ parse_bitfield_cons (exp, nbytes)
              *input_line_pointer = cache;
              demand_empty_rest_of_line ();
              return;
-           }                   /* too complex */
+           }                   /* Too complex.  */
 
          value |= ((~(-1 << width) & exp->X_add_number)
                    << ((BITS_PER_CHAR * nbytes) - bits_available));
@@ -3732,22 +3891,23 @@ parse_bitfield_cons (exp, nbytes)
              || *input_line_pointer != ',')
            {
              break;
-           }                   /* all the bitfields we're gonna get */
+           }                   /* All the bitfields we're gonna get.  */
 
          hold = ++input_line_pointer;
          (void) expression (exp);
-       }                       /* forever loop */
+       }
 
       exp->X_add_number = value;
       exp->X_op = O_constant;
       exp->X_unsigned = 1;
-    }                          /* if looks like a bitfield */
-}                              /* parse_bitfield_cons() */
+    }
+}
 
 #endif /* BITFIELD_CONS_EXPRESSIONS */
 \f
 /* Handle an MRI style string expression.  */
 
+#ifdef TC_M68K
 static void
 parse_mri_cons (exp, nbytes)
      expressionS *exp;
@@ -3789,16 +3949,18 @@ parse_mri_cons (exp, nbytes)
          result = (result << 8) | (*input_line_pointer++);
        }
 
-      /* Left justify */
+      /* Left justify */
       while (scan < nbytes)
        {
          result <<= 8;
          scan++;
        }
-      /* Create correct expression */
+
+      /* Create correct expression.  */
       exp->X_op = O_constant;
       exp->X_add_number = result;
-      /* Fake it so that we can read the next char too */
+
+      /* Fake it so that we can read the next char too.  */
       if (input_line_pointer[0] != '\'' ||
          (input_line_pointer[0] == '\'' && input_line_pointer[1] == '\''))
        {
@@ -3810,6 +3972,7 @@ parse_mri_cons (exp, nbytes)
        input_line_pointer++;
     }
 }
+#endif /* TC_M68K */
 \f
 #ifdef REPEAT_CONS_EXPRESSIONS
 
@@ -3840,7 +4003,7 @@ parse_repeat_cons (exp, nbytes)
   if (count.X_op != O_constant
       || count.X_add_number <= 0)
     {
-      as_warn (_("Unresolvable or nonpositive repeat count; using 1"));
+      as_warn (_("unresolvable or nonpositive repeat count; using 1"));
       return;
     }
 
@@ -3891,7 +4054,7 @@ hex_float (float_type, bytes)
       break;
 
     default:
-      as_bad (_("Unknown floating type type '%c'"), float_type);
+      as_bad (_("unknown floating type type '%c'"), float_type);
       return -1;
     }
 
@@ -3904,7 +4067,7 @@ hex_float (float_type, bytes)
       int d;
 
       /* The MRI assembler accepts arbitrary underscores strewn about
-        through the hex constant, so we ignore them as well. */
+        through the hex constant, so we ignore them as well.  */
       if (*input_line_pointer == '_')
        {
          ++input_line_pointer;
@@ -3913,7 +4076,7 @@ hex_float (float_type, bytes)
 
       if (i >= length)
        {
-         as_warn (_("Floating point constant too large"));
+         as_warn (_("floating point constant too large"));
          return -1;
        }
       d = hex_value (*input_line_pointer) << 4;
@@ -3943,34 +4106,31 @@ hex_float (float_type, bytes)
   return length;
 }
 
-/*
- *                     float_cons()
- *
- * CONStruct some more frag chars of .floats .ffloats etc.
- * Makes 0 or more new frags.
- * If need_pass_2 == 1, no frags are emitted.
- * This understands only floating literals, not expressions. Sorry.
- *
- * A floating constant is defined by atof_generic(), except it is preceded
- * by 0d 0f 0g or 0h. After observing the STRANGE way my BSD AS does its
- * reading, I decided to be incompatible. This always tries to give you
- * rounded bits to the precision of the pseudo-op. Former AS did premature
- * truncatation, restored noisy bits instead of trailing 0s AND gave you
- * a choice of 2 flavours of noise according to which of 2 floating-point
- * scanners you directed AS to use.
- *
- * In: input_line_pointer->whitespace before, or '0' of flonum.
- *
- */
+/*                     float_cons()
+
+   CONStruct some more frag chars of .floats .ffloats etc.
+   Makes 0 or more new frags.
+   If need_pass_2 == 1, no frags are emitted.
+   This understands only floating literals, not expressions. Sorry.
+
+   A floating constant is defined by atof_generic(), except it is preceded
+   by 0d 0f 0g or 0h. After observing the STRANGE way my BSD AS does its
+   reading, I decided to be incompatible. This always tries to give you
+   rounded bits to the precision of the pseudo-op. Former AS did premature
+   truncatation, restored noisy bits instead of trailing 0s AND gave you
+   a choice of 2 flavours of noise according to which of 2 floating-point
+   scanners you directed AS to use.
+
+   In: input_line_pointer->whitespace before, or '0' of flonum.  */
 
 void
 float_cons (float_type)
-     /* Clobbers input_line-pointer, checks end-of-line. */
-     register int float_type;  /* 'f':.ffloat ... 'F':.float ... */
+     /* Clobbers input_line-pointer, checks end-of-line.  */
+     register int float_type;  /* 'f':.ffloat ... 'F':.float ...  */
 {
   register char *p;
-  int length;                  /* Number of chars in an object. */
-  register char *err;          /* Error from scanning floating literal. */
+  int length;                  /* Number of chars in an object.  */
+  register char *err;          /* Error from scanning floating literal.  */
   char temp[MAXIMUM_NUMBER_OF_CHARS_FOR_FLOAT];
 
   if (is_it_end_of_statement ())
@@ -3985,16 +4145,15 @@ float_cons (float_type)
 
   do
     {
-      /* input_line_pointer->1st char of a flonum (we hope!). */
+      /* input_line_pointer->1st char of a flonum (we hope!).  */
       SKIP_WHITESPACE ();
 
       /* Skip any 0{letter} that may be present. Don't even check if the
-       * letter is legal. Someone may invent a "z" format and this routine
-       * has no use for such information. Lusers beware: you get
-       * diagnostics if your input is ill-conditioned.
-       */
+         letter is legal. Someone may invent a "z" format and this routine
+         has no use for such information. Lusers beware: you get
+         diagnostics if your input is ill-conditioned.  */
       if (input_line_pointer[0] == '0'
-         && isalpha ((unsigned char) input_line_pointer[1]))
+         && ISALPHA (input_line_pointer[1]))
        input_line_pointer += 2;
 
       /* Accept :xxxx, where the x's are hex digits, for a floating
@@ -4016,7 +4175,7 @@ float_cons (float_type)
          know (length > 0);
          if (err)
            {
-             as_bad (_("Bad floating literal: %s"), err);
+             as_bad (_("bad floating literal: %s"), err);
              ignore_rest_of_line ();
              return;
            }
@@ -4035,11 +4194,10 @@ float_cons (float_type)
 
              ++input_line_pointer;
              expression (&count_exp);
+
              if (count_exp.X_op != O_constant
                  || count_exp.X_add_number <= 0)
-               {
-                 as_warn (_("unresolvable or nonpositive repeat count; using 1"));
-               }
+               as_warn (_("unresolvable or nonpositive repeat count; using 1"));
              else
                count = count_exp.X_add_number;
            }
@@ -4055,11 +4213,12 @@ float_cons (float_type)
     }
   while (*input_line_pointer++ == ',');
 
-  --input_line_pointer;                /* Put terminator back into stream.  */
+  /* Put terminator back into stream.  */
+  --input_line_pointer;
   demand_empty_rest_of_line ();
-}                              /* float_cons() */
+}
 \f
-/* Return the size of a LEB128 value */
+/* Return the size of a LEB128 value */
 
 static inline int
 sizeof_sleb128 (value)
@@ -4165,7 +4324,7 @@ output_uleb128 (p, value)
   return p - orig;
 }
 
-inline int
+int
 output_leb128 (p, value, sign)
      char *p;
      valueT value;
@@ -4181,7 +4340,7 @@ output_leb128 (p, value, sign)
    we don't output for NULL values of P.  It isn't really as critical as
    for "normal" values that this be streamlined.  */
 
-static int
+static inline int
 output_big_sleb128 (p, bignum, size)
      char *p;
      LITTLENUM_TYPE *bignum;
@@ -4193,7 +4352,7 @@ output_big_sleb128 (p, bignum, size)
   unsigned byte;
 
   /* Strip leading sign extensions off the bignum.  */
-  while (size > 0 && bignum[size-1] == (LITTLENUM_TYPE)-1)
+  while (size > 0 && bignum[size - 1] == (LITTLENUM_TYPE) -1)
     size--;
 
   do
@@ -4213,7 +4372,7 @@ output_big_sleb128 (p, bignum, size)
       if (size == 0)
        {
          if ((val == 0 && (byte & 0x40) == 0)
-             || (~(val | ~(((valueT)1 << loaded) - 1)) == 0
+             || (~(val | ~(((valueT) 1 << loaded) - 1)) == 0
                  && (byte & 0x40) != 0))
            byte |= 0x80;
        }
@@ -4227,7 +4386,7 @@ output_big_sleb128 (p, bignum, size)
   return p - orig;
 }
 
-static int
+static inline int
 output_big_uleb128 (p, bignum, size)
      char *p;
      LITTLENUM_TYPE *bignum;
@@ -4240,7 +4399,7 @@ output_big_uleb128 (p, bignum, size)
 
   /* Strip leading zeros off the bignum.  */
   /* XXX: Is this needed?  */
-  while (size > 0 && bignum[size-1] == 0)
+  while (size > 0 && bignum[size - 1] == 0)
     size--;
 
   do
@@ -4269,7 +4428,7 @@ output_big_uleb128 (p, bignum, size)
   return p - orig;
 }
 
-static inline int
+static int
 output_big_leb128 (p, bignum, size, sign)
      char *p;
      LITTLENUM_TYPE *bignum;
@@ -4285,11 +4444,12 @@ output_big_leb128 (p, bignum, size, sign)
    leb128 value.  */
 
 void
-emit_leb128_expr(exp, sign)
+emit_leb128_expr (exp, sign)
      expressionS *exp;
      int sign;
 {
   operatorT op = exp->X_op;
+  int nbytes;
 
   if (op == O_absent || op == O_illegal)
     {
@@ -4299,7 +4459,7 @@ emit_leb128_expr(exp, sign)
     }
   else if (op == O_big && exp->X_add_number <= 0)
     {
-      as_bad (_("floating point number invalid; zero assumed"));
+      as_bad (_("floating point number invalid"));
       exp->X_add_number = 0;
       op = O_constant;
     }
@@ -4309,6 +4469,17 @@ emit_leb128_expr(exp, sign)
       op = O_constant;
     }
 
+  /* Let check_eh_frame know that data is being emitted.  nbytes == -1 is
+     a signal that this is leb128 data.  It shouldn't optimize this away.  */
+  nbytes = -1;
+  if (check_eh_frame (exp, &nbytes))
+    abort ();
+
+  /* Let the backend know that subsequent data may be byte aligned.  */
+#ifdef md_cons_align
+  md_cons_align (1);
+#endif
+
   if (op == O_constant)
     {
       /* If we've got a constant, emit the thing directly right now.  */
@@ -4334,10 +4505,10 @@ emit_leb128_expr(exp, sign)
     }
   else
     {
-      /* Otherwise, we have to create a variable sized fragment and 
+      /* Otherwise, we have to create a variable sized fragment and
         resolve things later.  */
 
-      frag_var (rs_leb128, sizeof_uleb128 (~(valueT)0), 0, sign,
+      frag_var (rs_leb128, sizeof_uleb128 (~(valueT) 0), 0, sign,
                make_expr_symbol (exp), 0, (char *) NULL);
     }
 }
@@ -4350,28 +4521,25 @@ s_leb128 (sign)
 {
   expressionS exp;
 
-  do {
-    expression (&exp);
-    emit_leb128_expr (&exp, sign);
-  } while (*input_line_pointer++ == ',');
+  do
+    {
+      expression (&exp);
+      emit_leb128_expr (&exp, sign);
+    }
+  while (*input_line_pointer++ == ',');
 
   input_line_pointer--;
   demand_empty_rest_of_line ();
 }
 \f
-/*
- *                     stringer()
- *
- * We read 0 or more ',' seperated, double-quoted strings.
- *
- * Caller should have checked need_pass_2 is FALSE because we don't check it.
- */
-
-
-void 
-stringer (append_zero)         /* Worker to do .ascii etc statements. */
-     /* Checks end-of-line. */
-     register int append_zero; /* 0: don't append '\0', else 1 */
+/* We read 0 or more ',' separated, double-quoted strings.
+   Caller should have checked need_pass_2 is FALSE because we don't
+   check it.  */
+
+void
+stringer (append_zero)         /* Worker to do .ascii etc statements.  */
+     /* Checks end-of-line.  */
+     register int append_zero; /* 0: don't append '\0', else 1.  */
 {
   register unsigned int c;
   char *start;
@@ -4380,30 +4548,36 @@ stringer (append_zero)          /* Worker to do .ascii etc statements. */
   md_flush_pending_output ();
 #endif
 
-  /*
-   * The following awkward logic is to parse ZERO or more strings,
-   * comma seperated. Recall a string expression includes spaces
-   * before the opening '\"' and spaces after the closing '\"'.
-   * We fake a leading ',' if there is (supposed to be)
-   * a 1st, expression. We keep demanding expressions for each
-   * ','.
-   */
+  /* The following awkward logic is to parse ZERO or more strings,
+     comma separated. Recall a string expression includes spaces
+     before the opening '\"' and spaces after the closing '\"'.
+     We fake a leading ',' if there is (supposed to be)
+     a 1st, expression. We keep demanding expressions for each ','.  */
   if (is_it_end_of_statement ())
     {
-      c = 0;                   /* Skip loop. */
-      ++input_line_pointer;    /* Compensate for end of loop. */
+      c = 0;                   /* Skip loop.  */
+      ++input_line_pointer;    /* Compensate for end of loop.  */
     }
   else
     {
-      c = ',';                 /* Do loop. */
+      c = ',';                 /* Do loop.  */
+    }
+  /* If we have been switched into the abs_section then we
+     will not have an obstack onto which we can hang strings.  */
+  if (now_seg == absolute_section)
+    {
+      as_bad (_("strings must be placed into a section"));
+      c = 0;
+      ignore_rest_of_line ();
     }
+  
   while (c == ',' || c == '<' || c == '"')
     {
       SKIP_WHITESPACE ();
       switch (*input_line_pointer)
        {
        case '\"':
-         ++input_line_pointer; /*->1st char of string. */
+         ++input_line_pointer; /*->1st char of string.  */
          start = input_line_pointer;
          while (is_a_char (c = next_char_of_string ()))
            {
@@ -4442,7 +4616,7 @@ stringer (append_zero)            /* Worker to do .ascii etc statements. */
          FRAG_APPEND_1_CHAR (c);
          if (*input_line_pointer != '>')
            {
-             as_bad (_("Expected <nn>"));
+             as_bad (_("expected <nn>"));
            }
          input_line_pointer++;
          break;
@@ -4460,9 +4634,9 @@ stringer (append_zero)            /* Worker to do .ascii etc statements. */
 /* FIXME-SOMEDAY: I had trouble here on characters with the
     high bits set.  We'll probably also have trouble with
     multibyte chars, wide chars, etc.  Also be careful about
-    returning values bigger than 1 byte.  xoxorich. */
+    returning values bigger than 1 byte.  xoxorich.  */
 
-unsigned int 
+unsigned int
 next_char_of_string ()
 {
   register unsigned int c;
@@ -4475,7 +4649,7 @@ next_char_of_string ()
       break;
 
     case '\n':
-      as_warn (_("Unterminated string: Newline inserted."));
+      as_warn (_("unterminated string; newline inserted"));
       bump_line_counters ();
       break;
 
@@ -4509,7 +4683,7 @@ next_char_of_string ()
 
        case '\\':
        case '"':
-         break;                /* As itself. */
+         break;                /* As itself.  */
 
        case '0':
        case '1':
@@ -4525,10 +4699,13 @@ next_char_of_string ()
            long number;
            int i;
 
-           for (i = 0, number = 0; isdigit (c) && i < 3; c = *input_line_pointer++, i++)
+           for (i = 0, number = 0;
+                ISDIGIT (c) && i < 3;
+                c = *input_line_pointer++, i++)
              {
                number = number * 8 + c - '0';
              }
+
            c = number & 0xff;
          }
          --input_line_pointer;
@@ -4541,11 +4718,11 @@ next_char_of_string ()
 
            number = 0;
            c = *input_line_pointer++;
-           while (isxdigit (c))
+           while (ISXDIGIT (c))
              {
-               if (isdigit (c))
+               if (ISDIGIT (c))
                  number = number * 16 + c - '0';
-               else if (isupper (c))
+               else if (ISUPPER (c))
                  number = number * 16 + c - 'A' + 10;
                else
                  number = number * 16 + c - 'a' + 10;
@@ -4557,8 +4734,8 @@ next_char_of_string ()
          break;
 
        case '\n':
-         /* To be compatible with BSD 4.2 as: give the luser a linefeed!! */
-         as_warn (_("Unterminated string: Newline inserted."));
+         /* To be compatible with BSD 4.2 as: give the luser a linefeed!!  */
+         as_warn (_("unterminated string; newline inserted"));
          c = '\n';
          bump_line_counters ();
          break;
@@ -4566,20 +4743,20 @@ next_char_of_string ()
        default:
 
 #ifdef ONLY_STANDARD_ESCAPES
-         as_bad (_("Bad escaped character in string, '?' assumed"));
+         as_bad (_("bad escaped character in string"));
          c = '?';
 #endif /* ONLY_STANDARD_ESCAPES */
 
          break;
-       }                       /* switch on escaped char */
+       }
       break;
 #endif /* ! defined (NO_STRING_ESCAPES) */
 
     default:
       break;
-    }                          /* switch on char */
+    }
   return (c);
-}                              /* next_char_of_string() */
+}
 \f
 static segT
 get_segmented_expression (expP)
@@ -4592,7 +4769,7 @@ get_segmented_expression (expP)
       || expP->X_op == O_absent
       || expP->X_op == O_big)
     {
-      as_bad (_("expected address expression; zero assumed"));
+      as_bad (_("expected address expression"));
       expP->X_op = O_constant;
       expP->X_add_number = 0;
       retval = absolute_section;
@@ -4600,7 +4777,7 @@ get_segmented_expression (expP)
   return retval;
 }
 
-static segT 
+static segT
 get_known_segmented_expression (expP)
      register expressionS *expP;
 {
@@ -4622,38 +4799,42 @@ get_known_segmented_expression (expP)
     }
   know (retval == absolute_section || SEG_NORMAL (retval));
   return (retval);
-}                              /* get_known_segmented_expression() */
+}
+
+offsetT
+get_absolute_expr (exp)
+     expressionS *exp;
+{
+  expression (exp);
+  if (exp->X_op != O_constant)
+    {
+      if (exp->X_op != O_absent)
+       as_bad (_("bad or irreducible absolute expression"));
+      exp->X_add_number = 0;
+    }
+  return exp->X_add_number;
+}
 
 offsetT
 get_absolute_expression ()
 {
   expressionS exp;
 
-  expression (&exp);
-  if (exp.X_op != O_constant)
-    {
-      if (exp.X_op != O_absent)
-       as_bad (_("bad or irreducible absolute expression; zero assumed"));
-      exp.X_add_number = 0;
-    }
-  return exp.X_add_number;
+  return get_absolute_expr (&exp);
 }
 
-char                           /* return terminator */
+char                           /* Return terminator.  */
 get_absolute_expression_and_terminator (val_pointer)
-     long *val_pointer;                /* return value of expression */
+     long *val_pointer;                /* Return value of expression.  */
 {
   /* FIXME: val_pointer should probably be offsetT *.  */
   *val_pointer = (long) get_absolute_expression ();
   return (*input_line_pointer++);
 }
 \f
-/*
- *                     demand_copy_C_string()
- *
- * Like demand_copy_string, but return NULL if the string contains any '\0's.
- * Give a warning if that happens.
- */
+/* Like demand_copy_string, but return NULL if the string contains any '\0's.
+   Give a warning if that happens.  */
+
 char *
 demand_copy_C_string (len_pointer)
      int *len_pointer;
@@ -4671,19 +4852,17 @@ demand_copy_C_string (len_pointer)
              s = 0;
              len = 1;
              *len_pointer = 0;
-             as_bad (_("This string may not contain \'\\0\'"));
+             as_bad (_("this string may not contain \'\\0\'"));
            }
        }
     }
+
   return s;
 }
 \f
-/*
- *                     demand_copy_string()
- *
- * Demand string, but return a safe (=private) copy of the string.
- * Return NULL if we can't read a string here.
- */
+/* Demand string, but return a safe (=private) copy of the string.
+   Return NULL if we can't read a string here.  */
+
 char *
 demand_copy_string (lenP)
      int *lenP;
@@ -4696,7 +4875,7 @@ demand_copy_string (lenP)
   SKIP_WHITESPACE ();
   if (*input_line_pointer == '\"')
     {
-      input_line_pointer++;    /* Skip opening quote. */
+      input_line_pointer++;    /* Skip opening quote.  */
 
       while (is_a_char (c = next_char_of_string ()))
        {
@@ -4704,42 +4883,39 @@ demand_copy_string (lenP)
          len++;
        }
       /* JF this next line is so demand_copy_C_string will return a
-        null terminated string. */
+        null terminated string.  */
       obstack_1grow (&notes, '\0');
       retval = obstack_finish (&notes);
     }
   else
     {
-      as_warn (_("Missing string"));
+      as_warn (_("missing string"));
       retval = NULL;
       ignore_rest_of_line ();
     }
   *lenP = len;
   return (retval);
-}                              /* demand_copy_string() */
+}
 \f
-/*
- *             is_it_end_of_statement()
- *
- * In: Input_line_pointer->next character.
- *
- * Do: Skip input_line_pointer over all whitespace.
- *
- * Out:        1 if input_line_pointer->end-of-line.
-*/
-int 
+/* In: Input_line_pointer->next character.
+
+   Do: Skip input_line_pointer over all whitespace.
+
+   Out:        1 if input_line_pointer->end-of-line.  */
+
+int
 is_it_end_of_statement ()
 {
   SKIP_WHITESPACE ();
   return (is_end_of_line[(unsigned char) *input_line_pointer]);
-}                              /* is_it_end_of_statement() */
+}
 
-void 
+void
 equals (sym_name, reassign)
      char *sym_name;
      int reassign;
 {
-  register symbolS *symbolP;   /* symbol we are working with */
+  register symbolS *symbolP;   /* Symbol we are working with.  */
   char *stop = NULL;
   char stopc;
 
@@ -4755,7 +4931,7 @@ equals (sym_name, reassign)
 
   if (sym_name[0] == '.' && sym_name[1] == '\0')
     {
-      /* Turn '. = mumble' into a .org mumble */
+      /* Turn '. = mumble' into a .org mumble */
       register segT segment;
       expressionS exp;
 
@@ -4765,33 +4941,166 @@ equals (sym_name, reassign)
     }
   else
     {
+#ifdef OBJ_COFF
+      int local;
+
+      symbolP = symbol_find (sym_name);
+      local = symbolP == NULL;
+      if (local)
+#endif /* OBJ_COFF */
       symbolP = symbol_find_or_make (sym_name);
       /* Permit register names to be redefined.  */
-      if (! reassign
+      if (!reassign
          && S_IS_DEFINED (symbolP)
          && S_GET_SEGMENT (symbolP) != reg_section)
-       as_bad (_("symbol `%s' already defined"), S_GET_NAME (symbolP));
+       as_bad (_("symbol `%s' is already defined"), S_GET_NAME (symbolP));
+
+#ifdef OBJ_COFF
+      /* "set" symbols are local unless otherwise specified.  */
+      if (local)
+       SF_SET_LOCAL (symbolP);
+#endif /* OBJ_COFF */
+
       pseudo_set (symbolP);
     }
 
   if (flag_mri)
-    mri_comment_end (stop, stopc);
-}                              /* equals() */
+    {
+      /* Check garbage after the expression.  */
+      ignore_rest_of_line ();
+      mri_comment_end (stop, stopc);
+    }
+}
+
+/* .incbin -- include a file verbatim at the current location.  */
+
+void
+s_incbin (x)
+     int x ATTRIBUTE_UNUSED;
+{
+  FILE * binfile;
+  char * path;
+  char * filename;
+  char * binfrag;
+  long   skip = 0;
+  long   count = 0;
+  long   bytes;
+  int    len;
+
+#ifdef md_flush_pending_output
+  md_flush_pending_output ();
+#endif
+
+  SKIP_WHITESPACE ();
+  filename = demand_copy_string (& len);
+  if (filename == NULL)
+    return;
+
+  SKIP_WHITESPACE ();
+
+  /* Look for optional skip and count.  */
+  if (* input_line_pointer == ',')
+    {
+      ++ input_line_pointer;
+      skip = get_absolute_expression ();
+
+      SKIP_WHITESPACE ();
+
+      if (* input_line_pointer == ',')
+       {
+         ++ input_line_pointer;
+
+         count = get_absolute_expression ();
+         if (count == 0)
+           as_warn (_(".incbin count zero, ignoring `%s'"), filename);
+
+         SKIP_WHITESPACE ();
+       }
+    }
+
+  demand_empty_rest_of_line ();
+
+  /* Try opening absolute path first, then try include dirs.  */
+  binfile = fopen (filename, FOPEN_RB);
+  if (binfile == NULL)
+    {
+      int i;
+
+      path = xmalloc ((unsigned long) len + include_dir_maxlen + 5);
+
+      for (i = 0; i < include_dir_count; i++)
+       {
+         sprintf (path, "%s/%s", include_dirs[i], filename);
 
-/* .include -- include a file at this point. */
+         binfile = fopen (path, FOPEN_RB);
+         if (binfile != NULL)
+           break;
+       }
+
+      if (binfile == NULL)
+       as_bad (_("file not found: %s"), filename);
+    }
+  else
+    path = xstrdup (filename);
 
-/* ARGSUSED */
-void 
+  if (binfile)
+    {
+      long   file_len;
+
+      register_dependency (path);
+
+      /* Compute the length of the file.  */
+      if (fseek (binfile, 0, SEEK_END) != 0)
+       {
+         as_bad (_("seek to end of .incbin file failed `%s'"), path);
+         goto done;
+       }
+      file_len = ftell (binfile);
+
+      /* If a count was not specified use the size of the file.  */
+      if (count == 0)
+       count = file_len;
+
+      if (skip + count > file_len)
+       {
+         as_bad (_("skip (%ld) + count (%ld) larger than file size (%ld)"),
+                 skip, count, file_len);
+         goto done;
+       }
+
+      if (fseek (binfile, skip, SEEK_SET) != 0)
+       {
+         as_bad (_("could not skip to %ld in file `%s'"), skip, path);
+         goto done;
+       }
+
+      /* Allocate frag space and store file contents in it.  */
+      binfrag = frag_more (count);
+
+      bytes = fread (binfrag, 1, count, binfile);
+      if (bytes < count)
+       as_warn (_("truncated file `%s', %ld of %ld bytes read"),
+                path, bytes, count);
+    }
+done:
+  if (binfile != NULL)
+    fclose (binfile);
+  if (path)
+    free (path);
+}
+
+/* .include -- include a file at this point.  */
+
+void
 s_include (arg)
-     int arg;
+     int arg ATTRIBUTE_UNUSED;
 {
-  char *newbuf;
   char *filename;
   int i;
   FILE *try;
   char *path;
 
-  if (! flag_m68k_mri)
+  if (!flag_m68k_mri)
     {
       filename = demand_copy_string (&i);
       if (filename == NULL)
@@ -4805,7 +5114,7 @@ s_include (arg)
     {
       SKIP_WHITESPACE ();
       i = 0;
-      while (! is_end_of_line[(unsigned char) *input_line_pointer]
+      while (!is_end_of_line[(unsigned char) *input_line_pointer]
             && *input_line_pointer != ' '
             && *input_line_pointer != '\t')
        {
@@ -4813,34 +5122,37 @@ s_include (arg)
          ++input_line_pointer;
          ++i;
        }
+
       obstack_1grow (&notes, '\0');
       filename = obstack_finish (&notes);
-      while (! is_end_of_line[(unsigned char) *input_line_pointer])
+      while (!is_end_of_line[(unsigned char) *input_line_pointer])
        ++input_line_pointer;
     }
+
   demand_empty_rest_of_line ();
   path = xmalloc ((unsigned long) i + include_dir_maxlen + 5 /* slop */ );
+
   for (i = 0; i < include_dir_count; i++)
     {
       strcpy (path, include_dirs[i]);
       strcat (path, "/");
       strcat (path, filename);
-      if (0 != (try = fopen (path, "r")))
+      if (0 != (try = fopen (path, FOPEN_RT)))
        {
          fclose (try);
          goto gotit;
        }
     }
+
   free (path);
   path = filename;
 gotit:
-  /* malloc Storage leak when file is found on path.  FIXME-SOMEDAY. */
+  /* malloc Storage leak when file is found on path.  FIXME-SOMEDAY.  */
   register_dependency (path);
-  newbuf = input_scrub_include_file (path, input_line_pointer);
-  buffer_limit = input_scrub_next_buffer (&input_line_pointer);
-}                              /* s_include() */
+  input_scrub_insert_file (path);
+}
 
-void 
+void
 add_include_dir (path)
      char *path;
 {
@@ -4849,23 +5161,24 @@ add_include_dir (path)
   if (include_dir_count == 0)
     {
       include_dirs = (char **) xmalloc (2 * sizeof (*include_dirs));
-      include_dirs[0] = ".";   /* Current dir */
+      include_dirs[0] = ".";   /* Current dir */
       include_dir_count = 2;
     }
   else
     {
       include_dir_count++;
-      include_dirs = (char **) realloc (include_dirs,
-                               include_dir_count * sizeof (*include_dirs));
+      include_dirs =
+       (char **) realloc (include_dirs,
+                          include_dir_count * sizeof (*include_dirs));
     }
 
-  include_dirs[include_dir_count - 1] = path;  /* New one */
+  include_dirs[include_dir_count - 1] = path;  /* New one */
 
   i = strlen (path);
   if (i > include_dir_maxlen)
     include_dir_maxlen = i;
-}                              /* add_include_dir() */
-
+}
+\f
 /* Output debugging information to denote the source file.  */
 
 static void
@@ -4877,38 +5190,126 @@ generate_file_debug ()
 
 /* Output line number debugging information for the current source line.  */
 
-static void
+void
 generate_lineno_debug ()
 {
-  if (debug_type == DEBUG_STABS)
-    stabs_generate_asm_lineno ();
-
-#ifdef OBJ_GENERATE_ASM_LINENO
-#ifdef ECOFF_DEBUGGING
-  /* ECOFF assemblers automatically generate
-     debugging information.  FIXME: This should
-     probably be handled elsewhere.  */
-  if (debug_type == DEBUG_NONE)
+  switch (debug_type)
     {
-      if (ecoff_no_current_file ())
-       debug_type = DEBUG_ECOFF;
+    case DEBUG_UNSPECIFIED:
+    case DEBUG_NONE:
+    case DEBUG_DWARF:
+      break;
+    case DEBUG_STABS:
+      stabs_generate_asm_lineno ();
+      break;
+    case DEBUG_ECOFF:
+      ecoff_generate_asm_lineno ();
+      break;
+    case DEBUG_DWARF2:
+      /* ??? We could here indicate to dwarf2dbg.c that something
+        has changed.  However, since there is additional backend
+        support that is required (calling dwarf2_emit_insn), we
+        let dwarf2dbg.c call as_where on its own.  */
+      break;
     }
+}
+
+/* Output debugging information to mark a function entry point or end point.
+   END_P is zero for .func, and non-zero for .endfunc.  */
+
+void
+s_func (end_p)
+     int end_p;
+{
+  do_s_func (end_p, NULL);
+}
 
-  if (debug_type == DEBUG_ECOFF)
+/* Subroutine of s_func so targets can choose a different default prefix.
+   If DEFAULT_PREFIX is NULL, use the target's "leading char".  */
+
+void
+do_s_func (end_p, default_prefix)
+     int end_p;
+     const char *default_prefix;
+{
+  /* Record the current function so that we can issue an error message for
+     misplaced .func,.endfunc, and also so that .endfunc needs no
+     arguments.  */
+  static char *current_name;
+  static char *current_label;
+
+  if (end_p)
     {
-      unsigned int lineno;
-      char *s;
+      if (current_name == NULL)
+       {
+         as_bad (_("missing .func"));
+         ignore_rest_of_line ();
+         return;
+       }
+
+      if (debug_type == DEBUG_STABS)
+       stabs_generate_asm_endfunc (current_name, current_label);
 
-      as_where (&s, &lineno);
-      OBJ_GENERATE_ASM_LINENO (s, lineno);
+      current_name = current_label = NULL;
     }
+  else /* ! end_p */
+    {
+      char *name, *label;
+      char delim1, delim2;
+
+      if (current_name != NULL)
+       {
+         as_bad (_(".endfunc missing for previous .func"));
+         ignore_rest_of_line ();
+         return;
+       }
+
+      name = input_line_pointer;
+      delim1 = get_symbol_end ();
+      name = xstrdup (name);
+      *input_line_pointer = delim1;
+      SKIP_WHITESPACE ();
+      if (*input_line_pointer != ',')
+       {
+         if (default_prefix)
+           asprintf (&label, "%s%s", default_prefix, name);
+         else
+           {
+             char leading_char = 0;
+#ifdef BFD_ASSEMBLER
+             leading_char = bfd_get_symbol_leading_char (stdoutput);
 #endif
-#endif
-}
+             /* Missing entry point, use function's name with the leading
+                char prepended.  */
+             if (leading_char)
+               asprintf (&label, "%c%s", leading_char, name);
+             else
+               label = name;
+           }
+       }
+      else
+       {
+         ++input_line_pointer;
+         SKIP_WHITESPACE ();
+         label = input_line_pointer;
+         delim2 = get_symbol_end ();
+         label = xstrdup (label);
+         *input_line_pointer = delim2;
+       }
 
-void 
+      if (debug_type == DEBUG_STABS)
+       stabs_generate_asm_func (name, label);
+
+      current_name = name;
+      current_label = label;
+    }
+
+  demand_empty_rest_of_line ();
+}
+\f
+void
 s_ignore (arg)
-     int arg;
+     int arg ATTRIBUTE_UNUSED;
 {
   while (!is_end_of_line[(unsigned char) *input_line_pointer])
     {
@@ -4917,7 +5318,6 @@ s_ignore (arg)
   ++input_line_pointer;
 }
 
-
 void
 read_print_statistics (file)
      FILE *file;
@@ -4925,4 +5325,34 @@ read_print_statistics (file)
   hash_print_statistics (file, "pseudo-op table", po_hash);
 }
 
-/* end of read.c */
+/* Inserts the given line into the input stream.
+
+   This call avoids macro/conditionals nesting checking, since the contents of
+   the line are assumed to replace the contents of a line already scanned.
+
+   An appropriate use of this function would be substition of input lines when
+   called by md_start_line_hook().  The given line is assumed to already be
+   properly scrubbed.  */
+
+void
+input_scrub_insert_line (line)
+     const char *line;
+{
+  sb newline;
+  sb_new (&newline);
+  sb_add_string (&newline, line);
+  input_scrub_include_sb (&newline, input_line_pointer, 0);
+  sb_kill (&newline);
+  buffer_limit = input_scrub_next_buffer (&input_line_pointer);
+}
+
+/* Insert a file into the input stream; the path must resolve to an actual
+   file; no include path searching or dependency registering is performed.  */
+
+void
+input_scrub_insert_file (path)
+     char *path;
+{
+  input_scrub_include_file (path, input_line_pointer);
+  buffer_limit = input_scrub_next_buffer (&input_line_pointer);
+}
This page took 0.100524 seconds and 4 git commands to generate.