]> Git Repo - binutils.git/blobdiff - gas/expr.c
a belated reflection of changed MPW files
[binutils.git] / gas / expr.c
index 871ada0cea3a0d258cc3445d018e09e4f16dac85..f35f8eee7769695f7bec00afbc0ea6d8005feeea 100644 (file)
@@ -1,18 +1,18 @@
 /* expr.c -operands, expressions-
-   Copyright (C) 1987, 1990, 1991, 1992 Free Software Foundation, Inc.
-   
+   Copyright (C) 1987, 1990, 1991, 1992, 1993 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. */
 
 #include "obstack.h"
 
-#if __STDC__ == 1
-static void clean_up_expression(expressionS *expressionP);
-#else /* __STDC__ */
-static void clean_up_expression();     /* Internal. */
-#endif /* not __STDC__ */
-extern const char EXP_CHARS[]; /* JF hide MD floating pt stuff all the same place */
-extern const char FLT_CHARS[];
+static void floating_constant PARAMS ((expressionS * expressionP));
+static void integer_constant PARAMS ((int radix, expressionS * expressionP));
+static void clean_up_expression PARAMS ((expressionS * expressionP));
+static symbolS *make_expr_symbol 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.  */
 
+static symbolS *
+make_expr_symbol (expressionP)
+     expressionS *expressionP;
+{
+  const char *fake;
+  symbolS *symbolP;
+
+  /* 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_new (fake,
+                       (expressionP->X_op == O_constant
+                        ? absolute_section
+                        : expr_section),
+                       0, &zero_address_frag);
+  symbolP->sy_value = *expressionP;
+  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
-floating_constant(expressionP)
-expressionS *expressionP;
+static void
+floating_constant (expressionP)
+     expressionS *expressionP;
 {
   /* input_line_pointer->*/
   /* floating-point constant. */
   int error_code;
 
-  error_code = atof_generic
-   (& input_line_pointer, ".", EXP_CHARS,
-    & generic_floating_point_number);
+  error_code = atof_generic (&input_line_pointer, ".", EXP_CHARS,
+                            &generic_floating_point_number);
 
   if (error_code)
-  {
-    if (error_code == ERROR_EXPONENT_OVERFLOW)
     {
-      as_bad("bad floating-point constant: exponent overflow, probably assembling junk");
-    }
-    else
-    {
-      as_bad("bad floating-point constant: unknown error code=%d.", error_code);
+      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_seg = SEG_BIG;
+  expressionP->X_op = O_big;
   /* input_line_pointer->just after constant, */
   /* which may point to whitespace. */
-  expressionP->X_add_number =-1;
-
+  expressionP->X_add_number = -1;
 }
 
-
-
-integer_constant(radix, expressionP)
-int radix;
-expressionS *expressionP;
-
-
+static void
+integer_constant (radix, expressionP)
+     int radix;
+     expressionS *expressionP;
 {
-  register char *      digit_2; /*->2nd digit of number. */  
+  char *start;         /* start of number. */
   char c;
 
-  register valueT      number; /* offset or (absolute) value */
-  register short int digit; /* value of next digit in current radix */
-  register short int maxdig = 0; /* highest permitted digit value. */
-  register int too_many_digits = 0; /* if we see >= this number of */
-  register char *name; /* points to name of symbol */
-  register symbolS *   symbolP; /* points to symbol */
-
-  int small; /* true if fits in 32 bits. */
-  extern  char hex_value[]; /* in hex_value.c */
-
-  /* may be bignum, or may fit in 32 bits. */
-  /*
-   * most numbers fit into 32 bits, and we want this case to be fast.
-   * so we pretend it will fit into 32 bits. if, after making up a 32
-   * bit number, we realise that we have scanned more digits than
-   * comfortably fit into 32 bits, we re-scan the digits coding
-   * them into a bignum. for decimal and octal numbers we are conservative: some
-   * numbers may be assumed bignums when in fact they do fit into 32 bits.
-   * numbers of any radix can have excess leading zeros: we strive
-   * to recognise this and cast them back into 32 bits.
-   * we must check that the bignum really is more than 32
-   * bits, and change it back to a 32-bit number if it fits.
-   * the number we are looking for is expected to be positive, but
-   * if it fits into 32 bits as an unsigned number, we let it be a 32-bit
-   * number. the cavalier approach is for speed in ordinary cases.
-   */
+  valueT number;       /* offset or (absolute) value */
+  short int digit;     /* value of next digit in current radix */
+  short int maxdig = 0;/* highest permitted digit value. */
+  int too_many_digits = 0;     /* if we see >= this number of */
+  char *name;          /* points to name of symbol */
+  symbolS *symbolP;    /* points to symbol */
+
+  int small;                   /* true if fits in 32 bits. */
+  extern const char hex_value[]; /* in hex_value.c */
+
+  /* May be bignum, or may fit in 32 bits. */
+  /* Most numbers fit into 32 bits, and we want this case to be fast.
+     so we pretend it will fit into 32 bits.  If, after making up a 32
+     bit number, we realise that we have scanned more digits than
+     comfortably fit into 32 bits, we re-scan the digits coding them
+     into a bignum.  For decimal and octal numbers we are
+     conservative: Some numbers may be assumed bignums when in fact
+     they do fit into 32 bits.  Numbers of any radix can have excess
+     leading zeros: We strive to recognise this and cast them back
+     into 32 bits.  We must check that the bignum really is more than
+     32 bits, and change it back to a 32-bit number if it fits.  The
+     number we are looking for is expected to be positive, but if it
+     fits into 32 bits as an unsigned number, we let it be a 32-bit
+     number.  The cavalier approach is for speed in ordinary cases. */
+  /* This has been extended for 64 bits.  We blindly assume that if
+     you're compiling in 64-bit mode, the target is a 64-bit machine.
+     This should be cleaned up.  */
+
+#ifdef BFD64
+#define valuesize 64
+#else /* includes non-bfd case, mostly */
+#define valuesize 32
+#endif
 
-  switch (radix) 
-  {
-    
-  case 2:
-    maxdig = 2;
-    too_many_digits = 33;
-    break;
-  case 8:
-     maxdig = radix = 8;
-    too_many_digits = 11;
-    break;
-  case 16:
-    
-
-    maxdig = radix = 16;
-    too_many_digits = 9;
-    break;
-  case 10:
-    maxdig = radix = 10;
-    too_many_digits = 11;
-  }
-  c = *input_line_pointer;
-  input_line_pointer++;
-  digit_2 = input_line_pointer;
-  for (number=0;  (digit=hex_value[c])<maxdig;  c = * input_line_pointer ++)
-  {
-    number = number * radix + digit;
-  }
-  /* c contains character after number. */
-  /* input_line_pointer->char after c. */
-  small = input_line_pointer - digit_2 < too_many_digits;
-  if (! small)
-  {
-    /*
-     * 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 ++)
+  switch (radix)
     {
-      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;
+    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 */
     }
-    else
+#undef valuesize
+  start = input_line_pointer;
+  c = *input_line_pointer++;
+  for (number = 0;
+       (digit = hex_value[(unsigned char) c]) < maxdig;
+       c = *input_line_pointer++)
     {
-      number = leader - generic_bignum + 1; /* number of littlenums in the bignum. */
+      number = number * radix + digit;
     }
-  }
-  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 (number<10)
+  /* c contains character after number. */
+  /* input_line_pointer->char after c. */
+  small = (input_line_pointer - start - 1) < too_many_digits;
+  if (!small)
     {
-      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)));
+      /*
+       * 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[(unsigned char) c]) < maxdig;
+          c = *input_line_pointer++)
+       {
+         for (pointer = generic_bignum;
+              pointer <= leader;
+              pointer++)
+           {
+             long work;
 
-         expressionP->X_add_symbol = symbolP;
-         expressionP->X_add_number = 0;
-         expressionP->X_seg = S_GET_SEGMENT(symbolP);
+             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;
+               }
+           }
        }
-       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;
+      /* 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. */
+         number =
+           ((generic_bignum[1] & LITTLENUM_MASK) << LITTLENUM_NUMBER_OF_BITS)
+           | (generic_bignum[0] & LITTLENUM_MASK);
+         small = 1;
        }
