]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gas/expr.c
* config/sh/tm-sh.h (BELIEVE_PCC_PROMOTION): Define, so that
[thirdparty/binutils-gdb.git] / gas / expr.c
index bfcb8e658f5ec2129425c8999fc6fbdec2b00188..3408f71d9f69b10f15f56a831dd266a893f9ff34 100644 (file)
@@ -1,5 +1,6 @@
 /* expr.c -operands, expressions-
-   Copyright (C) 1987, 1990, 1991, 1992 Free Software Foundation, Inc.
+   Copyright (C) 1987, 90, 91, 92, 93, 94, 95, 96, 97, 1998
+   Free Software Foundation, Inc.
 
    This file is part of GAS, the GNU Assembler.
 
@@ -14,8 +15,9 @@
    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, 675 Mass Ave, Cambridge, MA 02139, USA. */
+   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. */
 
 /*
  * This is really a branch office of as-read.c. I split it out to clearly
 
 #include <ctype.h>
 #include <string.h>
+#define min(a, b)       ((a) < (b) ? (a) : (b))
 
 #include "as.h"
-
 #include "obstack.h"
 
-#if __STDC__ == 1
-static void clean_up_expression (expressionS * expressionP);
-#else /* __STDC__ */
-static void clean_up_expression ();    /* Internal. */
-#endif /* not __STDC__ */
-extern const char EXP_CHARS[]; /* JF hide MD floating pt stuff all the same place */
-extern const char FLT_CHARS[];
+static void floating_constant PARAMS ((expressionS * expressionP));
+static void integer_constant PARAMS ((int radix, expressionS * expressionP));
+static void mri_char_constant PARAMS ((expressionS *));
+static void current_location PARAMS ((expressionS *));
+static void clean_up_expression PARAMS ((expressionS * expressionP));
+static segT operand PARAMS ((expressionS *));
+static operatorT operator PARAMS ((void));
+
+extern const char EXP_CHARS[], FLT_CHARS[];
+
+/* We keep a mapping of expression symbols to file positions, so that
+   we can provide better error messages.  */
+
+struct expr_symbol_line
+{
+  struct expr_symbol_line *next;
+  symbolS *sym;
+  char *file;
+  unsigned int line;
+};
+
+static struct expr_symbol_line *expr_symbol_lines;
+\f
+/* Build a dummy symbol to hold a complex expression.  This is how we
+   build expressions up out of other expressions.  The symbol is put
+   into the fake section expr_section.  */
+
+symbolS *
+make_expr_symbol (expressionP)
+     expressionS *expressionP;
+{
+  expressionS zero;
+  const char *fake;
+  symbolS *symbolP;
+  struct expr_symbol_line *n;
+
+  if (expressionP->X_op == O_symbol
+      && expressionP->X_add_number == 0)
+    return expressionP->X_add_symbol;
+
+  if (expressionP->X_op == O_big)
+    {
+      /* This won't work, because the actual value is stored in
+         generic_floating_point_number or generic_bignum, and we are
+         going to lose it if we haven't already.  */
+      if (expressionP->X_add_number > 0)
+       as_bad (_("bignum invalid; zero assumed"));
+      else
+       as_bad (_("floating point number invalid; zero assumed"));
+      zero.X_op = O_constant;
+      zero.X_add_number = 0;
+      zero.X_unsigned = 0;
+      clean_up_expression (&zero);
+      expressionP = &zero;
+    }
+
+  fake = FAKE_LABEL_NAME;
+
+  /* Putting constant symbols in absolute_section rather than
+     expr_section is convenient for the old a.out code, for which
+     S_GET_SEGMENT does not always retrieve the value put in by
+     S_SET_SEGMENT.  */
+  symbolP = symbol_create (fake,
+                          (expressionP->X_op == O_constant
+                           ? absolute_section
+                           : expr_section),
+                          0, &zero_address_frag);
+  symbolP->sy_value = *expressionP;
+
+  if (expressionP->X_op == O_constant)
+    resolve_symbol_value (symbolP, 1);
+
+  n = (struct expr_symbol_line *) xmalloc (sizeof *n);
+  n->sym = symbolP;
+  as_where (&n->file, &n->line);
+  n->next = expr_symbol_lines;
+  expr_symbol_lines = n;
+
+  return symbolP;
+}
+
+/* Return the file and line number for an expr symbol.  Return
+   non-zero if something was found, 0 if no information is known for
+   the symbol.  */
+
+int
+expr_symbol_where (sym, pfile, pline)
+     symbolS *sym;
+     char **pfile;
+     unsigned int *pline;
+{
+  register struct expr_symbol_line *l;
+
+  for (l = expr_symbol_lines; l != NULL; l = l->next)
+    {
+      if (l->sym == sym)
+       {
+         *pfile = l->file;
+         *pline = l->line;
+         return 1;
+       }
+    }
+
+  return 0;
+}
+\f
+/* Utilities for building expressions.
+   Since complex expressions are recorded as symbols for use in other
+   expressions these return a symbolS * and not an expressionS *.
+   These explicitly do not take an "add_number" argument.  */
+/* ??? For completeness' sake one might want expr_build_symbol.
+   It would just return its argument.  */
+
+/* Build an expression for an unsigned constant.
+   The corresponding one for signed constants is missing because
+   there's currently no need for it.  One could add an unsigned_p flag
+   but that seems more clumsy.  */
+
+symbolS *
+expr_build_uconstant (value)
+     offsetT value;
+{
+  expressionS e;
+
+  e.X_op = O_constant;
+  e.X_add_number = value;
+  e.X_unsigned = 1;
+  return make_expr_symbol (&e);
+}
+
+/* Build an expression for OP s1.  */
+
+symbolS *
+expr_build_unary (op, s1)
+     operatorT op;
+     symbolS *s1;
+{
+  expressionS e;
+
+  e.X_op = op;
+  e.X_add_symbol = s1;
+  e.X_add_number = 0;
+  return make_expr_symbol (&e);
+}
+
+/* Build an expression for s1 OP s2.  */
+
+symbolS *
+expr_build_binary (op, s1, s2)
+     operatorT op;
+     symbolS *s1;
+     symbolS *s2;
+{
+  expressionS e;
+
+  e.X_op = op;
+  e.X_add_symbol = s1;
+  e.X_op_symbol = s2;
+  e.X_add_number = 0;
+  return make_expr_symbol (&e);
+}
+
+/* Build an expression for the current location ('.').  */
+
+symbolS *
+expr_build_dot ()
+{
+  expressionS e;
 
+  current_location (&e);
+  return make_expr_symbol (&e);
+}
+\f
 /*
  * Build any floating-point literal here.
  * Also build any bignum literal here.
  */
 
