OSDN Git Service

* cppfiles.c (search_from): Special case the empty string.
[pf3gnuchains/gcc-fork.git] / gcc / genattrtab.c
index cd5ae43..c69ad53 100644 (file)
@@ -1,5 +1,6 @@
 /* Generate code from machine description to compute values of attributes.
-   Copyright (C) 1991, 1993, 1994 Free Software Foundation, Inc.
+   Copyright (C) 1991, 1993, 1994, 1995, 1996, 1997, 1998,
+   1999, 2000 Free Software Foundation, Inc.
    Contributed by Richard Kenner (kenner@vlsi1.ultra.nyu.edu)
 
 This file is part of GNU CC.
@@ -16,7 +17,8 @@ GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
 along with GNU CC; see the file COPYING.  If not, write to
-the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
+the Free Software Foundation, 59 Temple Place - Suite 330,
+Boston, MA 02111-1307, USA.  */
 
 /* This program handles insn attributes and the DEFINE_DELAY and
    DEFINE_FUNCTION_UNIT definitions.
@@ -33,9 +35,9 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
    If the attribute `alternative', or a random C expression is present,
    `constrain_operands' is called.  If either of these cases of a reference to
-   an operand is found, `insn_extract' is called.
+   an operand is found, `extract_insn' is called.
 
-   The special attribute `length' is also recognized.  For this operand, 
+   The special attribute `length' is also recognized.  For this operand,
    expressions involving the address of an operand or the current insn,
    (address (pc)), are valid.  In this case, an initial pass is made to
    set all lengths that do not depend on address.  Those that do are set to
@@ -50,7 +52,7 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
    parameters as it does not depend on any particular insn.  Constant
    attributes are typically used to specify which variety of processor is
    used.
-   
+
    Internal attributes are defined to handle DEFINE_DELAY and
    DEFINE_FUNCTION_UNIT.  Special routines are output for these cases.
 
@@ -93,46 +95,28 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
    `volatil' (MEM_VOLATILE_P): During simplify_by_exploding the value of an
       EQ_ATTR rtx is true if !volatil and false if volatil.  */
 
-
 #include "hconfig.h"
-/* varargs must always be included after *config.h.  */
-#ifdef __STDC__
-#include <stdarg.h>
-#else
-#include <varargs.h>
-#endif
+#include "system.h"
 #include "rtl.h"
-#include "insn-config.h"       /* For REGISTER_CONSTRAINTS */
-#include <stdio.h>
+#include "ggc.h"
+#include "gensupport.h"
 
-#ifndef VMS
-#ifndef USG
-#include <sys/time.h>
-#include <sys/resource.h>
-#endif
+#ifdef HAVE_SYS_RESOURCE_H
+# include <sys/resource.h>
 #endif
 
 /* We must include obstack.h after <sys/time.h>, to avoid lossage with
    /usr/include/sys/stdtypes.h on Sun OS 4.x.  */
 #include "obstack.h"
+#include "errors.h"
 
-static struct obstack obstack, obstack1, obstack2;
-struct obstack *rtl_obstack = &obstack;
+static struct obstack obstack1, obstack2;
 struct obstack *hash_obstack = &obstack1;
 struct obstack *temp_obstack = &obstack2;
 
 #define obstack_chunk_alloc xmalloc
 #define obstack_chunk_free free
 
-/* Define this so we can link with print-rtl.o to get debug_rtx function.  */
-char **insn_name_ptr = 0;
-
-extern void free ();
-extern rtx read_rtx ();
-
-static void fatal ();
-void fancy_abort ();
-
 /* enough space to reserve for printing out ints */
 #define MAX_DIGITS (HOST_BITS_PER_INT * 3 / 10 + 3)
 
@@ -145,12 +129,13 @@ void fancy_abort ();
 
 struct insn_def
 {
-  int insn_code;               /* Instruction number. */
-  int insn_index;              /* Expression numer in file, for errors. */
-  struct insn_def *next;       /* Next insn in chain. */
-  rtx def;                     /* The DEFINE_... */
+  struct insn_def *next;       /* Next insn in chain.  */
+  rtx def;                     /* The DEFINE_...  */
+  int insn_code;               /* Instruction number.  */
+  int insn_index;              /* Expression numer in file, for errors.  */
+  int lineno;                  /* Line number.  */
   int num_alternatives;                /* Number of alternatives.  */
-  int vec_idx;                 /* Index of attribute vector in `def'. */
+  int vec_idx;                 /* Index of attribute vector in `def'.  */
 };
 
 /* Once everything has been read in, we store in each attribute value a list
@@ -159,9 +144,10 @@ struct insn_def
 
 struct insn_ent
 {
+  struct insn_ent *next;       /* Next in chain.  */
   int insn_code;               /* Instruction number.  */
   int insn_index;              /* Index of definition in file */
-  struct insn_ent *next;       /* Next in chain.  */
+  int lineno;                  /* Line number.  */
 };
 
 /* Each value of an attribute (either constant or computed) is assigned a
@@ -181,15 +167,18 @@ struct attr_value
 
 struct attr_desc
 {
-  char *name;                  /* Name of attribute. */
-  struct attr_desc *next;      /* Next attribute. */
-  int is_numeric;              /* Values of this attribute are numeric. */
-  int negative_ok;             /* Allow negative numeric values.  */
-  int unsigned_p;              /* Make the output function unsigned int.  */
-  int is_const;                        /* Attribute value constant for each run.  */
-  int is_special;              /* Don't call `write_attr_set'. */
-  struct attr_value *first_value; /* First value of this attribute. */
-  struct attr_value *default_val; /* Default value for this attribute. */
+  char *name;                  /* Name of attribute.  */
+  struct attr_desc *next;      /* Next attribute.  */
+  unsigned is_numeric  : 1;    /* Values of this attribute are numeric.  */
+  unsigned negative_ok : 1;    /* Allow negative numeric values.  */
+  unsigned unsigned_p  : 1;    /* Make the output function unsigned int.  */
+  unsigned is_const    : 1;    /* Attribute value constant for each run.  */
+  unsigned is_special  : 1;    /* Don't call `write_attr_set'.  */
+  unsigned func_units_p        : 1;    /* this is the function_units attribute */
+  unsigned blockage_p  : 1;    /* this is the blockage range function */
+  struct attr_value *first_value; /* First value of this attribute.  */
+  struct attr_value *default_val; /* Default value for this attribute.  */
+  int lineno;                  /* Line number.  */
 };
 
 #define NULL_ATTR (struct attr_desc *) NULL
@@ -207,8 +196,9 @@ struct range
 struct delay_desc
 {
   rtx def;                     /* DEFINE_DELAY expression.  */
-  struct delay_desc *next;     /* Next DEFINE_DELAY. */
+  struct delay_desc *next;     /* Next DEFINE_DELAY.  */
   int num;                     /* Number of DEFINE_DELAY, starting at 1.  */
+  int lineno;                  /* Line number.  */
 };
 
 /* Record information about each DEFINE_FUNCTION_UNIT.  */
@@ -222,6 +212,7 @@ struct function_unit_op
   int issue_delay;             /* Cost until unit can accept another insn.  */
   rtx conflict_exp;            /* Expression TRUE for insns incurring issue delay.  */
   rtx issue_exp;               /* Expression computing issue delay.  */
+  int lineno;                  /* Line number.  */
 };
 
 /* Record information about each function unit mentioned in a
@@ -229,13 +220,13 @@ struct function_unit_op
 
 struct function_unit
 {
-  char *name;                  /* Function unit name.  */
+  const char *name;            /* Function unit name.  */
   struct function_unit *next;  /* Next function unit.  */
   int num;                     /* Ordinal of this unit type.  */
   int multiplicity;            /* Number of units of this type.  */
   int simultaneity;            /* Maximum number of simultaneous insns
                                   on this function unit or 0 if unlimited.  */
-  rtx condexp;                 /* Expression TRUE for insn needing unit. */
+  rtx condexp;                 /* Expression TRUE for insn needing unit.  */
   int num_opclasses;           /* Number of different operation types.  */
   struct function_unit_op *ops;        /* Pointer to first operation type.  */
   int needs_conflict_function; /* Nonzero if a conflict function required.  */
@@ -244,6 +235,7 @@ struct function_unit
   rtx default_cost;            /* Conflict cost, if constant.  */
   struct range issue_delay;    /* Range of issue delay values.  */
   int max_blockage;            /* Maximum time an insn blocks the unit.  */
+  int first_lineno;            /* First seen line number.  */
 };
 
 /* Listheads of above structures.  */
@@ -262,7 +254,7 @@ static struct function_unit *units;
    computed and becomes the corresponding value.  To do this, we must be
    able to enumerate all values for each attribute used in the expression
    (currently, we give up if we find a numeric attribute).
-   
+
    If the set of EQ_ATTR tests used in an expression tests the value of N
    different attributes, the list of all possible combinations can be made
    by walking the N-dimensional attribute space defined by those
@@ -290,7 +282,7 @@ static struct function_unit *units;
    Once the dimensions are created, the algorithm enumerates all possible
    values and computes the current value of the given expression.  */
 
-struct dimension 
+struct dimension
 {
   struct attr_desc *attr;      /* Attribute for this dimension.  */
   rtx values;                  /* List of attribute values used.  */
@@ -298,7 +290,7 @@ struct dimension
   int num_values;              /* Length of the values list.  */
 };
 
-/* Other variables. */
+/* Other variables.  */
 
 static int insn_code_number;
 static int insn_index_number;
@@ -309,11 +301,12 @@ static int address_used;
 static int length_used;
 static int num_delays;
 static int have_annul_true, have_annul_false;
-static int num_units;
+static int num_units, num_unit_opclasses;
+static int num_insn_ents;
 
 /* Used as operand to `operate_exp':  */
 
-enum operator {PLUS_OP, MINUS_OP, POS_MINUS_OP, EQ_OP, OR_OP, MAX_OP, MIN_OP, RANGE_OP};
+enum operator {PLUS_OP, MINUS_OP, POS_MINUS_OP, EQ_OP, OR_OP, ORX_OP, MAX_OP, MIN_OP, RANGE_OP};
 
 /* Stores, for each insn code, the number of constraint alternatives.  */
 
@@ -328,7 +321,7 @@ static int *insn_alternatives;
    This is the hashed, unique string for the numeral
    whose value is chosen alternative.  */
 
-static char *current_alternative_string;
+static const char *current_alternative_string;
 
 /* Used to simplify expressions.  */
 
@@ -338,12 +331,22 @@ static rtx true_rtx, false_rtx;
 
 static char *alternative_name;
 
+/* Indicate that REG_DEAD notes are valid if dead_or_set_p is ever
+   called.  */
+
+int reload_completed = 0;
+
+/* Some machines test `optimize' in macros called from rtlanal.c, so we need
+   to define it here.  */
+
+int optimize = 0;
+
 /* Simplify an expression.  Only call the routine if there is something to
    simplify.  */
 #define SIMPLIFY_TEST_EXP(EXP,INSN_CODE,INSN_INDEX)    \
   (RTX_UNCHANGING_P (EXP) || MEM_IN_STRUCT_P (EXP) ? (EXP)     \
    : simplify_test_exp (EXP, INSN_CODE, INSN_INDEX))
-  
+
 /* Simplify (eq_attr ("alternative") ...)
    when we are working with a particular alternative.  */
 #define SIMPLIFY_ALTERNATIVE(EXP)                              \
@@ -356,99 +359,110 @@ static char *alternative_name;
 /* These are referenced by rtlanal.c and hence need to be defined somewhere.
    They won't actually be used.  */
 
-rtx frame_pointer_rtx, hard_frame_pointer_rtx, stack_pointer_rtx;
-rtx arg_pointer_rtx;
-
-static rtx attr_rtx            PVPROTO((enum rtx_code, ...));
-#ifdef HAVE_VPRINTF
-static char *attr_printf       PVPROTO((int, char *, ...));
-#else
-static char *attr_printf ();
+rtx global_rtl[GR_MAX];
+rtx pic_offset_table_rtx;
+
+static void attr_hash_add_rtx  PARAMS ((int, rtx));
+static void attr_hash_add_string PARAMS ((int, char *));
+static rtx attr_rtx            PARAMS ((enum rtx_code, ...));
+static char *attr_printf       PARAMS ((int, const char *, ...))
+  ATTRIBUTE_PRINTF_2;
+static char *attr_string        PARAMS ((const char *, int));
+static rtx check_attr_test     PARAMS ((rtx, int, int));
+static rtx check_attr_value    PARAMS ((rtx, struct attr_desc *));
+static rtx convert_set_attr_alternative PARAMS ((rtx, struct insn_def *));
+static rtx convert_set_attr    PARAMS ((rtx, struct insn_def *));
+static void check_defs         PARAMS ((void));
+#if 0
+static rtx convert_const_symbol_ref PARAMS ((rtx, struct attr_desc *));
 #endif
-
-static char *attr_string        PROTO((char *, int));
-static rtx check_attr_test     PROTO((rtx, int));
-static rtx check_attr_value    PROTO((rtx, struct attr_desc *));
-static rtx convert_set_attr_alternative PROTO((rtx, int, int, int));
-static rtx convert_set_attr    PROTO((rtx, int, int, int));
-static void check_defs         PROTO((void));
-static rtx convert_const_symbol_ref PROTO((rtx, struct attr_desc *));
-static rtx make_canonical      PROTO((struct attr_desc *, rtx));
-static struct attr_value *get_attr_value PROTO((rtx, struct attr_desc *, int));
-static rtx copy_rtx_unchanging PROTO((rtx));
-static rtx copy_boolean                PROTO((rtx));
-static void expand_delays      PROTO((void));
-static rtx operate_exp         PROTO((enum operator, rtx, rtx));
-static void expand_units       PROTO((void));
-static rtx simplify_knowing    PROTO((rtx, rtx));
-static rtx encode_units_mask   PROTO((rtx));
-static void fill_attr          PROTO((struct attr_desc *));
+static rtx make_canonical      PARAMS ((struct attr_desc *, rtx));
+static struct attr_value *get_attr_value PARAMS ((rtx, struct attr_desc *, int));
+static rtx copy_rtx_unchanging PARAMS ((rtx));
+static rtx copy_boolean                PARAMS ((rtx));
+static void expand_delays      PARAMS ((void));
+static rtx operate_exp         PARAMS ((enum operator, rtx, rtx));
+static void expand_units       PARAMS ((void));
+static rtx simplify_knowing    PARAMS ((rtx, rtx));
+static rtx encode_units_mask   PARAMS ((rtx));
+static void fill_attr          PARAMS ((struct attr_desc *));
 /* dpx2 compiler chokes if we specify the arg types of the args.  */