-      }
       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. */
+         number = leader - generic_bignum + 1; /* number of littlenums in the bignum. */
+       }
+    }
+  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!
+       */
+
+      switch (c)
+       {
+
+#ifdef LOCAL_LABELS_FB
+       case 'b':
+         {
+           /*
+            * backward ref to local label.
+            * because it is backward, expect it to be defined.
+            */
+           /* Construct a local label.  */
+           name = fb_label_name ((int) number, 0);
+
+           /* seen before, or symbol is defined: ok */
+           symbolP = symbol_find (name);
+           if ((symbolP != NULL) && (S_IS_DEFINED (symbolP)))
+             {
+
+               /* local labels are never absolute. don't waste time
+                  checking absoluteness. */
+               know (SEG_NORMAL (S_GET_SEGMENT (symbolP)));
+
+               expressionP->X_op = O_symbol;
+               expressionP->X_add_symbol = symbolP;
+
+             }
+           else
+             {
+               /* either not seen or not defined. */
+               /* @@ Should print out the original string instead of
+                  the parsed number.  */
+               as_bad ("backw. ref to unknown label \"%d:\", 0 assumed.",
+                       (int) number);
+               expressionP->X_op = O_constant;
+             }
+
+           expressionP->X_add_number = 0;
+           break;
+         }                     /* case 'b' */
+
+       case 'f':
+         {
+           /*
+            * forward reference. expect symbol to be undefined or
+            * unknown. undefined: seen it before. unknown: never seen
+            * it before.
+            * construct a local label name, then an undefined symbol.
+            * don't create a xseg frag for it: caller may do that.
+            * just return it as never seen before.
+            */
+           name = fb_label_name ((int) number, 1);
+           symbolP = symbol_find_or_make (name);
+           /* we have no need to check symbol properties. */
 #ifndef many_segments
-         /* since "know" puts its arg into a "string", we
-            can't have newlines in the argument.  */
-         know(S_GET_SEGMENT(symbolP) == SEG_UNKNOWN || S_GET_SEGMENT(symbolP) == SEG_TEXT || S_GET_SEGMENT(symbolP) == SEG_DATA);
+           /* since "know" puts its arg into a "string", we
+              can't have newlines in the argument.  */
+           know (S_GET_SEGMENT (symbolP) == undefined_section || S_GET_SEGMENT (symbolP) == text_section || S_GET_SEGMENT (symbolP) == data_section);
 #endif
-         expressionP->X_add_symbol      = symbolP;
-         expressionP->X_seg             = SEG_UNKNOWN;
-         expressionP->X_subtract_symbol = NULL;
-         expressionP->X_add_number      = 0;
-       }
-       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') */
+           expressionP->X_op = O_symbol;
+           expressionP->X_add_symbol = symbolP;
+           expressionP->X_add_number = 0;
+
+           break;
+         }                     /* case 'f' */
+
+#endif /* LOCAL_LABELS_FB */
+
+#ifdef LOCAL_LABELS_DOLLAR
+
+       case '$':
+         {
+
+           /* 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;
+
+           break;
+         }                     /* case '$' */
+
+#endif /* LOCAL_LABELS_DOLLAR */
+
+       default:
+         {
+           expressionP->X_op = O_constant;
+           expressionP->X_add_number = number;
+           input_line_pointer--;       /* restore following character. */
+           break;
+         }                     /* really just a number */
+
+       }                       /* switch on char following the number */
+
     }
