* expr.c (operand): Handle 08 and 09 in MRI mode.
[binutils-gdb.git] / gas / expr.c
index 601feafd765d60fdddac08ae746493779f951c58..ebc580f572f66b3e5c6a25a7d6152dc04565763f 100644 (file)
@@ -1,21 +1,21 @@
 /* expr.c -operands, expressions-
-   Copyright (C) 1987, 1990, 1991 Free Software Foundation, Inc.
-   
+   Copyright (C) 1987, 1990, 1991, 1992, 1993, 1994 Free Software Foundation, Inc.
+
    This file is part of GAS, the GNU Assembler.
-   
+
    GAS is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 2, or (at your option)
    any later version.
-   
+
    GAS is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    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. */
+   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 <string.h>
 
 #include "as.h"
-
+#include "libiberty.h"
 #include "obstack.h"
 
-#ifdef __STDC__
-static void clean_up_expression(expressionS *expressionP);
-#else /* __STDC__ */
-static void clean_up_expression();     /* Internal. */
-#endif /* __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 clean_up_expression PARAMS ((expressionS * expressionP));
 
-#ifdef LOCAL_LABELS_DOLLAR
-extern int local_label_defined[];
-#endif
+extern const char EXP_CHARS[], FLT_CHARS[];
+\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;
+{
+  const char *fake;
+  symbolS *symbolP;
+
+  if (expressionP->X_op == O_symbol
+      && expressionP->X_add_number == 0)
+    return expressionP->X_add_symbol;
+
+  /* FIXME: This should be something which decode_local_label_name
+     will handle.  */
+  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);
+
+  return symbolP;
+}
+\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.
-   */
-LITTLENUM_TYPE generic_bignum [SIZE_OF_LARGE_NUMBER+6];
-FLONUM_TYPE    generic_floating_point_number =
+   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) */
-    0,                         /* leader */
-    0,                         /* exponent */
-    0                          /* sign */
-    };
+  &generic_bignum[6],          /* low (JF: Was 0) */
+  &generic_bignum[SIZE_OF_LARGE_NUMBER + 6 - 1], /* high JF: (added +6) */
+  0,                           /* leader */
+  0,                           /* exponent */
+  0                            /* sign */
+};
 /* If nonzero, we've been asked to assemble nan, +inf or -inf */
 int generic_floating_point_magic;
 \f