-static rtx substitute_address  PROTO((rtx, rtx (*) (), rtx (*) ()));
-static void make_length_attrs  PROTO((void));
-static rtx identity_fn         PROTO((rtx));
-static rtx zero_fn             PROTO((rtx));
-static rtx one_fn              PROTO((rtx));
-static rtx max_fn              PROTO((rtx));
-static rtx simplify_cond       PROTO((rtx, int, int));
-static rtx simplify_by_alternatives PROTO((rtx, int, int));
-static rtx simplify_by_exploding PROTO((rtx));
-static int find_and_mark_used_attributes PROTO((rtx, rtx *, int *));
-static void unmark_used_attributes PROTO((rtx, struct dimension *, int));
-static int add_values_to_cover PROTO((struct dimension *));
-static int increment_current_value PROTO((struct dimension *, int));
-static rtx test_for_current_value PROTO((struct dimension *, int));
-static rtx simplify_with_current_value PROTO((rtx, struct dimension *, int));
-static rtx simplify_with_current_value_aux PROTO((rtx));
-static void clear_struct_flag PROTO((rtx));
-static int count_sub_rtxs    PROTO((rtx, int));
-static void remove_insn_ent  PROTO((struct attr_value *, struct insn_ent *));
-static void insert_insn_ent  PROTO((struct attr_value *, struct insn_ent *));
-static rtx insert_right_side   PROTO((enum rtx_code, rtx, rtx, int, int));
-static rtx make_alternative_compare PROTO((int));
-static int compute_alternative_mask PROTO((rtx, enum rtx_code));
-static rtx evaluate_eq_attr    PROTO((rtx, rtx, int, int));
-static rtx simplify_and_tree   PROTO((rtx, rtx *, int, int));
-static rtx simplify_or_tree    PROTO((rtx, rtx *, int, int));
-static rtx simplify_test_exp   PROTO((rtx, int, int));
-static void optimize_attrs     PROTO((void));
-static void gen_attr           PROTO((rtx));
-static int count_alternatives  PROTO((rtx));
-static int compares_alternatives_p PROTO((rtx));
-static int contained_in_p      PROTO((rtx, rtx));
-static void gen_insn           PROTO((rtx));
-static void gen_delay          PROTO((rtx));
-static void gen_unit           PROTO((rtx));
-static void write_test_expr    PROTO((rtx, int));
-static int max_attr_value      PROTO((rtx));
-static void walk_attr_value    PROTO((rtx));
-static void write_attr_get     PROTO((struct attr_desc *));
-static rtx eliminate_known_true PROTO((rtx, rtx, int, int));
-static void write_attr_set     PROTO((struct attr_desc *, int, rtx, char *,
-                                      char *, rtx, int, int));
-static void write_attr_case    PROTO((struct attr_desc *, struct attr_value *,
-                                      int, char *, char *, int, rtx));
-static void write_attr_valueq  PROTO((struct attr_desc *, char *));
-static void write_attr_value   PROTO((struct attr_desc *, rtx));
-static void write_upcase       PROTO((char *));
-static void write_indent       PROTO((int));
-static void write_eligible_delay PROTO((char *));
-static void write_function_unit_info PROTO((void));
-static void write_complex_function PROTO((struct function_unit *, char *,
-                                         char *));
-static int n_comma_elts                PROTO((char *));
-static char *next_comma_elt    PROTO((char **));
-static struct attr_desc *find_attr PROTO((char *, int));
-static void make_internal_attr PROTO((char *, rtx, int));
-static struct attr_value *find_most_used  PROTO((struct attr_desc *));
-static rtx find_single_value   PROTO((struct attr_desc *));
-static rtx make_numeric_value  PROTO((int));
-static void extend_range       PROTO((struct range *, int, int));
-char *xrealloc                 PROTO((char *, unsigned));
-char *xmalloc                  PROTO((unsigned));
+static rtx substitute_address  PARAMS ((rtx, rtx (*) (rtx), rtx (*) (rtx)));
+static void make_length_attrs  PARAMS ((void));
+static rtx identity_fn         PARAMS ((rtx));
+static rtx zero_fn             PARAMS ((rtx));
+static rtx one_fn              PARAMS ((rtx));
+static rtx max_fn              PARAMS ((rtx));
+static void write_length_unit_log PARAMS ((void));
+static rtx simplify_cond       PARAMS ((rtx, int, int));
+#if 0
+static rtx simplify_by_alternatives PARAMS ((rtx, int, int));
+#endif
+static rtx simplify_by_exploding PARAMS ((rtx));
+static int find_and_mark_used_attributes PARAMS ((rtx, rtx *, int *));
+static void unmark_used_attributes PARAMS ((rtx, struct dimension *, int));
+static int add_values_to_cover PARAMS ((struct dimension *));
+static int increment_current_value PARAMS ((struct dimension *, int));
+static rtx test_for_current_value PARAMS ((struct dimension *, int));
+static rtx simplify_with_current_value PARAMS ((rtx, struct dimension *, int));
+static rtx simplify_with_current_value_aux PARAMS ((rtx));
+static void clear_struct_flag PARAMS ((rtx));
+static int count_sub_rtxs    PARAMS ((rtx, int));
+static void remove_insn_ent  PARAMS ((struct attr_value *, struct insn_ent *));
+static void insert_insn_ent  PARAMS ((struct attr_value *, struct insn_ent *));
+static rtx insert_right_side   PARAMS ((enum rtx_code, rtx, rtx, int, int));
+static rtx make_alternative_compare PARAMS ((int));
+static int compute_alternative_mask PARAMS ((rtx, enum rtx_code));
+static rtx evaluate_eq_attr    PARAMS ((rtx, rtx, int, int));
+static rtx simplify_and_tree   PARAMS ((rtx, rtx *, int, int));
+static rtx simplify_or_tree    PARAMS ((rtx, rtx *, int, int));
+static rtx simplify_test_exp   PARAMS ((rtx, int, int));
+static void optimize_attrs     PARAMS ((void));
+static void gen_attr           PARAMS ((rtx, int));
+static int count_alternatives  PARAMS ((rtx));
+static int compares_alternatives_p PARAMS ((rtx));
+static int contained_in_p      PARAMS ((rtx, rtx));
+static void gen_insn           PARAMS ((rtx, int));
+static void gen_delay          PARAMS ((rtx, int));
+static void gen_unit           PARAMS ((rtx, int));
+static void write_test_expr    PARAMS ((rtx, int));
+static int max_attr_value      PARAMS ((rtx, int*));
+static int or_attr_value       PARAMS ((rtx, int*));
+static void walk_attr_value    PARAMS ((rtx));
+static void write_attr_get     PARAMS ((struct attr_desc *));
+static rtx eliminate_known_true PARAMS ((rtx, rtx, int, int));
+static void write_attr_set     PARAMS ((struct attr_desc *, int, rtx,
+                                      const char *, const char *, rtx,
+                                      int, int));
+static void write_attr_case    PARAMS ((struct attr_desc *, struct attr_value *,
+                                      int, const char *, const char *, int, rtx));
+static void write_unit_name    PARAMS ((const char *, int, const char *));
+static void write_attr_valueq  PARAMS ((struct attr_desc *, const char *));
+static void write_attr_value   PARAMS ((struct attr_desc *, rtx));
+static void write_upcase       PARAMS ((const char *));
+static void write_indent       PARAMS ((int));
+static void write_eligible_delay PARAMS ((const char *));
+static void write_function_unit_info PARAMS ((void));
+static void write_complex_function PARAMS ((struct function_unit *, const char *,
+                                         const char *));
+static int write_expr_attr_cache PARAMS ((rtx, struct attr_desc *));
+static void write_toplevel_expr        PARAMS ((rtx));
+static void write_const_num_delay_slots PARAMS ((void));
+static int n_comma_elts                PARAMS ((const char *));
+static char *next_comma_elt    PARAMS ((const char **));
+static struct attr_desc *find_attr PARAMS ((const char *, int));
+static void make_internal_attr PARAMS ((const char *, rtx, int));
+static struct attr_value *find_most_used  PARAMS ((struct attr_desc *));
+static rtx find_single_value   PARAMS ((struct attr_desc *));
+static rtx make_numeric_value  PARAMS ((int));
+static void extend_range       PARAMS ((struct range *, int, int));
+static rtx attr_eq             PARAMS ((const char *, const char *));
+static const char *attr_numeral        PARAMS ((int));
+static int attr_equal_p                PARAMS ((rtx, rtx));
+static rtx attr_copy_rtx       PARAMS ((rtx));
+static int attr_rtx_cost       PARAMS ((rtx));
 
 #define oballoc(size) obstack_alloc (hash_obstack, size)
-
 \f
 /* Hash table for sharing RTL and strings.  */
 
@@ -479,7 +493,7 @@ struct attr_hash *attr_hash_table[RTL_HASH_SIZE];
 
 /* Here is how primitive or already-shared RTL's hash
    codes are made.  */
-#define RTL_HASH(RTL) ((HOST_WIDE_INT) (RTL) & 0777777)
+#define RTL_HASH(RTL) ((long) (RTL) & 0777777)
 
 /* Add an entry to the hash table for RTL with hash code HASHCODE.  */
 
@@ -525,24 +539,23 @@ attr_hash_add_string (hashcode, str)
 
    rtx attr_rtx (code, [element1, ..., elementn])  */
 
-/*VARARGS1*/
 static rtx
-attr_rtx VPROTO((enum rtx_code code, ...))
+attr_rtx VPARAMS ((enum rtx_code code, ...))
 {
-#ifndef __STDC__
+#ifndef ANSI_PROTOTYPES
   enum rtx_code code;
 #endif
   va_list p;
   register int i;              /* Array indices...                     */
-  register char *fmt;          /* Current rtx's format...              */
-  register rtx rt_val;         /* RTX to return to caller...           */
+  register const char *fmt;    /* Current rtx's format...              */
+  register rtx rt_val = NULL_RTX;/* RTX to return to caller...         */
   int hashcode;
   register struct attr_hash *h;
   struct obstack *old_obstack = rtl_obstack;
 
   VA_START (p, code);
 
-#ifndef __STDC__
+#ifndef ANSI_PROTOTYPES
   code = va_arg (p, enum rtx_code);
 #endif
 
@@ -613,7 +626,7 @@ attr_rtx VPROTO((enum rtx_code code, ...))
   else if (GET_RTX_LENGTH (code) == 1
           && GET_RTX_FORMAT (code)[0] == 's')
     {
-      char * arg0 = va_arg (p, char *);
+      char *arg0 = va_arg (p, char *);
 
       if (code == SYMBOL_REF)
        arg0 = attr_string (arg0, strlen (arg0));
@@ -659,16 +672,22 @@ attr_rtx VPROTO((enum rtx_code code, ...))
     {
       HOST_WIDE_INT arg0 = va_arg (p, HOST_WIDE_INT);
       if (arg0 == 0)
-       return false_rtx;
+       {
+         va_end (p);
+         return false_rtx;
+       }
       if (arg0 == 1)
-       return true_rtx;
+       {
+         va_end (p);
+         return true_rtx;
+       }
       goto nohash;
     }
   else
     {
     nohash:
       rt_val = rtx_alloc (code);       /* Allocate the storage space.  */
-      
+
       fmt = GET_RTX_FORMAT (code);     /* Find the right format...  */
       for (i = 0; i < GET_RTX_LENGTH (code); i++)
        {
@@ -699,7 +718,7 @@ attr_rtx VPROTO((enum rtx_code code, ...))
              break;
 
            default:
-             abort();
+             abort ();
            }
        }
       va_end (p);
@@ -722,24 +741,21 @@ attr_rtx VPROTO((enum rtx_code code, ...))
 
    rtx attr_printf (len, format, [arg1, ..., argn])  */
 
-#ifdef HAVE_VPRINTF
-
-/*VARARGS2*/
 static char *
-attr_printf VPROTO((register int len, char *fmt, ...))
+attr_printf VPARAMS ((register int len, const char *fmt, ...))
 {
-#ifndef __STDC__
+#ifndef ANSI_PROTOTYPES
   register int len;
-  char *fmt;
+  const char *fmt;
 #endif
   va_list p;
   register char *str;
 
   VA_START (p, fmt);
 
-#ifndef __STDC__
+#ifndef ANSI_PROTOTYPES
   len = va_arg (p, int);
-  fmt = va_arg (p, char*);
+  fmt = va_arg (p, const char *);
 #endif
 
   /* Print the string into a temporary location.  */
@@ -750,33 +766,15 @@ attr_printf VPROTO((register int len, char *fmt, ...))
   return attr_string (str, strlen (str));
 }
 
-#else /* not HAVE_VPRINTF */
-
-static char *
-attr_printf (len, fmt, arg1, arg2, arg3)
-     int len;
-     char *fmt;
-     char *arg1, *arg2, *arg3; /* also int */
-{
-  register char *str;
-
-  /* Print the string into a temporary location.  */
-  str = (char *) alloca (len);
-  sprintf (str, fmt, arg1, arg2, arg3);
-
-  return attr_string (str, strlen (str));
-}
-#endif /* not HAVE_VPRINTF */
-
-rtx
+static rtx
 attr_eq (name, value)
-     char *name, *value;
+     const char *name, *value;
 {
   return attr_rtx (EQ_ATTR, attr_string (name, strlen (name)),
                   attr_string (value, strlen (value)));
 }
 
-char *
+static const char *
 attr_numeral (n)
      int n;
 {
@@ -788,7 +786,7 @@ attr_numeral (n)
 
 static char *
 attr_string (str, len)
-     char *str;
+     const char *str;
      int len;
 {
   register struct attr_hash *h;
@@ -797,9 +795,9 @@ attr_string (str, len)
   register char *new_str;
 
   /* Compute the hash code.  */
-  hashcode = (len + 1) * 613 + (unsigned)str[0];
+  hashcode = (len + 1) * 613 + (unsigned) str[0];
   for (i = 1; i <= len; i += 2)
-    hashcode = ((hashcode * 613) + (unsigned)str[i]);
+    hashcode = ((hashcode * 613) + (unsigned) str[i]);
   if (hashcode < 0)
     hashcode = -hashcode;
 
@@ -811,7 +809,7 @@ attr_string (str, len)
 
   /* Not found; create a permanent copy and add it to the hash table.  */
   new_str = (char *) obstack_alloc (hash_obstack, len + 1);
-  bcopy (str, new_str, len);
+  memcpy (new_str, str, len);
   new_str[len] = '\0';
   attr_hash_add_string (hashcode, new_str);
 
@@ -822,7 +820,7 @@ attr_string (str, len)
    taking advantage of the fact that if both are hashed
    then they can't be equal unless they are the same object.  */
 
-int
+static int
 attr_equal_p (x, y)
      rtx x, y;
 {
@@ -834,14 +832,14 @@ attr_equal_p (x, y)
    descending to all depths, but not copying any
    permanent hashed subexpressions.  */
 
-rtx
+static rtx
 attr_copy_rtx (orig)
      register rtx orig;
 {
   register rtx copy;
   register int i, j;
   register RTX_CODE code;
-  register char *format_ptr;
+  register const char *format_ptr;
 
   /* No need to copy a permanent object.  */
   if (RTX_INTEGRATED_P (orig))
@@ -860,6 +858,9 @@ attr_copy_rtx (orig)
     case PC:
     case CC0:
       return orig;
+
+    default:
+      break;
     }
 
   copy = rtx_alloc (code);
@@ -868,7 +869,7 @@ attr_copy_rtx (orig)
   copy->volatil = orig->volatil;
   copy->unchanging = orig->unchanging;
   copy->integrated = orig->integrated;
-  
+
   format_ptr = GET_RTX_FORMAT (GET_CODE (copy));
 
   for (i = 0; i < GET_RTX_LENGTH (GET_CODE (copy)); i++)
@@ -928,13 +929,14 @@ attr_copy_rtx (orig)
    Return the new expression, if any.   */
 
 static rtx
-check_attr_test (exp, is_const)
+check_attr_test (exp, is_const, lineno)
      rtx exp;
      int is_const;
+     int lineno;
 {
   struct attr_desc *attr;
   struct attr_value *av;
-  char *name_ptr, *p;
+  const char *name_ptr, *p;
   rtx orexp, newexp;
 
   switch (GET_CODE (exp))
@@ -945,7 +947,7 @@ check_attr_test (exp, is_const)
        return check_attr_test (attr_rtx (NOT,
                                          attr_eq (XSTR (exp, 0),
                                                   &XSTR (exp, 1)[1])),
-                               is_const);
+                               is_const, lineno);
 
       else if (n_comma_elts (XSTR (exp, 1)) == 1)
        {
@@ -960,12 +962,12 @@ check_attr_test (exp, is_const)
                  return exp;
                }
              else
-               fatal ("Unknown attribute `%s' in EQ_ATTR", XEXP (exp, 0));
+               fatal ("Unknown attribute `%s' in EQ_ATTR", XSTR (exp, 0));
            }
 
          if (is_const && ! attr->is_const)
            fatal ("Constant expression uses insn attribute `%s' in EQ_ATTR",
-                  XEXP (exp, 0));
+                  XSTR (exp, 0));
 
          /* Copy this just to make it permanent,
             so expressions using it can be permanent too.  */
@@ -973,7 +975,7 @@ check_attr_test (exp, is_const)
 
          /* It shouldn't be possible to simplify the value given to a
             constant attribute, so don't expand this until it's time to
-            write the test expression.  */            
+            write the test expression.  */
          if (attr->is_const)
            RTX_UNCHANGING_P (exp) = 1;
 
@@ -981,8 +983,8 @@ check_attr_test (exp, is_const)
            {
              for (p = XSTR (exp, 1); *p; p++)
                if (*p < '0' || *p > '9')
-                  fatal ("Attribute `%s' takes only numeric values", 
-                         XEXP (exp, 0));
+                 fatal ("Attribute `%s' takes only numeric values",
+                        XSTR (exp, 0));
            }
          else
            {
@@ -993,7 +995,7 @@ check_attr_test (exp, is_const)
 
              if (av == NULL)
                fatal ("Unknown value `%s' for `%s' attribute",
-                      XEXP (exp, 1), XEXP (exp, 0));
+                      XSTR (exp, 1), XSTR (exp, 0));
            }
        }
       else
@@ -1007,7 +1009,7 @@ check_attr_test (exp, is_const)
              orexp = insert_right_side (IOR, orexp, newexp, -2, -2);
            }
 
-         return check_attr_test (orexp, is_const);
+         return check_attr_test (orexp, is_const, lineno);
        }
       break;
 
@@ -1023,18 +1025,19 @@ check_attr_test (exp, is_const)
 
     case IOR:
     case AND:
-      XEXP (exp, 0) = check_attr_test (XEXP (exp, 0), is_const);
-      XEXP (exp, 1) = check_attr_test (XEXP (exp, 1), is_const);
+      XEXP (exp, 0) = check_attr_test (XEXP (exp, 0), is_const, lineno);
+      XEXP (exp, 1) = check_attr_test (XEXP (exp, 1), is_const, lineno);
       break;
 
     case NOT:
-      XEXP (exp, 0) = check_attr_test (XEXP (exp, 0), is_const);
+      XEXP (exp, 0) = check_attr_test (XEXP (exp, 0), is_const, lineno);
       break;
 