-/* LITTLENUM_TYPE      generic_buffer [6]; *//* JF this is a hack */
 /* Seems atof_machine can backscan through generic_bignum and hit whatever
    happens to be loaded before it in memory.  And its way too complicated
    for me to fix right.  Thus a hack.  JF:  Just make generic_bignum bigger,
    and never write into the early words, thus they'll always be zero.
-   I hate Dean's floating-point code.  Bleh.
-   */
+   I hate Dean's floating-point code.  Bleh.  */
 LITTLENUM_TYPE generic_bignum[SIZE_OF_LARGE_NUMBER + 6];
 FLONUM_TYPE generic_floating_point_number =
 {
   &generic_bignum[6],          /* low (JF: Was 0) */
-  &generic_bignum[SIZE_OF_LARGE_NUMBER + 6 - 1],       /* high JF: (added +6) */
+  &generic_bignum[SIZE_OF_LARGE_NUMBER + 6 - 1], /* high JF: (added +6) */
   0,                           /* leader */
   0,                           /* exponent */
   0                            /* sign */
@@ -63,6 +228,7 @@ FLONUM_TYPE generic_floating_point_number =
 /* If nonzero, we've been asked to assemble nan, +inf or -inf */
 int generic_floating_point_magic;
 \f
+static void
 floating_constant (expressionP)
      expressionS *expressionP;
 {
@@ -70,95 +236,246 @@ floating_constant (expressionP)
   /* floating-point constant. */
   int error_code;
 
-  error_code = atof_generic
-    (&input_line_pointer, ".", EXP_CHARS,
-     &generic_floating_point_number);
+  error_code = atof_generic (&input_line_pointer, ".", EXP_CHARS,
+                            &generic_floating_point_number);
 
   if (error_code)
     {
       if (error_code == ERROR_EXPONENT_OVERFLOW)
        {
-         as_bad ("bad floating-point constant: exponent overflow, probably assembling junk");
+         as_bad (_("bad floating-point constant: exponent overflow, probably assembling junk"));
        }
       else
        {
-         as_bad ("bad floating-point constant: unknown error code=%d.", error_code);
+         as_bad (_("bad floating-point constant: unknown error code=%d."), error_code);
        }
     }
-  expressionP->X_seg = SEG_BIG;
+  expressionP->X_op = O_big;
   /* input_line_pointer->just after constant, */
   /* which may point to whitespace. */
   expressionP->X_add_number = -1;
 }
 
+static valueT 
+generic_bignum_to_int32 () 
+{
+  valueT number =
+          ((generic_bignum[1] & LITTLENUM_MASK) << LITTLENUM_NUMBER_OF_BITS)
+          | (generic_bignum[0] & LITTLENUM_MASK);
+  number &= 0xffffffff;
+  return number;
+}
 
+#ifdef BFD64
+static valueT 
+generic_bignum_to_int64 () 
+{
+  valueT number = 
+          ((((((((valueT) generic_bignum[3] & LITTLENUM_MASK)
+                << LITTLENUM_NUMBER_OF_BITS)
+               | ((valueT) generic_bignum[2] & LITTLENUM_MASK))
+              << LITTLENUM_NUMBER_OF_BITS)
+             | ((valueT) generic_bignum[1] & LITTLENUM_MASK))
+            << LITTLENUM_NUMBER_OF_BITS)
+           | ((valueT) generic_bignum[0] & LITTLENUM_MASK));
+  return number;
+}
+#endif
 
+static void
 integer_constant (radix, expressionP)
      int radix;
      expressionS *expressionP;
 {
-  register char *digit_2;      /*->2nd digit of number. */
+  char *start;         /* start of number. */
+  char *suffix = NULL;
   char c;
-
-  register valueT number;      /* offset or (absolute) value */
-  register short int digit;    /* value of next digit in current radix */
-  register short int maxdig = 0;/* highest permitted digit value. */
-  register int too_many_digits = 0;    /* if we see >= this number of */
-  register char *name;         /* points to name of symbol */
-  register symbolS *symbolP;   /* points to symbol */
+  valueT number;       /* offset or (absolute) value */
+  short int digit;     /* value of next digit in current radix */
+  short int maxdig = 0;/* highest permitted digit value. */
+  int too_many_digits = 0;     /* if we see >= this number of */
+  char *name;          /* points to name of symbol */
+  symbolS *symbolP;    /* points to symbol */
 
   int small;                   /* true if fits in 32 bits. */
-  extern char hex_value[];     /* in hex_value.c */
 
-  /* may be bignum, or may fit in 32 bits. */
-  /*
-   * most numbers fit into 32 bits, and we want this case to be fast.
-   * so we pretend it will fit into 32 bits. if, after making up a 32
-   * bit number, we realise that we have scanned more digits than
-   * comfortably fit into 32 bits, we re-scan the digits coding
-   * them into a bignum. for decimal and octal numbers we are conservative: some
-   * numbers may be assumed bignums when in fact they do fit into 32 bits.
-   * numbers of any radix can have excess leading zeros: we strive
-   * to recognise this and cast them back into 32 bits.
-   * we must check that the bignum really is more than 32
-   * bits, and change it back to a 32-bit number if it fits.
-   * the number we are looking for is expected to be positive, but
-   * if it fits into 32 bits as an unsigned number, we let it be a 32-bit
-   * number. the cavalier approach is for speed in ordinary cases.
-   */
+  /* May be bignum, or may fit in 32 bits. */
+  /* Most numbers fit into 32 bits, and we want this case to be fast.
+     so we pretend it will fit into 32 bits.  If, after making up a 32
+     bit number, we realise that we have scanned more digits than
+     comfortably fit into 32 bits, we re-scan the digits coding them
+     into a bignum.  For decimal and octal numbers we are
+     conservative: Some numbers may be assumed bignums when in fact
+     they do fit into 32 bits.  Numbers of any radix can have excess
+     leading zeros: We strive to recognise this and cast them back
+     into 32 bits.  We must check that the bignum really is more than
+     32 bits, and change it back to a 32-bit number if it fits.  The
+     number we are looking for is expected to be positive, but if it
+     fits into 32 bits as an unsigned number, we let it be a 32-bit
+     number.  The cavalier approach is for speed in ordinary cases. */
+  /* This has been extended for 64 bits.  We blindly assume that if
+     you're compiling in 64-bit mode, the target is a 64-bit machine.
+     This should be cleaned up.  */
+
+#ifdef BFD64
+#define valuesize 64
+#else /* includes non-bfd case, mostly */
+#define valuesize 32
+#endif
 
-  switch (radix)
+  if (flag_m68k_mri && radix == 0)
     {
+      int flt = 0;
+
+      /* In MRI mode, the number may have a suffix indicating the
+         radix.  For that matter, it might actually be a floating
+         point constant.  */
+      for (suffix = input_line_pointer;
+          isalnum ((unsigned char) *suffix);
+          suffix++)
+       {
+         if (*suffix == 'e' || *suffix == 'E')
+           flt = 1;
+       }
+
+      if (suffix == input_line_pointer)
+       {
+         radix = 10;
+         suffix = NULL;
+       }
+      else
+       {
+         c = *--suffix;
+         if (islower ((unsigned char) c))
+           c = toupper (c);
+         if (c == 'B')
+           radix = 2;
+         else if (c == 'D')
+           radix = 10;
+         else if (c == 'O' || c == 'Q')
+           radix = 8;
+         else if (c == 'H')
+           radix = 16;
+         else if (suffix[1] == '.' || c == 'E' || flt)
+           {
+             floating_constant (expressionP);
+             return;
+           }
+         else
+           {
+             radix = 10;
+             suffix = NULL;
+           }
+       }
+    }
 
+  switch (radix)
+    {
     case 2:
       maxdig = 2;
-      too_many_digits = 33;
+      too_many_digits = valuesize + 1;
       break;
     case 8:
       maxdig = radix = 8;
-      too_many_digits = 11;
+      too_many_digits = (valuesize + 2) / 3 + 1;
       break;
     case 16:
-
-
       maxdig = radix = 16;
-      too_many_digits = 9;
+      too_many_digits = (valuesize + 3) / 4 + 1;
       break;
     case 10:
       maxdig = radix = 10;
-      too_many_digits = 11;
+      too_many_digits = (valuesize + 12) / 4; /* very rough */
     }
-  c = *input_line_pointer;
-  input_line_pointer++;
-  digit_2 = input_line_pointer;
-  for (number = 0; (digit = hex_value[c]) < maxdig; c = *input_line_pointer++)
+#undef valuesize
+  start = input_line_pointer;
+  c = *input_line_pointer++;
+  for (number = 0;
+       (digit = hex_value (c)) < maxdig;
+       c = *input_line_pointer++)
     {
       number = number * radix + digit;
     }
   /* c contains character after number. */
   /* input_line_pointer->char after c. */
-  small = input_line_pointer - digit_2 < too_many_digits;
-  if (!small)
+  small = (input_line_pointer - start - 1) < too_many_digits;
+
+  if (radix == 16 && c == '_') 
+    {
+      /* This is literal of the form 0x333_0_12345678_1.
+         This example is equivalent to 0x00000333000000001234567800000001.  */
+
+      int num_little_digits = 0;
+      int i;
+      input_line_pointer = start;      /*->1st digit. */
+
+      know (LITTLENUM_NUMBER_OF_BITS == 16);
+
+      for (c = '_'; c == '_'; num_little_digits+=2)
+       {
+
+         /* Convert one 64-bit word. */
+         int ndigit = 0; 
+         number = 0;
+         for (c = *input_line_pointer++;
+              (digit = hex_value (c)) < maxdig;
+              c = *(input_line_pointer++))
+           {
+             number = number * radix + digit;
+             ndigit++;
+           }
+
+         /* Check for 8 digit per word max.  */
+         if (ndigit > 8) 
+           as_bad (_("A bignum with underscores may not have more than 8 hex digits in any word."));
+
+         /* Add this chunk to the bignum.  Shift things down 2 little digits.*/
+         know (LITTLENUM_NUMBER_OF_BITS == 16);
+         for (i = min (num_little_digits + 1, SIZE_OF_LARGE_NUMBER - 1); i >= 2; i--)
+           generic_bignum[i] = generic_bignum[i-2];
+
+         /* Add the new digits as the least significant new ones. */
+         generic_bignum[0] = number & 0xffffffff;
+         generic_bignum[1] = number >> 16;
+       }
+
+      /* Again, c is char after number, input_line_pointer->after c. */
+
+      if (num_little_digits > SIZE_OF_LARGE_NUMBER - 1)
+       num_little_digits = SIZE_OF_LARGE_NUMBER - 1;
+
+      assert (num_little_digits >= 4);
+
+      if (num_little_digits != 8)
+       as_bad (_("A bignum with underscores must have exactly 4 words."));
+
+      /* We might have some leading zeros.  These can be trimmed to give
+       * us a change to fit this constant into a small number.
+       */
+      while (generic_bignum[num_little_digits-1] == 0 && num_little_digits > 1)
+       num_little_digits--;
+       
+      if (num_little_digits <= 2)
+       {
+         /* will fit into 32 bits. */
+         number = generic_bignum_to_int32 ();
+         small = 1;
+       }
+#ifdef BFD64
+      else if (num_little_digits <= 4)
+       {
+         /* Will fit into 64 bits.  */
+         number = generic_bignum_to_int64 ();
+         small = 1;
+       }
+#endif
+      else
+       {
+         small = 0;
+         number = num_little_digits; /* number of littlenums in the bignum. */
+       }
+    }
+  else if (!small)
     {
       /*
        * we saw a lot of digits. manufacture a bignum the hard way.
@@ -170,10 +487,13 @@ integer_constant (radix, expressionP)
       leader = generic_bignum;
       generic_bignum[0] = 0;
       generic_bignum[1] = 0;
-      /* we could just use digit_2, but lets be mnemonic. */
-      input_line_pointer = --digit_2;  /*->1st digit. */
+      generic_bignum[2] = 0;
+      generic_bignum[3] = 0;
+      input_line_pointer = start;      /*->1st digit. */
       c = *input_line_pointer++;
-      for (; (carry = hex_value[c]) < maxdig; c = *input_line_pointer++)
+      for (;
+          (carry = hex_value (c)) < maxdig;
+          c = *input_line_pointer++)
        {
          for (pointer = generic_bignum;
               pointer <= leader;
@@ -188,28 +508,38 @@ integer_constant (radix, expressionP)
          if (carry)
            {
              if (leader < generic_bignum + SIZE_OF_LARGE_NUMBER - 1)
-               {               /* room to grow a longer bignum. */
+               {
+                 /* room to grow a longer bignum. */
                  *++leader = carry;
                }
            }
        }
       /* again, c is char after number, */
       /* input_line_pointer->after c. */
-      know (sizeof (int) * 8 == 32);
       know (LITTLENUM_NUMBER_OF_BITS == 16);
-      /* hence the constant "2" in the next line. */
       if (leader < generic_bignum + 2)
-       {                       /* will fit into 32 bits. */
-         number =
-           ((generic_bignum[1] & LITTLENUM_MASK) << LITTLENUM_NUMBER_OF_BITS)
-           | (generic_bignum[0] & LITTLENUM_MASK);
+       {
+         /* will fit into 32 bits. */
+         number = generic_bignum_to_int32 ();
+         small = 1;
+       }
+#ifdef BFD64
+      else if (leader < generic_bignum + 4)
+       {
+         /* Will fit into 64 bits.  */
+         number = generic_bignum_to_int64 ();
          small = 1;
        }
+#endif
       else
        {
          number = leader - generic_bignum + 1; /* number of littlenums in the bignum. */
        }
     }
+
+  if (flag_m68k_mri && suffix != NULL && input_line_pointer - 1 == suffix)
+    c = *input_line_pointer++;
+
   if (small)
     {
       /*
@@ -219,121 +549,216 @@ integer_constant (radix, expressionP)
        * than checking for strict canonical form. syntax sux!
        */
 
-      switch (c)
+      if (LOCAL_LABELS_FB && c == 'b')
        {
+         /*
+          * backward ref to local label.
+          * because it is backward, expect it to be defined.
+          */
+         /* Construct a local label.  */
+         name = fb_label_name ((int) number, 0);
 
-#ifdef LOCAL_LABELS_FB
-       case 'b':
-         {
-           /*
-            * backward ref to local label.
-            * because it is backward, expect it to be defined.
-            */
-           /* Construct a local label.  */
-           name = fb_label_name ((int) number, 0);
-
-           /* seen before, or symbol is defined: ok */
-           symbolP = symbol_find (name);
-           if ((symbolP != NULL) && (S_IS_DEFINED (symbolP)))
-             {
-
-               /* local labels are never absolute. don't waste time
-                  checking absoluteness. */
-               know (SEG_NORMAL (S_GET_SEGMENT (symbolP)));
-
-               expressionP->X_add_symbol = symbolP;
-               expressionP->X_seg = S_GET_SEGMENT (symbolP);
-
-             }
-           else
-             {                 /* either not seen or not defined. */
-               as_bad ("backw. ref to unknown label \"%d:\", 0 assumed.", number);
-               expressionP->X_seg = SEG_ABSOLUTE;
-             }
+         /* seen before, or symbol is defined: ok */
+         symbolP = symbol_find (name);
+         if ((symbolP != NULL) && (S_IS_DEFINED (symbolP)))
+           {
+             /* local labels are never absolute. don't waste time
+                checking absoluteness. */
+             know (SEG_NORMAL (S_GET_SEGMENT (symbolP)));
 
-           expressionP->X_add_number = 0;
-           break;
-         }                     /* case 'b' */
+             expressionP->X_op = O_symbol;
+             expressionP->X_add_symbol = symbolP;
+           }
+         else
+           {
+             /* either not seen or not defined. */
+             /* @@ Should print out the original string instead of
+                the parsed number.  */
+             as_bad (_("backw. ref to unknown label \"%d:\", 0 assumed."),
+                     (int) number);
+             expressionP->X_op = O_constant;
+           }
 
-       case 'f':
-         {
-           /*
-            * forward reference. expect symbol to be undefined or
-            * unknown. undefined: seen it before. unknown: never seen
-            * it before.
-            * construct a local label name, then an undefined symbol.
-            * don't create a xseg frag for it: caller may do that.
-            * just return it as never seen before.
-            */
-           name = fb_label_name ((int) number, 1);
-           symbolP = symbol_find_or_make (name);
-           /* we have no need to check symbol properties. */
+         expressionP->X_add_number = 0;
+       }                       /* case 'b' */
+      else if (LOCAL_LABELS_FB && c == 'f')
+       {
+         /*
+          * forward reference. expect symbol to be undefined or
+          * unknown. undefined: seen it before. unknown: never seen
+          * it before.
+          * construct a local label name, then an undefined symbol.
+          * don't create a xseg frag for it: caller may do that.
+          * just return it as never seen before.
+          */
+         name = fb_label_name ((int) number, 1);
+         symbolP = symbol_find_or_make (name);
+         /* we have no need to check symbol properties. */
 #ifndef many_segments
-           /* since "know" puts its arg into a "string", we
-              can't have newlines in the argument.  */
-           know (S_GET_SEGMENT (symbolP) == SEG_UNKNOWN || S_GET_SEGMENT (symbolP) == SEG_TEXT || S_GET_SEGMENT (symbolP) == SEG_DATA);
+         /* since "know" puts its arg into a "string", we
+            can't have newlines in the argument.  */
+         know (S_GET_SEGMENT (symbolP) == undefined_section || S_GET_SEGMENT (symbolP) == text_section || S_GET_SEGMENT (symbolP) == data_section);
 #endif
-           expressionP->X_add_symbol = symbolP;
-           expressionP->X_seg = SEG_UNKNOWN;
-           expressionP->X_subtract_symbol = NULL;
-           expressionP->X_add_number = 0;
+         expressionP->X_op = O_symbol;
+         expressionP->X_add_symbol = symbolP;
+         expressionP->X_add_number = 0;
+       }                       /* case 'f' */
+      else if (LOCAL_LABELS_DOLLAR && c == '$')
+       {
+         /* If the dollar label is *currently* defined, then this is just
+            another reference to it.  If it is not *currently* defined,
+            then this is a fresh instantiation of that number, so create
+            it.  */
 
-           break;
-         }                     /* case 'f' */
+         if (dollar_label_defined ((long) number))
+           {
+             name = dollar_label_name ((long) number, 0);
+             symbolP = symbol_find (name);
+             know (symbolP != NULL);
+           }
+         else
+           {
+             name = dollar_label_name ((long) number, 1);
+             symbolP = symbol_find_or_make (name);
+           }
 
-#endif /* LOCAL_LABELS_FB */
+         expressionP->X_op = O_symbol;
+         expressionP->X_add_symbol = symbolP;
+         expressionP->X_add_number = 0;
+       }                       /* case '$' */
+      else
+       {
+         expressionP->X_op = O_constant;
+#ifdef TARGET_WORD_SIZE
+         /* Sign extend NUMBER.  */
+         number |= (-(number >> (TARGET_WORD_SIZE - 1))) << (TARGET_WORD_SIZE - 1);
+#endif
+         expressionP->X_add_number = number;
+         input_line_pointer--; /* restore following character. */
+       }                       /* really just a number */
+    }
+  else
+    {
+      /* not a small number */
+      expressionP->X_op = O_big;
+      expressionP->X_add_number = number;      /* number of littlenums */
+      input_line_pointer--;    /*->char following number. */
+    }
+}
 
-#ifdef LOCAL_LABELS_DOLLAR
+/* Parse an MRI multi character constant.  */
 
-       case '$':
-         {
+static void
+mri_char_constant (expressionP)
+     expressionS *expressionP;
+{
+  int i;
 
-           /* If the dollar label is *currently* defined, then this is just
-              another reference to it.  If it is not *currently* defined,
-              then this is a fresh instantiation of that number, so create
-              it.  */
+  if (*input_line_pointer == '\''
+      && input_line_pointer[1] != '\'')
+    {
+      expressionP->X_op = O_constant;
+      expressionP->X_add_number = 0;
+      return;
+    }
 
-           if (dollar_label_defined (number))
-             {
-               name = dollar_label_name (number, 0);
-               symbolP = symbol_find (name);
-               know (symbolP != NULL);
-             }
-           else
-             {
-               name = dollar_label_name (number, 1);
-               symbolP = symbol_find_or_make (name);
-             }
+  /* In order to get the correct byte ordering, we must build the
+     number in reverse.  */
+  for (i = SIZE_OF_LARGE_NUMBER - 1; i >= 0; i--)
+    {
+      int j;
 
-           expressionP->X_add_symbol = symbolP;
-           expressionP->X_add_number = 0;
-           expressionP->X_seg = S_GET_SEGMENT (symbolP);
+      generic_bignum[i] = 0;
+      for (j = 0; j < CHARS_PER_LITTLENUM; j++)
+       {
+         if (*input_line_pointer == '\'')
+           {
+             if (input_line_pointer[1] != '\'')
+               break;
+             ++input_line_pointer;
+           }
+         generic_bignum[i] <<= 8;
+         generic_bignum[i] += *input_line_pointer;
+         ++input_line_pointer;
+       }
 
-           break;
-         }                     /* case '$' */
+      if (i < SIZE_OF_LARGE_NUMBER - 1)
+       {
+         /* If there is more than one littlenum, left justify the
+             last one to make it match the earlier ones.  If there is
+             only one, we can just use the value directly.  */
+         for (; j < CHARS_PER_LITTLENUM; j++)
+           generic_bignum[i] <<= 8;
+       }
 
-#endif /* LOCAL_LABELS_DOLLAR */
+      if (*input_line_pointer == '\''
+         && input_line_pointer[1] != '\'')
+       break;
+    }
 
-       default:
-         {
-           expressionP->X_add_number = number;
-           expressionP->X_seg = SEG_ABSOLUTE;
-           input_line_pointer--;       /* restore following character. */
-           break;
-         }                     /* really just a number */
+  if (i < 0)
+    {
+      as_bad (_("Character constant too large"));
+      i = 0;
+    }
 
-       }                       /* switch on char following the number */
+  if (i > 0)
+    {
+      int c;
+      int j;
 
+      c = SIZE_OF_LARGE_NUMBER - i;
+      for (j = 0; j < c; j++)
+       generic_bignum[j] = generic_bignum[i + j];
+      i = c;
+    }
 
+  know (LITTLENUM_NUMBER_OF_BITS == 16);
+  if (i > 2)
+    {
+      expressionP->X_op = O_big;
+      expressionP->X_add_number = i;
     }
   else
-    {                          /* not a small number */
-      expressionP->X_add_number = number;
-      expressionP->X_seg = SEG_BIG;
-      input_line_pointer--;    /*->char following number. */
-    }                          /* if (small) */
-}                              /* integer_constant() */
+    {
+      expressionP->X_op = O_constant;
+      if (i < 2)
+       expressionP->X_add_number = generic_bignum[0] & LITTLENUM_MASK;
+      else
+       expressionP->X_add_number =
+         (((generic_bignum[1] & LITTLENUM_MASK)
+           << LITTLENUM_NUMBER_OF_BITS)
+          | (generic_bignum[0] & LITTLENUM_MASK));
+    }
+
+  /* Skip the final closing quote.  */
+  ++input_line_pointer;
+}
+
+/* Return an expression representing the current location.  This
+   handles the magic symbol `.'.  */
 
+static void
+current_location (expressionp)
+     expressionS *expressionp;
+{
+  if (now_seg == absolute_section)
+    {
+      expressionp->X_op = O_constant;
+      expressionp->X_add_number = abs_section_offset;
+    }
+  else
+    {
+      symbolS *symbolp;
+
+      symbolp = symbol_new (FAKE_LABEL_NAME, now_seg,
+                           (valueT) frag_now_fix (),
+                           frag_now);
+      expressionp->X_op = O_symbol;
+      expressionp->X_add_symbol = symbolp;
+      expressionp->X_add_number = 0;
+    }
+}
 
 /*
  * Summary of operand().
@@ -341,27 +766,27 @@ integer_constant (radix, expressionP)
  * in: Input_line_pointer points to 1st char of operand, which may
  *     be a space.
  *
- * out:        A expressionS. X_seg determines how to understand the rest of the
- *     expressionS.
- *     The operand may have been empty: in this case X_seg == SEG_ABSENT.
+ * out:        A expressionS.
+ *     The operand may have been empty: in this case X_op == O_absent.
  *     Input_line_pointer->(next non-blank) char after operand.
- *
  */
-\f
-
 
 static segT
 operand (expressionP)
-     register expressionS *expressionP;
+     expressionS *expressionP;
 {
-  register char c;
-  register symbolS *symbolP;   /* points to symbol */
-  register char *name;         /* points to name of symbol */
-  /* invented for humans only, hope */
-  /* optimising compiler flushes it! */
-  register short int radix;    /* 2, 8, 10 or 16, 0 when floating */
-  /* 0 means we saw start of a floating- */
-  /* point constant. */
+  char c;
+  symbolS *symbolP;    /* points to symbol */
+  char *name;          /* points to name of symbol */
+  segT segment;
+
+  /* All integers are regarded as unsigned unless they are negated.
+     This is because the only thing which cares whether a number is
+     unsigned is the code in emit_expr which extends constants into
+     bignums.  It should only sign extend negative numbers, so that
+     something like ``.quad 0x80000000'' is not sign extended even
+     though it appears negative if valueT is 32 bits.  */
+  expressionP->X_unsigned = 1;
 
   /* digits, assume it is a bignum. */
 
@@ -370,17 +795,6 @@ operand (expressionP)
 
   switch (c)
     {
-#ifdef MRI
-    case '%':
-      integer_constant (2, expressionP);
-      break;
-    case '@':
-      integer_constant (8, expressionP);
-      break;
-    case '$':
-      integer_constant (16, expressionP);
-      break;
-#endif
     case '1':
     case '2':
     case '3':
@@ -392,52 +806,93 @@ operand (expressionP)
     case '9':
       input_line_pointer--;
 
-      integer_constant (10, expressionP);
+      integer_constant (flag_m68k_mri ? 0 : 10, expressionP);
       break;
 
     case '0':
       /* non-decimal radix */
 
+      if (flag_m68k_mri)
+       {
+         char *s;
+
+         /* Check for a hex constant.  */
+         for (s = input_line_pointer; hex_p (*s); s++)
+           ;
+         if (*s == 'h' || *s == 'H')
+           {
+             --input_line_pointer;
+             integer_constant (0, expressionP);
+             break;
+           }
+       }
 
       c = *input_line_pointer;
       switch (c)
        {
-
+       case 'o':
+       case 'O':
+       case 'q':
+       case 'Q':
+       case '8':
+       case '9':
+         if (flag_m68k_mri)
+           {
+             integer_constant (0, expressionP);
+             break;
+           }
+         /* Fall through.  */
        default:
+       default_case:
          if (c && strchr (FLT_CHARS, c))
            {
              input_line_pointer++;
              floating_constant (expressionP);
+             expressionP->X_add_number =
+               - (isupper ((unsigned char) c) ? tolower (c) : c);
            }
          else
            {
              /* The string was only zero */
-             expressionP->X_add_symbol = 0;
+             expressionP->X_op = O_constant;
              expressionP->X_add_number = 0;
-             expressionP->X_seg = SEG_ABSOLUTE;
            }
 
          break;
 
        case 'x':
        case 'X':
+         if (flag_m68k_mri)
+           goto default_case;
          input_line_pointer++;
          integer_constant (16, expressionP);
          break;
 
        case 'b':
-#ifdef LOCAL_LABELS_FB
-         if (!*input_line_pointer
-             || (!strchr ("+-.0123456789", *input_line_pointer)
-                 && !strchr (EXP_CHARS, *input_line_pointer)))
+         if (LOCAL_LABELS_FB && ! flag_m68k_mri)
            {
-             input_line_pointer--;
-             integer_constant (10, expressionP);
-             break;
+             /* This code used to check for '+' and '-' here, and, in
+                some conditions, fall through to call
+                integer_constant.  However, that didn't make sense,
+                as integer_constant only accepts digits.  */
+             /* Some of our code elsewhere does permit digits greater
+                than the expected base; for consistency, do the same
+                here.  */
+             if (input_line_pointer[1] < '0'
+                 || input_line_pointer[1] > '9')
+               {
+                 /* Parse this as a back reference to label 0.  */
+                 input_line_pointer--;
+                 integer_constant (10, expressionP);
+                 break;
+               }
+             /* Otherwise, parse this as a binary number.  */
            }
-#endif
+         /* Fall through.  */
        case 'B':
          input_line_pointer++;
+         if (flag_m68k_mri)
+           goto default_case;
          integer_constant (2, expressionP);
          break;
 
@@ -449,155 +904,288 @@ operand (expressionP)
        case '5':
        case '6':
        case '7':
-         integer_constant (8, expressionP);
+         integer_constant (flag_m68k_mri ? 0 : 8, expressionP);
          break;
 
        case 'f':
-#ifdef LOCAL_LABELS_FB
-         /* if it says '0f' and the line ends or it doesn't look like
-            a floating point #, its a local label ref.  dtrt */
-         /* likewise for the b's.  xoxorich. */
-         if (c == 'f'
-             && (!*input_line_pointer ||
-                 (!strchr ("+-.0123456789", *input_line_pointer) &&
-                  !strchr (EXP_CHARS, *input_line_pointer))))
+         if (LOCAL_LABELS_FB)
            {
-             input_line_pointer -= 1;
+             /* If it says "0f" and it could possibly be a floating point
+                number, make it one.  Otherwise, make it a local label,
+                and try to deal with parsing the rest later.  */
+             if (!input_line_pointer[1]
+                 || (is_end_of_line[0xff & input_line_pointer[1]]))
+               goto is_0f_label;
+             {
+               char *cp = input_line_pointer + 1;
+               int r = atof_generic (&cp, ".", EXP_CHARS,
+                                     &generic_floating_point_number);
+               switch (r)
+                 {
+                 case 0:
+                 case ERROR_EXPONENT_OVERFLOW:
+                   if (cp == input_line_pointer + 1)
+                     /* looks like a difference expression */
+                     goto is_0f_label;
+                   else
+                     goto is_0f_float;
+                 default:
+                   as_fatal (_("expr.c(operand): bad atof_generic return val %d"),
+                             r);
+                 }
+             }
+
+             /* Okay, now we've sorted it out.  We resume at one of these
+                two labels, depending on what we've decided we're probably
+                looking at.  */
+           is_0f_label:
+             input_line_pointer--;
              integer_constant (10, expressionP);
              break;
+
+           is_0f_float:
+             /* fall through */
+             ;
            }
-#endif
 
        case 'd':
        case 'D':
+         if (flag_m68k_mri)
+           {
+             integer_constant (0, expressionP);
+             break;
+           }
+         /* Fall through.  */
        case 'F':
        case 'r':
        case 'e':
        case 'E':
        case 'g':
        case 'G':
-
          input_line_pointer++;
          floating_constant (expressionP);
+         expressionP->X_add_number =
+           - (isupper ((unsigned char) c) ? tolower (c) : c);
          break;
 
-#ifdef LOCAL_LABELS_DOLLAR
        case '$':
-         integer_constant (10, expressionP);
-         break;
-#endif
-       }
-
-      break;
+         if (LOCAL_LABELS_DOLLAR)
+           {
+             integer_constant (10, expressionP);
+             break;
+           }
+         else
+           goto default_case;
+       }
+
+      break;
+
     case '(':
+    case '[':
       /* didn't begin with digit & not a name */
-      {
-       (void) expression (expressionP);
-       /* Expression() will pass trailing whitespace */
-       if (*input_line_pointer++ != ')')
-         {
-           as_bad ("Missing ')' assumed");
-           input_line_pointer--;
-         }
-       /* here with input_line_pointer->char after "(...)" */
-      }
-      return expressionP->X_seg;
+      segment = expression (expressionP);
+      /* Expression() will pass trailing whitespace */
+      if ((c == '(' && *input_line_pointer++ != ')')
+         || (c == '[' && *input_line_pointer++ != ']'))
+       {
+         as_bad (_("Missing ')' assumed"));
+         input_line_pointer--;
+       }
+      SKIP_WHITESPACE ();
+      /* here with input_line_pointer->char after "(...)" */
+      return segment;
+
+    case 'E':
+      if (! flag_m68k_mri || *input_line_pointer != '\'')
+       goto de_fault;
+      as_bad (_("EBCDIC constants are not supported"));
+      /* Fall through.  */
+    case 'A':
+      if (! flag_m68k_mri || *input_line_pointer != '\'')
+       goto de_fault;
+      ++input_line_pointer;
+      /* Fall through.  */
+    case '\'':
+      if (! flag_m68k_mri)
+       {
+         /* Warning: to conform to other people's assemblers NO
+            ESCAPEMENT is permitted for a single quote. The next
+            character, parity errors and all, is taken as the value
+            of the operand. VERY KINKY.  */
+         expressionP->X_op = O_constant;
+         expressionP->X_add_number = *input_line_pointer++;
+         break;
+       }
 
+      mri_char_constant (expressionP);
+      break;
 
-    case '\'':
-      /* Warning: to conform to other people's assemblers NO ESCAPEMENT is
-        permitted for a single quote. The next character, parity errors and
-        all, is taken as the value of the operand. VERY KINKY.  */
-      expressionP->X_add_number = *input_line_pointer++;
-      expressionP->X_seg = SEG_ABSOLUTE;
+    case '+':
+      (void) operand (expressionP);
       break;
 
+    case '"':
+      /* Double quote is the bitwise not operator in MRI mode.  */
+      if (! flag_m68k_mri)
+       goto de_fault;
+      /* Fall through.  */
     case '~':
+      /* ~ is permitted to start a label on the Delta.  */
+      if (is_name_beginner (c))
+       goto isname;
+    case '!':
     case '-':
-    case '+':
-
       {
-       /* unary operator: hope for SEG_ABSOLUTE */
-       switch (operand (expressionP))
+       operand (expressionP);
+       if (expressionP->X_op == O_constant)
          {
-         case SEG_ABSOLUTE:
            /* input_line_pointer -> char after operand */
            if (c == '-')
              {
-               expressionP->X_add_number = -expressionP->X_add_number;
+               expressionP->X_add_number = - expressionP->X_add_number;
                /* Notice: '-' may overflow: no warning is given. This is
                   compatible with other people's assemblers. Sigh.  */
+               expressionP->X_unsigned = 0;
              }
+           else if (c == '~' || c == '"')
+             expressionP->X_add_number = ~ expressionP->X_add_number;
            else
-             {
-               expressionP->X_add_number = ~expressionP->X_add_number;
-             }
-           break;
-
-         case SEG_TEXT:
-         case SEG_DATA:
-         case SEG_BSS:
-         case SEG_PASS1:
-         case SEG_UNKNOWN:
+             expressionP->X_add_number = ! expressionP->X_add_number;
+         }
+       else if (expressionP->X_op != O_illegal
+                && expressionP->X_op != O_absent)
+         {
+           expressionP->X_add_symbol = make_expr_symbol (expressionP);
            if (c == '-')
-             {                 /* JF I hope this hack works */
-               expressionP->X_subtract_symbol = expressionP->X_add_symbol;
-               expressionP->X_add_symbol = 0;
-               expressionP->X_seg = SEG_DIFFERENCE;
-               break;
-             }
-         default:              /* unary on non-absolute is unsuported */
-           as_warn ("Unary operator %c ignored because bad operand follows", c);
-           break;
-           /* Expression undisturbed from operand(). */
+             expressionP->X_op = O_uminus;
+           else if (c == '~' || c == '"')
+             expressionP->X_op = O_bit_not;
+           else
+             expressionP->X_op = O_logical_not;
+           expressionP->X_add_number = 0;
          }
+       else
+         as_warn (_("Unary operator %c ignored because bad operand follows"),
+                  c);
       }
+      break;
 
+    case '$':
+      /* $ is the program counter when in MRI mode, or when DOLLAR_DOT
+         is defined.  */
+#ifndef DOLLAR_DOT
+      if (! flag_m68k_mri)
+       goto de_fault;
+#endif
+      if (flag_m68k_mri && hex_p (*input_line_pointer))
+       {
+         /* In MRI mode, $ is also used as the prefix for a
+             hexadecimal constant.  */
+         integer_constant (16, expressionP);
+         break;
+       }
 
+      if (is_part_of_name (*input_line_pointer))
+       goto isname;
 
+      current_location (expressionP);
       break;
 
     case '.':
       if (!is_part_of_name (*input_line_pointer))
        {
-         char *fake;
-         extern struct obstack frags;
-
-         /* JF: '.' is pseudo symbol with value of current location
-            in current segment.  */
-#ifdef DOT_LABEL_PREFIX
-         fake = ".L0\001";
-#else
-         fake = "L0\001";
-#endif
-         symbolP = symbol_new (fake,
-                               now_seg,
-              (valueT) (obstack_next_free (&frags) - frag_now->fr_literal),
-                               frag_now);
-
-         expressionP->X_add_number = 0;
-         expressionP->X_add_symbol = symbolP;
-         expressionP->X_seg = now_seg;
+         current_location (expressionP);
          break;
+       }
+      else if ((strncasecmp (input_line_pointer, "startof.", 8) == 0
+               && ! is_part_of_name (input_line_pointer[8]))
+              || (strncasecmp (input_line_pointer, "sizeof.", 7) == 0
+                  && ! is_part_of_name (input_line_pointer[7])))
+       {
+         int start;
+
+         start = (input_line_pointer[1] == 't'
+                  || input_line_pointer[1] == 'T');
+         input_line_pointer += start ? 8 : 7;
+         SKIP_WHITESPACE ();
+         if (*input_line_pointer != '(')
+           as_bad (_("syntax error in .startof. or .sizeof."));
+         else
+           {
+             char *buf;
+
+             ++input_line_pointer;
+             SKIP_WHITESPACE ();
+             name = input_line_pointer;
+             c = get_symbol_end ();
+
+             buf = (char *) xmalloc (strlen (name) + 10);
+             if (start)
+               sprintf (buf, ".startof.%s", name);
+             else
+               sprintf (buf, ".sizeof.%s", name);
+             symbolP = symbol_make (buf);
+             free (buf);
 
+             expressionP->X_op = O_symbol;
+             expressionP->X_add_symbol = symbolP;
+             expressionP->X_add_number = 0;
+
+             *input_line_pointer = c;
+             SKIP_WHITESPACE ();
+             if (*input_line_pointer != ')')
+               as_bad (_("syntax error in .startof. or .sizeof."));
+             else
+               ++input_line_pointer;
+           }
+         break;
        }
       else
        {
          goto isname;
-
-
        }
     case ',':
     case '\n':
+    case '\0':
     eol:
       /* can't imagine any other kind of operand */
-      expressionP->X_seg = SEG_ABSENT;
+      expressionP->X_op = O_absent;
       input_line_pointer--;
-      md_operand (expressionP);
+      break;
+
+    case '%':
+      if (! flag_m68k_mri)
+       goto de_fault;
+      integer_constant (2, expressionP);
+      break;
+
+    case '@':
+      if (! flag_m68k_mri)
+       goto de_fault;
+      integer_constant (8, expressionP);
+      break;
+
+    case ':':
+      if (! flag_m68k_mri)
+       goto de_fault;
+
+      /* In MRI mode, this is a floating point constant represented
+         using hexadecimal digits.  */
+
+      ++input_line_pointer;
+      integer_constant (16, expressionP);
+      break;
+
+    case '*':
+      if (! flag_m68k_mri || is_part_of_name (*input_line_pointer))
+       goto de_fault;
+
+      current_location (expressionP);
       break;
 
     default:
-      if (is_end_of_line[c])
+    de_fault:
+      if (is_end_of_line[(unsigned char) c])
        goto eol;
       if (is_name_beginner (c))        /* here if did not begin with a digit */
        {
@@ -608,40 +1196,101 @@ operand (expressionP)
        isname:
          name = --input_line_pointer;
          c = get_symbol_end ();
-         symbolP = symbol_find_or_make (name);
-         /* If we have an absolute symbol or a reg, then we know its value
-            now.  */
-         expressionP->X_seg = S_GET_SEGMENT (symbolP);
-         switch (expressionP->X_seg)
+
+#ifdef md_parse_name
+         /* This is a hook for the backend to parse certain names
+             specially in certain contexts.  If a name always has a
+             specific value, it can often be handled by simply
+             entering it in the symbol table.  */
+         if (md_parse_name (name, expressionP))
            {
-           case SEG_ABSOLUTE:
-           case SEG_REGISTER:
-             expressionP->X_add_number = S_GET_VALUE (symbolP);
+             *input_line_pointer = c;
              break;
+           }
+#endif
+
+#ifdef TC_I960
+         /* The MRI i960 assembler permits
+                lda sizeof code,g13
+            FIXME: This should use md_parse_name.  */
+         if (flag_mri
+             && (strcasecmp (name, "sizeof") == 0
+                 || strcasecmp (name, "startof") == 0))
+           {
+             int start;
+             char *buf;
+
+             start = (name[1] == 't'
+                      || name[1] == 'T');
+
+             *input_line_pointer = c;
+             SKIP_WHITESPACE ();
 
-           default:
+             name = input_line_pointer;
+             c = get_symbol_end ();
+
+             buf = (char *) xmalloc (strlen (name) + 10);
+             if (start)
+               sprintf (buf, ".startof.%s", name);
+             else
+               sprintf (buf, ".sizeof.%s", name);
+             symbolP = symbol_make (buf);
+             free (buf);
+
+             expressionP->X_op = O_symbol;
+             expressionP->X_add_symbol = symbolP;
              expressionP->X_add_number = 0;
+
+             *input_line_pointer = c;
+             SKIP_WHITESPACE ();
+
+             break;
+           }         
+#endif
+
+         symbolP = symbol_find_or_make (name);
+
+         /* If we have an absolute symbol or a reg, then we know its
+            value now.  */
+         segment = S_GET_SEGMENT (symbolP);
+         if (segment == absolute_section)
+           {
+             expressionP->X_op = O_constant;
+             expressionP->X_add_number = S_GET_VALUE (symbolP);
+           }
+         else if (segment == reg_section)
+           {
+             expressionP->X_op = O_register;
+             expressionP->X_add_number = S_GET_VALUE (symbolP);
+           }
+         else
+           {
+             expressionP->X_op = O_symbol;
              expressionP->X_add_symbol = symbolP;
+             expressionP->X_add_number = 0;
            }
          *input_line_pointer = c;
-         expressionP->X_subtract_symbol = NULL;
        }
       else
        {
-         as_bad ("Bad expression");
-         expressionP->X_add_number = 0;
-         expressionP->X_seg = SEG_ABSOLUTE;
-
+         /* Let the target try to parse it.  Success is indicated by changing
+            the X_op field to something other than O_absent and pointing
+            input_line_pointer passed the expression.  If it can't parse the
+            expression, X_op and input_line_pointer should be unchanged.  */
+         expressionP->X_op = O_absent;
+         --input_line_pointer;
+         md_operand (expressionP);
+         if (expressionP->X_op == O_absent)
+           {
+             ++input_line_pointer;
+             as_bad (_("Bad expression"));
+             expressionP->X_op = O_constant;
+             expressionP->X_add_number = 0;
+           }
        }
-
+      break;
     }
 
-
-
-
-
-
-
   /*
    * It is more 'efficient' to clean up the expressionS when they are created.
    * Doing it here saves lines of code.
@@ -649,162 +1298,75 @@ operand (expressionP)
   clean_up_expression (expressionP);
   SKIP_WHITESPACE ();          /*->1st char after operand. */
   know (*input_line_pointer != ' ');
-  return (expressionP->X_seg);
+
+  /* The PA port needs this information.  */
+  if (expressionP->X_add_symbol)
+    expressionP->X_add_symbol->sy_used = 1;
+
+  switch (expressionP->X_op)
+    {
+    default:
+      return absolute_section;
+    case O_symbol:
+      return S_GET_SEGMENT (expressionP->X_add_symbol);
+    case O_register:
+      return reg_section;
+    }
 }                              /* operand() */
 \f
-
 /* Internal. Simplify a struct expression for use by expr() */
 
 /*
  * In: address of a expressionS.
- *     The X_seg field of the expressionS may only take certain values.
- *     Now, we permit SEG_PASS1 to make code smaller & faster.
+ *     The X_op field of the expressionS may only take certain values.
  *     Elsewise we waste time special-case testing. Sigh. Ditto SEG_ABSENT.
  * Out:        expressionS may have been modified:
  *     'foo-foo' symbol references cancelled to 0,
- *             which changes X_seg from SEG_DIFFERENCE to SEG_ABSOLUTE;
+ *             which changes X_op from O_subtract to O_constant.
  *     Unused fields zeroed to help expr().
  */
 
 static void
 clean_up_expression (expressionP)
-     register expressionS *expressionP;
+     expressionS *expressionP;
 {
-  switch (expressionP->X_seg)
+  switch (expressionP->X_op)
     {
-    case SEG_ABSENT:
-    case SEG_PASS1:
-      expressionP->X_add_symbol = NULL;
-      expressionP->X_subtract_symbol = NULL;
+    case O_illegal:
+    case O_absent:
       expressionP->X_add_number = 0;
-      break;
-
-    case SEG_BIG:
-    case SEG_ABSOLUTE:
-      expressionP->X_subtract_symbol = NULL;
+      /* Fall through.  */
+    case O_big:
+    case O_constant:
+    case O_register:
       expressionP->X_add_symbol = NULL;
+      /* Fall through.  */
+    case O_symbol:
+    case O_uminus:
+    case O_bit_not:
+      expressionP->X_op_symbol = NULL;
       break;
-
-    case SEG_UNKNOWN:
-      expressionP->X_subtract_symbol = NULL;
-      break;
-
-    case SEG_DIFFERENCE:
-      /*
-        * It does not hurt to 'cancel' NULL==NULL
-        * when comparing symbols for 'eq'ness.
-        * It is faster to re-cancel them to NULL
-        * than to check for this special case.
-        */
-      if (expressionP->X_subtract_symbol == expressionP->X_add_symbol
-         || (expressionP->X_subtract_symbol
-             && expressionP->X_add_symbol
-             && expressionP->X_subtract_symbol->sy_frag == expressionP->X_add_symbol->sy_frag
-             && S_GET_VALUE (expressionP->X_subtract_symbol) == S_GET_VALUE (expressionP->X_add_symbol)))
+    case O_subtract:
+      if (expressionP->X_op_symbol == expressionP->X_add_symbol
+         || ((expressionP->X_op_symbol->sy_frag
+              == expressionP->X_add_symbol->sy_frag)
+             && SEG_NORMAL (S_GET_SEGMENT (expressionP->X_add_symbol))
+             && (S_GET_VALUE (expressionP->X_op_symbol)
+                 == S_GET_VALUE (expressionP->X_add_symbol))))
        {
-         expressionP->X_subtract_symbol = NULL;
+         addressT diff = (S_GET_VALUE (expressionP->X_add_symbol)
+                          - S_GET_VALUE (expressionP->X_op_symbol));
+
+         expressionP->X_op = O_constant;
          expressionP->X_add_symbol = NULL;
-         expressionP->X_seg = SEG_ABSOLUTE;
+         expressionP->X_op_symbol = NULL;
+         expressionP->X_add_number += diff;
        }
       break;
-
-    case SEG_REGISTER:
-      expressionP->X_add_symbol = NULL;
-      expressionP->X_subtract_symbol = NULL;
-      break;
-
     default:
-      if (SEG_NORMAL (expressionP->X_seg))
-       {
-         expressionP->X_subtract_symbol = NULL;
-       }
-      else
-       {
-         BAD_CASE (expressionP->X_seg);
-       }
       break;
     }
-}                              /* clean_up_expression() */
-\f
-/*
- *                     expr_part ()
- *
- * Internal. Made a function because this code is used in 2 places.
- * Generate error or correct X_?????_symbol of expressionS.
- */
-
-/*
- * symbol_1 += symbol_2 ... well ... sort of.
- */
-
-static segT
-expr_part (symbol_1_PP, symbol_2_P)
-     symbolS **symbol_1_PP;
-     symbolS *symbol_2_P;
-{
-  segT return_value;
-#ifndef MANY_SEGMENTS
-  know ((*symbol_1_PP) == NULL || (S_GET_SEGMENT (*symbol_1_PP) == SEG_TEXT) || (S_GET_SEGMENT (*symbol_1_PP) == SEG_DATA) || (S_GET_SEGMENT (*symbol_1_PP) == SEG_BSS) || (!S_IS_DEFINED (*symbol_1_PP)));
-  know (symbol_2_P == NULL || (S_GET_SEGMENT (symbol_2_P) == SEG_TEXT) || (S_GET_SEGMENT (symbol_2_P) == SEG_DATA) || (S_GET_SEGMENT (symbol_2_P) == SEG_BSS) || (!S_IS_DEFINED (symbol_2_P)));
-#endif
-  if (*symbol_1_PP)
-    {
-      if (!S_IS_DEFINED (*symbol_1_PP))
-       {
-         if (symbol_2_P)
-           {
-             return_value = SEG_PASS1;
-             *symbol_1_PP = NULL;
-           }
-         else
-           {
-             know (!S_IS_DEFINED (*symbol_1_PP));
-             return_value = SEG_UNKNOWN;
-           }
-       }
-      else
-       {
-         if (symbol_2_P)
-           {
-             if (!S_IS_DEFINED (symbol_2_P))
-               {
-                 *symbol_1_PP = NULL;
-                 return_value = SEG_PASS1;
-               }
-             else
-               {
-                 /* {seg1} - {seg2} */
-                 as_bad ("Expression too complex, 2 symbolS forgotten: \"%s\" \"%s\"",
-                       S_GET_NAME (*symbol_1_PP), S_GET_NAME (symbol_2_P));
-                 *symbol_1_PP = NULL;
-                 return_value = SEG_ABSOLUTE;
-               }
-           }
-         else
-           {
-             return_value = S_GET_SEGMENT (*symbol_1_PP);
-           }
-       }
-    }
-  else
-    {                          /* (* symbol_1_PP) == NULL */
-      if (symbol_2_P)
-       {
-         *symbol_1_PP = symbol_2_P;
-         return_value = S_GET_SEGMENT (symbol_2_P);
-       }
-      else
-       {
-         *symbol_1_PP = NULL;
-         return_value = SEG_ABSOLUTE;
-       }
-    }
-#ifndef MANY_SEGMENTS
-  know (return_value == SEG_ABSOLUTE || return_value == SEG_TEXT || return_value == SEG_DATA || return_value == SEG_BSS || return_value == SEG_UNKNOWN || return_value == SEG_PASS1);
-#endif
-  know ((*symbol_1_PP) == NULL || (S_GET_SEGMENT (*symbol_1_PP) == return_value));
-  return (return_value);
-}                              /* expr_part() */
+}
 \f
 /* Expression parser. */
 