-/*
- * Summary of operand().
- *
- * 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.
- *     Input_line_pointer->(next non-blank) char after operand.
- *
- */
-\f
-static segT
-    operand (expressionP)
-register expressionS * expressionP;
+static void
+floating_constant (expressionP)
+     expressionS *expressionP;
 {
-    register char c;
-    register char *name;       /* points to name of symbol */
-    register symbolS * symbolP; /* Points to symbol */
-    
-    extern  char hex_value[];  /* In hex_value.c */
-    
-    SKIP_WHITESPACE();         /* Leading whitespace is part of operand. */
-    c = * input_line_pointer ++;       /* Input_line_pointer->past char in c. */
-    if (isdigit(c) || (c == 'H' && input_line_pointer[0] == '\''))
+  /* input_line_pointer->*/
+  /* floating-point constant. */
+  int error_code;
+
+  error_code = atof_generic (&input_line_pointer, ".", EXP_CHARS,
+                            &generic_floating_point_number);
+
+  if (error_code)
     {
-       register valueT number; /* offset or (absolute) value */
-       register short int digit;       /* value of next digit in current radix */
-       /* invented for humans only, hope */
-       /* optimising compiler flushes it! */
-       register short int radix;       /* 2, 8, 10 or 16 */
-       /* 0 means we saw start of a floating- */
-       /* point constant. */
-       register short int maxdig = 0;/* Highest permitted digit value. */
-       register int too_many_digits = 0; /* If we see >= this number of */
-       /* digits, assume it is a bignum. */
-       register char * digit_2; /*->2nd digit of number. */
-       int small;      /* TRUE if fits in 32 bits. */
-       
-       
-       if (c == 'H' || c == '0') {                     /* non-decimal radix */
-           if ((c = *input_line_pointer ++)=='x' || c=='X' || c=='\'') {
-               c = *input_line_pointer ++; /* read past "0x" or "0X" or H' */
-               maxdig = radix = 16;
-               too_many_digits = 9;
-           } else {
-               /* 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' || c == 'b' || c == 'B')
-                   && (!*input_line_pointer ||
-                       (!strchr("+-.0123456789",*input_line_pointer) &&
-                        !strchr(EXP_CHARS,*input_line_pointer)))) {
-                   maxdig = radix = 10;
-                   too_many_digits = 11;
-                   c = '0';
-                   input_line_pointer -= 2;
-                   
-               } else if (c == 'b' || c == 'B') {
-                   c = *input_line_pointer++;
-                   maxdig = radix = 2;
-                   too_many_digits = 33;
-                   
-               } else if (c && strchr(FLT_CHARS,c)) {
-                   radix = 0;  /* Start of floating-point constant. */
-                   /* input_line_pointer->1st char of number. */
-                   expressionP->X_add_number =  -(isupper(c) ? tolower(c) : c);
-                   
-               } else {                /* By elimination, assume octal radix. */
-                   radix = maxdig = 8;
-                   too_many_digits = 11;
-               }
-           } /* c == char after "0" or "0x" or "0X" or "0e" etc. */
-       } else {
-           maxdig = radix = 10;
-           too_many_digits = 11;
-       } /* if operand starts with a zero */
-       
-       if (radix) {                    /* Fixed-point integer constant. */
-           /* 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.
-            */
-           digit_2 = input_line_pointer;
-           for (number=0;  (digit=hex_value[c])<maxdig;  c = * input_line_pointer ++)
+      if (error_code == ERROR_EXPONENT_OVERFLOW)
+       {
+         as_bad ("bad floating-point constant: exponent overflow, probably assembling junk");
+       }
+      else
+       {
+         as_bad ("bad floating-point constant: unknown error code=%d.", error_code);
+       }
+    }
+  expressionP->X_op = O_big;
+  /* input_line_pointer->just after constant, */
+  /* which may point to whitespace. */
+  expressionP->X_add_number = -1;
+}
+
+static void
+integer_constant (radix, expressionP)
+     int radix;
+     expressionS *expressionP;
+{
+  char *start;         /* start of number. */
+  char *suffix = NULL;
+  char c;
+  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. */
+
+  /* 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
+
+  if (flag_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 (*suffix); suffix++)
+       {
+         if (*suffix == 'e' || *suffix == 'E')
+           flt = 1;
+       }
+
+      if (suffix == input_line_pointer)
+       {
+         radix = 10;
+         suffix = NULL;
+       }
+      else
+       {
+         c = *--suffix;
+         if (islower (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)
            {
-               number = number * radix + digit;
+             floating_constant (expressionP);
+             return;
            }
-           /* C contains character after number. */
-           /* Input_line_pointer->char after C. */
-           small = input_line_pointer - digit_2 < too_many_digits;
-           if (! small)
+         else
            {
-               /*
-                * We saw a lot of digits. Manufacture a bignum the hard way.
-                */
-               LITTLENUM_TYPE *        leader; /*->high order littlenum of the bignum. */
-               LITTLENUM_TYPE *        pointer; /*->littlenum we are frobbing now. */
-               long carry;
-               
-               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. */
-               c = *input_line_pointer ++;
-               for (;   (carry = hex_value [c]) < maxdig;   c = * input_line_pointer ++)
-               {
-                   for (pointer = generic_bignum;
-                        pointer <= leader;
-                        pointer ++)
-                   {
-                       long work;
-                       
-                       work = carry + radix * * pointer;
-                       * pointer = work & LITTLENUM_MASK;
-                       carry = work >> LITTLENUM_NUMBER_OF_BITS;
-                   }
-                   if (carry)
-                   {
-                       if (leader < generic_bignum + SIZE_OF_LARGE_NUMBER - 1)
-                       {       /* 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);
-                   small = 1;
-               }
-               else
-               {
-                   number = leader - generic_bignum + 1;       /* Number of littlenums in the bignum. */
-               }
+             radix = 10;
+             suffix = NULL;
            }
-           if (small)
+       }
+    }
+
+  switch (radix)
+    {
+    case 2:
+      maxdig = 2;
+      too_many_digits = valuesize + 1;
+      break;
+    case 8:
+      maxdig = radix = 8;
+      too_many_digits = (valuesize + 2) / 3 + 1;
+      break;
+    case 16:
+      maxdig = radix = 16;
+      too_many_digits = (valuesize + 3) / 4 + 1;
+      break;
+    case 10:
+      maxdig = radix = 10;
+      too_many_digits = (valuesize + 12) / 4; /* very rough */
+    }
+#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 - start - 1) < too_many_digits;
+  if (!small)
+    {
+      /*
+       * we saw a lot of digits. manufacture a bignum the hard way.
+       */
+      LITTLENUM_TYPE *leader;  /*->high order littlenum of the bignum. */
+      LITTLENUM_TYPE *pointer; /*->littlenum we are frobbing now. */
+      long carry;
+
+      leader = generic_bignum;
+      generic_bignum[0] = 0;
+      generic_bignum[1] = 0;
+      input_line_pointer = start;      /*->1st digit. */
+      c = *input_line_pointer++;
+      for (;
+          (carry = hex_value (c)) < maxdig;
+          c = *input_line_pointer++)
+       {
+         for (pointer = generic_bignum;
+              pointer <= leader;
+              pointer++)
            {
-               /*
-                * Here with number, in correct radix. c is the next char.
-                * Note that unlike Un*x, we allow "011f" "0x9f" to
-                * both mean the same as the (conventional) "9f". This is simply easier
-                * than checking for strict canonical form. Syntax sux!
-                */
-               if (number<10)
-               {
-                   if (0
-#ifdef LOCAL_LABELS_FB
-                       || c=='b'
-#endif
-#ifdef LOCAL_LABELS_DOLLAR
-                       || (c=='$' && local_label_defined[number])
-#endif
-                       )
-                   {
-                       /*
-                        * Backward ref to local label.
-                        * Because it is backward, expect it to be DEFINED.
-                        */
-                       /*
-                        * Construct a local label.
-                        */
-                       name = local_label_name ((int)number, 0);
-                       if (((symbolP = symbol_find(name)) != NULL) /* seen before */
-                           && (S_IS_DEFINED(symbolP))) /* symbol is defined: OK */
-                       {               /* Expected path: symbol defined. */
-                           /* 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_add_number = 0;
-                           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_add_number = 0;
-                           expressionP->X_seg        = SEG_ABSOLUTE;
-                       }
-                   }
-                   else
-                   {
-                       if (0
-#ifdef LOCAL_LABELS_FB
-                           || c == 'f'
-#endif
-#ifdef LOCAL_LABELS_DOLLAR
-                           || (c=='$' && !local_label_defined[number])
-#endif
-                           )
-                       {
-                           /*
-                            * Forward reference. Expect symbol to be undefined or
-                            * unknown. Undefined: seen it before. Unknown: never seen
-                            * it in this pass.
-                            * 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 = local_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);
-#endif
-                           expressionP->X_add_symbol      = symbolP;
-                           expressionP->X_seg             = SEG_UNKNOWN;
-                           expressionP->X_subtract_symbol = NULL;
-                           expressionP->X_add_number      = 0;
-                       }
-                       else
-                       {               /* Really a number, not a local label. */
-                           expressionP->X_add_number = number;
-                           expressionP->X_seg        = SEG_ABSOLUTE;
-                           input_line_pointer --; /* Restore following character. */
-                       }               /* if (c=='f') */
-                   }                   /* if (c=='b') */
-               }
-               else
-               {                       /* Really a number. */
-                   expressionP->X_add_number = number;
-                   expressionP->X_seg        = SEG_ABSOLUTE;
-                   input_line_pointer --; /* Restore following character. */
-               }                       /* if (number<10) */
+             long work;
+
+             work = carry + radix * *pointer;
+             *pointer = work & LITTLENUM_MASK;
+             carry = work >> LITTLENUM_NUMBER_OF_BITS;
            }
-           else
-           {
-               expressionP->X_add_number = number;
-               expressionP->X_seg = SEG_BIG;
-               input_line_pointer --; /*->char following number. */
-           }                   /* if (small) */
-       }                       /* (If integer constant) */
-       else
-       {                       /* input_line_pointer->*/
-           /* floating-point constant. */
-           int error_code;
-           
-           error_code = atof_generic
-               (& input_line_pointer, ".", EXP_CHARS,
-                & generic_floating_point_number);
-           
-           if (error_code)
+         if (carry)
            {
-               if (error_code == ERROR_EXPONENT_OVERFLOW)
-               {
-                   as_bad("Bad floating-point constant: exponent overflow, probably assembling junk");
-               }
-               else
+             if (leader < generic_bignum + SIZE_OF_LARGE_NUMBER - 1)
                {
-                   as_bad("Bad floating-point constant: unknown error code=%d.", error_code);
+                 /* room to grow a longer bignum. */
+                 *++leader = carry;
                }
            }
-           expressionP->X_seg = SEG_BIG;
-           /* input_line_pointer->just after constant, */
-           /* which may point to whitespace. */
-           know(expressionP->X_add_number < 0); /* < 0 means "floating point". */
-       }                       /* if (not floating-point constant) */
-    }
-    else if(c=='.' && !is_part_of_name(*input_line_pointer)) {
-       extern struct obstack frags;
-       
-       /*
-         JF:  '.' is pseudo symbol with value of current location in current
-         segment. . .
-         */
-       symbolP = symbol_new("L0\001",
-                            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;
-       
-    } else if (is_name_beginner(c)) /* here if did not begin with a digit */
-    {
-       /*
-        * Identifier begins here.
-        * This is kludged for speed, so code is repeated.
-        */
-       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)
-       {
-       case SEG_ABSOLUTE:
-       case SEG_REGISTER:
-           expressionP->X_add_number = S_GET_VALUE(symbolP);
-           break;
-           
-       default:
-           expressionP->X_add_number  = 0;
-           expressionP->X_add_symbol  = symbolP;
        }
-       * input_line_pointer = c;
-       expressionP->X_subtract_symbol = NULL;
-    }
-    else if (c=='(')/* didn't begin with digit & not a name */
-    {
-       (void)expression(expressionP);
-       /* Expression() will pass trailing whitespace */
-       if (* input_line_pointer ++ != ')')
+      /* again, c is char after number, */
+      /* input_line_pointer->after c. */
+      know (LITTLENUM_NUMBER_OF_BITS == 16);
+      if (leader < generic_bignum + 2)
        {
-           as_bad("Missing ')' assumed");
-           input_line_pointer --;
+         /* will fit into 32 bits. */
+         number =
+           ((generic_bignum[1] & LITTLENUM_MASK) << LITTLENUM_NUMBER_OF_BITS)
+           | (generic_bignum[0] & LITTLENUM_MASK);
+         small = 1;
+       }
+      else
+       {
+         number = leader - generic_bignum + 1; /* number of littlenums in the bignum. */
        }
-       /* here with input_line_pointer->char after "(...)" */
     }
-    else if (c == '~' || c == '-' || c == '+') {
-       /* unary operator: hope for SEG_ABSOLUTE */
-       switch (operand (expressionP)) {
-       case SEG_ABSOLUTE:
-           /* input_line_pointer->char after operand */
-           if (c=='-') {
-               expressionP->X_add_number = - expressionP->X_add_number;
-               /*
-                * Notice: '-' may  overflow: no warning is given. This is compatible
-                * with other people's assemblers. Sigh.
-                */
-           } else if (c == '~') {
-               expressionP->X_add_number = ~ expressionP->X_add_number;
-           } else if (c != '+') {
-               know(0);
-           } /* switch on unary operator */
-           break;
-           
-       default:                /* unary on non-absolute is unsuported */
-           if (!SEG_NORMAL(operand(expressionP))) 
+
+  if (flag_mri && suffix != NULL && input_line_pointer - 1 == suffix)
+    c = *input_line_pointer++;
+
+  if (small)
+    {
+      /*
+       * here with number, in correct radix. c is the next char.
+       * note that unlike un*x, we allow "011f" "0x9f" to
+       * both mean the same as the (conventional) "9f". this is simply easier
+       * than checking for strict canonical form. syntax sux!
+       */
+
+      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);
+
+         /* seen before, or symbol is defined: ok */
+         symbolP = symbol_find (name);
+         if ((symbolP != NULL) && (S_IS_DEFINED (symbolP)))
            {
-               as_bad("Unary operator %c ignored because bad operand follows", c);
-               break;
+             /* local labels are never absolute. don't waste time
+                checking absoluteness. */
+             know (SEG_NORMAL (S_GET_SEGMENT (symbolP)));
+
+             expressionP->X_op = O_symbol;
+             expressionP->X_add_symbol = symbolP;
            }
-           /* Fall through for normal segments ****/
-       case SEG_PASS1:
-       case SEG_UNKNOWN:
-           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;
+         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;
            }
-           /* Expression undisturbed from operand(). */
-       }
-    }
-    else if (c=='\'')
-    {
-       /*
-        * 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;
+
+         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) == undefined_section || S_GET_SEGMENT (symbolP) == text_section || S_GET_SEGMENT (symbolP) == data_section);
+#endif
+         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.  */
+
+         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);
+           }
+
+         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
+  else
     {
-       /* can't imagine any other kind of operand */
-       expressionP->X_seg = SEG_ABSENT;
-       input_line_pointer --;
-       md_operand (expressionP);
+      /* not a small number */
+      expressionP->X_op = O_big;
+      expressionP->X_add_number = number;      /* number of littlenums */
+      input_line_pointer--;    /*->char following number. */
     }