-    else
-    { /* really a number. */
-      expressionP->X_add_number = number;
-      expressionP->X_seg        = SEG_ABSOLUTE;
-      input_line_pointer --; /* restore following character. */
-    } /* if (number<10) */
-  }
   else
-  {
-    expressionP->X_add_number = number;
-    expressionP->X_seg = SEG_BIG;
-    input_line_pointer --; /*->char following number. */
-  } /* if (small) */
-} 
+    {
+      /* not a small number */
+      expressionP->X_op = O_big;
+      expressionP->X_add_number = number;      /* number of littlenums */
+      input_line_pointer--;    /*->char following number. */
+    }
+}
 
 
 /*
@@ -324,90 +375,46 @@ expressionS *expressionP;
  * in: Input_line_pointer points to 1st char of operand, which may
  *     be a space.
  *
- * out:        A expressionS. X_seg determines how to understand the rest of the
- *     expressionS.
- *     The operand may have been empty: in this case X_seg == SEG_ABSENT.
+ * out:        A expressionS.
+ *     The operand may have been empty: in this case X_op == O_absent.
  *     Input_line_pointer->(next non-blank) char after operand.
- *
  */
-\f
-
 
 static segT
 operand (expressionP)
-     register expressionS *    expressionP;
+     expressionS *expressionP;
 {
-  register char c;
-  register symbolS *   symbolP; /* points to symbol */
-  register char *name; /* points to name of symbol */
-  /* invented for humans only, hope */
-  /* optimising compiler flushes it! */
-  register short int radix; /* 2, 8, 10 or 16, 0 when floating */
-  /* 0 means we saw start of a floating- */
-  /* point constant. */
+  char c;
+  symbolS *symbolP;    /* points to symbol */
+  char *name;          /* points to name of symbol */
+  segT segment;
+
+  /* All integers are regarded as unsigned unless they are negated.
+     This is because the only thing which cares whether a number is
+     unsigned is the code in emit_expr which extends constants into
+     bignums.  It should only sign extend negative numbers, so that
+     something like ``.quad 0x80000000'' is not sign extended even
+     though it appears negative if valueT is 32 bits.  */
+  expressionP->X_unsigned = 1;
 
   /* digits, assume it is a bignum. */
 
-
-
-
-  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. */
 
   switch (c)
-  {
-#ifdef MRI
-  case '%':
-    integer_constant(2, expressionP);
-    break;
-  case '@':
-    integer_constant(8, expressionP);
-    break;
-  case '$':
-    integer_constant(16, expressionP);
-    break;     
-#endif
-  case '1':
-  case '2':
-  case '3':
-  case '4':
-  case '5':
-  case '6':
-  case '7':
-  case '8':
-  case '9':
-    input_line_pointer--;
-    
-    integer_constant(10, expressionP);
-    break;
-
-  case '0':
-    /* non-decimal radix */
-
-      
-    c = *input_line_pointer;
-    switch (c) 
     {
-
-    default:
-      /* The string was only zero */
-      expressionP->X_add_symbol = 0;
-      expressionP->X_add_number = 0;
-      expressionP->X_seg = SEG_ABSOLUTE;
+#ifdef MRI
+    case '%':
+      integer_constant (2, expressionP);
       break;
-      
-    case 'x':
-    case 'X':
-      input_line_pointer++;
-      integer_constant(16, expressionP);
+    case '@':
+      integer_constant (8, expressionP);
       break;
-    case 'B':
-    case 'b':
-      input_line_pointer++;
-      integer_constant(2, expressionP);
+    case '$':
+      integer_constant (16, expressionP);
       break;
-
-    case '0':
+#endif
     case '1':
     case '2':
     case '3':
@@ -415,347 +422,315 @@ operand (expressionP)
     case '5':
     case '6':
     case '7':
-      integer_constant(8, expressionP);
-      break;
+    case '8':
+    case '9':
+      input_line_pointer--;
 
-    case 'f':      
-      /* 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)))) 
-      {
-       input_line_pointer -= 2;
-       integer_constant(10, expressionP);
-       break;
-      } 
-
-    case 'd':
-    case 'D':
-    case 'F':
-      
-    case 'e':
-    case 'E':
-    case 'g':
-    case 'G':
-      
-      input_line_pointer++;  
-      floating_constant(expressionP);
+      integer_constant (10, expressionP);
       break;
-    }
-  
-    break;
-  case  '(':
-    /* didn't begin with digit & not a name */
-  {
-    (void)expression(expressionP);
-    /* Expression() will pass trailing whitespace */
-    if (* input_line_pointer ++ != ')')
-    {
-      as_bad("Missing ')' assumed");
-      input_line_pointer --;
-    }
-    /* here with input_line_pointer->char after "(...)" */
-  }
-    return;
-
-
-  case '\'':
-    /*
-     * Warning: to conform to other people's assemblers NO ESCAPEMENT is permitted
-     * for a single quote. The next character, parity errors and all, is taken
-     * as the value of the operand. VERY KINKY.
-     */
-    expressionP->X_add_number = * input_line_pointer ++;
-    expressionP->X_seg        = SEG_ABSOLUTE;
-    break;
-
-  case  '~':
-  case  '-':
-  case  '+':
-
-  {
-    /* unary operator: hope for SEG_ABSOLUTE */
-    switch(operand (expressionP)) {
-      case SEG_ABSOLUTE:
-       /* input_line_pointer -> char after operand */
-       if ( c=='-' )
+
+    case '0':
+      /* non-decimal radix */
+
+      c = *input_line_pointer;
+      switch (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.
-          */
+
+       default:
+         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':
+         input_line_pointer++;
+         integer_constant (16, expressionP);
+         break;
+
+       case 'b':
+#ifdef LOCAL_LABELS_FB
+         if (!input_line_pointer[1]
+             /* Strictly speaking, we should only need to check for
+                "+-01", since that's all you'd normally have in a
+                binary constant.  But some of our code does permit
+                digits greater than the base we're expecting.  */
+             || !strchr ("+-0123456789", input_line_pointer[1]))
+           {
+             input_line_pointer--;
+             integer_constant (10, expressionP);
+             break;
+           }
+#endif
+       case 'B':
+         input_line_pointer++;
+         integer_constant (2, expressionP);
+         break;
+
+       case '0':
+       case '1':
+       case '2':
+       case '3':
+       case '4':
+       case '5':
+       case '6':
+       case '7':
+         integer_constant (8, expressionP);
+         break;
+
+       case 'f':
+#ifdef LOCAL_LABELS_FB
+         /* if it says '0f' and the line ends or it doesn't look like
+            a floating point #, its a local label ref.  dtrt */
+         /* likewise for the b's.  xoxorich. */
+         if (c == 'f'
+             && (!input_line_pointer[1]
+                 || (!strchr ("+-.0123456789", input_line_pointer[1])
+                     && !strchr (EXP_CHARS, input_line_pointer[1]))))
+           {
+             input_line_pointer -= 1;
+             integer_constant (10, expressionP);
+             break;
+           }
+#endif
+
+       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;
+
+#ifdef LOCAL_LABELS_DOLLAR
+       case '$':
+         integer_constant (10, expressionP);
+         break;
+#endif
        }
-       else
+
+      break;
+
+    case '(':
+      /* didn't begin with digit & not a name */
+      segment = expression (expressionP);
+      /* Expression() will pass trailing whitespace */
+      if (*input_line_pointer++ != ')')
        {
-         expressionP -> X_add_number = ~ expressionP -> X_add_number;
+         as_bad ("Missing ')' assumed");
+         input_line_pointer--;
        }
-       break;
-
-      case SEG_TEXT:
-      case SEG_DATA:
-      case SEG_BSS:
-      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;
-         }
-      default: /* unary on non-absolute is unsuported */
-       as_warn("Unary operator %c ignored because bad operand follows", c);
-       break;
-       /* Expression undisturbed from operand(). */
-      }
-  }
-    
-
-  
-    break;  
+      /* here with input_line_pointer->char after "(...)" */
+      return segment;
+
+    case '\'':
+      /* Warning: to conform to other people's assemblers NO ESCAPEMENT is
+        permitted for a single quote. The next character, parity errors and
+        all, is taken as the value of the operand. VERY KINKY.  */
+      expressionP->X_op = O_constant;
+      expressionP->X_add_number = *input_line_pointer++;
+      break;
 
-  case '.':
-    if( !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;
+    case '+':
+      (void) operand (expressionP);
       break;
-      
-    }
-    else 
-    {
-      goto isname;
-      
 
-    }
-  case ',':    
-  case '\n':
-       /* can't imagine any other kind of operand */
-       expressionP->X_seg = SEG_ABSENT;
-       input_line_pointer --;
-       md_operand (expressionP);
-    break;    
-    /* Fall through */
-  default:
-    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.
-       */
-      expressionP->X_seg = S_GET_SEGMENT(symbolP);
-      switch (expressionP->X_seg)
+    case '~':
+    case '-':
       {
-      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;
+       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);
       }
-      * input_line_pointer = c;
-      expressionP->X_subtract_symbol = NULL;
-    }
-    else 
-    {
-      as_bad("Bad expression");
-      expressionP->X_add_number = 0;
-      expressionP->X_seg = SEG_ABSOLUTE;
+      break;
 
-    }
-    
-  }
-  
-  
-  
+    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
+       {
+         goto isname;
+       }
+    case ',':
+    case '\n':
+    case '\0':
+    eol:
+      /* can't imagine any other kind of operand */
+      expressionP->X_op = O_absent;
+      input_line_pointer--;
+      md_operand (expressionP);
+      break;
 