@@ -824,27 +1386,12 @@ expr_part (symbol_1_PP, symbol_2_P)
  * After expr(RANK,resultP) input_line_pointer->operator of rank <= RANK.
  * Also, we have consumed any leading or trailing spaces (operand does that)
  * and done all intervening operators.
+ *
+ * This returns the segment of the result, which will be
+ * absolute_section or the segment of a symbol.
  */
 
-typedef enum
-{
-  O_illegal,                   /* (0)  what we get for illegal op */
-
-  O_multiply,                  /* (1)  * */
-  O_divide,                    /* (2)  / */
-  O_modulus,                   /* (3)  % */
-  O_left_shift,                        /* (4)  < */
-  O_right_shift,               /* (5)  > */
-  O_bit_inclusive_or,          /* (6)  | */
-  O_bit_or_not,                        /* (7)  ! */
-  O_bit_exclusive_or,          /* (8)  ^ */
-  O_bit_and,                   /* (9)  & */
-  O_add,                       /* (10) + */
-  O_subtract                   /* (11) - */
-}
-
-operatorT;
-
+#undef __
 #define __ O_illegal
 
 static const operatorT op_encoding[256] =
@@ -856,7 +1403,7 @@ static const operatorT op_encoding[256] =
   __, O_bit_or_not, __, __, __, O_modulus, O_bit_and, __,
   __, __, O_multiply, O_add, __, O_subtract, __, O_divide,
   __, __, __, __, __, __, __, __,