+    case MATCH_INSN:
     case MATCH_OPERAND:
       if (is_const)
        fatal ("RTL operator \"%s\" not valid in constant attribute test",
-              GET_RTX_NAME (MATCH_OPERAND));
+              GET_RTX_NAME (GET_CODE (exp)));
       /* These cases can't be simplified.  */
       RTX_UNCHANGING_P (exp) = 1;
       break;
@@ -1080,20 +1083,29 @@ check_attr_value (exp, attr)
      struct attr_desc *attr;
 {
   struct attr_value *av;
-  char *p;
+  const char *p;
   int i;
 
   switch (GET_CODE (exp))
     {
     case CONST_INT:
       if (attr && ! attr->is_numeric)
-       fatal ("CONST_INT not valid for non-numeric `%s' attribute",
-              attr->name);
-
-      if (INTVAL (exp) < 0)
-       fatal ("Negative numeric value specified for `%s' attribute",
-              attr->name);
+       {
+         message_with_line (attr->lineno,
+                            "CONST_INT not valid for non-numeric attribute %s",
+                            attr->name);
+         have_error = 1;
+         break;
+       }
 
+      if (INTVAL (exp) < 0 && ! attr->negative_ok)
+       {
+         message_with_line (attr->lineno,
+                            "negative numeric value specified for attribute %s",
+                            attr->name);
+         have_error = 1;
+         break;
+       }
       break;
 
     case CONST_STRING:
@@ -1107,8 +1119,13 @@ check_attr_value (exp, attr)
            p++;
          for (; *p; p++)
            if (*p > '9' || *p < '0')
-             fatal ("Non-numeric value for numeric `%s' attribute",
-                    attr ? attr->name : "internal");
+             {
+               message_with_line (attr ? attr->lineno : 0,
+                                  "non-numeric value for numeric attribute %s",
+                                  attr ? attr->name : "internal");
+               have_error = 1;
+               break;
+             }
          break;
        }
 
@@ -1118,26 +1135,61 @@ check_attr_value (exp, attr)
          break;
 
       if (av == NULL)
-       fatal ("Unknown value `%s' for `%s' attribute",
-              XSTR (exp, 0), attr ? attr->name : "internal");
-
+       {
+         message_with_line (attr->lineno,
+                            "unknown value `%s' for `%s' attribute",
+                            XSTR (exp, 0), attr ? attr->name : "internal");
+         have_error = 1;
+       }
       break;
 
     case IF_THEN_ELSE:
       XEXP (exp, 0) = check_attr_test (XEXP (exp, 0),
-                                      attr ? attr->is_const : 0);
+                                      attr ? attr->is_const : 0,
+                                      attr ? attr->lineno : 0);
       XEXP (exp, 1) = check_attr_value (XEXP (exp, 1), attr);
       XEXP (exp, 2) = check_attr_value (XEXP (exp, 2), attr);
       break;
 
+    case PLUS:
+    case MINUS:
+    case MULT:
+    case DIV:
+    case MOD:
+      if (attr && !attr->is_numeric)
+       {
+         message_with_line (attr->lineno,
+                            "invalid operation `%s' for non-numeric attribute value",
+                            GET_RTX_NAME (GET_CODE (exp)));
+         have_error = 1;
+         break;
+       }
+      /* FALLTHRU */
+
+    case IOR:
+    case AND:
+      XEXP (exp, 0) = check_attr_value (XEXP (exp, 0), attr);
+      XEXP (exp, 1) = check_attr_value (XEXP (exp, 1), attr);
+      break;
+
+    case FFS:
+      XEXP (exp, 0) = check_attr_value (XEXP (exp, 0), attr);
+      break;
+
     case COND:
       if (XVECLEN (exp, 0) % 2 != 0)
-       fatal ("First operand of COND must have even length");
+       {
+         message_with_line (attr->lineno,
+                            "first operand of COND must have even length");
+         have_error = 1;
+         break;
+       }
 
       for (i = 0; i < XVECLEN (exp, 0); i += 2)
        {
          XVECEXP (exp, 0, i) = check_attr_test (XVECEXP (exp, 0, i),
-                                                attr ? attr->is_const : 0);
+                                                attr ? attr->is_const : 0,
+                                                attr ? attr->lineno : 0);
          XVECEXP (exp, 0, i + 1)
            = check_attr_value (XVECEXP (exp, 0, i + 1), attr);
        }
@@ -1145,16 +1197,47 @@ check_attr_value (exp, attr)
       XEXP (exp, 1) = check_attr_value (XEXP (exp, 1), attr);
       break;
 
+    case ATTR:
+      {
+       struct attr_desc *attr2 = find_attr (XSTR (exp, 0), 0);
+       if (attr2 == NULL)
+         {
+           message_with_line (attr ? attr->lineno : 0,
+                              "unknown attribute `%s' in ATTR",
+                              XSTR (exp, 0));
+           have_error = 1;
+         }
+       else if (attr && attr->is_const && ! attr2->is_const)
+         {
+           message_with_line (attr->lineno,
+               "non-constant attribute `%s' referenced from `%s'",
+               XSTR (exp, 0), attr->name);
+           have_error = 1;
+         }
+       else if (attr
+                && (attr->is_numeric != attr2->is_numeric
+                    || (! attr->negative_ok && attr2->negative_ok)))
+         {
+           message_with_line (attr->lineno,
+               "numeric attribute mismatch calling `%s' from `%s'",
+               XSTR (exp, 0), attr->name);
+           have_error = 1;
+         }
+      }
+      break;
+
     case SYMBOL_REF:
-      if (attr && attr->is_const)
-       /* A constant SYMBOL_REF is valid as a constant attribute test and
-          is expanded later by make_canonical into a COND.  */
-       return attr_rtx (SYMBOL_REF, XSTR (exp, 0));
-      /* Otherwise, fall through... */
+      /* A constant SYMBOL_REF is valid as a constant attribute test and
+         is expanded later by make_canonical into a COND.  In a non-constant
+         attribute test, it is left be.  */
+      return attr_rtx (SYMBOL_REF, XSTR (exp, 0));
 
     default:
-      fatal ("Illegal operation `%s' for attribute value",
-            GET_RTX_NAME (GET_CODE (exp)));
+      message_with_line (attr ? attr->lineno : 0,
+                        "invalid operation `%s' for attribute value",
+                        GET_RTX_NAME (GET_CODE (exp)));
+      have_error = 1;
+      break;
     }
 
   return exp;
@@ -1164,17 +1247,21 @@ check_attr_value (exp, attr)
    It becomes a COND with each test being (eq_attr "alternative "n") */
 
 static rtx
-convert_set_attr_alternative (exp, num_alt, insn_code, insn_index)
+convert_set_attr_alternative (exp, id)
      rtx exp;
-     int num_alt;
-     int insn_code, insn_index;
+     struct insn_def *id;
 {
+  int num_alt = id->num_alternatives;
   rtx condexp;
   int i;
 
   if (XVECLEN (exp, 1) != num_alt)
-    fatal ("Bad number of entries in SET_ATTR_ALTERNATIVE for insn %d",
-          insn_index);
+    {
+      message_with_line (id->lineno,
+                        "bad number of entries in SET_ATTR_ALTERNATIVE");
+      have_error = 1;
+      return NULL_RTX;
+    }
 
   /* Make a COND with all tests but the last.  Select the last value via the
      default.  */
@@ -1183,16 +1270,10 @@ convert_set_attr_alternative (exp, num_alt, insn_code, insn_index)
 
   for (i = 0; i < num_alt - 1; i++)
     {
-      char *p;
+      const char *p;
       p = attr_numeral (i);
 
       XVECEXP (condexp, 0, 2 * i) = attr_eq (alternative_name, p);
-#if 0
-      /* Sharing this EQ_ATTR rtl causes trouble.  */   
-      XVECEXP (condexp, 0, 2 * i) = rtx_alloc (EQ_ATTR);
-      XSTR (XVECEXP (condexp, 0, 2 * i), 0) = alternative_name;
-      XSTR (XVECEXP (condexp, 0, 2 * i), 1) = p;
-#endif
       XVECEXP (condexp, 0, 2 * i + 1) = XVECEXP (exp, 1, i);
     }
 
@@ -1205,13 +1286,12 @@ convert_set_attr_alternative (exp, num_alt, insn_code, insn_index)
    list of values is given, convert to SET_ATTR_ALTERNATIVE first.  */
 
 static rtx
-convert_set_attr (exp, num_alt, insn_code, insn_index)
+convert_set_attr (exp, id)
      rtx exp;
-     int num_alt;
-     int insn_code, insn_index;
+     struct insn_def *id;
 {
   rtx newexp;
-  char *name_ptr;
+  const char *name_ptr;
   char *p;
   int n;
 
@@ -1232,12 +1312,12 @@ convert_set_attr (exp, num_alt, insn_code, insn_index)
   while ((p = next_comma_elt (&name_ptr)) != NULL)
     XVECEXP (newexp, 1, n++) = attr_rtx (CONST_STRING, p);
 
-  return convert_set_attr_alternative (newexp, num_alt, insn_code, insn_index);
+  return convert_set_attr_alternative (newexp, id);
 }
 \f
 /* Scan all definitions, checking for validity.  Also, convert any SET_ATTR
    and SET_ATTR_ALTERNATIVE expressions to the corresponding SET
-   expressions. */
+   expressions.  */
 
 static void
 check_defs ()
@@ -1259,29 +1339,37 @@ check_defs ()
            {
            case SET:
              if (GET_CODE (XEXP (value, 0)) != ATTR)
-               fatal ("Bad attribute set in pattern %d", id->insn_index);
+               {
+                 message_with_line (id->lineno, "bad attribute set");
+                 have_error = 1;
+                 value = NULL_RTX;
+               }
              break;
 
            case SET_ATTR_ALTERNATIVE:
-             value = convert_set_attr_alternative (value,
-                                                   id->num_alternatives,
-                                                   id->insn_code,
-                                                   id->insn_index);
+             value = convert_set_attr_alternative (value, id);
              break;
 
            case SET_ATTR:
-             value = convert_set_attr (value, id->num_alternatives,
-                                       id->insn_code, id->insn_index);
+             value = convert_set_attr (value, id);
              break;
 
            default:
-             fatal ("Invalid attribute code `%s' for pattern %d",
-                    GET_RTX_NAME (GET_CODE (value)), id->insn_index);
+             message_with_line (id->lineno, "invalid attribute code %s",
+                                GET_RTX_NAME (GET_CODE (value)));
+             have_error = 1;
+             value = NULL_RTX;
            }
+         if (value == NULL_RTX)
+           continue;
 
          if ((attr = find_attr (XSTR (XEXP (value, 0), 0), 0)) == NULL)
-           fatal ("Unknown attribute `%s' for pattern number %d",
-                  XSTR (XEXP (value, 0), 0), id->insn_index);
+           {
+             message_with_line (id->lineno, "unknown attribute %s",
+                                XSTR (XEXP (value, 0), 0));
+             have_error = 1;
+             continue;
+           }
 
          XVECEXP (id->def, id->vec_idx, i) = value;
          XEXP (value, 1) = check_attr_value (XEXP (value, 1), attr);
@@ -1289,6 +1377,7 @@ check_defs ()
     }
 }
 \f
+#if 0
 /* Given a constant SYMBOL_REF expression, convert to a COND that
    explicitly tests each enumerated value.  */
 
@@ -1326,12 +1415,11 @@ convert_const_symbol_ref (exp, attr)
       strcat (p, "_");
       strcat (p, XSTR (av->value, 0));
       for (; *p != '\0'; p++)
-       if (*p >= 'a' && *p <= 'z')
-         *p -= 'a' - 'A';
+       *p = TOUPPER (*p);
 
       value = attr_rtx (SYMBOL_REF, string);
       RTX_UNCHANGING_P (value) = 1;
-      
+
       XVECEXP (condexp, 0, 2 * i) = attr_rtx (EQ, exp, value);
 
       XVECEXP (condexp, 0, 2 * i + 1) = av->value;
@@ -1339,6 +1427,7 @@ convert_const_symbol_ref (exp, attr)
 
   return condexp;
 }
+#endif
 \f
 /* Given a valid expression for an attribute value, remove any IF_THEN_ELSE
    expressions by converting them into a COND.  This removes cases from this
@@ -1376,13 +1465,21 @@ make_canonical (attr, exp)
         This makes the COND something that won't be considered an arbitrary
         expression by walk_attr_value.  */
       RTX_UNCHANGING_P (exp) = 1;
+#if 0
+      /* ??? Why do we do this?  With attribute values { A B C D E }, this
+         tends to generate (!(x==A) && !(x==B) && !(x==C) && !(x==D)) rather
+        than (x==E).  */
       exp = convert_const_symbol_ref (exp, attr);
       RTX_UNCHANGING_P (exp) = 1;
       exp = check_attr_value (exp, attr);
       /* Goto COND case since this is now a COND.  Note that while the
-         new expression is rescanned, all symbol_ref notes are mared as
+         new expression is rescanned, all symbol_ref notes are marked as
         unchanging.  */
       goto cond;
+#else
+      exp = check_attr_value (exp, attr);
+      break;
+#endif
 
     case IF_THEN_ELSE:
       newexp = rtx_alloc (COND);
@@ -1396,12 +1493,11 @@ make_canonical (attr, exp)
       /* Fall through to COND case since this is now a COND.  */
 
     case COND:
-    cond:
       {
        int allsame = 1;
        rtx defval;
 
-       /* First, check for degenerate COND. */
+       /* First, check for degenerate COND.  */
        if (XVECLEN (exp, 0) == 0)
          return make_canonical (attr, XEXP (exp, 1));
        defval = XEXP (exp, 1) = make_canonical (attr, XEXP (exp, 1));
@@ -1416,8 +1512,11 @@ make_canonical (attr, exp)
          }
        if (allsame)
          return defval;
-       break;
       }
+      break;
+
+    default:
+      break;
     }
 
   return exp;
@@ -1534,44 +1633,46 @@ expand_delays ()
   /* For each delay possibility and delay slot, compute an eligibility
      attribute for non-annulled insns and for each type of annulled (annul
      if true and annul if false).  */
- for (delay = delays; delay; delay = delay->next)
-   {
-     for (i = 0; i < XVECLEN (delay->def, 1); i += 3)
-       {
-        condexp = XVECEXP (delay->def, 1, i);
-        if (condexp == 0) condexp = false_rtx;
-        newexp = attr_rtx (IF_THEN_ELSE, condexp,
-                           make_numeric_value (1), make_numeric_value (0));
-
-        p = attr_printf (sizeof ("*delay__") + MAX_DIGITS*2, "*delay_%d_%d",
-                         delay->num, i / 3);
-        make_internal_attr (p, newexp, 1);
-
-        if (have_annul_true)
-          {
-            condexp = XVECEXP (delay->def, 1, i + 1);
-            if (condexp == 0) condexp = false_rtx;
-            newexp = attr_rtx (IF_THEN_ELSE, condexp,
-                               make_numeric_value (1),
-                               make_numeric_value (0));
-            p = attr_printf (sizeof ("*annul_true__") + MAX_DIGITS*2,
-                             "*annul_true_%d_%d", delay->num, i / 3);
-            make_internal_attr (p, newexp, 1);
-          }
-
-        if (have_annul_false)
-          {
-            condexp = XVECEXP (delay->def, 1, i + 2);
-            if (condexp == 0) condexp = false_rtx;
-            newexp = attr_rtx (IF_THEN_ELSE, condexp,
-                               make_numeric_value (1),
-                               make_numeric_value (0));
-            p = attr_printf (sizeof ("*annul_false__") + MAX_DIGITS*2,
-                             "*annul_false_%d_%d", delay->num, i / 3);
-            make_internal_attr (p, newexp, 1);
-          }
-       }
-   }
+  for (delay = delays; delay; delay = delay->next)
+    {
+      for (i = 0; i < XVECLEN (delay->def, 1); i += 3)
+       {
+         condexp = XVECEXP (delay->def, 1, i);
+         if (condexp == 0)
+           condexp = false_rtx;
+         newexp = attr_rtx (IF_THEN_ELSE, condexp,
+                            make_numeric_value (1), make_numeric_value (0));
+
+         p = attr_printf (sizeof ("*delay__") + MAX_DIGITS * 2,
+                          "*delay_%d_%d",
+                          delay->num, i / 3);
+         make_internal_attr (p, newexp, 1);
+
+         if (have_annul_true)
+           {
+             condexp = XVECEXP (delay->def, 1, i + 1);
+             if (condexp == 0) condexp = false_rtx;
+             newexp = attr_rtx (IF_THEN_ELSE, condexp,
+                                make_numeric_value (1),
+                                make_numeric_value (0));
+             p = attr_printf (sizeof ("*annul_true__") + MAX_DIGITS * 2,
+                              "*annul_true_%d_%d", delay->num, i / 3);
+             make_internal_attr (p, newexp, 1);
+           }
+
+         if (have_annul_false)
+           {
+             condexp = XVECEXP (delay->def, 1, i + 2);
+             if (condexp == 0) condexp = false_rtx;
+             newexp = attr_rtx (IF_THEN_ELSE, condexp,
+                                make_numeric_value (1),
+                                make_numeric_value (0));
+             p = attr_printf (sizeof ("*annul_false__") + MAX_DIGITS * 2,
+                              "*annul_false_%d_%d", delay->num, i / 3);
+             make_internal_attr (p, newexp, 1);
+           }
+       }
+    }
 }
 \f
 /* This function is given a left and right side expression and an operator.
@@ -1617,6 +1718,7 @@ operate_exp (op, left, right)
              break;
 
            case OR_OP:
+           case ORX_OP:
              i = left_value | right_value;
              break;
 
@@ -1646,6 +1748,10 @@ operate_exp (op, left, right)
              abort ();
            }
 
+         if (i == left_value)
+           return left;
+         if (i == right_value)
+           return right;
          return make_numeric_value (i);
        }
       else if (GET_CODE (right) == IF_THEN_ELSE)
@@ -1672,25 +1778,14 @@ operate_exp (op, left, right)
              XVECEXP (newexp, 0, i + 1)
                = operate_exp (op, left, XVECEXP (right, 0, i + 1));
              if (! rtx_equal_p (XVECEXP (newexp, 0, i + 1),
-                                defval))     
+                                defval))
                allsame = 0;
            }
 
          /* If the resulting cond is trivial (all alternatives
             give the same value), optimize it away.  */
          if (allsame)