-    /*
-     * It is more 'efficient' to clean up the expressions when they are created.
-     * Doing it here saves lines of code.
-     */
-    clean_up_expression (expressionP);
-    SKIP_WHITESPACE();         /*->1st char after operand. */
-    know(* input_line_pointer != ' ');
-    return (expressionP->X_seg);
-} /* 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.
- *     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;
- *     Unused fields zeroed to help expr().
- */
+/* Parse an MRI multi character constant.  */
 
 static void
-    clean_up_expression (expressionP)
-register expressionS * expressionP;
+mri_char_constant (expressionP)
+     expressionS *expressionP;
 {
-    switch (expressionP->X_seg)
+  int i;
+
+  if (*input_line_pointer == '\''
+      && input_line_pointer[1] != '\'')
     {
-    case SEG_ABSENT:
-    case SEG_PASS1:
-       expressionP->X_add_symbol       = NULL;
-       expressionP->X_subtract_symbol  = NULL;
-       expressionP->X_add_number       = 0;
-       break;
-       
-    case SEG_BIG:
-    case SEG_ABSOLUTE:
-       expressionP->X_subtract_symbol  = NULL;
-       expressionP->X_add_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))) {
-           expressionP->X_subtract_symbol      = NULL;
-           expressionP->X_add_symbol           = NULL;
-           expressionP->X_seg                  = SEG_ABSOLUTE;
-       }
-       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;
+      expressionP->X_op = O_constant;
+      expressionP->X_add_number = 0;
+      return;
+    }
+
+  /* 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;
+
+      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;
        }
-       else {
-           BAD_CASE (expressionP->X_seg);
+
+      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;
        }
+
+      if (*input_line_pointer == '\''
+         && input_line_pointer[1] != '\'')
        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.
- */
+
+  if (i < 0)
+    {
+      as_bad ("Character constant too large");
+      i = 0;
+    }
+
+  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
+    {
+      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;
+}
 
 /*
- * symbol_1 += symbol_2 ... well ... sort of.
+ * Summary of operand().
+ *
+ * in: Input_line_pointer points to 1st char of operand, which may
+ *     be a space.
+ *
+ * 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.
  */
 
 static segT