-  __, __, __, __, O_left_shift, __, O_right_shift, __,
+  __, __, __, __, O_lt, __, O_gt, __,
   __, __, __, __, __, __, __, __,
   __, __, __, __, __, __, __, __,
   __, __, __, __, __, __, __, __,
@@ -880,237 +1427,388 @@ static const operatorT op_encoding[256] =
 /*
  *     Rank    Examples
  *     0       operand, (expression)
- *     1       + -
- *     2       & ^ ! |
- *     3       * / % << >>
+ *     1       ||
+ *     2       &&
+ *     3       = <> < <= >= >
+ *     4       + -
+ *     5       used for * / % in MRI mode
+ *     6       & ^ ! |
+ *     7       * / % << >>
+ *     8       unary - unary ~
  */
-static const operator_rankT
-  op_rank[] =
-{0, 3, 3, 3, 3, 3, 2, 2, 2, 2, 1, 1};
+static operator_rankT op_rank[] =
+{
+  0,   /* O_illegal */
+  0,   /* O_absent */
+  0,   /* O_constant */
+  0,   /* O_symbol */
+  0,   /* O_symbol_rva */
+  0,   /* O_register */
+  0,   /* O_bit */
+  8,   /* O_uminus */
+  8,   /* O_bit_not */
+  8,   /* O_logical_not */
+  7,   /* O_multiply */
+  7,   /* O_divide */
+  7,   /* O_modulus */
+  7,   /* O_left_shift */
+  7,   /* O_right_shift */
+  6,   /* O_bit_inclusive_or */
+  6,   /* O_bit_or_not */
+  6,   /* O_bit_exclusive_or */
+  6,   /* O_bit_and */
+  4,   /* O_add */
+  4,   /* O_subtract */
+  3,   /* O_eq */
+  3,   /* O_ne */
+  3,   /* O_lt */
+  3,   /* O_le */
+  3,   /* O_ge */
+  3,   /* O_gt */
+  2,   /* O_logical_and */
+  1    /* O_logical_or */
+};
+
+/* Unfortunately, in MRI mode for the m68k, multiplication and
+   division have lower precedence than the bit wise operators.  This
+   function sets the operator precedences correctly for the current
+   mode.  Also, MRI uses a different bit_not operator, and this fixes
+   that as well.  */
+
+#define STANDARD_MUL_PRECEDENCE (7)
+#define MRI_MUL_PRECEDENCE (5)
+
+void
+expr_set_precedence ()
+{
+  if (flag_m68k_mri)
+    {
+      op_rank[O_multiply] = MRI_MUL_PRECEDENCE;
+      op_rank[O_divide] = MRI_MUL_PRECEDENCE;
+      op_rank[O_modulus] = MRI_MUL_PRECEDENCE;
+    }
+  else
+    {
+      op_rank[O_multiply] = STANDARD_MUL_PRECEDENCE;
+      op_rank[O_divide] = STANDARD_MUL_PRECEDENCE;
+      op_rank[O_modulus] = STANDARD_MUL_PRECEDENCE;
+    }
+}
+
+/* Initialize the expression parser.  */
+
+void
+expr_begin ()
+{
+  expr_set_precedence ();
+
+  /* Verify that X_op field is wide enough.  */
+  {
+    expressionS e;
+    e.X_op = O_max;
+    assert (e.X_op == O_max);
+  }
+}
 \f