-           {
-             obstack_free (rtl_obstack, newexp);
-             return operate_exp (op, left, XEXP (right, 1));
-           }
-
-         /* If the result is the same as the RIGHT operand,
-            just use that.  */
-         if (rtx_equal_p (newexp, right))
-           {
-             obstack_free (rtl_obstack, newexp);
-             return right;
-           }
+           return operate_exp (op, left, XEXP (right, 1));
 
          return newexp;
        }
@@ -1698,6 +1793,13 @@ operate_exp (op, left, right)
        fatal ("Badly formed attribute value");
     }
 
+  /* A hack to prevent expand_units from completely blowing up: ORX_OP does
+     not associate through IF_THEN_ELSE.  */
+  else if (op == ORX_OP && GET_CODE (right) == IF_THEN_ELSE)
+    {
+      return attr_rtx (IOR, left, right);
+    }
+
   /* Otherwise, do recursion the other way.  */
   else if (GET_CODE (left) == IF_THEN_ELSE)
     {
@@ -1722,25 +1824,19 @@ operate_exp (op, left, right)
          XVECEXP (newexp, 0, i + 1)
            = operate_exp (op, XVECEXP (left, 0, i + 1), right);
          if (! rtx_equal_p (XVECEXP (newexp, 0, i + 1),
-                            defval))     
+                            defval))
            allsame = 0;
        }
 
       /* If the cond is trivial (all alternatives give the same value),
         optimize it away.  */
       if (allsame)
-       {
-         obstack_free (rtl_obstack, newexp);
-         return operate_exp (op, XEXP (left, 1), right);
-       }
+       return operate_exp (op, XEXP (left, 1), right);
 
       /* If the result is the same as the LEFT operand,
         just use that.  */
       if (rtx_equal_p (newexp, left))
-       {
-         obstack_free (rtl_obstack, newexp);
-         return left;
-       }
+       return left;
 
       return newexp;
     }
@@ -1791,7 +1887,7 @@ expand_units ()
   rtx unitsmask;
   rtx readycost;
   rtx newexp;
-  char *str;
+  const char *str;
   int i, j, u, num, nvalues;
 
   /* Rebuild the condition for the unit to share the RTL expressions.
@@ -1802,7 +1898,7 @@ expand_units ()
 
   for (unit = units; unit; unit = unit->next)
     {
-      unit->condexp = check_attr_test (unit->condexp, 0);
+      unit->condexp = check_attr_test (unit->condexp, 0, unit->first_lineno);
 
       for (op = unit->ops; op; op = op->next)
        {
@@ -1830,7 +1926,7 @@ expand_units ()
            }
 
          /* Validate the condition.  */
-         op->condexp = check_attr_test (op->condexp, 0);
+         op->condexp = check_attr_test (op->condexp, 0, op->lineno);
        }
     }
 
@@ -1840,19 +1936,49 @@ expand_units ()
   newexp = rtx_alloc (IF_THEN_ELSE);
   XEXP (newexp, 2) = make_numeric_value (0);
 
-  /* Merge each function unit into the unit mask attributes.  */
-  for (unit = units; unit; unit = unit->next)
+  /* If we have just a few units, we may be all right expanding the whole
+     thing.  But the expansion is 2**N in space on the number of opclasses,
+     so we can't do this for very long -- Alpha and MIPS in particular have
+     problems with this.  So in that situation, we fall back on an alternate
+     implementation method.  */
+#define NUM_UNITOP_CUTOFF 20
+
+  if (num_unit_opclasses < NUM_UNITOP_CUTOFF)
+    {
+      /* Merge each function unit into the unit mask attributes.  */
+      for (unit = units; unit; unit = unit->next)
+       {
+         XEXP (newexp, 0) = unit->condexp;
+         XEXP (newexp, 1) = make_numeric_value (1 << unit->num);
+         unitsmask = operate_exp (OR_OP, unitsmask, newexp);
+       }
+    }
+  else
     {
-      XEXP (newexp, 0) = unit->condexp;
-      XEXP (newexp, 1) = make_numeric_value (1 << unit->num);
-      unitsmask = operate_exp (OR_OP, unitsmask, newexp);
+      /* Merge each function unit into the unit mask attributes.  */
+      for (unit = units; unit; unit = unit->next)
+       {
+         XEXP (newexp, 0) = unit->condexp;
+         XEXP (newexp, 1) = make_numeric_value (1 << unit->num);
+         unitsmask = operate_exp (ORX_OP, unitsmask, attr_copy_rtx (newexp));
+       }
     }
 
   /* Simplify the unit mask expression, encode it, and make an attribute
      for the function_units_used function.  */
   unitsmask = simplify_by_exploding (unitsmask);
-  unitsmask = encode_units_mask (unitsmask);
-  make_internal_attr ("*function_units_used", unitsmask, 2);
+
+  if (num_unit_opclasses < NUM_UNITOP_CUTOFF)
+    unitsmask = encode_units_mask (unitsmask);
+  else
+    {
+      /* We can no longer encode unitsmask at compile time, so emit code to
+         calculate it at runtime.  Rather, put a marker for where we'd do
+        the code, and actually output it in write_attr_get().  */
+      unitsmask = attr_rtx (FFS, unitsmask);
+    }
+
+  make_internal_attr ("*function_units_used", unitsmask, 10);
 
   /* Create an array of ops for each unit.  Add an extra unit for the
      result_ready_cost function that has the ops of all other units.  */
@@ -1900,11 +2026,11 @@ expand_units ()
       /* Sort the array of ops into increasing ready cost order.  */
       for (i = 0; i < num; i++)
        for (j = num - 1; j > i; j--)
-         if (op_array[j-1]->ready < op_array[j]->ready)
+         if (op_array[j - 1]->ready < op_array[j]->ready)
            {
              op = op_array[j];
-             op_array[j] = op_array[j-1];
-             op_array[j-1] = op;
+             op_array[j] = op_array[j - 1];
+             op_array[j - 1] = op;
            }
 
       /* Determine how many distinct non-default ready cost values there
@@ -1927,7 +2053,9 @@ expand_units ()
          XVEC (readycost, 0) = rtvec_alloc (nvalues * 2);
          XEXP (readycost, 1) = make_numeric_value (1);
 
-         nvalues = 0; orexp = false_rtx; value = op_array[0]->ready;
+         nvalues = 0;
+         orexp = false_rtx;
+         value = op_array[0]->ready;
          for (i = 0; i < num; i++)
            {
              op = op_array[i];
@@ -2001,21 +2129,7 @@ expand_units ()
 
          for (op = unit->ops; op; op = op->next)
            {
-             rtx blockage = operate_exp (POS_MINUS_OP, readycost,
-                                         make_numeric_value (1));
-
-             if (unit->simultaneity != 0)
-               {
-                 rtx filltime = make_numeric_value ((unit->simultaneity - 1)
-                                                    * unit->issue_delay.min);
-                 blockage = operate_exp (MIN_OP, blockage, filltime);
-               }
-
-             blockage = operate_exp (POS_MINUS_OP,
-                                     make_numeric_value (op->ready),
-                                     blockage);
-
-             blockage = operate_exp (MAX_OP, blockage, op->issue_exp);
+             rtx blockage = op->issue_exp;
              blockage = simplify_knowing (blockage, unit->condexp);
 
              /* Add this op's contribution to MAX (BLOCKAGE (E,*)) and
@@ -2041,7 +2155,10 @@ expand_units ()
            }
 
          /* Record MAX (BLOCKAGE (*,*)).  */
-         unit->max_blockage = max_attr_value (max_blockage);
+         {
+           int unknown;
+           unit->max_blockage = max_attr_value (max_blockage, &unknown);
+         }
 
          /* See if the upper and lower bounds of BLOCKAGE (E,*) are the
             same.  If so, the blockage function carries no additional
@@ -2053,7 +2170,7 @@ expand_units ()
               || atoi (XSTR (newexp, 0)) != 1);
 
          /* If the all values of BLOCKAGE (E,C) have the same value,
-            neither blockage function is written.  */    
+            neither blockage function is written.  */
          unit->needs_range_function
            = (unit->needs_blockage_function
               || GET_CODE (max_blockage) != CONST_STRING);
@@ -2061,13 +2178,13 @@ expand_units ()
          if (unit->needs_range_function)
            {
              /* Compute the blockage range function and make an attribute
-                for writing it's value.  */
+                for writing its value.  */
              newexp = operate_exp (RANGE_OP, min_blockage, max_blockage);
              newexp = simplify_knowing (newexp, unit->condexp);
 
              str = attr_printf (strlen (unit->name) + sizeof ("*_unit_blockage_range"),
                                 "*%s_unit_blockage_range", unit->name);
-             make_internal_attr (str, newexp, 4);
+             make_internal_attr (str, newexp, 20);
            }
 
          str = attr_printf (strlen (unit->name) + sizeof ("*_unit_ready_cost"),
@@ -2126,9 +2243,14 @@ simplify_knowing (exp, known_true)
 {
   if (GET_CODE (exp) != CONST_STRING)
     {
-      exp = attr_rtx (IF_THEN_ELSE, known_true, exp,
-                     make_numeric_value (max_attr_value (exp)));
-      exp = simplify_by_exploding (exp);
+      int unknown = 0, max;
+      max = max_attr_value (exp, &unknown);
+      if (! unknown)
+       {
+         exp = attr_rtx (IF_THEN_ELSE, known_true, exp,
+                         make_numeric_value (max));
+         exp = simplify_by_exploding (exp);
+       }
     }
   return exp;
 }
@@ -2145,7 +2267,7 @@ encode_units_mask (x)
   register int i;
   register int j;
   register enum rtx_code code;
-  register char *fmt;
+  register const char *fmt;
 
   code = GET_CODE (x);
 
@@ -2154,7 +2276,8 @@ encode_units_mask (x)
     case CONST_STRING:
       i = atoi (XSTR (x, 0));
       if (i < 0)
-       abort (); /* The sign bit encodes a one's compliment mask.  */
+       /* The sign bit encodes a one's compliment mask.  */
+       abort ();
       else if (i != 0 && i == (i & -i))
        /* Only one bit is set, so yield that unit number.  */
        for (j = 0; (i >>= 1) != 0; j++)
@@ -2173,6 +2296,9 @@ encode_units_mask (x)
     case CC0:
     case EQ_ATTR:
       return x;
+
+    default:
+      break;
     }
 
   /* Compare the elements.  If any pair of corresponding elements
@@ -2223,7 +2349,7 @@ fill_attr (attr)
       value = NULL;
       if (XVEC (id->def, id->vec_idx))
        for (i = 0; i < XVECLEN (id->def, id->vec_idx); i++)
-         if (! strcmp (XSTR (XEXP (XVECEXP (id->def, id->vec_idx, i), 0), 0), 
+         if (! strcmp (XSTR (XEXP (XVECEXP (id->def, id->vec_idx, i), 0), 0),
                        attr->name))
            value = XEXP (XVECEXP (id->def, id->vec_idx, i), 1);
 
@@ -2249,8 +2375,8 @@ fill_attr (attr)
 static rtx
 substitute_address (exp, no_address_fn, address_fn)
      rtx exp;
-     rtx (*no_address_fn) ();
-     rtx (*address_fn) ();
+     rtx (*no_address_fn) PARAMS ((rtx));
+     rtx (*address_fn) PARAMS ((rtx));
 {
   int i;
   rtx newexp;
@@ -2323,12 +2449,12 @@ substitute_address (exp, no_address_fn, address_fn)
 static void
 make_length_attrs ()
 {
-  static char *new_names[] = {"*insn_default_length",
-                             "*insn_variable_length_p",
-                             "*insn_current_length"};
-  static rtx (*no_address_fn[]) PROTO((rtx)) = {identity_fn, zero_fn, zero_fn};
-  static rtx (*address_fn[]) PROTO((rtx)) = {max_fn, one_fn, identity_fn};
-  int i;
+  static const char *new_names[] = {"*insn_default_length",
+                                     "*insn_variable_length_p",
+                                     "*insn_current_length"};
+  static rtx (*no_address_fn[]) PARAMS ((rtx)) = {identity_fn, zero_fn, zero_fn};
+  static rtx (*address_fn[]) PARAMS ((rtx)) = {max_fn, one_fn, identity_fn};
+  size_t i;
   struct attr_desc *length_attr, *new_attr;
   struct attr_value *av, *new_av;
   struct insn_ent *ie, *new_ie;
@@ -2346,7 +2472,7 @@ make_length_attrs ()
   length_attr->is_special = 1;
 
   /* Make each new attribute, in turn.  */
-  for (i = 0; i < sizeof new_names / sizeof new_names[0]; i++)
+  for (i = 0; i < ARRAY_SIZE (new_names); i++)
     {
       make_internal_attr (new_names[i],
                          substitute_address (length_attr->default_val->value,
@@ -2379,14 +2505,14 @@ identity_fn (exp)
 
 static rtx
 zero_fn (exp)
-     rtx exp;
+     rtx exp ATTRIBUTE_UNUSED;
 {
   return make_numeric_value (0);
 }
 
 static rtx
 one_fn (exp)
-     rtx exp;
+     rtx exp ATTRIBUTE_UNUSED;
 {
   return make_numeric_value (1);
 }
@@ -2395,7 +2521,35 @@ static rtx
 max_fn (exp)
      rtx exp;
 {
-  return make_numeric_value (max_attr_value (exp));
+  int unknown;
+  return make_numeric_value (max_attr_value (exp, &unknown));
+}
+
+static void
+write_length_unit_log ()
+{
+  struct attr_desc *length_attr = find_attr ("length", 0);
+  struct attr_value *av;
+  struct insn_ent *ie;
+  unsigned int length_unit_log, length_or;
+  int unknown = 0;
+
+  if (length_attr == 0)
+    return;
+  length_or = or_attr_value (length_attr->default_val->value, &unknown);
+  for (av = length_attr->first_value; av; av = av->next)
+    for (ie = av->first_insn; ie; ie = ie->next)
+      length_or |= or_attr_value (av->value, &unknown);
+
+  if (unknown)
+    length_unit_log = 0;
+  else
+    {
+      length_or = ~length_or;
+      for (length_unit_log = 0; length_or & 1; length_or >>= 1)
+       length_unit_log++;
+    }
+  printf ("int length_unit_log = %u;\n", length_unit_log);
 }
 \f
 /* Take a COND expression and see if any of the conditions in it can be
@@ -2416,7 +2570,6 @@ simplify_cond (exp, insn_code, insn_index)
      then build a new expression if they don't match EXP.  */
   rtx defval = XEXP (exp, 1);
   rtx new_defval = XEXP (exp, 1);
-
   int len = XVECLEN (exp, 0);
   rtx *tests = (rtx *) alloca (len * sizeof (rtx));
   int allsame = 1;
@@ -2425,7 +2578,7 @@ simplify_cond (exp, insn_code, insn_index)
   /* This lets us free all storage allocated below, if appropriate.  */
   first_spacer = (char *) obstack_finish (rtl_obstack);
 
-  bcopy ((char *) &XVECEXP (exp, 0, 0), (char *) tests, len * sizeof (rtx));
+  memcpy (tests, XVEC (exp, 0)->elem, len * sizeof (rtx));
 
   /* See if default value needs simplification.  */
   if (GET_CODE (defval) == COND)
@@ -2502,23 +2655,18 @@ simplify_cond (exp, insn_code, insn_index)
 
   if (len == 0)
     {
-      obstack_free (rtl_obstack, first_spacer);
       if (GET_CODE (defval) == COND)
        return simplify_cond (defval, insn_code, insn_index);
       return defval;
     }
   else if (allsame)
-    {
-      obstack_free (rtl_obstack, first_spacer);
-      return exp;
-    }
+    return exp;
   else
     {
       rtx newexp = rtx_alloc (COND);
 
       XVEC (newexp, 0) = rtvec_alloc (len);
-      bcopy ((char *) tests, (char *) &XVECEXP (newexp, 0, 0),
-            len * sizeof (rtx));
+      memcpy (XVEC (newexp, 0)->elem, tests, len * sizeof (rtx));
       XEXP (newexp, 1) = new_defval;
       return newexp;
     }
@@ -2545,6 +2693,8 @@ remove_insn_ent (av, ie)
   av->num_insns--;
   if (ie->insn_code == -1)
     av->has_asm_insn = 0;
+
+  num_insn_ents--;
 }
 
 /* Insert an insn entry in an attribute value list.  */
@@ -2559,6 +2709,8 @@ insert_insn_ent (av, ie)
   av->num_insns++;
   if (ie->insn_code == -1)
     av->has_asm_insn = 1;
+
+  num_insn_ents++;
 }
 \f
 /* This is a utility routine to take an expression that is a tree of either
@@ -2641,7 +2793,7 @@ compute_alternative_mask (exp, code)
      rtx exp;
      enum rtx_code code;
 {
-  char *string;
+  const char *string;
   if (GET_CODE (exp) == code)
     return compute_alternative_mask (XEXP (exp, 0), code)
           | compute_alternative_mask (XEXP (exp, 1), code);
@@ -2687,7 +2839,7 @@ make_alternative_compare (mask)
    of "attr" for this insn code.  From that value, we can compute a test
    showing when the EQ_ATTR will be true.  This routine performs that
    computation.  If a test condition involves an address, we leave the EQ_ATTR
-   intact because addresses are only valid for the `length' attribute. 
+   intact because addresses are only valid for the `length' attribute.
 
    EXP is the EQ_ATTR expression and VALUE is the value of that attribute
    for the insn corresponding to INSN_CODE and INSN_INDEX.  */
@@ -2710,6 +2862,25 @@ evaluate_eq_attr (exp, value, insn_code, insn_index)
       else
        newexp = false_rtx;
     }
