Update the address and phone number of the FSF
[deliverable/binutils-gdb.git] / gas / expr.c
index fe2aa6204e1f8bedd6cea489b551068bbf37ea8b..7e37010949d9f2bc277465ff48665560974a8c94 100644 (file)
@@ -1,5 +1,6 @@
 /* expr.c -operands, expressions-
-   Copyright (C) 1987, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 2000
+   Copyright 1987, 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
+   1999, 2000, 2001, 2002, 2003, 2004, 2005
    Free Software Foundation, Inc.
 
    This file is part of GAS, the GNU Assembler.
 
    You should have received a copy of the GNU General Public License
    along with GAS; see the file COPYING.  If not, write to the Free
-   Software Foundation, 59 Temple Place - Suite 330, Boston, MA
-   02111-1307, USA. */
+   Software Foundation, 51 Franklin Street - Fifth Floor, Boston, MA
+   02110-1301, USA.  */
 
-/*
- * This is really a branch office of as-read.c. I split it out to clearly
- * distinguish the world of expressions from the world of statements.
- * (It also gives smaller files to re-compile.)
- * Here, "operand"s are of expressions, not instructions.
- */
+/* This is really a branch office of as-read.c. I split it out to clearly
+   distinguish the world of expressions from the world of statements.
+   (It also gives smaller files to re-compile.)
+   Here, "operand"s are of expressions, not instructions.  */
 
-#include <ctype.h>
 #include <string.h>
 #define min(a, b)       ((a) < (b) ? (a) : (b))
 
 #include "as.h"
+#include "safe-ctype.h"
 #include "obstack.h"
 
-static void floating_constant PARAMS ((expressionS * expressionP));
-static valueT generic_bignum_to_int32 PARAMS ((void));
+static void floating_constant (expressionS * expressionP);
+static valueT generic_bignum_to_int32 (void);
 #ifdef BFD64
-static valueT generic_bignum_to_int64 PARAMS ((void));
+static valueT generic_bignum_to_int64 (void);
 #endif