-/* Return resultP->X_seg. */
-segT 
+/* Return the encoding for the operator at INPUT_LINE_POINTER.
+   Advance INPUT_LINE_POINTER to the last character in the operator
+   (i.e., don't change it for a single character operator).  */
+
+static inline operatorT
+operator ()
+{
+  int c;
+  operatorT ret;
+
+  c = *input_line_pointer;
+
+  switch (c)
+    {
+    default:
+      return op_encoding[c];
+
+    case '<':
+      switch (input_line_pointer[1])
+       {
+       default:
+         return op_encoding[c];
+       case '<':
+         ret = O_left_shift;
+         break;
+       case '>':
+         ret = O_ne;
+         break;
+       case '=':
+         ret = O_le;
+         break;
+       }
+      ++input_line_pointer;
+      return ret;
+
+    case '=':
+      if (input_line_pointer[1] != '=')
+       return op_encoding[c];
+
+      ++input_line_pointer;
+      return O_eq;
+
+    case '>':
+      switch (input_line_pointer[1])
+       {
+       default:
+         return op_encoding[c];
+       case '>':
+         ret = O_right_shift;
+         break;
+       case '=':
+         ret = O_ge;
+         break;
+       }
+      ++input_line_pointer;
+      return ret;
+
+    case '!':
+      /* We accept !! as equivalent to ^ for MRI compatibility.  */
+      if (input_line_pointer[1] != '!')
+       {
+         if (flag_m68k_mri)
+           return O_bit_inclusive_or;
+         return op_encoding[c];
+       }
+      ++input_line_pointer;
+      return O_bit_exclusive_or;
+
+    case '|':
+      if (input_line_pointer[1] != '|')
+       return op_encoding[c];
+
+      ++input_line_pointer;
+      return O_logical_or;
+
+    case '&':
+      if (input_line_pointer[1] != '&')
+       return op_encoding[c];
+
+      ++input_line_pointer;
+      return O_logical_and;
+    }
+
+  /*NOTREACHED*/
+}
+
+/* Parse an expression.  */
+
+segT
 expr (rank, resultP)