+  else if (GET_CODE (value) == SYMBOL_REF)
+    {
+      char *p, *string;
+
+      if (GET_CODE (exp) != EQ_ATTR)
+       abort ();
+
+      string = (char *) alloca (2 + strlen (XSTR (exp, 0))
+                               + strlen (XSTR (exp, 1)));
+      strcpy (string, XSTR (exp, 0));
+      strcat (string, "_");
+      strcat (string, XSTR (exp, 1));
+      for (p = string; *p; p++)
+       *p = TOUPPER (*p);
+
+      newexp = attr_rtx (EQ, value,
+                        attr_rtx (SYMBOL_REF,
+                                  attr_string (string, strlen (string))));
+    }
   else if (GET_CODE (value) == COND)
     {
       /* We construct an IOR of all the cases for which the requested attribute
@@ -2717,12 +2888,12 @@ evaluate_eq_attr (exp, value, insn_code, insn_index)
         FALSE will be returned.
 
         Each case is the AND of the NOT's of the previous conditions with the
-        current condition; in the default case the current condition is TRUE. 
+        current condition; in the default case the current condition is TRUE.
 
         For each possible COND value, call ourselves recursively.
 
         The extra TRUE and FALSE expressions will be eliminated by another
-        call to the simplification routine. */
+        call to the simplification routine.  */
 
       orexp = false_rtx;
       andexp = true_rtx;
@@ -2786,7 +2957,7 @@ evaluate_eq_attr (exp, value, insn_code, insn_index)
    can be replaced with TRUE or FALSE, respectively.
 
    Note that (eq_attr "att" "v1") and (eq_attr "att" "v2") cannot both
-   be true and hence are complementary.  
+   be true and hence are complementary.
 
    There is one special case:  If we see
        (and (not (eq_attr "att" "v1"))
@@ -2808,7 +2979,7 @@ simplify_and_tree (exp, pterm, insn_code, insn_index)
 
   if (GET_CODE (exp) == AND)
     {
-      left = simplify_and_tree (XEXP (exp, 0), pterm,  insn_code, insn_index);
+      left  = simplify_and_tree (XEXP (exp, 0), pterm, insn_code, insn_index);
       right = simplify_and_tree (XEXP (exp, 1), pterm, insn_code, insn_index);
       if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
        {
@@ -2823,7 +2994,7 @@ simplify_and_tree (exp, pterm, insn_code, insn_index)
       /* For the IOR case, we do the same as above, except that we can
          only eliminate `term' if both sides of the IOR would do so.  */
       temp = *pterm;
-      left = simplify_and_tree (XEXP (exp, 0), &temp,  insn_code, insn_index);
+      left = simplify_and_tree (XEXP (exp, 0), &temp, insn_code, insn_index);
       left_eliminates_term = (temp == true_rtx);
 
       temp = *pterm;
@@ -2927,7 +3098,7 @@ simplify_or_tree (exp, pterm, insn_code, insn_index)
 
   if (GET_CODE (exp) == IOR)
     {
-      left = simplify_or_tree (XEXP (exp, 0), pterm,  insn_code, insn_index);
+      left  = simplify_or_tree (XEXP (exp, 0), pterm, insn_code, insn_index);
       right = simplify_or_tree (XEXP (exp, 1), pterm, insn_code, insn_index);
       if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
        {
@@ -2942,7 +3113,7 @@ simplify_or_tree (exp, pterm, insn_code, insn_index)
       /* For the AND case, we do the same as above, except that we can
          only eliminate `term' if both sides of the AND would do so.  */
       temp = *pterm;
-      left = simplify_or_tree (XEXP (exp, 0), &temp,  insn_code, insn_index);
+      left = simplify_or_tree (XEXP (exp, 0), &temp, insn_code, insn_index);
       left_eliminates_term = (temp == false_rtx);
 
       temp = *pterm;
@@ -2981,12 +3152,59 @@ simplify_or_tree (exp, pterm, insn_code, insn_index)
 
   return exp;
 }
+/* Compute approximate cost of the expression.  Used to decide whether
+   expression is cheap enought for inline.  */
+static int
+attr_rtx_cost (x)
+     rtx x;
+{
+  int cost = 0;
+  enum rtx_code code;
+  if (!x)
+    return 0;
+  code = GET_CODE (x);
+  switch (code)
+    {
+    case MATCH_OPERAND:
+      if (XSTR (x, 1)[0])
+       return 10;
+      else
+       return 0;
+    case EQ_ATTR:
+      /* Alternatives don't result into function call.  */
+      if (!strcmp (XSTR (x, 0), "alternative"))
+       return 0;
+      else
+       return 5;
+    default:
+      {
+       int i, j;
+       const char *fmt = GET_RTX_FORMAT (code);
+       for (i = GET_RTX_LENGTH (code) - 1; i >= 0; i--)
+         {
+           switch (fmt[i])
+             {
+             case 'V':
+             case 'E':
+               for (j = 0; j < XVECLEN (x, i); j++)
+                 cost += attr_rtx_cost (XVECEXP (x, i, j));
+               break;
+             case 'e':
+               cost += attr_rtx_cost (XEXP (x, i));
+               break;
+             }
+         }
+      }
+      break;
+    }
+  return cost;
+}
 \f
 /* Given an expression, see if it can be simplified for a particular insn
    code based on the values of other attributes being tested.  This can
    eliminate nested get_attr_... calls.
 
-   Note that if an endless recursion is specified in the patterns, the 
+   Note that if an endless recursion is specified in the patterns, the
    optimization will loop.  However, it will do so in precisely the cases where
    an infinite recursion loop could occur during compilation.  It's better that
    it occurs here!  */
@@ -3002,7 +3220,6 @@ simplify_test_exp (exp, insn_code, insn_index)
   struct insn_ent *ie;
   int i;
   rtx newexp = exp;
-  char *spacer = (char *) obstack_finish (rtl_obstack);
 
   /* Don't re-simplify something we already simplified.  */
   if (RTX_UNCHANGING_P (exp) || MEM_IN_STRUCT_P (exp))
@@ -3014,17 +3231,11 @@ simplify_test_exp (exp, insn_code, insn_index)
       left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
       SIMPLIFY_ALTERNATIVE (left);
       if (left == false_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return false_rtx;
-       }
+       return false_rtx;
       right = SIMPLIFY_TEST_EXP (XEXP (exp, 1), insn_code, insn_index);
       SIMPLIFY_ALTERNATIVE (right);
       if (left == false_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return false_rtx;
-       }
+       return false_rtx;
 
       /* If either side is an IOR and we have (eq_attr "alternative" ..")
         present on both sides, apply the distributive law since this will
@@ -3053,10 +3264,7 @@ simplify_test_exp (exp, insn_code, insn_index)
        left = simplify_and_tree (left, &right, insn_code, insn_index);
 
       if (left == false_rtx || right == false_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return false_rtx;
-       }
+       return false_rtx;
       else if (left == true_rtx)
        {
          return right;
@@ -3080,10 +3288,10 @@ simplify_test_exp (exp, insn_code, insn_index)
        {
          i = compute_alternative_mask (exp, AND);
          if (i & ~insn_alternatives[insn_code])
-           fatal ("Illegal alternative specified for pattern number %d",
+           fatal ("Invalid alternative specified for pattern number %d",
                   insn_index);
 
-         /* If all alternatives are excluded, this is false. */
+         /* If all alternatives are excluded, this is false.  */
          i ^= insn_alternatives[insn_code];
          if (i == 0)
            return false_rtx;
@@ -3112,27 +3320,18 @@ simplify_test_exp (exp, insn_code, insn_index)
       left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
       SIMPLIFY_ALTERNATIVE (left);
       if (left == true_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return true_rtx;
-       }
+       return true_rtx;
       right = SIMPLIFY_TEST_EXP (XEXP (exp, 1), insn_code, insn_index);
       SIMPLIFY_ALTERNATIVE (right);
       if (right == true_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return true_rtx;
-       }
+       return true_rtx;
 
       right = simplify_or_tree (right, &left, insn_code, insn_index);
       if (left == XEXP (exp, 0) && right == XEXP (exp, 1))
        left = simplify_or_tree (left, &right, insn_code, insn_index);
 
       if (right == true_rtx || left == true_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return true_rtx;
-       }
+       return true_rtx;
       else if (left == false_rtx)
        {
          return right;
@@ -3150,7 +3349,7 @@ simplify_test_exp (exp, insn_code, insn_index)
        */
 
       else if (GET_CODE (left) == AND && GET_CODE (right) == AND
-         && attr_equal_p (XEXP (left, 0), XEXP (right, 0)))
+              && attr_equal_p (XEXP (left, 0), XEXP (right, 0)))
        {
          newexp = attr_rtx (IOR, XEXP (left, 1), XEXP (right, 1));
 
@@ -3164,19 +3363,19 @@ simplify_test_exp (exp, insn_code, insn_index)
         in this tree.  Optimize if so.  */
 
       else if (insn_code >= 0
-         && (GET_CODE (left) == IOR
-             || (GET_CODE (left) == EQ_ATTR
-                 && XSTR (left, 0) == alternative_name)
-             || GET_CODE (right) == IOR
-             || (GET_CODE (right) == EQ_ATTR
-                 && XSTR (right, 0) == alternative_name)))
+              && (GET_CODE (left) == IOR
+                  || (GET_CODE (left) == EQ_ATTR
+                      && XSTR (left, 0) == alternative_name)
+                  || GET_CODE (right) == IOR
+                  || (GET_CODE (right) == EQ_ATTR
+                      && XSTR (right, 0) == alternative_name)))
        {
          i = compute_alternative_mask (exp, IOR);
          if (i & ~insn_alternatives[insn_code])
-           fatal ("Illegal alternative specified for pattern number %d",
+           fatal ("Invalid alternative specified for pattern number %d",
                   insn_index);
 
-         /* If all alternatives are included, this is true. */
+         /* If all alternatives are included, this is true.  */
          i ^= insn_alternatives[insn_code];
          if (i == 0)
            return true_rtx;
@@ -3216,15 +3415,9 @@ simplify_test_exp (exp, insn_code, insn_index)
        return XEXP (left, 0);
 
       if (left == false_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return true_rtx;
-       }
+       return true_rtx;
       else if (left == true_rtx)
-       {
-         obstack_free (rtl_obstack, spacer);
-         return false_rtx;
-       }
+       return false_rtx;
 
       /* Try to apply De`Morgan's laws.  */
       else if (GET_CODE (left) == IOR)
@@ -3253,7 +3446,7 @@ simplify_test_exp (exp, insn_code, insn_index)
       if (current_alternative_string && XSTR (exp, 0) == alternative_name)
        return (XSTR (exp, 1) == current_alternative_string
                ? true_rtx : false_rtx);
-       
+
       /* Look at the value for this insn code in the specified attribute.
         We normally can replace this comparison with the condition that
         would give this insn the values being tested for.   */
@@ -3262,7 +3455,20 @@ simplify_test_exp (exp, insn_code, insn_index)
        for (av = attr->first_value; av; av = av->next)
          for (ie = av->first_insn; ie; ie = ie->next)
            if (ie->insn_code == insn_code)
-             return evaluate_eq_attr (exp, av->value, insn_code, insn_index);
+             {
+               rtx x;
+               struct obstack *old = rtl_obstack;
+               rtl_obstack = temp_obstack;
+               x = evaluate_eq_attr (exp, av->value, insn_code, insn_index);
+               x = SIMPLIFY_TEST_EXP (x, insn_code, insn_index);
+               rtl_obstack = old;
+               if (attr_rtx_cost(x) < 20)
+                 return attr_copy_rtx (x);
+             }
+      break;
+
+    default:
+      break;
     }
 
   /* We have already simplified this expression.  Simplifying it again
@@ -3288,40 +3494,56 @@ optimize_attrs ()
   rtx newexp;
   int something_changed = 1;
   int i;
-  struct attr_value_list { struct attr_value *av;
-                          struct insn_ent *ie;
-                          struct attr_desc * attr;
-                          struct attr_value_list *next; };
+  struct attr_value_list
+  {
+    struct attr_value *av;
+    struct insn_ent *ie;
+    struct attr_desc *attr;
+    struct attr_value_list *next;
+  };
   struct attr_value_list **insn_code_values;
+  struct attr_value_list *ivbuf;
   struct attr_value_list *iv;
 
   /* For each insn code, make a list of all the insn_ent's for it,
      for all values for all attributes.  */
 
+  if (num_insn_ents == 0)
+    return;
+
   /* Make 2 extra elements, for "code" values -2 and -1.  */
   insn_code_values
     = (struct attr_value_list **) alloca ((insn_code_number + 2)
                                          * sizeof (struct attr_value_list *));
-  bzero ((char *) insn_code_values,
+  memset ((char *) insn_code_values, 0,
         (insn_code_number + 2) * sizeof (struct attr_value_list *));
 
   /* Offset the table address so we can index by -2 or -1.  */
   insn_code_values += 2;
 
+  /* Allocate the attr_value_list structures using xmalloc rather than
+     alloca, because using alloca can overflow the maximum permitted
+     stack limit on SPARC Lynx.  */
+  iv = ivbuf = ((struct attr_value_list *)
+               xmalloc (num_insn_ents * sizeof (struct attr_value_list)));
+
   for (i = 0; i < MAX_ATTRS_INDEX; i++)
     for (attr = attrs[i]; attr; attr = attr->next)
       for (av = attr->first_value; av; av = av->next)
        for (ie = av->first_insn; ie; ie = ie->next)
          {
-           iv = ((struct attr_value_list *)
-                 alloca (sizeof (struct attr_value_list)));
            iv->attr = attr;
            iv->av = av;
            iv->ie = ie;
            iv->next = insn_code_values[ie->insn_code];
            insn_code_values[ie->insn_code] = iv;
+           iv++;
          }
 
+  /* Sanity check on num_insn_ents.  */
+  if (iv != ivbuf + num_insn_ents)
+    abort ();
+
   /* Process one insn code at a time.  */
   for (i = -2; i < insn_code_number; i++)
     {
@@ -3338,7 +3560,6 @@ optimize_attrs ()
          for (iv = insn_code_values[i]; iv; iv = iv->next)
            {
              struct obstack *old = rtl_obstack;
-             char *spacer = (char *) obstack_finish (temp_obstack);
 
              attr = iv->attr;
              av = iv->av;
@@ -3367,10 +3588,11 @@ optimize_attrs ()
                  insert_insn_ent (av, ie);
                  something_changed = 1;
                }
-             obstack_free (temp_obstack, spacer);
            }
        }
     }
+
+  free (ivbuf);
 }
 
 #if 0
@@ -3384,7 +3606,6 @@ simplify_by_alternatives (exp, insn_code, insn_index)
   rtx newexp = rtx_alloc (COND);
   rtx ultimate;
 
-
   XVEC (newexp, 0) = rtvec_alloc (len * 2);
 
   /* It will not matter what value we use as the default value
@@ -3415,7 +3636,7 @@ static rtx
 simplify_by_exploding (exp)
      rtx exp;
 {
-  rtx list = 0, link, condexp, defval;
+  rtx list = 0, link, condexp, defval = NULL_RTX;
   struct dimension *space;
   rtx *condtest, *condval;
   int i, j, total, ndim = 0;
@@ -3441,7 +3662,7 @@ simplify_by_exploding (exp)
     {
       /* Pull the first attribute value from the list and record that
         attribute as another dimension in the attribute space.  */
-      char *name = XSTR (XEXP (list, 0), 0);
+      const char *name = XSTR (XEXP (list, 0), 0);
       rtx *prev;
 
       if ((space[ndim].attr = find_attr (name, 0)) == 0
@@ -3481,8 +3702,8 @@ simplify_by_exploding (exp)
        {
          struct dimension tmp;
          tmp = space[j];
-         space[j] = space[j-1];
-         space[j-1] = tmp;
+         space[j] = space[j - 1];
+         space[j - 1] = tmp;
        }
 
   /* Establish the initial current value.  */
@@ -3578,7 +3799,10 @@ find_and_mark_used_attributes (exp, terms, nterms)
          *nterms += 1;
          MEM_VOLATILE_P (exp) = 1;
        }
+      return 1;
+
     case CONST_STRING:
+    case CONST_INT:
       return 1;
 
     case IF_THEN_ELSE:
@@ -3600,9 +3824,10 @@ find_and_mark_used_attributes (exp, terms, nterms)
       if (! find_and_mark_used_attributes (XEXP (exp, 1), terms, nterms))
        return 0;
       return 1;
-    }
 
-  return 0;
+    default:
+      return 0;
+    }
 }
 
 /* Clear the MEM_VOLATILE_P flag in all EQ_ATTR expressions on LIST and
@@ -3646,7 +3871,8 @@ add_values_to_cover (dim)
   if (nalt < dim->num_values)
     abort ();
   else if (nalt == dim->num_values)
-    ; /* Ok.  */
+    /* OK.  */
+    ;
   else if (nalt * 2 < dim->num_values * 3)
     {
       /* Most all the values of the attribute are used, so add all the unused
@@ -3783,6 +4009,7 @@ simplify_with_current_value_aux (exp)
       else
        return true_rtx;
     case CONST_STRING:
+    case CONST_INT:
       return exp;
 
     case IF_THEN_ELSE:
@@ -3838,8 +4065,10 @@ simplify_with_current_value_aux (exp)
                         have been selected.  */
        }
       return simplify_with_current_value_aux (XEXP (exp, 1));
+
+    default:
+      abort ();
     }