-  
+    default:
+      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
+       {
+         as_bad ("Bad expression");
+         expressionP->X_op = O_constant;
+         expressionP->X_add_number = 0;
+       }
+    }
 
   /*
    * 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() */
+  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_seg field of the expressionS may only take certain values.
- *     Now, we permit SEG_PASS1 to make code smaller & faster.
+ *     The X_op field of the expressionS may only take certain values.
  *     Elsewise we waste time special-case testing. Sigh. Ditto SEG_ABSENT.
  * Out:        expressionS may have been modified:
  *     'foo-foo' symbol references cancelled to 0,
- *             which changes X_seg from SEG_DIFFERENCE to SEG_ABSOLUTE;
+ *             which changes X_op from O_subtract to O_constant.
  *     Unused fields zeroed to help expr().
  */
 
 static void
 clean_up_expression (expressionP)
-     register expressionS * expressionP;
+     expressionS *expressionP;
 {
-  switch (expressionP->X_seg)
-      {
-      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;
-       }
-       else {
-           BAD_CASE (expressionP->X_seg);
-       }
-       break;
-      }
-} /* clean_up_expression() */
-\f
-/*
- *                     expr_part ()
- *
- * Internal. Made a function because this code is used in 2 places.
- * Generate error or correct X_?????_symbol of expressionS.
- */
-
-/*
- * symbol_1 += symbol_2 ... well ... sort of.
- */
-
-static segT
-expr_part (symbol_1_PP, symbol_2_P)
-     symbolS **        symbol_1_PP;
-     symbolS * symbol_2_P;
-{
-    segT                       return_value;
-#ifndef MANY_SEGMENTS
-    know((* symbol_1_PP) == NULL || (S_GET_SEGMENT(*symbol_1_PP) == SEG_TEXT) || (S_GET_SEGMENT(*symbol_1_PP) == SEG_DATA) || (S_GET_SEGMENT(*symbol_1_PP) == SEG_BSS) || (!S_IS_DEFINED(* symbol_1_PP)));
-    know(symbol_2_P == NULL || (S_GET_SEGMENT(symbol_2_P) == SEG_TEXT) || (S_GET_SEGMENT(symbol_2_P) == SEG_DATA) || (S_GET_SEGMENT(symbol_2_P) == SEG_BSS) || (!S_IS_DEFINED(symbol_2_P)));
-#endif
-  if (* symbol_1_PP)
+  switch (expressionP->X_op)
     {
-      if (!S_IS_DEFINED(* symbol_1_PP))
-       {
-         if (symbol_2_P)
-           {
-               return_value = SEG_PASS1;
-               * symbol_1_PP = NULL;
-           }
-         else
-           {
-               know(!S_IS_DEFINED(* symbol_1_PP));
-               return_value = SEG_UNKNOWN;
-           }
-       }
-      else
-       {
-         if (symbol_2_P)
-           {
-             if (!S_IS_DEFINED(symbol_2_P))
-               {
-                 * symbol_1_PP = NULL;
-                 return_value = SEG_PASS1;
-               }
-             else
-               {
-                 /* {seg1} - {seg2} */
-                 as_bad("Expression too complex, 2 symbolS forgotten: \"%s\" \"%s\"",
-                         S_GET_NAME(* symbol_1_PP), S_GET_NAME(symbol_2_P));
-                 * symbol_1_PP = NULL;
-                 return_value = SEG_ABSOLUTE;
-               }
-           }
-         else
-           {
-             return_value = S_GET_SEGMENT(* symbol_1_PP);
-           }
-       }
-    }
-  else
-    {                          /* (* symbol_1_PP) == NULL */
-      if (symbol_2_P)
-       {
-         * symbol_1_PP = symbol_2_P;
-         return_value = S_GET_SEGMENT(symbol_2_P);
-       }
-      else
+    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))))
        {
-         * symbol_1_PP = NULL;
-         return_value = SEG_ABSOLUTE;
+         expressionP->X_op = O_constant;
+         expressionP->X_add_symbol = NULL;
+         expressionP->X_op_symbol = NULL;
        }