-     register operator_rankT rank;     /* Larger # is higher rank. */
-     register expressionS *resultP;    /* Deliver result here. */
+     operator_rankT rank;      /* Larger # is higher rank. */
+     expressionS *resultP;     /* Deliver result here. */
 {
+  segT retval;
   expressionS right;
-  register operatorT op_left;
-  register char c_left;                /* 1st operator character. */
-  register operatorT op_right;
-  register char c_right;
+  operatorT op_left;
+  operatorT op_right;
 
   know (rank >= 0);
-  (void) operand (resultP);
+
+  retval = operand (resultP);
+
   know (*input_line_pointer != ' ');   /* Operand() gobbles spaces. */
-  c_left = *input_line_pointer;        /* Potential operator character. */
-  op_left = op_encoding[c_left];
+
+  op_left = operator ();
   while (op_left != O_illegal && op_rank[(int) op_left] > rank)
     {
+      segT rightseg;
+
       input_line_pointer++;    /*->after 1st character of operator. */
-      /* Operators "<<" and ">>" have 2 characters. */
-      if (*input_line_pointer == c_left && (c_left == '<' || c_left == '>'))
-       {
-         input_line_pointer++;
-       }                       /*->after operator. */
-      if (SEG_ABSENT == expr (op_rank[(int) op_left], &right))
+
+      rightseg = expr (op_rank[(int) op_left], &right);
+      if (right.X_op == O_absent)
        {
-         as_warn ("Missing operand value assumed absolute 0.");
-         resultP->X_add_number = 0;
-         resultP->X_subtract_symbol = NULL;
-         resultP->X_add_symbol = NULL;
-         resultP->X_seg = SEG_ABSOLUTE;
+         as_warn (_("missing operand; zero assumed"));
+         right.X_op = O_constant;
+         right.X_add_number = 0;
+         right.X_add_symbol = NULL;
+         right.X_op_symbol = NULL;
        }
+
       know (*input_line_pointer != ' ');
-      c_right = *input_line_pointer;
-      op_right = op_encoding[c_right];
-      if (*input_line_pointer == c_right && (c_right == '<' || c_right == '>'))
+
+      if (retval == undefined_section)
        {
-         input_line_pointer++;
-       }                       /*->after operator. */
-      know ((int) op_right == 0 || op_rank[(int) op_right] <= op_rank[(int) op_left]);
+         if (SEG_NORMAL (rightseg))
+           retval = rightseg;
+       }
+      else if (! SEG_NORMAL (retval))
+       retval = rightseg;
+      else if (SEG_NORMAL (rightseg)
+              && retval != rightseg
+#ifdef DIFF_EXPR_OK
+              && op_left != O_subtract
+#endif
+              )
+       as_bad (_("operation combines symbols in different segments"));
+
+      op_right = operator ();
+
+      know (op_right == O_illegal || op_rank[(int) op_right] <= op_rank[(int) op_left]);
+      know ((int) op_left >= (int) O_multiply
+           && (int) op_left <= (int) O_logical_or);
+
       /* input_line_pointer->after right-hand quantity. */
       /* left-hand quantity in resultP */
       /* right-hand quantity in right. */
       /* operator in op_left. */
-      if (resultP->X_seg == SEG_PASS1 || right.X_seg == SEG_PASS1)
+
+      if (resultP->X_op == O_big)
        {
-         resultP->X_seg = SEG_PASS1;
+         if (resultP->X_add_number > 0)
+           as_warn (_("left operand is a bignum; integer 0 assumed"));
+         else
+           as_warn (_("left operand is a float; integer 0 assumed"));
+         resultP->X_op = O_constant;
+         resultP->X_add_number = 0;
+         resultP->X_add_symbol = NULL;
+         resultP->X_op_symbol = NULL;
        }
-      else
+      if (right.X_op == O_big)
        {
-         if (resultP->X_seg == SEG_BIG)
-           {
-             as_warn ("Left operand of %c is a %s.  Integer 0 assumed.",
-                   c_left, resultP->X_add_number > 0 ? "bignum" : "float");
-             resultP->X_seg = SEG_ABSOLUTE;
-             resultP->X_add_symbol = 0;
-             resultP->X_subtract_symbol = 0;
-             resultP->X_add_number = 0;
-           }
-         if (right.X_seg == SEG_BIG)
+         if (right.X_add_number > 0)
+         as_warn (_("right operand is a bignum; integer 0 assumed"));
+         as_warn (_("right operand is a float; integer 0 assumed"));
+         right.X_op = O_constant;
+         right.X_add_number = 0;
+         right.X_add_symbol = NULL;
+         right.X_op_symbol = NULL;
+       }
+
+      /* Optimize common cases.  */
+      if (op_left == O_add && right.X_op == O_constant)
+       {
+         /* X + constant.  */
+         resultP->X_add_number += right.X_add_number;
+       }
+      /* This case comes up in PIC code.  */
+      else if (op_left == O_subtract
+              && right.X_op == O_symbol
+              && resultP->X_op == O_symbol
+              && (right.X_add_symbol->sy_frag
+                  == resultP->X_add_symbol->sy_frag)
+              && SEG_NORMAL (S_GET_SEGMENT (right.X_add_symbol)))
+
+       {
+         resultP->X_add_number -= right.X_add_number;
+         resultP->X_add_number += (S_GET_VALUE (resultP->X_add_symbol)
+                                   - S_GET_VALUE (right.X_add_symbol));
+         resultP->X_op = O_constant;
+         resultP->X_add_symbol = 0;
+       }
+      else if (op_left == O_subtract && right.X_op == O_constant)
+       {
+         /* X - constant.  */
+         resultP->X_add_number -= right.X_add_number;
+       }
+      else if (op_left == O_add && resultP->X_op == O_constant)
+       {
+         /* Constant + X.  */
+         resultP->X_op = right.X_op;
+         resultP->X_add_symbol = right.X_add_symbol;
+         resultP->X_op_symbol = right.X_op_symbol;
+         resultP->X_add_number += right.X_add_number;
+         retval = rightseg;
+       }
+      else if (resultP->X_op == O_constant && right.X_op == O_constant)
+       {
+         /* Constant OP constant.  */
+         offsetT v = right.X_add_number;
+         if (v == 0 && (op_left == O_divide || op_left == O_modulus))
            {
-             as_warn ("Right operand of %c is a %s.  Integer 0 assumed.",
-                      c_left, right.X_add_number > 0 ? "bignum" : "float");
-             right.X_seg = SEG_ABSOLUTE;
-             right.X_add_symbol = 0;
-             right.X_subtract_symbol = 0;
-             right.X_add_number = 0;
+             as_warn (_("division by zero"));
+             v = 1;
            }
-         if (op_left == O_subtract)
+         switch (op_left)
            {
-             /*
-              * Convert - into + by exchanging symbolS and negating number.
-              * I know -infinity can't be negated in 2's complement:
-              * but then it can't be subtracted either. This trick
-              * does not cause any further inaccuracy.
-              */
-
-             register symbolS *symbolP;
-
-             right.X_add_number = -right.X_add_number;
-             symbolP = right.X_add_symbol;
-             right.X_add_symbol = right.X_subtract_symbol;
-             right.X_subtract_symbol = symbolP;
-             if (symbolP)
-               {
-                 right.X_seg = SEG_DIFFERENCE;
-               }
-             op_left = O_add;
+           default:                    abort ();
+           case O_multiply:            resultP->X_add_number *= v; break;
+           case O_divide:              resultP->X_add_number /= v; break;
+           case O_modulus:             resultP->X_add_number %= v; break;
+           case O_left_shift:          resultP->X_add_number <<= v; break;
+           case O_right_shift:
+             /* We always use unsigned shifts, to avoid relying on
+                 characteristics of the compiler used to compile gas.  */
+             resultP->X_add_number =
+               (offsetT) ((valueT) resultP->X_add_number >> (valueT) v);
+             break;
+           case O_bit_inclusive_or:    resultP->X_add_number |= v; break;
+           case O_bit_or_not:          resultP->X_add_number |= ~v; break;
+           case O_bit_exclusive_or:    resultP->X_add_number ^= v; break;
+           case O_bit_and:             resultP->X_add_number &= v; break;
+           case O_add:                 resultP->X_add_number += v; break;
+           case O_subtract:            resultP->X_add_number -= v; break;
+           case O_eq:
+             resultP->X_add_number =
+               resultP->X_add_number == v ? ~ (offsetT) 0 : 0;
+             break;
+           case O_ne:
+             resultP->X_add_number =
+               resultP->X_add_number != v ? ~ (offsetT) 0 : 0;
+             break;
+           case O_lt:
+             resultP->X_add_number =
+               resultP->X_add_number <  v ? ~ (offsetT) 0 : 0;
+             break;
+           case O_le:
+             resultP->X_add_number =
+               resultP->X_add_number <= v ? ~ (offsetT) 0 : 0;
+             break;
+           case O_ge:
+             resultP->X_add_number =
+               resultP->X_add_number >= v ? ~ (offsetT) 0 : 0;
+             break;
+           case O_gt:
+             resultP->X_add_number =
+               resultP->X_add_number >  v ? ~ (offsetT) 0 : 0;
+             break;
+           case O_logical_and:
+             resultP->X_add_number = resultP->X_add_number && v;
+             break;
+           case O_logical_or:
+             resultP->X_add_number = resultP->X_add_number || v;
+             break;
            }
-\f
+       }
+      else if (resultP->X_op == O_symbol
+              && right.X_op == O_symbol
+              && (op_left == O_add
+                  || op_left == O_subtract
+                  || (resultP->X_add_number == 0
+                      && right.X_add_number == 0)))
+       {
+         /* Symbol OP symbol.  */
+         resultP->X_op = op_left;
+         resultP->X_op_symbol = right.X_add_symbol;
          if (op_left == O_add)
-           {
-             segT seg1;
-             segT seg2;
-#ifndef MANY_SEGMENTS
-
-             know (resultP->X_seg == SEG_DATA || resultP->X_seg == SEG_TEXT || resultP->X_seg == SEG_BSS || resultP->X_seg == SEG_UNKNOWN || resultP->X_seg == SEG_DIFFERENCE || resultP->X_seg == SEG_ABSOLUTE || resultP->X_seg == SEG_PASS1 || resultP->X_seg == SEG_REGISTER);
-
-             know (right.X_seg == SEG_DATA || right.X_seg == SEG_TEXT || right.X_seg == SEG_BSS || right.X_seg == SEG_UNKNOWN || right.X_seg == SEG_DIFFERENCE || right.X_seg == SEG_ABSOLUTE || right.X_seg == SEG_PASS1);
-#endif
-             clean_up_expression (&right);
-             clean_up_expression (resultP);
-
-             seg1 = expr_part (&resultP->X_add_symbol, right.X_add_symbol);
-             seg2 = expr_part (&resultP->X_subtract_symbol, right.X_subtract_symbol);
-             if (seg1 == SEG_PASS1 || seg2 == SEG_PASS1)
-               {
-                 need_pass_2 = 1;
-                 resultP->X_seg = SEG_PASS1;
-               }
-             else if (seg2 == SEG_ABSOLUTE)
-               resultP->X_seg = seg1;
-             else if (seg1 != SEG_UNKNOWN
-                      && seg1 != SEG_ABSOLUTE
-                      && seg2 != SEG_UNKNOWN
-                      && seg1 != seg2)
-               {
-                 know (seg2 != SEG_ABSOLUTE);
-                 know (resultP->X_subtract_symbol);
-#ifndef MANY_SEGMENTS
-                 know (seg1 == SEG_TEXT || seg1 == SEG_DATA || seg1 == SEG_BSS);
-                 know (seg2 == SEG_TEXT || seg2 == SEG_DATA || seg2 == SEG_BSS);
-#endif
-                 know (resultP->X_add_symbol);
-                 know (resultP->X_subtract_symbol);
-                 as_bad ("Expression too complex: forgetting %s - %s",
-                         S_GET_NAME (resultP->X_add_symbol),
-                         S_GET_NAME (resultP->X_subtract_symbol));
-                 resultP->X_seg = SEG_ABSOLUTE;
-                 /* Clean_up_expression() will do the rest. */
-               }
-             else
-               resultP->X_seg = SEG_DIFFERENCE;
+           resultP->X_add_number += right.X_add_number;
+         else if (op_left == O_subtract)
+           resultP->X_add_number -= right.X_add_number;
+       }
+      else
+       {
+         /* The general case.  */
+         resultP->X_add_symbol = make_expr_symbol (resultP);
+         resultP->X_op_symbol = make_expr_symbol (&right);
+         resultP->X_op = op_left;
+         resultP->X_add_number = 0;
+         resultP->X_unsigned = 1;
+       }
 
-             resultP->X_add_number += right.X_add_number;
-             clean_up_expression (resultP);
-           }
-         else
-           {                   /* Not +. */
-             if (resultP->X_seg == SEG_UNKNOWN || right.X_seg == SEG_UNKNOWN)
-               {
-                 resultP->X_seg = SEG_PASS1;
-                 need_pass_2 = 1;
-               }
-             else
-               {
-                 resultP->X_subtract_symbol = NULL;
-                 resultP->X_add_symbol = NULL;
-                 /* Will be SEG_ABSOLUTE. */
-                 if (resultP->X_seg != SEG_ABSOLUTE || right.X_seg != SEG_ABSOLUTE)
-                   {
-                     as_bad ("Relocation error. Absolute 0 assumed.");
-                     resultP->X_seg = SEG_ABSOLUTE;
-                     resultP->X_add_number = 0;
-                   }
-                 else
-                   {
-                     switch (op_left)
-                       {
-                       case O_bit_inclusive_or:
-                         resultP->X_add_number |= right.X_add_number;
-                         break;
-
-                       case O_modulus:
-                         if (right.X_add_number)
-                           {
-                             resultP->X_add_number %= right.X_add_number;
-                           }
-                         else
-                           {
-                             as_warn ("Division by 0. 0 assumed.");
-                             resultP->X_add_number = 0;
-                           }
-                         break;
-
-                       case O_bit_and:
-                         resultP->X_add_number &= right.X_add_number;
-                         break;
-
-                       case O_multiply:
-                         resultP->X_add_number *= right.X_add_number;
-                         break;
-
-                       case O_divide:
-                         if (right.X_add_number)
-                           {
-                             resultP->X_add_number /= right.X_add_number;
-                           }
-                         else
-                           {
-                             as_warn ("Division by 0. 0 assumed.");
-                             resultP->X_add_number = 0;
-                           }
-                         break;
-
-                       case O_left_shift:
-                         resultP->X_add_number <<= right.X_add_number;
-                         break;
-
-                       case O_right_shift:
-                         resultP->X_add_number >>= right.X_add_number;
-                         break;
-
-                       case O_bit_exclusive_or:
-                         resultP->X_add_number ^= right.X_add_number;
-                         break;
-
-                       case O_bit_or_not:
-                         resultP->X_add_number |= ~right.X_add_number;
-                         break;
-
-                       default:
-                         BAD_CASE (op_left);
-                         break;
-                       }       /* switch(operator) */
-                   }
-               }               /* If we have to force need_pass_2. */
-           }                   /* If operator was +. */
-       }                       /* If we didn't set need_pass_2. */
       op_left = op_right;
     }                          /* While next operator is >= this rank. */
-  return (resultP->X_seg);
+
+  /* The PA port needs this information.  */
+  if (resultP->X_add_symbol)
+    resultP->X_add_symbol->sy_used = 1;
+
+  return resultP->X_op == O_constant ? absolute_section : retval;
 }
 \f
 /*
@@ -1131,16 +1829,20 @@ expr (rank, resultP)
 char
 get_symbol_end ()
 {
-  register char c;
+  char c;
 
-  while (is_part_of_name (c = *input_line_pointer++))
-    ;
+  /* We accept \001 in a name in case this is being called with a
+     constructed string.  */
+  if (is_name_beginner (c = *input_line_pointer++) || c == '\001')
+    while (is_part_of_name (c = *input_line_pointer++)
+          || c == '\001')
+      ;
   *--input_line_pointer = 0;
   return (c);
 }
 
 
-unsigned int 
+unsigned int
 get_single_number ()
 {
   expressionS exp;