-  abort ();
 }
 \f
 /* Clear the MEM_IN_STRUCT_P flag in EXP and its subexpressions.  */
@@ -3851,7 +4080,7 @@ clear_struct_flag (x)
   register int i;
   register int j;
   register enum rtx_code code;
-  register char *fmt;
+  register const char *fmt;
 
   MEM_IN_STRUCT_P (x) = 0;
   if (RTX_UNCHANGING_P (x))
@@ -3872,6 +4101,9 @@ clear_struct_flag (x)
     case EQ_ATTR:
     case ATTR_FLAG:
       return;
+
+    default:
+      break;
     }
 
   /* Compare the elements.  If any pair of corresponding elements
@@ -3896,7 +4128,7 @@ clear_struct_flag (x)
 }
 
 /* Return the number of RTX objects making up the expression X.
-   But if we count more more than MAX objects, stop counting.  */
+   But if we count more than MAX objects, stop counting.  */
 
 static int
 count_sub_rtxs (x, max)
@@ -3906,7 +4138,7 @@ count_sub_rtxs (x, max)
   register int i;
   register int j;
   register enum rtx_code code;
-  register char *fmt;
+  register const char *fmt;
   int total = 0;
 
   code = GET_CODE (x);
@@ -3924,6 +4156,9 @@ count_sub_rtxs (x, max)
     case EQ_ATTR:
     case ATTR_FLAG:
       return 1;
+
+    default:
+      break;
     }
 
   /* Compare the elements.  If any pair of corresponding elements
@@ -3955,22 +4190,30 @@ count_sub_rtxs (x, max)
 /* Create table entries for DEFINE_ATTR.  */
 
 static void
-gen_attr (exp)
+gen_attr (exp, lineno)
      rtx exp;
+     int lineno;
 {
   struct attr_desc *attr;
   struct attr_value *av;
-  char *name_ptr;
+  const char *name_ptr;
   char *p;
 
   /* Make a new attribute structure.  Check for duplicate by looking at
      attr->default_val, since it is initialized by this routine.  */
   attr = find_attr (XSTR (exp, 0), 1);
   if (attr->default_val)
-    fatal ("Duplicate definition for `%s' attribute", attr->name);
+    {
+      message_with_line (lineno, "duplicate definition for attribute %s",
+                        attr->name);
+      message_with_line (attr->lineno, "previous definition");
+      have_error = 1;
+      return;
+    }
+  attr->lineno = lineno;
 
   if (*XSTR (exp, 1) == '\0')
-      attr->is_numeric = 1;
+    attr->is_numeric = 1;
   else
     {
       name_ptr = XSTR (exp, 1);
@@ -3990,15 +4233,24 @@ gen_attr (exp)
     {
       attr->is_const = 1;
       if (attr->is_numeric)
-       fatal ("Constant attributes may not take numeric values");
+       {
+         message_with_line (lineno,
+                            "constant attributes may not take numeric values");
+         have_error = 1;
+       }
+
       /* Get rid of the CONST node.  It is allowed only at top-level.  */
       XEXP (exp, 2) = XEXP (XEXP (exp, 2), 0);
     }
 
   if (! strcmp (attr->name, "length") && ! attr->is_numeric)
-    fatal ("`length' attribute must take numeric values");
+    {
+      message_with_line (lineno,
+                        "`length' attribute must take numeric values");
+      have_error = 1;
+    }
 
-  /* Set up the default value. */
+  /* Set up the default value.  */
   XEXP (exp, 2) = check_attr_value (XEXP (exp, 2), attr);
   attr->default_val = get_attr_value (XEXP (exp, 2), attr, -2);
 }
@@ -4012,8 +4264,8 @@ count_alternatives (exp)
      rtx exp;
 {
   int i, j, n;
-  char *fmt;
-  
+  const char *fmt;
+
   if (GET_CODE (exp) == MATCH_OPERAND)
     return n_comma_elts (XSTR (exp, 2));
 
@@ -4050,7 +4302,7 @@ compares_alternatives_p (exp)
      rtx exp;
 {
   int i, j;
-  char *fmt;
+  const char *fmt;
 
   if (GET_CODE (exp) == EQ_ATTR && XSTR (exp, 0) == alternative_name)
     return 1;
@@ -4083,7 +4335,7 @@ contained_in_p (inner, exp)
      rtx exp;
 {
   int i, j;
-  char *fmt;
+  const char *fmt;
 
   if (rtx_equal_p (inner, exp))
     return 1;
@@ -4107,12 +4359,13 @@ contained_in_p (inner, exp)
 
   return 0;
 }
-\f      
+\f
 /* Process DEFINE_PEEPHOLE, DEFINE_INSN, and DEFINE_ASM_ATTRIBUTES.  */
 
 static void
-gen_insn (exp)
+gen_insn (exp, lineno)
      rtx exp;
+     int lineno;
 {
   struct insn_def *id;
 
@@ -4120,12 +4373,13 @@ gen_insn (exp)
   id->next = defs;
   defs = id;
   id->def = exp;
+  id->lineno = lineno;
 
   switch (GET_CODE (exp))
     {
     case DEFINE_INSN:
-      id->insn_code = insn_code_number++;
-      id->insn_index = insn_index_number++;
+      id->insn_code = insn_code_number;
+      id->insn_index = insn_index_number;
       id->num_alternatives = count_alternatives (exp);
       if (id->num_alternatives == 0)
        id->num_alternatives = 1;
@@ -4133,8 +4387,8 @@ gen_insn (exp)
       break;
 
     case DEFINE_PEEPHOLE:
-      id->insn_code = insn_code_number++;
-      id->insn_index = insn_index_number++;
+      id->insn_code = insn_code_number;
+      id->insn_index = insn_index_number;
       id->num_alternatives = count_alternatives (exp);
       if (id->num_alternatives == 0)
        id->num_alternatives = 1;
@@ -4148,6 +4402,9 @@ gen_insn (exp)
       id->vec_idx = 0;
       got_define_asm_attributes = 1;
       break;
+
+    default:
+      abort ();
     }
 }
 \f
@@ -4155,14 +4412,20 @@ gen_insn (exp)
    true or annul false is specified, and make a `struct delay_desc'.  */
 
 static void
-gen_delay (def)
+gen_delay (def, lineno)
      rtx def;
+     int lineno;
 {
   struct delay_desc *delay;
   int i;
 
   if (XVECLEN (def, 1) % 3 != 0)
-    fatal ("Number of elements in DEFINE_DELAY must be multiple of three.");
+    {
+      message_with_line (lineno,
+                        "number of elements in DEFINE_DELAY must be multiple of three");
+      have_error = 1;
+      return;
+    }
 
   for (i = 0; i < XVECLEN (def, 1); i += 3)
     {
@@ -4171,27 +4434,29 @@ gen_delay (def)
       if (XVECEXP (def, 1, i + 2))
        have_annul_false = 1;
     }
-  
+
   delay = (struct delay_desc *) oballoc (sizeof (struct delay_desc));
   delay->def = def;
   delay->num = ++num_delays;
   delay->next = delays;
+  delay->lineno = lineno;
   delays = delay;
 }
 \f
-/* Process a DEFINE_FUNCTION_UNIT.  
+/* Process a DEFINE_FUNCTION_UNIT.
 
    This gives information about a function unit contained in the CPU.
    We fill in a `struct function_unit_op' and a `struct function_unit'
    with information used later by `expand_unit'.  */
 
 static void
-gen_unit (def)
+gen_unit (def, lineno)
      rtx def;
+     int lineno;
 {
   struct function_unit *unit;
   struct function_unit_op *op;
-  char *name = XSTR (def, 0);
+  const char *name = XSTR (def, 0);
   int multiplicity = XINT (def, 1);
   int simultaneity = XINT (def, 2);
   rtx condexp = XEXP (def, 3);
@@ -4206,8 +4471,14 @@ gen_unit (def)
       {
        if (unit->multiplicity != multiplicity
            || unit->simultaneity != simultaneity)
-         fatal ("Differing specifications given for `%s' function unit.",
-                unit->name);
+         {
+           message_with_line (lineno,
+                              "differing specifications given for function unit %s",
+                              unit->name);
+           message_with_line (unit->first_lineno, "previous definition");
+           have_error = 1;
+           return;
+         }
        break;
       }
 
@@ -4223,6 +4494,7 @@ gen_unit (def)
       unit->condexp = false_rtx;
       unit->ops = 0;
       unit->next = units;
+      unit->first_lineno = lineno;
       units = unit;
     }
 
@@ -4233,7 +4505,9 @@ gen_unit (def)
   op->ready = ready_cost;
   op->issue_delay = issue_delay;
   op->next = unit->ops;
+  op->lineno = lineno;
   unit->ops = op;
+  num_unit_opclasses++;
 
   /* Set our issue expression based on whether or not an optional conflict
      vector was specified.  */
@@ -4260,15 +4534,18 @@ gen_unit (def)
   unit->condexp = insert_right_side (IOR, unit->condexp, op->condexp, -2, -2);
 }
 \f
-/* Given a piece of RTX, print a C expression to test it's truth value.
-   We use AND and IOR both for logical and bit-wise operations, so 
+/* Given a piece of RTX, print a C expression to test its truth value.
+   We use AND and IOR both for logical and bit-wise operations, so
    interpret them as logical unless they are inside a comparison expression.
-   The second operand of this function will be non-zero in that case.  */
+   The first bit of FLAGS will be non-zero in that case.
+
+   Set the second bit of FLAGS to make references to attribute values use
+   a cached local variable instead of calling a function.  */
 
 static void
-write_test_expr (exp, in_comparison)
+write_test_expr (exp, flags)
      rtx exp;
-     int in_comparison;
+     int flags;
 {
   int comparison_operator = 0;
   RTX_CODE code;
@@ -4290,9 +4567,9 @@ write_test_expr (exp, in_comparison)
     case PLUS:   case MINUS:  case MULT:     case DIV:      case MOD:
     case AND:    case IOR:    case XOR:
     case ASHIFT: case LSHIFTRT: case ASHIFTRT:
-      write_test_expr (XEXP (exp, 0), in_comparison || comparison_operator);
+      write_test_expr (XEXP (exp, 0), flags | comparison_operator);
       switch (code)
-        {
+       {
        case EQ:
          printf (" == ");
          break;
@@ -4339,13 +4616,13 @@ write_test_expr (exp, in_comparison)
          printf (" %% ");
          break;
        case AND:
-         if (in_comparison)
+         if (flags & 1)
            printf (" & ");
          else
            printf (" && ");
          break;
        case IOR:
-         if (in_comparison)
+         if (flags & 1)
            printf (" | ");
          else
            printf (" || ");
@@ -4360,14 +4637,16 @@ write_test_expr (exp, in_comparison)
        case ASHIFTRT:
          printf (" >> ");
          break;
-        }
+       default:
+         abort ();
+       }
 
-      write_test_expr (XEXP (exp, 1), in_comparison || comparison_operator);
+      write_test_expr (XEXP (exp, 1), flags | comparison_operator);
       break;
 
     case NOT:
       /* Special-case (not (eq_attrq "alternative" "x")) */
-      if (! in_comparison && GET_CODE (XEXP (exp, 0)) == EQ_ATTR
+      if (! (flags & 1) && GET_CODE (XEXP (exp, 0)) == EQ_ATTR
          && XSTR (XEXP (exp, 0), 0) == alternative_name)
        {
          printf ("which_alternative != %s", XSTR (XEXP (exp, 0), 1));
@@ -4376,12 +4655,12 @@ write_test_expr (exp, in_comparison)
 
       /* Otherwise, fall through to normal unary operator.  */
 
-    /* Unary operators.  */   
+    /* Unary operators.  */
     case ABS:  case NEG:
       switch (code)
        {
        case NOT:
-         if (in_comparison)
+         if (flags & 1)
            printf ("~ ");
          else
            printf ("! ");
@@ -4392,16 +4671,18 @@ write_test_expr (exp, in_comparison)
        case NEG:
          printf ("-");
          break;
+       default:
+         abort ();
        }
 
-      write_test_expr (XEXP (exp, 0), in_comparison);
+      write_test_expr (XEXP (exp, 0), flags);
       break;
 
     /* Comparison test of an attribute with a value.  Most of these will
        have been removed by optimization.   Handle "alternative"
        specially and give error if EQ_ATTR present inside a comparison.  */
     case EQ_ATTR:
-      if (in_comparison)
+      if (flags & 1)
        fatal ("EQ_ATTR not valid inside comparison");
 
       if (XSTR (exp, 0) == alternative_name)
@@ -4411,25 +4692,30 @@ write_test_expr (exp, in_comparison)
        }
 
       attr = find_attr (XSTR (exp, 0), 0);
-      if (! attr) abort ();
+      if (! attr)
+       abort ();
 
       /* Now is the time to expand the value of a constant attribute.  */
       if (attr->is_const)
        {
          write_test_expr (evaluate_eq_attr (exp, attr->default_val->value,
                                             -2, -2),
-                          in_comparison);
+                          flags);
        }
       else
        {
-         printf ("get_attr_%s (insn) == ", attr->name);
-         write_attr_valueq (attr, XSTR (exp, 1)); 
+         if (flags & 2)
+           printf ("attr_%s", attr->name);
+         else
+           printf ("get_attr_%s (insn)", attr->name);
+         printf (" == ");
+         write_attr_valueq (attr, XSTR (exp, 1));
        }
       break;
 
     /* Comparison test of flags for define_delays.  */
     case ATTR_FLAG:
-      if (in_comparison)
+      if (flags & 1)
        fatal ("ATTR_FLAG not valid inside comparison");
       printf ("(flags & ATTR_FLAG_%s) != 0", XSTR (exp, 0));
       break;
@@ -4438,7 +4724,7 @@ write_test_expr (exp, in_comparison)
     case MATCH_OPERAND:
       /* If only a mode is given, just ensure the mode matches the operand.
         If neither a mode nor predicate is given, error.  */
-     if (XSTR (exp, 1) == NULL || *XSTR (exp, 1) == '\0')
+      if (XSTR (exp, 1) == NULL || *XSTR (exp, 1) == '\0')
        {
          if (GET_MODE (exp) == VOIDmode)
            fatal ("Null MATCH_OPERAND specified as test");
@@ -4451,31 +4737,45 @@ write_test_expr (exp, in_comparison)
                XSTR (exp, 1), XINT (exp, 0), GET_MODE_NAME (GET_MODE (exp)));
       break;
 
-    /* Constant integer. */
+    case MATCH_INSN:
+      printf ("%s (insn)", XSTR (exp, 0));
+      break;
+
+    /* Constant integer.  */
     case CONST_INT:
-#if HOST_BITS_PER_WIDE_INT == HOST_BITS_PER_INT
-      printf ("%d", XWINT (exp, 0));
-#else
-      printf ("%ld", XWINT (exp, 0));
-#endif
+      printf (HOST_WIDE_INT_PRINT_DEC, XWINT (exp, 0));
       break;
 
-    /* A random C expression. */
+    /* A random C expression.  */
     case SYMBOL_REF:
       printf ("%s", XSTR (exp, 0));
       break;
 
     /* The address of the branch target.  */
     case MATCH_DUP:
-      printf ("insn_addresses[INSN_UID (JUMP_LABEL (insn))]");
+      printf ("INSN_ADDRESSES_SET_P () ? INSN_ADDRESSES (INSN_UID (GET_CODE (operands[%d]) == LABEL_REF ? XEXP (operands[%d], 0) : operands[%d])) : 0",
+             XINT (exp, 0), XINT (exp, 0), XINT (exp, 0));
       break;
 
-    /* The address of the current insn.  It would be more consistent with
-       other usage to make this the address of the NEXT insn, but this gets
-       too confusing because of the ambiguity regarding the length of the
-       current insn.  */
     case PC:
-      printf ("insn_current_address");
+      /* The address of the current insn.  We implement this actually as the
+        address of the current insn for backward branches, but the last
+        address of the next insn for forward branches, and both with
+        adjustments that account for the worst-case possible stretching of
+        intervening alignments between this insn and its destination.  */
+      printf ("insn_current_reference_address (insn)");
+      break;
+
+    case CONST_STRING:
+      printf ("%s", XSTR (exp, 0));
+      break;
+
+    case IF_THEN_ELSE:
+      write_test_expr (XEXP (exp, 0), flags & 2);
+      printf (" ? ");
+      write_test_expr (XEXP (exp, 1), flags | 1);
+      printf (" : ");
+      write_test_expr (XEXP (exp, 2), flags | 1);
       break;
 
     default:
@@ -4487,45 +4787,84 @@ write_test_expr (exp, in_comparison)
 }
 \f
 /* Given an attribute value, return the maximum CONST_STRING argument
-   encountered.  It is assumed that they are all numeric.  */
+   encountered.  Set *UNKNOWNP and return INT_MAX if the value is unknown.  */
 
 static int