-    expr_part (symbol_1_PP, symbol_2_P)
-symbolS **     symbol_1_PP;
-symbolS *      symbol_2_P;
+operand (expressionP)
+     expressionS *expressionP;
 {
-    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)
+  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. */
+
+  SKIP_WHITESPACE ();          /* leading whitespace is part of operand. */
+  c = *input_line_pointer++;   /* input_line_pointer->past char in c. */
+
+  switch (c)
     {
-       if (!S_IS_DEFINED(* symbol_1_PP))
+    case '1':
+    case '2':
+    case '3':
+    case '4':
+    case '5':
+    case '6':
+    case '7':
+    case '8':
+    case '9':
+      input_line_pointer--;
+
+      integer_constant (flag_mri ? 0 : 10, expressionP);
+      break;
+
+    case '0':
+      /* non-decimal radix */
+
+      if (flag_mri)
        {
-           if (symbol_2_P)
-           {
-               return_value = SEG_PASS1;
-               * symbol_1_PP = NULL;
-           }
-           else
+         char *s;
+
+         /* Check for a hex constant.  */
+         for (s = input_line_pointer; hex_p (*s); s++)
+           ;
+         if (*s == 'h' || *s == 'H')
            {
-               know(!S_IS_DEFINED(* symbol_1_PP));
-               return_value = SEG_UNKNOWN;
+             --input_line_pointer;
+             integer_constant (0, expressionP);
+             break;
            }
        }
-       else
+
+      c = *input_line_pointer;
+      switch (c)
        {
-           if (symbol_2_P)
+       case '8':
+       case '9':
+         if (flag_mri)
            {
-               if (!S_IS_DEFINED(symbol_2_P))
-               {
-                   * symbol_1_PP = NULL;
-                   return_value = SEG_PASS1;
-               }
-               else
+             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 (c) ? tolower (c) : c);
+           }
+         else
+           {
+             /* The string was only zero */
+             expressionP->X_op = O_constant;
+             expressionP->X_add_number = 0;
+           }
+
+         break;
+
+       case 'x':
+       case 'X':
+         if (flag_mri)
+           goto default_case;
+         input_line_pointer++;
+         integer_constant (16, expressionP);
+         break;
+
+       case 'b':
+         if (LOCAL_LABELS_FB)
+           {
+             switch (input_line_pointer[1])
                {
-                   /* {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;
+               case '+':
+               case '-':
+                 /* If unambiguously a difference expression, treat
+                    it as one by indicating a label; otherwise, it's
+                    always a binary number.  */
+                 {
+                   char *cp = input_line_pointer + 1;
+                   while (strchr ("0123456789", *++cp))
+                     ;
+                   if (*cp == 'b' || *cp == 'f')
+                     goto is_0b_label;
+                 }
+                 goto is_0b_binary;
+               case '0':    case '1':
+                 /* Some of our code elsewhere does permit digits
+                    greater than the expected base; for consistency,
+                    do the same here.  */
+               case '2':    case '3':    case '4':    case '5':
+               case '6':    case '7':    case '8':    case '9':
+                 goto is_0b_binary;
+               case 0:
+                 goto is_0b_label;
+               default:
+                 goto is_0b_label;
                }
+           is_0b_label:
+             input_line_pointer--;
+             integer_constant (10, expressionP);
+             break;
+           is_0b_binary:
+             ;
            }
-           else
+       case 'B':
+         input_line_pointer++;
+         if (flag_mri)
+           goto default_case;
+         integer_constant (2, expressionP);
+         break;
+
+       case '0':
+       case '1':
+       case '2':
+       case '3':
+       case '4':
+       case '5':
+       case '6':
+       case '7':
+         integer_constant (flag_mri ? 0 : 8, expressionP);
+         break;
+
+       case 'f':
+         if (LOCAL_LABELS_FB)
+           {
+             /* 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 == 'f' || *cp == 'b')
+                     /* 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 */
+             ;
+           }
+
+       case 'd':
+       case 'D':
+       case 'F':
+       case 'r':
+       case 'e':
+       case 'E':
+       case 'g':
+       case 'G':
+         input_line_pointer++;
+         floating_constant (expressionP);
+         expressionP->X_add_number = -(isupper (c) ? tolower (c) : c);
+         break;
+
+       case '$':
+         if (LOCAL_LABELS_DOLLAR)
            {
-               return_value = S_GET_SEGMENT(* symbol_1_PP);
+             integer_constant (10, expressionP);
+             break;
            }
+         else
+           goto default_case;
        }