+      break;
+    default:
+      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() */
+}
 \f
 /* Expression parser. */
 
@@ -775,55 +750,42 @@ expr_part (symbol_1_PP, symbol_2_P)
  * After expr(RANK,resultP) input_line_pointer->operator of rank <= RANK.
  * Also, we have consumed any leading or trailing spaces (operand does that)
  * and done all intervening operators.
+ *
+ * This returns the segment of the result, which will be
+ * absolute_section or the segment of a symbol.
  */
 
-typedef enum
-{
-    O_illegal,                 /* (0)  what we get for illegal op */
-    
-    O_multiply,                        /* (1)  * */
-    O_divide,                  /* (2)  / */
-    O_modulus,                 /* (3)  % */
-    O_left_shift,                      /* (4)  < */
-    O_right_shift,                     /* (5)  > */
-    O_bit_inclusive_or,                /* (6)  | */
-    O_bit_or_not,                      /* (7)  ! */
-    O_bit_exclusive_or,                /* (8)  ^ */
-    O_bit_and,                 /* (9)  & */
-    O_add,                             /* (10) + */
-    O_subtract                 /* (11) - */
-    }
-operatorT;
-
+#undef __
 #define __ O_illegal
 
-static const operatorT op_encoding [256] = {   /* 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 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, __, __, __,
+
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __,
+  __, __, __, __, __, __, __, __, __, __, __, __, __, __, __, __
+};
 
 
 /*
@@ -832,226 +794,197 @@ static const operatorT op_encoding [256] = {    /* maps ASCII->operators */
  *     1       + -
  *     2       & ^ ! |
  *     3       * / % << >>
+ *     4       unary - unary ~
  */
-static const operator_rankT
-op_rank [] = { 0, 3, 3, 3, 3, 3, 2, 2, 2, 2, 1, 1 };
+static const operator_rankT op_rank[] =
+{
+  0,   /* O_illegal */
+  0,   /* O_absent */
+  0,   /* O_constant */
+  0,   /* O_symbol */
+  0,   /* O_register */
+  0,   /* O_bit */
+  4,   /* O_uminus */
+  4,   /* O_bit_now */
+  3,   /* O_multiply */
+  3,   /* O_divide */
+  3,   /* O_modulus */
+  3,   /* O_left_shift */
+  3,   /* O_right_shift */
+  2,   /* O_bit_inclusive_or */
+  2,   /* O_bit_or_not */
+  2,   /* O_bit_exclusive_or */
+  2,   /* O_bit_and */
+  1,   /* O_add */
+  1,   /* O_subtract */
+};
 \f
-/* Return resultP->X_seg. */
-segT expr(rank, resultP)
-register operator_rankT        rank; /* Larger # is higher rank. */
-register expressionS *resultP; /* Deliver result here. */
+segT
+expr (rank, resultP)
+     operator_rankT rank;      /* Larger # is higher rank. */
+     expressionS *resultP;     /* Deliver result here. */
 {
-  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)
+  segT retval;
+  expressionS right;
+  operatorT op_left;
+  char c_left;         /* 1st operator character. */
+  operatorT op_right;
+  char c_right;
+
+  know (rank >= 0);
+
+  retval = operand (resultP);
+
+  know (*input_line_pointer != ' ');   /* Operand() gobbles spaces. */
+
+  c_left = *input_line_pointer;        /* Potential operator character. */
+  op_left = op_encoding[(unsigned char) c_left];
+  while (op_left != O_illegal && op_rank[(int) op_left] > rank)
     {
-      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))
+      segT rightseg;
+
+      input_line_pointer++;    /*->after 1st character of operator. */
+      /* Operators "<<" and ">>" have 2 characters. */
+      if (*input_line_pointer == c_left && (c_left == '<' || c_left == '>'))
+       ++input_line_pointer;
+
+      rightseg = expr (op_rank[(int) op_left], &right);
+      if (right.X_op == O_absent)
        {
-         as_warn("Missing operand value assumed absolute 0.");
-         resultP->X_add_number = 0;
-         resultP->X_subtract_symbol    = NULL;
-         resultP->X_add_symbol = NULL;
-         resultP->X_seg = SEG_ABSOLUTE;
+         as_warn ("missing operand; zero assumed");
+         right.X_op = O_constant;
+         right.X_add_number = 0;
+         right.X_add_symbol = NULL;
+         right.X_op_symbol = NULL;
        }