-max_attr_value (exp)
+max_attr_value (exp, unknownp)
      rtx exp;
+     int *unknownp;
 {
-  int current_max = 0;
-  int n;
-  int i;
-
-  if (GET_CODE (exp) == CONST_STRING)
-    return atoi (XSTR (exp, 0));
+  int current_max;
+  int i, n;
 
-  else if (GET_CODE (exp) == COND)
+  switch (GET_CODE (exp))
     {
+    case CONST_STRING:
+      current_max = atoi (XSTR (exp, 0));
+      break;
+
+    case COND:
+      current_max = max_attr_value (XEXP (exp, 1), unknownp);
       for (i = 0; i < XVECLEN (exp, 0); i += 2)
        {
-         n = max_attr_value (XVECEXP (exp, 0, i + 1));
+         n = max_attr_value (XVECEXP (exp, 0, i + 1), unknownp);
          if (n > current_max)
            current_max = n;
        }
+      break;
 
-      n = max_attr_value (XEXP (exp, 1));
+    case IF_THEN_ELSE:
+      current_max = max_attr_value (XEXP (exp, 1), unknownp);
+      n = max_attr_value (XEXP (exp, 2), unknownp);
       if (n > current_max)
        current_max = n;
+      break;
+
+    default:
+      *unknownp = 1;
+      current_max = INT_MAX;
+      break;
     }
 
-  else if (GET_CODE (exp) == IF_THEN_ELSE)
+  return current_max;
+}
+
+/* Given an attribute value, return the result of ORing together all
+   CONST_STRING arguments encountered.  Set *UNKNOWNP and return -1
+   if the numeric value is not known.  */
+
+static int
+or_attr_value (exp, unknownp)
+     rtx exp;
+     int *unknownp;
+{
+  int current_or;
+  int i;
+
+  switch (GET_CODE (exp))
     {
-      current_max = max_attr_value (XEXP (exp, 1));
-      n = max_attr_value (XEXP (exp, 2));
-      if (n > current_max)
-       current_max = n;
-    }
+    case CONST_STRING:
+      current_or = atoi (XSTR (exp, 0));
+      break;
 
-  else
-    abort ();
+    case COND:
+      current_or = or_attr_value (XEXP (exp, 1), unknownp);
+      for (i = 0; i < XVECLEN (exp, 0); i += 2)
+       current_or |= or_attr_value (XVECEXP (exp, 0, i + 1), unknownp);
+      break;
 
-  return current_max;
+    case IF_THEN_ELSE:
+      current_or = or_attr_value (XEXP (exp, 1), unknownp);
+      current_or |= or_attr_value (XEXP (exp, 2), unknownp);
+      break;
+
+    default:
+      *unknownp = 1;
+      current_or = -1;
+      break;
+    }
+
+  return current_or;
 }
 \f
 /* Scan an attribute value, possibly a conditional, and record what actions
@@ -4543,7 +4882,7 @@ walk_attr_value (exp)
      rtx exp;
 {
   register int i, j;
-  register char *fmt;
+  register const char *fmt;
   RTX_CODE code;
 
   if (exp == NULL)
@@ -4572,12 +4911,19 @@ walk_attr_value (exp)
       return;
 
     case MATCH_DUP:
+      must_extract = 1;
+      address_used = 1;
+      return;
+
     case PC:
       address_used = 1;
       return;
 
     case ATTR_FLAG:
       return;
+
+    default:
+      break;
     }
 
   for (i = 0, fmt = GET_RTX_FORMAT (code); i < GET_RTX_LENGTH (code); i++)
@@ -4605,9 +4951,24 @@ write_attr_get (attr)
   struct attr_value *av, *common_av;
 
   /* Find the most used attribute value.  Handle that as the `default' of the
-     switch we will generate. */
+     switch we will generate.  */
   common_av = find_most_used (attr);
 
+  /* Write out prototype of function.  */
+  if (!attr->is_numeric)
+    printf ("extern enum attr_%s ", attr->name);
+  else if (attr->unsigned_p)
+    printf ("extern unsigned int ");
+  else
+    printf ("extern int ");
+  /* If the attribute name starts with a star, the remainder is the name of
+     the subroutine to use, instead of `get_attr_...'.  */
+  if (attr->name[0] == '*')
+    printf ("%s PARAMS ((rtx));\n", &attr->name[1]);
+  else
+    printf ("get_attr_%s PARAMS ((%s));\n", attr->name,
+           (attr->is_const ? "void" : "rtx"));
+
   /* Write out start of function, then all values with explicit `case' lines,
      then a `default', then the value with the most uses.  */
   if (!attr->is_numeric)
@@ -4637,17 +4998,40 @@ write_attr_get (attr)
       printf ("}\n\n");
       return;
     }
+
   printf ("     rtx insn;\n");
   printf ("{\n");
-  printf ("  switch (recog_memoized (insn))\n");
-  printf ("    {\n");
 
-  for (av = attr->first_value; av; av = av->next)
-    if (av != common_av)
-      write_attr_case (attr, av, 1, "return", ";", 4, true_rtx);
+  if (GET_CODE (common_av->value) == FFS)
+    {
+      rtx p = XEXP (common_av->value, 0);
 
-  write_attr_case (attr, common_av, 0, "return", ";", 4, true_rtx);
-  printf ("    }\n}\n\n");
+      /* No need to emit code to abort if the insn is unrecognized; the
+         other get_attr_foo functions will do that when we call them.  */
+
+      write_toplevel_expr (p);
+
+      printf ("\n  if (accum && accum == (accum & -accum))\n");
+      printf ("    {\n");
+      printf ("      int i;\n");
+      printf ("      for (i = 0; accum >>= 1; ++i) continue;\n");
+      printf ("      accum = i;\n");
+      printf ("    }\n  else\n");
+      printf ("    accum = ~accum;\n");
+      printf ("  return accum;\n}\n\n");
+    }
+  else
+    {
+      printf ("  switch (recog_memoized (insn))\n");
+      printf ("    {\n");
+
+      for (av = attr->first_value; av; av = av->next)
+       if (av != common_av)
+         write_attr_case (attr, av, 1, "return", ";", 4, true_rtx);
+
+      write_attr_case (attr, common_av, 0, "return", ";", 4, true_rtx);
+      printf ("    }\n}\n\n");
+    }
 }
 \f
 /* Given an AND tree of known true terms (because we are inside an `if' with
@@ -4692,19 +5076,12 @@ write_attr_set (attr, indent, value, prefix, suffix, known_true,
      struct attr_desc *attr;
      int indent;
      rtx value;
-     char *prefix;
-     char *suffix;
+     const char *prefix;
+     const char *suffix;
      rtx known_true;
      int insn_code, insn_index;
 {
-  if (GET_CODE (value) == CONST_STRING)
-    {
-      write_indent (indent);
-      printf ("%s ", prefix);
-      write_attr_value (attr, value);
-      printf ("%s\n", suffix);
-    }
-  else if (GET_CODE (value) == COND)
+  if (GET_CODE (value) == COND)
     {
       /* Assume the default value will be the default of the COND unless we
         find an always true expression.  */
@@ -4723,8 +5100,8 @@ write_attr_set (attr, indent, value, prefix, suffix, known_true,
                                          XVECEXP (value, 0, i),
                                          insn_code, insn_index);
          newexp = attr_rtx (NOT, testexp);
-         newexp  = insert_right_side (AND, our_known_true, newexp,
-                                      insn_code, insn_index);
+         newexp = insert_right_side (AND, our_known_true, newexp,
+                                     insn_code, insn_index);
 
          /* If the test expression is always true or if the next `known_true'
             expression is always false, this is the last case, so break
@@ -4752,7 +5129,7 @@ write_attr_set (attr, indent, value, prefix, suffix, known_true,
          write_indent (indent + 2);
          printf ("{\n");
 
-         write_attr_set (attr, indent + 4,  
+         write_attr_set (attr, indent + 4,
                          XVECEXP (value, 0, i + 1), prefix, suffix,
                          inner_true, insn_code, insn_index);
          write_indent (indent + 2);
@@ -4778,7 +5155,12 @@ write_attr_set (attr, indent, value, prefix, suffix, known_true,
        }
     }
   else
-    abort ();
+    {
+      write_indent (indent);
+      printf ("%s ", prefix);
+      write_attr_value (attr, value);
+      printf ("%s\n", suffix);
+    }
 }
 \f
 /* Write out the computation for one attribute value.  */