-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));
+static void integer_constant (int radix, expressionS * expressionP);
+static void mri_char_constant (expressionS *);
+static void current_location (expressionS *);
+static void clean_up_expression (expressionS * expressionP);
+static segT operand (expressionS *);
+static operatorT operator (int *);
 
 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 {
   struct expr_symbol_line *next;
   symbolS *sym;
   char *file;
@@ -65,11 +63,9 @@ static struct expr_symbol_line *expr_symbol_lines;
    into the fake section expr_section.  */
 
 symbolS *
-make_expr_symbol (expressionP)
-     expressionS *expressionP;
+make_expr_symbol (expressionS *expressionP)
 {
   expressionS zero;
-  const char *fake;
   symbolS *symbolP;
   struct expr_symbol_line *n;
 
@@ -80,12 +76,12 @@ make_expr_symbol (expressionP)
   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.  */
+        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"));
+       as_bad (_("bignum invalid"));
       else
-       as_bad (_("floating point number invalid; zero assumed"));
+       as_bad (_("floating point number invalid"));
       zero.X_op = O_constant;
       zero.X_add_number = 0;
       zero.X_unsigned = 0;
@@ -93,13 +89,11 @@ make_expr_symbol (expressionP)
       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,
+  symbolP = symbol_create (FAKE_LABEL_NAME,
                           (expressionP->X_op == O_constant
                            ? absolute_section
                            : expr_section),
@@ -107,7 +101,7 @@ make_expr_symbol (expressionP)
   symbol_set_value_expression (symbolP, expressionP);
 
   if (expressionP->X_op == O_constant)
-    resolve_symbol_value (symbolP, 1);
+    resolve_symbol_value (symbolP);
 
   n = (struct expr_symbol_line *) xmalloc (sizeof *n);
   n->sym = symbolP;
@@ -123,10 +117,7 @@ make_expr_symbol (expressionP)
    the symbol.  */
 
 int
-expr_symbol_where (sym, pfile, pline)
-     symbolS *sym;
-     char **pfile;
-     unsigned int *pline;
+expr_symbol_where (symbolS *sym, char **pfile, unsigned int *pline)
 {
   register struct expr_symbol_line *l;
 
@@ -156,8 +147,7 @@ expr_symbol_where (sym, pfile, pline)
    but that seems more clumsy.  */
 
 symbolS *
-expr_build_uconstant (value)
-     offsetT value;
+expr_build_uconstant (offsetT value)
 {
   expressionS e;
 
@@ -167,42 +157,10 @@ expr_build_uconstant (value)
   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 ()
+expr_build_dot (void)
 {
   expressionS e;
 
@@ -210,10 +168,8 @@ expr_build_dot ()
   return make_expr_symbol (&e);
 }
 \f
-/*
- * Build any floating-point literal here.
- * Also build any bignum literal here.
- */
+/* Build any floating-point literal here.
+   Also build any bignum literal here.  */
 
 /* Seems atof_machine can backscan through generic_bignum and hit whatever
    happens to be loaded before it in memory.  And its way too complicated
@@ -221,23 +177,20 @@ expr_build_dot ()
    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 =
-{
-  &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 */
+
+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 */
 };
-/* 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;
+floating_constant (expressionS *expressionP)
 {
-  /* input_line_pointer->*/
-  /* floating-point constant. */
+  /* input_line_pointer -> floating-point constant.  */
   int error_code;
 
   error_code = atof_generic (&input_line_pointer, ".", EXP_CHARS,
@@ -247,21 +200,22 @@ floating_constant (expressionP)
     {
       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"));
        }
       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_op = O_big;
-  /* input_line_pointer->just after constant, */
-  /* which may point to whitespace. */
+  /* input_line_pointer -> just after constant, which may point to
+     whitespace.  */
   expressionP->X_add_number = -1;
 }
 
-static valueT 
-generic_bignum_to_int32 (
+static valueT
+generic_bignum_to_int32 (void)
 {
   valueT number =
           ((generic_bignum[1] & LITTLENUM_MASK) << LITTLENUM_NUMBER_OF_BITS)
@@ -271,39 +225,37 @@ generic_bignum_to_int32 ()
 }
 
 #ifdef BFD64
-static valueT 
-generic_bignum_to_int64 (
+static valueT
+generic_bignum_to_int64 (void)
 {
-  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));
+  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;
+integer_constant (int radix, expressionS *expressionP)
 {
-  char *start;         /* start of number. */
+  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 */
+  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. */
+  int small;                   /* True if fits in 32 bits.  */
 
-  /* May be bignum, or may fit 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
@@ -316,7 +268,7 @@ integer_constant (radix, expressionP)
      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. */
+     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.  */
@@ -332,11 +284,9 @@ integer_constant (radix, expressionP)
       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++)
+        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;
@@ -350,8 +300,7 @@ integer_constant (radix, expressionP)
       else
        {
          c = *--suffix;
-         if (islower ((unsigned char) c))
-           c = toupper (c);
+         c = TOUPPER (c);
          if (c == 'B')
            radix = 2;
          else if (c == 'D')
@@ -389,7 +338,7 @@ integer_constant (radix, expressionP)
       break;
     case 10:
       maxdig = radix = 10;
-      too_many_digits = (valuesize + 11) / 4; /* very rough */
+      too_many_digits = (valuesize + 11) / 4; /* Very rough.  */
     }
 #undef valuesize
   start = input_line_pointer;
@@ -400,26 +349,26 @@ integer_constant (radix, expressionP)
     {
       number = number * radix + digit;
     }
-  /* c contains character after number. */
-  /* input_line_pointer->char after c. */
+  /* c contains character after number.  */
+  /* input_line_pointer->char after c.  */
   small = (input_line_pointer - start - 1) < too_many_digits;
 
-  if (radix == 16 && c == '_') 
+  if (radix == 16 && c == '_')
     {
       /* This is literal of the form 0x333_0_12345678_1.
-         This example is equivalent to 0x00000333000000001234567800000001.  */
+        This example is equivalent to 0x00000333000000001234567800000001.  */
 
       int num_little_digits = 0;
       int i;
-      input_line_pointer = start;      /*->1st digit. */
+      input_line_pointer = start;      /* -> 1st digit.  */
 
       know (LITTLENUM_NUMBER_OF_BITS == 16);
 
-      for (c = '_'; c == '_'; num_little_digits+=2)
+      for (c = '_'; c == '_'; num_little_digits += 2)
        {
 
-         /* Convert one 64-bit word. */
-         int ndigit = 0; 
+         /* Convert one 64-bit word.  */
+         int ndigit = 0;
          number = 0;
          for (c = *input_line_pointer++;
               (digit = hex_value (c)) < maxdig;
@@ -430,20 +379,23 @@ integer_constant (radix, expressionP)
            }
 
          /* 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."));
+         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.*/
+         /* 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];
+         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. */
+         /* 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. */
+      /* 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;
@@ -451,17 +403,17 @@ integer_constant (radix, expressionP)
       assert (num_little_digits >= 4);
 
       if (num_little_digits != 8)
-       as_bad (_("A bignum with underscores must have exactly 4 words."));
+       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)
+        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. */
+         /* will fit into 32 bits.  */
          number = generic_bignum_to_int32 ();
          small = 1;
        }
@@ -476,16 +428,16 @@ integer_constant (radix, expressionP)
       else
        {
          small = 0;
-         number = num_little_digits; /* number of littlenums in the bignum. */
+
+         /* Number of littlenums in the bignum.  */
+         number = num_little_digits;
        }
     }
   else 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. */
+      /* 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;
@@ -493,15 +445,11 @@ integer_constant (radix, expressionP)
       generic_bignum[1] = 0;
       generic_bignum[2] = 0;
       generic_bignum[3] = 0;
-      input_line_pointer = start;      /*->1st digit. */
+      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;
-              pointer++)
+         for (pointer = generic_bignum; pointer <= leader; pointer++)
            {
              long work;
 
@@ -513,17 +461,17 @@ integer_constant (radix, expressionP)
            {
              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. */
+      /* Again, c is char after number.  */
+      /* input_line_pointer -> after c.  */
       know (LITTLENUM_NUMBER_OF_BITS == 16);
       if (leader < generic_bignum + 2)
        {
-         /* will fit into 32 bits. */
+         /* Will fit into 32 bits.  */
          number = generic_bignum_to_int32 ();
          small = 1;
        }
@@ -537,7 +485,8 @@ integer_constant (radix, expressionP)
 #endif
       else
        {
-         number = leader - generic_bignum + 1; /* number of littlenums in the bignum. */
+         /* Number of littlenums in the bignum.  */
+         number = leader - generic_bignum + 1;
        }
     }
 
@@ -548,28 +497,25 @@ integer_constant (radix, expressionP)
 
   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!
-       */
+      /* 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.
-          */
+         /* 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 */
+         /* 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. */
+             /* Local labels are never absolute.  Don't waste time
+                checking absoluteness.  */
              know (SEG_NORMAL (S_GET_SEGMENT (symbolP)));
 
              expressionP->X_op = O_symbol;
@@ -577,10 +523,10 @@ integer_constant (radix, expressionP)
            }
          else
            {
-             /* either not seen or not defined. */
+             /* 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."),
+             as_bad (_("backward ref to unknown label \"%d:\""),
                      (int) number);
              expressionP->X_op = O_constant;
            }
@@ -589,19 +535,18 @@ integer_constant (radix, expressionP)
        }                       /* 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.
-          */
+         /* 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. */
+         /* We have no need to check symbol properties.  */
 #ifndef many_segments
-         /* since "know" puts its arg into a "string", we
+         /* 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
@@ -635,28 +580,23 @@ integer_constant (radix, expressionP)
       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 */
+         input_line_pointer--; /* Restore following character.  */
+       }                       /* Really just a number.  */
     }
   else
     {
-      /* not a small number */
+      /* Not a small number.  */
       expressionP->X_op = O_big;
-      expressionP->X_add_number = number;      /* number of littlenums */
-      input_line_pointer--;    /*->char following number. */
+      expressionP->X_add_number = number;      /* Number of littlenums.  */
+      input_line_pointer--;    /* -> char following number.  */
     }
 }
 
 /* Parse an MRI multi character constant.  */
 
 static void
-mri_char_constant (expressionP)
-     expressionS *expressionP;
+mri_char_constant (expressionS *expressionP)
 {
   int i;
 
@@ -691,8 +631,8 @@ mri_char_constant (expressionP)
       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.  */
+            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;
        }
@@ -704,7 +644,7 @@ mri_char_constant (expressionP)
 
   if (i < 0)
     {
-      as_bad (_("Character constant too large"));
+      as_bad (_("character constant too large"));
       i = 0;
     }
 
@@ -745,8 +685,7 @@ mri_char_constant (expressionP)
    handles the magic symbol `.'.  */
 
 static void
-current_location (expressionp)
-     expressionS *expressionp;
+current_location (expressionS *expressionp)
 {
   if (now_seg == absolute_section)
     {
@@ -755,35 +694,25 @@ current_location (expressionp)
     }
   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_symbol = symbol_temp_new_now ();
       expressionp->X_add_number = 0;
     }
 }
 
-/*
- * 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.
- */
+/* In: Input_line_pointer points to 1st char of operand, which may
+       be a space.
+
+   Out:        An 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
-operand (expressionP)
-     expressionS *expressionP;
+operand (expressionS *expressionP)
 {
   char c;
-  symbolS *symbolP;    /* points to symbol */
-  char *name;          /* points to name of symbol */
+  symbolS *symbolP;    /* Points to symbol.  */
+  char *name;          /* Points to name of symbol.  */
   segT segment;
 
   /* All integers are regarded as unsigned unless they are negated.
@@ -794,10 +723,10 @@ operand (expressionP)
      though it appears negative if valueT is 32 bits.  */
   expressionP->X_unsigned = 1;
 
-  /* digits, assume it is a bignum. */
+  /* 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. */
+  SKIP_WHITESPACE ();          /* Leading whitespace is part of operand.  */
+  c = *input_line_pointer++;   /* input_line_pointer -> past char in c.  */
 
   if (is_end_of_line[(unsigned char) c])
     goto eol;
@@ -816,33 +745,42 @@ operand (expressionP)
       input_line_pointer--;
 
       integer_constant ((NUMBERS_WITH_SUFFIX || flag_m68k_mri)
-                        ? 0 : 10,
-                        expressionP);
+                       ? 0 : 10,
+                       expressionP);
       break;
 
 #ifdef LITERAL_PREFIXDOLLAR_HEX
     case '$':
+      /* $L is the start of a local label, not a hex constant.  */
+      if (* input_line_pointer == 'L')
+      goto isname;
       integer_constant (16, expressionP);
       break;
 #endif
 
+#ifdef LITERAL_PREFIXPERCENT_BIN
+    case '%':
+      integer_constant (2, expressionP);
+      break;
+#endif
+
     case '0':
-      /* non-decimal radix */
+      /* Non-decimal radix.  */
 
       if (NUMBERS_WITH_SUFFIX || flag_m68k_mri)
        {
          char *s;
 
-         /* Check for a hex constant.  */
+         /* Check for a hex or float constant.  */
          for (s = input_line_pointer; hex_p (*s); s++)
            ;
-         if (*s == 'h' || *s == 'H')
+         if (*s == 'h' || *s == 'H' || *input_line_pointer == '.')
            {
              --input_line_pointer;
              integer_constant (0, expressionP);
              break;
            }
-        }
+       }
       c = *input_line_pointer;
       switch (c)
        {
@@ -864,12 +802,11 @@ operand (expressionP)
            {
              input_line_pointer++;
              floating_constant (expressionP);
-             expressionP->X_add_number =
-               - (isupper ((unsigned char) c) ? tolower (c) : c);
+             expressionP->X_add_number = - TOLOWER (c);
            }
          else
            {
-             /* The string was only zero */
+             /* The string was only zero */
              expressionP->X_op = O_constant;
              expressionP->X_add_number = 0;
            }
@@ -921,8 +858,8 @@ operand (expressionP)
        case '6':
        case '7':
          integer_constant ((flag_m68k_mri || NUMBERS_WITH_SUFFIX)
-                            ? 0 : 8, 
-                            expressionP);
+                           ? 0 : 8,
+                           expressionP);
          break;
 
        case 'f':
@@ -944,11 +881,11 @@ operand (expressionP)
                  case 0:
                  case ERROR_EXPONENT_OVERFLOW:
                    if (*cp == 'f' || *cp == 'b')
-                     /* looks like a difference expression */
+                     /* Looks like a difference expression.  */
                      goto is_0f_label;
                    else if (cp == input_line_pointer + 1)
                      /* No characters has been accepted -- looks like
-                         end of operand. */
+                        end of operand.  */
                      goto is_0f_label;
                    else
                      goto is_0f_float;
@@ -967,7 +904,7 @@ operand (expressionP)
              break;
 
            is_0f_float:
-             /* fall through */
+             /* Fall through.  */
              ;
            }
 
@@ -987,8 +924,7 @@ operand (expressionP)
        case 'G':
          input_line_pointer++;
          floating_constant (expressionP);
-         expressionP->X_add_number =
-           - (isupper ((unsigned char) c) ? tolower (c) : c);
+         expressionP->X_add_number = - TOLOWER (c);
          break;
 
        case '$':
@@ -1007,21 +943,16 @@ operand (expressionP)
 #ifndef NEED_INDEX_OPERATOR
     case '[':
 #endif
-      /* didn't begin with digit & not a name */
+      /* Didn't begin with digit & not a name.  */
       segment = expression (expressionP);
-      /* Expression() will pass trailing whitespace */
+      /* expression () will pass trailing whitespace.  */
       if ((c == '(' && *input_line_pointer != ')')
          || (c == '[' && *input_line_pointer != ']'))
-       {
-#ifdef RELAX_PAREN_GROUPING
-         if (c != '(')
-#endif
-           as_bad (_("Missing '%c' assumed"), c == '(' ? ')' : ']');
-       }
+       as_bad (_("missing '%c'"), c == '(' ? ')' : ']');
       else
-        input_line_pointer++;
+       input_line_pointer++;
       SKIP_WHITESPACE ();
-      /* here with input_line_pointer->char after "(...)" */
+      /* Here with input_line_pointer -> char after "(...)".  */
       return segment;
 
 #ifdef TC_M68K
@@ -1040,9 +971,9 @@ operand (expressionP)
       if (! flag_m68k_mri)
        {
          /* Warning: to conform to other people's assemblers NO
-            ESCAPEMENT is permitted for a single quote. The next
+            ESCAPEMENT is permitted for a single quote.  The next
             character, parity errors and all, is taken as the value
-            of the operand. VERY KINKY.  */
+            of the operand.  VERY KINKY.  */
          expressionP->X_op = O_constant;
          expressionP->X_add_number = *input_line_pointer++;
          break;
@@ -1051,10 +982,6 @@ operand (expressionP)
       mri_char_constant (expressionP);
       break;
 
-    case '+':
-      (void) operand (expressionP);
-      break;
-
 #ifdef TC_M68K
     case '"':
       /* Double quote is the bitwise not operator in MRI mode.  */
@@ -1063,39 +990,90 @@ operand (expressionP)
       /* Fall through.  */
 #endif
     case '~':
-      /* ~ is permitted to start a label on the Delta.  */
+      /* '~' is permitted to start a label on the Delta.  */
       if (is_name_beginner (c))
        goto isname;
     case '!':
     case '-':
+    case '+':
       {
+       /* Do not accept ++e or --e as +(+e) or -(-e)
+          Disabled, since the preprocessor removes whitespace.  */
+       if (0 && (c == '-' || c == '+') && *input_line_pointer == c)
+         goto target_op;
+       
        operand (expressionP);
        if (expressionP->X_op == O_constant)
          {
-           /* input_line_pointer -> char after operand */
+           /* 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.  */
+               /* 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
+           else if (c == '!')
              expressionP->X_add_number = ! expressionP->X_add_number;
          }
+       else if (expressionP->X_op == O_big
+                && expressionP->X_add_number <= 0
+                && c == '-'
+                && (generic_floating_point_number.sign == '+'
+                    || generic_floating_point_number.sign == 'P'))
+         {
+           /* Negative flonum (eg, -1.000e0).  */
+           if (generic_floating_point_number.sign == '+')
+             generic_floating_point_number.sign = '-';
+           else
+             generic_floating_point_number.sign = 'N';
+         }
+       else if (expressionP->X_op == O_big
+                && expressionP->X_add_number > 0)
+         {
+           int i;
+
+           if (c == '~' || c == '-')
+             {
+               for (i = 0; i < expressionP->X_add_number; ++i)
+                 generic_bignum[i] = ~generic_bignum[i];
+               if (c == '-')
+                 for (i = 0; i < expressionP->X_add_number; ++i)
+                   {
+                     generic_bignum[i] += 1;
+                     if (generic_bignum[i])
+                       break;
+                   }
+             }
+           else if (c == '!')
+             {
+               int nonzero = 0;
+               for (i = 0; i < expressionP->X_add_number; ++i)
+                 {
+                   if (generic_bignum[i])
+                     nonzero = 1;
+                   generic_bignum[i] = 0;
+                 }
+               generic_bignum[0] = nonzero;
+             }
+         }
        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 if (c == '~' || c == '"')
-             expressionP->X_op = O_bit_not;
-           else
-             expressionP->X_op = O_logical_not;
-           expressionP->X_add_number = 0;
+           if (c != '+')
+             {
+               expressionP->X_add_symbol = make_expr_symbol (expressionP);
+               if (c == '-')
+                 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"),
@@ -1105,16 +1083,16 @@ operand (expressionP)
 
 #if defined (DOLLAR_DOT) || defined (TC_M68K)
     case '$':
-      /* $ is the program counter when in MRI mode, or when DOLLAR_DOT
-         is defined.  */
+      /* '$' 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.  */
+         /* In MRI mode, '$' is also used as the prefix for a
+            hexadecimal constant.  */
          integer_constant (16, expressionP);
          break;
        }
@@ -1182,7 +1160,7 @@ operand (expressionP)
 
     case ',':
     eol:
-      /* can't imagine any other kind of operand */
+      /* Can't imagine any other kind of operand.  */
       expressionP->X_op = O_absent;
       input_line_pointer--;
       break;
@@ -1205,7 +1183,7 @@ operand (expressionP)
        goto de_fault;
 
       /* In MRI mode, this is a floating point constant represented
-         using hexadecimal digits.  */
+        using hexadecimal digits.  */
 
       ++input_line_pointer;
       integer_constant (16, expressionP);
@@ -1223,22 +1201,20 @@ operand (expressionP)
 #ifdef TC_M68K
     de_fault:
 #endif
-      if (is_name_beginner (c))        /* here if did not begin with a digit */
+      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.
-          */
+         /* Identifier begins here.
+            This is kludged for speed, so code is repeated.  */
        isname:
          name = --input_line_pointer;
          c = get_symbol_end ();
 
 #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))
+            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, &c))
            {
              *input_line_pointer = c;
              break;
@@ -1281,7 +1257,7 @@ operand (expressionP)
              SKIP_WHITESPACE ();
 
              break;
-           }         
+           }
 #endif
 
          symbolP = symbol_find_or_make (name);
@@ -1309,9 +1285,10 @@ operand (expressionP)
        }
       else
        {
+       target_op:
          /* 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
+            input_line_pointer past 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;
@@ -1319,7 +1296,7 @@ operand (expressionP)
          if (expressionP->X_op == O_absent)
            {
              ++input_line_pointer;
-             as_bad (_("Bad expression"));
+             as_bad (_("bad expression"));
              expressionP->X_op = O_constant;
              expressionP->X_add_number = 0;
            }
@@ -1327,12 +1304,10 @@ operand (expressionP)
       break;
     }
 
-  /*
-   * It is more 'efficient' to clean up the expressionS when they are created.
-   * Doing it here saves lines of code.
-   */
+  /* 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. */
+  SKIP_WHITESPACE ();          /* -> 1st char after operand.  */
   know (*input_line_pointer != ' ');
 
   /* The PA port needs this information.  */
@@ -1348,23 +1323,19 @@ operand (expressionP)
     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().
- */
+/* Internal.  Simplify a struct expression for use by expr ().  */
+
+/* In: address of an 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:
+       Unused fields zeroed to help expr ().  */
 
 static void
-clean_up_expression (expressionP)
-     expressionS *expressionP;
+clean_up_expression (expressionS *expressionP)
 {
   switch (expressionP->X_op)
     {
@@ -1382,57 +1353,37 @@ clean_up_expression (expressionP)
     case O_bit_not:
       expressionP->X_op_symbol = NULL;
       break;
-    case O_subtract:
-      if (expressionP->X_op_symbol == expressionP->X_add_symbol
-         || ((symbol_get_frag (expressionP->X_op_symbol)
-              == symbol_get_frag (expressionP->X_add_symbol))
-             && 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. */
-
-/*
- * We allow an empty expression, and just assume (absolute,0) silently.
- * Unary operators and parenthetical expressions are treated as operands.
- * As usual, Q==quantity==operand, O==operator, X==expression mnemonics.
- *
- * We used to do a aho/ullman shift-reduce parser, but the logic got so
- * warped that I flushed it and wrote a recursive-descent parser instead.
- * Now things are stable, would anybody like to write a fast parser?
- * Most expressions are either register (which does not even reach here)
- * or 1 symbol. Then "symbol+constant" and "symbol-symbol" are common.
- * So I guess it doesn't really matter how inefficient more complex expressions
- * are parsed.
- *
- * 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.
- */
+/* Expression parser.  */
+
+/* We allow an empty expression, and just assume (absolute,0) silently.
+   Unary operators and parenthetical expressions are treated as operands.
+   As usual, Q==quantity==operand, O==operator, X==expression mnemonics.
+
+   We used to do an aho/ullman shift-reduce parser, but the logic got so
+   warped that I flushed it and wrote a recursive-descent parser instead.
+   Now things are stable, would anybody like to write a fast parser?
+   Most expressions are either register (which does not even reach here)
+   or 1 symbol. Then "symbol+constant" and "symbol-symbol" are common.
+   So I guess it doesn't really matter how inefficient more complex expressions
+   are parsed.
+
+   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.  */
 
 #undef __
 #define __ O_illegal
 
-static const operatorT op_encoding[256] =
-{                              /* maps ASCII->operators */
-
+/* Maps ASCII -> operators.  */
+static const operatorT op_encoding[256] = {
   __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
   __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
 
@@ -1465,28 +1416,25 @@ static const operatorT op_encoding[256] =
   __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __
 };
 
-
-/*
- *     Rank    Examples
- *     0       operand, (expression)
- *     1       ||
- *     2       &&
- *     3       = <> < <= >= >
- *     4       + -
- *     5       used for * / % in MRI mode
- *     6       & ^ ! |
- *     7       * / % << >>
- *     8       unary - unary ~
- */
-static operator_rankT op_rank[] =
-{
+/* Rank        Examples
+   0   operand, (expression)
+   1   ||
+   2   &&
+   3   == <> < <= >= >
+   4   + -
+   5   used for * / % in MRI mode
+   6   & ^ ! |
+   7   * / % << >>
+   8   unary - unary ~
+*/
+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 */
+  0,   /* O_big */
   9,   /* O_uminus */
   9,   /* O_bit_not */
   9,   /* O_logical_not */
@@ -1534,11 +1482,11 @@ static operator_rankT op_rank[] =
    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)
+#define STANDARD_MUL_PRECEDENCE 8
+#define MRI_MUL_PRECEDENCE 6
 
 void
-expr_set_precedence ()
+expr_set_precedence (void)
 {
   if (flag_m68k_mri)
     {
@@ -1557,7 +1505,7 @@ expr_set_precedence ()
 /* Initialize the expression parser.  */
 
 void
-expr_begin ()
+expr_begin (void)
 {
   expr_set_precedence ();
 
@@ -1569,17 +1517,18 @@ expr_begin ()
   }
 }
 \f
-/* 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).  */
+/* Return the encoding for the operator at INPUT_LINE_POINTER, and
+   sets NUM_CHARS to the number of characters in the operator.
+   Does not advance INPUT_LINE_POINTER.  */
 
 static inline operatorT
-operator ()
+operator (int *num_chars)
 {
   int c;
   operatorT ret;
 
   c = *input_line_pointer & 0xff;
+  *num_chars = 1;
 
   if (is_end_of_line[c])
     return O_illegal;
@@ -1589,6 +1538,14 @@ operator ()
     default:
       return op_encoding[c];
 
+    case '+':
+    case '-':
+      /* Do not allow a++b and a--b to be a + (+b) and a - (-b)
+        Disabled, since the preprocessor removes whitespace.  */
+      if (1 || input_line_pointer[1] != c)
+       return op_encoding[c];
+      return O_illegal;
+
     case '<':
       switch (input_line_pointer[1])
        {
@@ -1604,14 +1561,14 @@ operator ()
          ret = O_le;
          break;
        }
-      ++input_line_pointer;
+      *num_chars = 2;
       return ret;
 
     case '=':
       if (input_line_pointer[1] != '=')
        return op_encoding[c];
 
-      ++input_line_pointer;
+      *num_chars = 2;
       return O_eq;
 
     case '>':
@@ -1626,7 +1583,7 @@ operator ()
          ret = O_ge;
          break;
        }
-      ++input_line_pointer;
+      *num_chars = 2;
       return ret;
 
     case '!':
@@ -1637,52 +1594,57 @@ operator ()
            return O_bit_inclusive_or;
          return op_encoding[c];
        }
-      ++input_line_pointer;
+      *num_chars = 2;
       return O_bit_exclusive_or;
 
     case '|':
       if (input_line_pointer[1] != '|')
        return op_encoding[c];
 
-      ++input_line_pointer;
+      *num_chars = 2;
       return O_logical_or;
 
     case '&':
       if (input_line_pointer[1] != '&')
        return op_encoding[c];
 
-      ++input_line_pointer;
+      *num_chars = 2;
       return O_logical_and;
     }
 
-  /*NOTREACHED*/
+  /* NOTREACHED  */
 }
 
 /* Parse an expression.  */
 
 segT
-expr (rankarg, resultP)
-     int rankarg;      /* Larger # is higher rank. */
-     expressionS *resultP;     /* Deliver result here. */
+expr (int rankarg,             /* Larger # is higher rank.  */
+      expressionS *resultP     /* Deliver result here.  */)
 {
   operator_rankT rank = (operator_rankT) rankarg;
   segT retval;
   expressionS right;
   operatorT op_left;
   operatorT op_right;
+  int op_chars;
 
   know (rank >= 0);
 
+  /* Save the value of dot for the fixup code.  */
+  if (rank == 0)
+    dot_value = frag_now_fix ();
+
   retval = operand (resultP);
 
-  know (*input_line_pointer != ' ');   /* Operand() gobbles spaces. */
+  /* operand () gobbles spaces.  */
+  know (*input_line_pointer != ' ');
 
-  op_left = operator ();
+  op_left = operator (&op_chars);
   while (op_left != O_illegal && op_rank[(int) op_left] > rank)
     {
       segT rightseg;
 
-      input_line_pointer++;    /*->after 1st character of operator. */
+      input_line_pointer += op_chars;  /* -> after operator.  */
 
       rightseg = expr (op_rank[(int) op_left], &right);
       if (right.X_op == O_absent)
@@ -1707,31 +1669,17 @@ expr (rankarg, resultP)
            }
        }
 
-      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 ();
+      op_right = operator (&op_chars);
 
-      know (op_right == O_illegal || op_rank[(int) op_right] <= op_rank[(int) op_left]);
+      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);
+           && (int) op_left <= (int) O_index);
 
-      /* input_line_pointer->after right-hand quantity. */
-      /* left-hand quantity in resultP */
-      /* right-hand quantity in right. */
-      /* operator in 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_op == O_big)
        {
@@ -1760,7 +1708,8 @@ expr (rankarg, resultP)
 #ifdef md_optimize_expr
       if (md_optimize_expr (resultP, op_left, &right))
        {
-         /* skip */;
+         /* Skip.  */
+         ;
        }
       else
 #endif
@@ -1775,8 +1724,8 @@ expr (rankarg, resultP)
               && resultP->X_op == O_symbol
               && (symbol_get_frag (right.X_add_symbol)
                   == symbol_get_frag (resultP->X_add_symbol))
-              && SEG_NORMAL (S_GET_SEGMENT (right.X_add_symbol)))
-
+              && (SEG_NORMAL (rightseg)
+                  || right.X_add_symbol == resultP->X_add_symbol))
        {
          resultP->X_add_number -= right.X_add_number;
          resultP->X_add_number += (S_GET_VALUE (resultP->X_add_symbol)
@@ -1816,7 +1765,7 @@ expr (rankarg, resultP)
            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.  */
+                characteristics of the compiler used to compile gas.  */
              resultP->X_add_number =
                (offsetT) ((valueT) resultP->X_add_number >> (valueT) v);
              break;
@@ -1871,7 +1820,14 @@ expr (rankarg, resultP)
          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;
+           {
+             resultP->X_add_number -= right.X_add_number;
+             if (retval == rightseg && SEG_NORMAL (retval))
+               {
+                 retval = absolute_section;
+                 rightseg = absolute_section;
+               }
+           }
        }
       else
        {
@@ -1883,8 +1839,23 @@ expr (rankarg, resultP)
          resultP->X_unsigned = 1;
        }
 
+      if (retval != rightseg)
+       {
+         if (! SEG_NORMAL (retval))
+           {
+             if (retval != undefined_section || SEG_NORMAL (rightseg))
+               retval = rightseg;
+           }
+         else if (SEG_NORMAL (rightseg)
+#ifdef DIFF_EXPR_OK
+                  && op_left != O_subtract
+#endif
+                  )
+           as_bad (_("operation combines symbols in different segments"));
+       }
+
       op_left = op_right;
-    }                          /* While next operator is >= this rank. */
+    }                          /* While next operator is >= this rank.  */
 
   /* The PA port needs this information.  */
   if (resultP->X_add_symbol)
@@ -1893,23 +1864,20 @@ expr (rankarg, resultP)
   return resultP->X_op == O_constant ? absolute_section : retval;
 }
 \f
-/*
- *                     get_symbol_end()
- *
- * This lives here because it belongs equally in expr.c & read.c.
- * Expr.c is just a branch office read.c anyway, and putting it
- * here lessens the crowd at read.c.
- *
- * Assume input_line_pointer is at start of symbol name.
- * Advance input_line_pointer past symbol name.
- * Turn that character into a '\0', returning its former value.
- * This allows a string compare (RMS wants symbol names to be strings)
- * of the symbol name.
- * There will always be a char following symbol name, because all good
- * lines end in end-of-line.
- */
+/* This lives here because it belongs equally in expr.c & read.c.
+   expr.c is just a branch office read.c anyway, and putting it
+   here lessens the crowd at read.c.
+
+   Assume input_line_pointer is at start of symbol name.
+   Advance input_line_pointer past symbol name.
+   Turn that character into a '\0', returning its former value.
+   This allows a string compare (RMS wants symbol names to be strings)
+   of the symbol name.
+   There will always be a char following symbol name, because all good
+   lines end in end-of-line.  */
+
 char
-get_symbol_end ()
+get_symbol_end (void)
 {
   char c;
 
@@ -1927,14 +1895,10 @@ get_symbol_end ()
   return (c);
 }
 
-
 unsigned int
-get_single_number ()
+get_single_number (void)
 {
   expressionS exp;
   operand (&exp);
   return exp.X_add_number;
-
 }
-
-/* end of expr.c */
This page took 0.079457 seconds and 4 git commands to generate.