-      know(* input_line_pointer != ' ');
-      c_right = * input_line_pointer;
-      op_right = op_encoding [c_right];
-      if (* input_line_pointer == c_right && (c_right == '<' || c_right == '>'))
+
+      know (*input_line_pointer != ' ');
+
+      if (retval == undefined_section)
        {
-         input_line_pointer ++;
-       }                       /*->after operator. */
-      know((int) op_right == 0 || op_rank [(int) op_right] <= op_rank[(int) op_left]);
+         if (SEG_NORMAL (rightseg))
+           retval = rightseg;
+       }
+      else if (! SEG_NORMAL (retval))
+       retval = rightseg;
+      else if (SEG_NORMAL (rightseg)
+              && retval != rightseg
+#ifdef DIFF_EXPR_OK
+              && op_left != O_subtract
+#endif
+              )
+       as_bad ("operation combines symbols in different segments");
+
+      c_right = *input_line_pointer;
+      op_right = op_encoding[(unsigned char) c_right];
+      if (*input_line_pointer == c_right && (c_right == '<' || c_right == '>'))
+       ++input_line_pointer;
+
+      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_seg == SEG_PASS1 || right . X_seg == SEG_PASS1)
+
+      if (resultP->X_op == O_big)
        {
-         resultP->X_seg = SEG_PASS1;
+         as_warn ("left operand of %c is a %s; integer 0 assumed",
+                  c_left, 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;
        }
-      else
+      if (right.X_op == O_big)
        {
-         if (resultP->X_seg == SEG_BIG)
-           {
-             as_warn("Left operand of %c is a %s.  Integer 0 assumed.",
-                     c_left, resultP->X_add_number > 0 ? "bignum" : "float");
-             resultP->X_seg = SEG_ABSOLUTE;
-             resultP->X_add_symbol = 0;
-             resultP->X_subtract_symbol = 0;
-             resultP->X_add_number = 0;
-           }
-         if (right . X_seg == SEG_BIG)
+         as_warn ("right operand of %c is a %s; integer 0 assumed",
+                  c_left, 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;
+       }
+      else if (op_left == O_subtract && right.X_op == O_constant)
+       {
+         /* X - constant.  */
+         resultP->X_add_number -= right.X_add_number;
+       }
+      else if (op_left == O_add && resultP->X_op == O_constant)
+       {
+         /* Constant + X.  */
+         resultP->X_op = right.X_op;
+         resultP->X_add_symbol = right.X_add_symbol;
+         resultP->X_op_symbol = right.X_op_symbol;
+         resultP->X_add_number += right.X_add_number;
+         retval = rightseg;
+       }
+      else if (resultP->X_op == O_constant && right.X_op == O_constant)
+       {
+         /* Constant OP constant.  */
+         offsetT v = right.X_add_number;
+         if (v == 0 && (op_left == O_divide || op_left == O_modulus))
            {
-             as_warn("Right operand of %c is a %s.  Integer 0 assumed.",
-                     c_left, right . X_add_number > 0 ? "bignum" : "float");
-             right . X_seg = SEG_ABSOLUTE;
-             right . X_add_symbol = 0;
-             right . X_subtract_symbol = 0;
-             right . X_add_number = 0;
+             as_warn ("division by zero");
+             v = 1;
            }
-         if (op_left == O_subtract)
+         switch (op_left)
            {
-             /*
-              * Convert - into + by exchanging symbolS and negating number.
-              * I know -infinity can't be negated in 2's complement:
-              * but then it can't be subtracted either. This trick
-              * does not cause any further inaccuracy.
-              */
-
-             register symbolS *        symbolP;
-
-             right . X_add_number      = - right . X_add_number;
-             symbolP                   = right . X_add_symbol;
-             right . X_add_symbol      = right . X_subtract_symbol;
-             right . X_subtract_symbol = symbolP;
-             if (symbolP)
-               {
-                 right . X_seg         = SEG_DIFFERENCE;
-               }
-             op_left = O_add;
+           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;
+           default:                    abort ();
            }
-\f
+       }
+      else if (resultP->X_op == O_symbol
+              && right.X_op == O_symbol
+              && (op_left == O_add
+                  || op_left == O_subtract
+                  || (resultP->X_add_number == 0
+                      && right.X_add_number == 0)))
+       {
+         /* Symbol OP symbol.  */
+         resultP->X_op = op_left;
+         resultP->X_op_symbol = right.X_add_symbol;
          if (op_left == O_add)
-           {
-             segT      seg1;
-             segT      seg2;
-#ifndef MANY_SEGMENTS
-             know(resultP->X_seg == SEG_DATA || resultP->X_seg == SEG_TEXT || resultP->X_seg == SEG_BSS || resultP->X_seg == SEG_UNKNOWN || resultP->X_seg == SEG_DIFFERENCE || resultP->X_seg == SEG_ABSOLUTE || resultP->X_seg == SEG_PASS1);
-             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);
-      }
-         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;
+           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
 /*
@@ -1070,29 +1003,24 @@ register expressionS *resultP; /* Deliver result here. */
  * 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;
+
+  while (is_part_of_name (c = *input_line_pointer++))
+    ;
+  *--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 */
This page took 0.07357 seconds and 4 git commands to generate.