@@ -4789,7 +5171,7 @@ write_attr_case (attr, av, write_case_lines, prefix, suffix, indent,
      struct attr_desc *attr;
      struct attr_value *av;
      int write_case_lines;
-     char *prefix, *suffix;
+     const char *prefix, *suffix;
      int indent;
      rtx known_true;
 {
@@ -4829,20 +5211,15 @@ write_attr_case (attr, av, write_case_lines, prefix, suffix, indent,
   must_extract = must_constrain = address_used = 0;
   walk_attr_value (av->value);
 
-  if (must_extract)
+  if (must_constrain)
     {
       write_indent (indent + 2);
-      printf ("insn_extract (insn);\n");
+      printf ("extract_constrain_insn_cached (insn);\n");
     }
-
-  if (must_constrain)
+  else if (must_extract)
     {
-#ifdef REGISTER_CONSTRAINTS
-      write_indent (indent + 2);
-      printf ("if (! constrain_operands (INSN_CODE (insn), reload_completed))\n");
       write_indent (indent + 2);
-      printf ("  fatal_insn_not_found (insn);\n");
-#endif
+      printf ("extract_insn_cached (insn);\n");
     }
 
   write_attr_set (attr, indent + 2, av->value, prefix, suffix,
@@ -4857,19 +5234,148 @@ write_attr_case (attr, av, write_case_lines, prefix, suffix, indent,
   printf ("\n");
 }
 \f
+/* Search for uses of non-const attributes and write code to cache them.  */
+
+static int
+write_expr_attr_cache (p, attr)
+     rtx p;
+     struct attr_desc *attr;
+{
+  const char *fmt;
+  int i, ie, j, je;
+
+  if (GET_CODE (p) == EQ_ATTR)
+    {
+      if (XSTR (p, 0) != attr->name)
+       return 0;
+
+      if (!attr->is_numeric)
+       printf ("  register enum attr_%s ", attr->name);
+      else if (attr->unsigned_p)
+       printf ("  register unsigned int ");
+      else
+       printf ("  register int ");
+
+      printf ("attr_%s = get_attr_%s (insn);\n", attr->name, attr->name);
+      return 1;
+    }
+
+  fmt = GET_RTX_FORMAT (GET_CODE (p));
+  ie = GET_RTX_LENGTH (GET_CODE (p));
+  for (i = 0; i < ie; i++)
+    {
+      switch (*fmt++)
+       {
+       case 'e':
+         if (write_expr_attr_cache (XEXP (p, i), attr))
+           return 1;
+         break;
+
+       case 'E':
+         je = XVECLEN (p, i);
+         for (j = 0; j < je; ++j)
+           if (write_expr_attr_cache (XVECEXP (p, i, j), attr))
+             return 1;
+         break;
+       }
+    }
+
+  return 0;
+}
+
+/* Evaluate an expression at top level.  A front end to write_test_expr,
+   in which we cache attribute values and break up excessively large
+   expressions to cater to older compilers.  */
+
+static void
+write_toplevel_expr (p)
+     rtx p;
+{
+  struct attr_desc *attr;
+  int i;
+
+  for (i = 0; i < MAX_ATTRS_INDEX; ++i)
+    for (attr = attrs[i]; attr; attr = attr->next)
+      if (!attr->is_const)
+       write_expr_attr_cache (p, attr);
+
+  printf ("  register unsigned long accum = 0;\n\n");
+
+  while (GET_CODE (p) == IOR)
+    {
+      rtx e;
+      if (GET_CODE (XEXP (p, 0)) == IOR)
+       e = XEXP (p, 1), p = XEXP (p, 0);
+      else
+       e = XEXP (p, 0), p = XEXP (p, 1);
+
+      printf ("  accum |= ");
+      write_test_expr (e, 3);
+      printf (";\n");
+    }
+  printf ("  accum |= ");
+  write_test_expr (p, 3);
+  printf (";\n");
+}
+\f
 /* Utilities to write names in various forms.  */
 
 static void
+write_unit_name (prefix, num, suffix)
+     const char *prefix;
+     int num;
+     const char *suffix;
+{
+  struct function_unit *unit;
+
+  for (unit = units; unit; unit = unit->next)
+    if (unit->num == num)
+      {
+       printf ("%s%s%s", prefix, unit->name, suffix);
+       return;
+      }
+
+  printf ("%s<unknown>%s", prefix, suffix);
+}
+
+static void
 write_attr_valueq (attr, s)
      struct attr_desc *attr;
-     char *s;
+     const char *s;
 {
   if (attr->is_numeric)
     {
-      printf ("%s", s);
-      /* Make the blockage range values easier to read.  */
-      if (strlen (s) > 1)
-       printf (" /* 0x%x */", atoi (s));
+      int num = atoi (s);
+
+      printf ("%d", num);
+
+      /* Make the blockage range values and function units used values easier
+         to read.  */
+      if (attr->func_units_p)
+       {
+         if (num == -1)
+           printf (" /* units: none */");
+         else if (num >= 0)
+           write_unit_name (" /* units: ", num, " */");
+         else
+           {
+             int i;
+             const char *sep = " /* units: ";
+             for (i = 0, num = ~num; num; i++, num >>= 1)
+               if (num & 1)
+                 {
+                   write_unit_name (sep, i, (num == 1) ? " */" : "");
+                   sep = ", ";
+                 }
+           }
+       }
+
+      else if (attr->blockage_p)
+       printf (" /* min %d, max %d */", num >> (HOST_BITS_PER_INT / 2),
+               num & ((1 << (HOST_BITS_PER_INT / 2)) - 1));
+
+      else if (num > 9 || num < 0)
+       printf (" /* 0x%x */", num);
     }
   else
     {
@@ -4884,21 +5390,69 @@ write_attr_value (attr, value)
      struct attr_desc *attr;
      rtx value;
 {
-  if (GET_CODE (value) != CONST_STRING)
-    abort ();
+  int op;
+
+  switch (GET_CODE (value))
+    {
+    case CONST_STRING:
+      write_attr_valueq (attr, XSTR (value, 0));
+      break;
+
+    case CONST_INT:
+      printf (HOST_WIDE_INT_PRINT_DEC, INTVAL (value));
+      break;
+
+    case SYMBOL_REF:
+      fputs (XSTR (value, 0), stdout);
+      break;
 
-  write_attr_valueq (attr, XSTR (value, 0));
+    case ATTR:
+      {
+       struct attr_desc *attr2 = find_attr (XSTR (value, 0), 0);
+       printf ("get_attr_%s (%s)", attr2->name,
+               (attr2->is_const ? "" : "insn"));
+      }
+      break;
+
+    case PLUS:
+      op = '+';
+      goto do_operator;
+    case MINUS:
+      op = '-';
+      goto do_operator;
+    case MULT:
+      op = '*';
+      goto do_operator;
+    case DIV:
+      op = '/';
+      goto do_operator;
+    case MOD:
+      op = '%';
+      goto do_operator;
+
+    do_operator:
+      write_attr_value (attr, XEXP (value, 0));
+      putchar (' ');
+      putchar (op);
+      putchar (' ');
+      write_attr_value (attr, XEXP (value, 1));
+      break;
+
+    default:
+      abort ();
+    }
 }
 
 static void
 write_upcase (str)
-     char *str;
+     const char *str;
 {
   while (*str)
-    if (*str < 'a' || *str > 'z')
-      printf ("%c", *str++);
-    else
-      printf ("%c", *str++ - 'a' + 'A');
+    {
+      /* The argument of TOUPPER should not have side effects.  */
+      putchar (TOUPPER(*str));
+      str++;
+    }
 }
 
 static void
@@ -4926,7 +5480,7 @@ write_indent (indent)
 
 static void
 write_eligible_delay (kind)
-     char *kind;
+     const char *kind;
 {
   struct delay_desc *delay;
   int max_slots;
@@ -4946,12 +5500,12 @@ write_eligible_delay (kind)
   /* Write function prelude.  */
 
   printf ("int\n");
-  printf ("eligible_for_%s (delay_insn, slot, candidate_insn, flags)\n", 
-          kind);
+  printf ("eligible_for_%s (delay_insn, slot, candidate_insn, flags)\n",
+         kind);
   printf ("     rtx delay_insn;\n");
   printf ("     int slot;\n");
   printf ("     rtx candidate_insn;\n");
-  printf ("     int flags;\n");
+  printf ("     int flags ATTRIBUTE_UNUSED;\n");
   printf ("{\n");
   printf ("  rtx insn;\n");
   printf ("\n");
@@ -4964,7 +5518,8 @@ write_eligible_delay (kind)
   if (num_delays > 1)
     {
       attr = find_attr ("*delay_type", 0);
-      if (! attr) abort ();
+      if (! attr)
+       abort ();
       common_av = find_most_used (attr);
 
       printf ("  insn = delay_insn;\n");
@@ -4992,7 +5547,8 @@ write_eligible_delay (kind)
       printf ("    {\n");
 
       attr = find_attr ("*delay_1_0", 0);
-      if (! attr) abort ();
+      if (! attr)
+       abort ();
       common_av = find_most_used (attr);
 
       for (av = attr->first_value; av; av = av->next)
@@ -5021,7 +5577,8 @@ write_eligible_delay (kind)
 
            sprintf (str, "*%s_%d_%d", kind, delay->num, i / 3);
            attr = find_attr (str, 0);
-           if (! attr) abort ();
+           if (! attr)
+             abort ();
            common_av = find_most_used (attr);
 
            for (av = attr->first_value; av; av = av->next)
@@ -5033,7 +5590,7 @@ write_eligible_delay (kind)
          }
 
       printf ("    default:\n");
-      printf ("      abort ();\n");     
+      printf ("      abort ();\n");
       printf ("    }\n");
     }
 
@@ -5115,7 +5672,7 @@ write_function_unit_info ()
 static void
 write_complex_function (unit, name, connection)
      struct function_unit *unit;
-     char *name, *connection;
+     const char *name, *connection;
 {
   struct attr_desc *case_attr, *attr;
   struct attr_value *av, *common_av;
@@ -5124,9 +5681,9 @@ write_complex_function (unit, name, connection)
   int using_case;
   int i;
 
+  printf ("static int %s_unit_%s PARAMS ((rtx, rtx));\n", unit->name, name);
   printf ("static int\n");
-  printf ("%s_unit_%s (executing_insn, candidate_insn)\n",
-         unit->name, name);
+  printf ("%s_unit_%s (executing_insn, candidate_insn)\n", unit->name, name);
   printf ("     rtx executing_insn;\n");
   printf ("     rtx candidate_insn;\n");
   printf ("{\n");
@@ -5140,7 +5697,8 @@ write_complex_function (unit, name, connection)
   str = (char *) alloca (strlen (unit->name) + strlen (name) + strlen (connection) + 10);
   sprintf (str, "*%s_cases", unit->name);
   case_attr = find_attr (str, 0);
-  if (! case_attr) abort ();
+  if (! case_attr)
+    abort ();
   common_av = find_most_used (case_attr);
 
   for (av = case_attr->first_value; av; av = av->next)
@@ -5173,7 +5731,8 @@ write_complex_function (unit, name, connection)
       printf ("    case %d:\n", i);
       sprintf (str, "*%s_%s_%d", unit->name, connection, i);
       attr = find_attr (str, 0);
-      if (! attr) abort ();
+      if (! attr)
+       abort ();
 
       /* If single value, just write it.  */
       value = find_single_value (attr);
@@ -5196,6 +5755,10 @@ write_complex_function (unit, name, connection)
        }
     }
 
+  /* This default case should not be needed, but gcc's analysis is not
+     good enough to realize that the default case is not needed for the
+     second switch statement.  */
+  printf ("    default:\n      abort ();\n");
   printf ("    }\n}\n\n");
 }
 \f
@@ -5206,7 +5769,7 @@ write_complex_function (unit, name, connection)
 
 static int
 n_comma_elts (s)
-     char *s;
+     const char *s;
 {
   int n;
 
@@ -5226,10 +5789,10 @@ n_comma_elts (s)
 
 static char *
 next_comma_elt (pstr)
-     char **pstr;
+     const char **pstr;
 {
   char *out_str;
-  char *p;
+  const char *p;
 
   if (**pstr == '\0')
     return NULL;
@@ -5252,7 +5815,7 @@ next_comma_elt (pstr)
 
 static struct attr_desc *
 find_attr (name, create)
-     char *name;
+     const char *name;
      int create;
 {
   struct attr_desc *attr;
@@ -5280,6 +5843,7 @@ find_attr (name, create)
   attr->name = attr_string (name, strlen (name));
   attr->first_value = attr->default_val = NULL;
   attr->is_numeric = attr->negative_ok = attr->is_const = attr->is_special = 0;
+  attr->unsigned_p = attr->func_units_p = attr->blockage_p = 0;
   attr->next = attrs[index];
   attrs[index] = attr;
 
@@ -5290,7 +5854,7 @@ find_attr (name, create)
 
 static void
 make_internal_attr (name, value, special)
-     char *name;
+     const char *name;
      rtx value;
      int special;
 {
@@ -5305,6 +5869,8 @@ make_internal_attr (name, value, special)
   attr->is_special = (special & 1) != 0;
   attr->negative_ok = (special & 2) != 0;
   attr->unsigned_p = (special & 4) != 0;
+  attr->func_units_p = (special & 8) != 0;
+  attr->blockage_p = (special & 16) != 0;
   attr->default_val = get_attr_value (value, attr, -2);
 }
 
@@ -5382,30 +5948,10 @@ extend_range (range, min, max)
      int min;
      int max;
 {
-  if (range->min > min) range->min = min;
-  if (range->max < max) range->max = max;
-}
-
-char *
-xrealloc (ptr, size)
-     char *ptr;
-     unsigned size;
-{
-  char *result = (char *) realloc (ptr, size);
-  if (!result)
-    fatal ("virtual memory exhausted");
-  return result;
-}
-
-char *
-xmalloc (size)
-     unsigned size;
-{
-  register char *val = (char *) malloc (size);
-
-  if (val == 0)
-    fatal ("virtual memory exhausted");
-  return val;
+  if (range->min > min)
+    range->min = min;
+  if (range->max < max)
+    range->max = max;
 }
 
 static rtx
@@ -5432,47 +5978,30 @@ copy_rtx_unchanging (orig)
     case SYMBOL_REF:
     case CODE_LABEL:
       return orig;
+
+    default:
+      break;
     }
 
   copy = rtx_alloc (code);
   PUT_MODE (copy, GET_MODE (orig));
   RTX_UNCHANGING_P (copy) = 1;
-  
-  bcopy ((char *) &XEXP (orig, 0), (char *) &XEXP (copy, 0),
-        GET_RTX_LENGTH (GET_CODE (copy)) * sizeof (rtx));
+
+  memcpy (&XEXP (copy, 0), &XEXP (orig, 0),
+         GET_RTX_LENGTH (GET_CODE (copy)) * sizeof (rtx));
   return copy;
 #endif
 }
 
-static void
-fatal (s, a1, a2)
-     char *s;
-{
-  fprintf (stderr, "genattrtab: ");
-  fprintf (stderr, s, a1, a2);
-  fprintf (stderr, "\n");
-  exit (FATAL_EXIT_CODE);
-}
-
-/* More 'friendly' abort that prints the line and file.
-   config.h can #define abort fancy_abort if you like that sort of thing.  */
-
-void
-fancy_abort ()
-{
-  fatal ("Internal gcc abort.");
-}
-
 /* Determine if an insn has a constant number of delay slots, i.e., the
    number of delay slots is not a function of the length of the insn.  */
 
-void
+static void
 write_const_num_delay_slots ()
 {
   struct attr_desc *attr = find_attr ("*num_delay_slots", 0);
   struct attr_value *av;
   struct insn_ent *ie;
-  int i;
 
   if (attr)
     {
@@ -5489,60 +6018,54 @@ write_const_num_delay_slots ()
          if (length_used)
            {
              for (ie = av->first_insn; ie; ie = ie->next)
-             if (ie->insn_code != -1)
-               printf ("    case %d:\n", ie->insn_code);
+               if (ie->insn_code != -1)
+                 printf ("    case %d:\n", ie->insn_code);
              printf ("      return 0;\n");
            }
        }
 
       printf ("    default:\n");
       printf ("      return 1;\n");
-      printf ("    }\n}\n");
+      printf ("    }\n}\n\n");
     }
 }
-
 \f
+extern int main PARAMS ((int, char **));
+
 int
 main (argc, argv)
      int argc;
      char **argv;
 {
   rtx desc;
-  FILE *infile;
-  register int c;
   struct attr_desc *attr;
   struct insn_def *id;
   rtx tem;
   int i;
 
-#ifdef RLIMIT_STACK
+  progname = "genattrtab";
+
+  if (argc <= 1)
+    fatal ("No input file name.");
+
+#if defined (RLIMIT_STACK) && defined (HAVE_GETRLIMIT) && defined (HAVE_SETRLIMIT)
   /* Get rid of any avoidable limit on stack size.  */
   {
     struct rlimit rlim;
 
-    /* Set the stack limit huge so that alloca does not fail. */
+    /* Set the stack limit huge so that alloca does not fail.  */
     getrlimit (RLIMIT_STACK, &rlim);
     rlim.rlim_cur = rlim.rlim_max;
     setrlimit (RLIMIT_STACK, &rlim);
   }
-#endif /* RLIMIT_STACK defined */
+#endif
+
+  if (init_md_reader (argv[1]) != SUCCESS_EXIT_CODE)
+    return (FATAL_EXIT_CODE);
 
-  obstack_init (rtl_obstack);
   obstack_init (hash_obstack);
   obstack_init (temp_obstack);
 
-  if (argc <= 1)
-    fatal ("No input file name.");
-
-  infile = fopen (argv[1], "r");
-  if (infile == 0)
-    {
-      perror (argv[1]);
-      exit (FATAL_EXIT_CODE);
-    }
-
-  init_rtl ();
-
   /* Set up true and false rtx's */
   true_rtx = rtx_alloc (CONST_INT);
   XWINT (true_rtx, 0) = 1;
@@ -5560,48 +6083,50 @@ from the machine description file `md'.  */\n\n");
 
   while (1)
     {
-      c = read_skip_spaces (infile);
-      if (c == EOF)
-       break;
-      ungetc (c, infile);
+      int lineno;
 
-      desc = read_rtx (infile);
-      if (GET_CODE (desc) == DEFINE_INSN
-         || GET_CODE (desc) == DEFINE_PEEPHOLE
-         || GET_CODE (desc) == DEFINE_ASM_ATTRIBUTES)
-       gen_insn (desc);
+      desc = read_md_rtx (&lineno, &insn_code_number);
+      if (desc == NULL)
+       break;
 
-      else if (GET_CODE (desc) == DEFINE_EXPAND)
-       insn_code_number++, insn_index_number++;
+      switch (GET_CODE (desc))
+       {
+       case DEFINE_INSN:
+       case DEFINE_PEEPHOLE:
+       case DEFINE_ASM_ATTRIBUTES:
+         gen_insn (desc, lineno);
+         break;
 
-      else if (GET_CODE (desc) == DEFINE_SPLIT)
-       insn_code_number++, insn_index_number++;
+       case DEFINE_ATTR:
+         gen_attr (desc, lineno);
+         break;
 
-      else if (GET_CODE (desc) == DEFINE_ATTR)
-       {
-         gen_attr (desc);
-         insn_index_number++;
-       }
+       case DEFINE_DELAY:
+         gen_delay (desc, lineno);
+         break;
 
-      else if (GET_CODE (desc) == DEFINE_DELAY)
-       {
-         gen_delay (desc);
-         insn_index_number++;
-       }
+       case DEFINE_FUNCTION_UNIT:
+         gen_unit (desc, lineno);
+         break;
 
-      else if (GET_CODE (desc) == DEFINE_FUNCTION_UNIT)
-       {
-         gen_unit (desc);
-         insn_index_number++;
+       default:
+         break;
        }
+      if (GET_CODE (desc) != DEFINE_ASM_ATTRIBUTES)
+       insn_index_number++;
     }
 
+  if (have_error)
+    return FATAL_EXIT_CODE;
+
+  insn_code_number++;
+
   /* If we didn't have a DEFINE_ASM_ATTRIBUTES, make a null one.  */
   if (! got_define_asm_attributes)
     {
       tem = rtx_alloc (DEFINE_ASM_ATTRIBUTES);
       XVEC (tem, 0) = rtvec_alloc (0);
-      gen_insn (tem);
+      gen_insn (tem, 0);
     }
 
   /* Expand DEFINE_DELAY information into new attribute.  */
@@ -5613,15 +6138,18 @@ from the machine description file `md'.  */\n\n");
     expand_units ();
 
   printf ("#include \"config.h\"\n");
+  printf ("#include \"system.h\"\n");
   printf ("#include \"rtl.h\"\n");
+  printf ("#include \"tm_p.h\"\n");
   printf ("#include \"insn-config.h\"\n");
   printf ("#include \"recog.h\"\n");
   printf ("#include \"regs.h\"\n");
   printf ("#include \"real.h\"\n");
   printf ("#include \"output.h\"\n");
   printf ("#include \"insn-attr.h\"\n");
-  printf ("\n");  
-  printf ("#define operands recog_operand\n\n");
+  printf ("#include \"toplev.h\"\n");
+  printf ("\n");
+  printf ("#define operands recog_data.operand\n\n");
 
   /* Make `insn_alternatives'.  */
   insn_alternatives = (int *) oballoc (insn_code_number * sizeof (int));
@@ -5639,18 +6167,23 @@ from the machine description file `md'.  */\n\n");
      away and building the attribute cases.  */
 
   check_defs ();
+
   for (i = 0; i < MAX_ATTRS_INDEX; i++)
     for (attr = attrs[i]; attr; attr = attr->next)
-      {
-       attr->default_val->value
-         = check_attr_value (attr->default_val->value, attr);
-       fill_attr (attr);
-      }
+      attr->default_val->value
+       = check_attr_value (attr->default_val->value, attr);
+
+  if (have_error)
+    return FATAL_EXIT_CODE;
+
+  for (i = 0; i < MAX_ATTRS_INDEX; i++)
+    for (attr = attrs[i]; attr; attr = attr->next)
+      fill_attr (attr);
 
   /* Construct extra attributes for `length'.  */
   make_length_attrs ();
 
-  /* Perform any possible optimizations to speed up compilation. */
+  /* Perform any possible optimizations to speed up compilation.  */
   optimize_attrs ();
 
   /* Now write out all the `gen_attr_...' routines.  Do these before the
@@ -5660,7 +6193,7 @@ from the machine description file `md'.  */\n\n");
   for (i = 0; i < MAX_ATTRS_INDEX; i++)
     for (attr = attrs[i]; attr; attr = attr->next)
       {
-       if (! attr->is_special)
+       if (! attr->is_special && ! attr->is_const)
          write_attr_get (attr);
       }
 
@@ -5683,8 +6216,16 @@ from the machine description file `md'.  */\n\n");
   /* Write out constant delay slot info */
   write_const_num_delay_slots ();
 
+  write_length_unit_log ();
+
   fflush (stdout);
-  exit (ferror (stdout) != 0 ? FATAL_EXIT_CODE : SUCCESS_EXIT_CODE);
-  /* NOTREACHED */
-  return 0;
+  return (ferror (stdout) != 0 ? FATAL_EXIT_CODE : SUCCESS_EXIT_CODE);
+}
+
+/* Define this so we can link with print-rtl.o to get debug_rtx function.  */
+const char *
+get_insn_name (code)
+     int code ATTRIBUTE_UNUSED;
+{
+  return NULL;
 }