-    }
-    else
-    {                          /* (* symbol_1_PP) == NULL */
-       if (symbol_2_P)
+
+      break;
+
+    case '(':
+    case '[':
+      /* didn't begin with digit & not a name */
+      segment = expression (expressionP);
+      /* Expression() will pass trailing whitespace */
+      if ((c == '(' && *input_line_pointer++ != ')')
+         || (c == '[' && *input_line_pointer++ != ']'))
+       {
+         as_bad ("Missing ')' assumed");
+         input_line_pointer--;
+       }
+      /* here with input_line_pointer->char after "(...)" */
+      return segment;
+
+    case 'E':
+      if (! flag_mri || *input_line_pointer != '\'')
+       goto de_fault;
+      as_bad ("EBCDIC constants are not supported");
+      /* Fall through.  */
+    case 'A':
+      if (! flag_mri || *input_line_pointer != '\'')
+       goto de_fault;
+      ++input_line_pointer;
+      /* Fall through.  */
+    case '\'':
+      if (! flag_mri)
        {
-           * symbol_1_PP = symbol_2_P;
-           return_value = S_GET_SEGMENT(symbol_2_P);
+         /* 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 '+':
+      (void) operand (expressionP);
+      break;
+
+    case '"':
+      /* Double quote is the logical not operator in MRI mode.  */
+      if (! flag_mri)
+       goto de_fault;
+      /* Fall through.  */
+    case '~':
+    case '-':
+      {
+       operand (expressionP);
+       if (expressionP->X_op == O_constant)
+         {
+           /* input_line_pointer -> char after operand */
+           if (c == '-')
+             {
+               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
+             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 == '-')
+             expressionP->X_op = O_uminus;
+           else
+             expressionP->X_op = O_bit_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_mri)
+       goto de_fault;
+#endif
+      if (flag_mri && hex_p (*input_line_pointer))
+       {
+         /* In MRI mode, $ is also used as the prefix for a
+             hexadecimal constant.  */
+         integer_constant (16, expressionP);
+         break;
+       }
+      /* Fall through.  */
+    case '.':
+      if (!is_part_of_name (*input_line_pointer))
+       {
+         const char *fake;
+
+         /* JF: '.' is pseudo symbol with value of current location
+            in current segment.  */
+         fake = FAKE_LABEL_NAME;
+         symbolP = symbol_new (fake,
+                               now_seg,
+                               (valueT) frag_now_fix (),
+                               frag_now);
+
+         expressionP->X_op = O_symbol;
+         expressionP->X_add_symbol = symbolP;
+         expressionP->X_add_number = 0;
+         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
        {
-           * symbol_1_PP = NULL;
-           return_value = SEG_ABSOLUTE;
+         goto isname;
        }
+    case ',':
+    case '\n':
+    case '\0':
+    eol:
+      /* can't imagine any other kind of operand */
+      expressionP->X_op = O_absent;
+      input_line_pointer--;
+      break;
+
+    case '%':
+      if (! flag_mri)
+       goto de_fault;
+      integer_constant (2, expressionP);
+      break;
+
+    case '@':
+      if (! flag_mri)
+       goto de_fault;
+      integer_constant (8, expressionP);
+      break;
+
+    case ':':
+      if (! flag_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;
+
+    default:
+    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 */
+       {
+         /*
+          * Identifier begins here.
+          * This is kludged for speed, so code is repeated.
+          */
+       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.  */
+         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;
+       }
+      else
+       {
+         /* 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;
     }
-#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() */
+
+  /*
+   * It is more 'efficient' to clean up the expressionS when they are created.
+   * Doing it here saves lines of code.
+   */
+  clean_up_expression (expressionP);
+  SKIP_WHITESPACE ();          /*->1st char after operand. */
+  know (*input_line_pointer != ' ');
+
+  /* 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_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_op from O_subtract to O_constant.
+ *     Unused fields zeroed to help expr().
+ */
+
+static void
+clean_up_expression (expressionP)
+     expressionS *expressionP;
+{
+  switch (expressionP->X_op)
+    {
+    case O_illegal:
+    case O_absent:
+      expressionP->X_add_number = 0;
+      /* 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 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))))
+       {
+         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_op_symbol = NULL;
+         expressionP->X_add_number += diff;
+       }
+      break;
+    default:
+      break;
+    }
+}
 \f
 /* Expression parser. */
 
@@ -633,283 +1059,362 @@ symbolS *      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] = {   /* maps ASCII->operators */
-    
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    
-    __, O_bit_or_not, __, __, __, O_modulus, O_bit_and, __,
-    __, __, O_multiply, O_add, __, O_subtract, __, O_divide,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, O_left_shift, __, O_right_shift, __,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, O_bit_exclusive_or, __,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __,
-    __, __, __, __, O_bit_inclusive_or, __, __, __,
-    
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
-    __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __
-    };
+static operatorT op_encoding[256] =
+{                              /* maps ASCII->operators */
+
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+
+  __, O_bit_or_not, __, __, __, O_modulus, O_bit_and, __,
+  __, __, O_multiply, O_add, __, O_subtract, __, O_divide,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, O_lt, __, O_gt, __,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, O_bit_exclusive_or, __,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __,
+  __, __, __, __, O_bit_inclusive_or, __, __, __,
+
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __
+};
 
 
 /*
  *     Rank    Examples
  *     0       operand, (expression)
- *     1       + -
- *     2       & ^ ! |
- *     3       * / % << >>
+ *     1       = <> < <= >= >
+ *     2       + -
+ *     3       used for * / % in MRI mode
+ *     4       & ^ ! |
+ *     5       * / % << >>
+ *     6       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_register */
+  0,   /* O_bit */
+  6,   /* O_uminus */
+  6,   /* O_bit_not */
+  5,   /* O_multiply */
+  5,   /* O_divide */
+  5,   /* O_modulus */
+  5,   /* O_left_shift */
+  5,   /* O_right_shift */
+  4,   /* O_bit_inclusive_or */
+  4,   /* O_bit_or_not */
+  4,   /* O_bit_exclusive_or */
+  4,   /* O_bit_and */
+  2,   /* O_add */
+  2,   /* O_subtract */
+  1,   /* O_eq */
+  1,   /* O_ne */
+  1,   /* O_lt */
+  1,   /* O_le */
+  1,   /* O_ge */
+  1    /* O_gt */
+};
+
+/* Initialize the expression parser.  */
+
+void
+expr_begin ()
+{
+  /* In MRI mode, multiplication and division have lower precedence
+     than the bit wise operators.  */
+  if (flag_mri)
+    {
+      op_rank[O_multiply] = 3;
+      op_rank[O_divide] = 3;
+      op_rank[O_modulus] = 3;
+      op_encoding['"'] = O_bit_not;
+    }
+}
 \f
-/* Return resultP->X_seg. */
-segT expr(rank, resultP)
-    register operator_rankT    rank; /* Larger # is higher rank. */
-    register expressionS *resultP; /* Deliver result here. */
+/* 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 ()
 {
-    expressionS                right;
-    register operatorT op_left;
-    register char c_left;      /* 1st operator character. */
-    register operatorT op_right;
-    register char c_right;
-    
-    know(rank >= 0);
-    (void)operand (resultP);
-    know(* input_line_pointer != ' '); /* Operand() gobbles spaces. */
-    c_left = * input_line_pointer; /* Potential operator character. */
-    op_left = op_encoding [c_left];
-    while (op_left != O_illegal && op_rank [(int) op_left] > rank)
+  int c;
+  operatorT ret;
+
+  c = *input_line_pointer;
+
+  switch (c)
     {
-       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))
-       {
-           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;
-       }
-       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 == '>'))
-       {
-           input_line_pointer ++;
-       }                       /*->after operator. */
-       know((int) op_right == 0 || op_rank [(int) op_right] <= op_rank[(int) op_left]);
-       /* 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)
-       {
-           resultP->X_seg = SEG_PASS1;
+    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;
        }
-       else
+      ++input_line_pointer;
+      return ret;
+
+    case '>':
+      switch (input_line_pointer[1])
        {
-           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)
-           {
-               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;
-           }
-           if (op_left == O_subtract)
+       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_mri)
+           return O_bit_inclusive_or;
+         return op_encoding[c];
+       }
+      ++input_line_pointer;
+      return O_bit_exclusive_or;
+    }
+
+  /*NOTREACHED*/
+}      
+
+/* Parse an expression.  */
+
+segT
+expr (rank, resultP)
+     operator_rankT rank;      /* Larger # is higher rank. */
+     expressionS *resultP;     /* Deliver result here. */
+{
+  segT retval;
+  expressionS right;
+  operatorT op_left;
+  operatorT op_right;
+
+  know (rank >= 0);
+
+  retval = operand (resultP);
+
+  know (*input_line_pointer != ' ');   /* Operand() gobbles spaces. */
+
+  op_left = operator ();
+  while (op_left != O_illegal && op_rank[(int) op_left] > rank)
+    {
+      segT rightseg;
+
+      input_line_pointer++;    /*->after 1st character of operator. */
+
+      rightseg = expr (op_rank[(int) op_left], &right);
+      if (right.X_op == O_absent)
+       {
+         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 != ' ');
+
+      if (retval == undefined_section)
+       {
+         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_subtract);
+
+      /* input_line_pointer->after right-hand quantity. */
+      /* left-hand quantity in resultP */
+      /* right-hand quantity in right. */
+      /* operator in op_left. */
+
+      if (resultP->X_op == O_big)
+       {
+         as_warn ("left operand is a %s; integer 0 assumed",
+                  resultP->X_add_number > 0 ? "bignum" : "float");
+         resultP->X_op = O_constant;
+         resultP->X_add_number = 0;
+         resultP->X_add_symbol = NULL;
+         resultP->X_op_symbol = NULL;
+       }
+      if (right.X_op == O_big)
+       {
+         as_warn ("right operand is a %s; integer 0 assumed",
+                  right.X_add_number > 0 ? "bignum" : "float");
+         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))
            {
-               /*
-                * 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;
+             as_warn ("division by zero");
+             v = 1;
            }
-           \f
-           if (op_left == O_add)
+         switch (op_left)
            {
-               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);
-               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;
-               clean_up_expression (resultP);
+           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:         resultP->X_add_number >>= 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;
            }
-           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;
+       }
+      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)
+           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;
+       }
+
+      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
 /*
@@ -928,29 +1433,27 @@ segT expr(rank, resultP)
  * lines end in end-of-line.
  */
 char
-    get_symbol_end()
+get_symbol_end ()
 {
-    register char c;
-    
-    while (is_part_of_name(c = * input_line_pointer ++))
-       ;
-    * -- input_line_pointer = 0;
-    return (c);
+  char c;
+
+  /* We accept \001 in a name in case this is being called with a
+     constructed string.  */
+  while (is_part_of_name (c = *input_line_pointer++)
+        || c == '\001')
+    ;
+  *--input_line_pointer = 0;
+  return (c);
 }
 
 
-unsigned int get_single_number()
+unsigned int
+get_single_number ()
 {
-    expressionS exp;
-    operand(&exp);
-    return exp.X_add_number;
-    
+  expressionS exp;
+  operand (&exp);
+  return exp.X_add_number;
+
 }
-/*
- * Local Variables:
- * comment-column: 0
- * fill-column: 131
- * End:
- */
 
 /* end of expr.c */