OSDN Git Service

2009-09-08 Kai Tietz <kai.tietz@onevision.com>
[pf3gnuchains/gcc-fork.git] / gcc / genattrtab.c
index 71c5957..4d6295a 100644 (file)
@@ -1,13 +1,14 @@
 /* Generate code from machine description to compute values of attributes.
-   Copyright (C) 1991, 1993, 1994, 1995, 1996, 1997, 1998,
-   1999, 2000, 2002 Free Software Foundation, Inc.
+   Copyright (C) 1991, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
+   2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009
+   Free Software Foundation, Inc.
    Contributed by Richard Kenner (kenner@vlsi1.ultra.nyu.edu)
 
 This file is part of GCC.
 
 GCC is free software; you can redistribute it and/or modify it under
 the terms of the GNU General Public License as published by the Free
-Software Foundation; either version 2, or (at your option) any later
+Software Foundation; either version 3, or (at your option) any later
 version.
 
 GCC is distributed in the hope that it will be useful, but WITHOUT ANY
@@ -16,12 +17,11 @@ FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
 for more details.
 
 You should have received a copy of the GNU General Public License
-along with GCC; see the file COPYING.  If not, write to the Free
-Software Foundation, 59 Temple Place - Suite 330, Boston, MA
-02111-1307, USA.  */
+along with GCC; see the file COPYING3.  If not see
+<http://www.gnu.org/licenses/>.  */
 
 /* This program handles insn attributes and the DEFINE_DELAY and
-   DEFINE_FUNCTION_UNIT definitions.
+   DEFINE_INSN_RESERVATION definitions.
 
    It produces a series of functions named `get_attr_...', one for each insn
    attribute.  Each of these is given the rtx for an insn and returns a member
@@ -54,7 +54,7 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    used.
 
    Internal attributes are defined to handle DEFINE_DELAY and
-   DEFINE_FUNCTION_UNIT.  Special routines are output for these cases.
+   DEFINE_INSN_RESERVATION.  Special routines are output for these cases.
 
    This program works by keeping a list of possible values for each attribute.
    These include the basic attribute choices, default values for attribute, and
@@ -70,9 +70,8 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
    indicates when the attribute has the specified value for the insn.  This
    avoids recursive calls during compilation.
 
-   The strategy used when processing DEFINE_DELAY and DEFINE_FUNCTION_UNIT
-   definitions is to create arbitrarily complex expressions and have the
-   optimization simplify them.
+   The strategy used when processing DEFINE_DELAY definitions is to create
+   arbitrarily complex expressions and have the optimization simplify them.
 
    Once optimization is complete, any required routines and definitions
    will be written.
@@ -90,39 +89,37 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
       independent of the insn code.
    `in_struct' (ATTR_CURR_SIMPLIFIED_P): This rtx is fully simplified
       for the insn code currently being processed (see optimize_attrs).
-   `integrated' (ATTR_PERMANENT_P): This rtx is permanent and unique
-      (see attr_rtx).
-   `volatil' (ATTR_EQ_ATTR_P): During simplify_by_exploding the value of an
-      EQ_ATTR rtx is true if !volatil and false if volatil.  */
+   `return_val' (ATTR_PERMANENT_P): This rtx is permanent and unique
+      (see attr_rtx).  */
 
 #define ATTR_IND_SIMPLIFIED_P(RTX) (RTX_FLAG((RTX), unchanging))
 #define ATTR_CURR_SIMPLIFIED_P(RTX) (RTX_FLAG((RTX), in_struct))
-#define ATTR_PERMANENT_P(RTX) (RTX_FLAG((RTX), integrated))
-#define ATTR_EQ_ATTR_P(RTX) (RTX_FLAG((RTX), volatil))
+#define ATTR_PERMANENT_P(RTX) (RTX_FLAG((RTX), return_val))
 
-#include "hconfig.h"
+#if 0
+#define strcmp_check(S1, S2) ((S1) == (S2)             \
+                             ? 0                       \
+                             : (gcc_assert (strcmp ((S1), (S2))), 1))
+#else
+#define strcmp_check(S1, S2) ((S1) != (S2))
+#endif
+
+#include "bconfig.h"
 #include "system.h"
+#include "coretypes.h"
+#include "tm.h"
 #include "rtl.h"
-#include "ggc.h"
 #include "gensupport.h"
-
-#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"
 
-#include "genattrtab.h"
+/* Flags for make_internal_attr's `special' parameter.  */
+#define ATTR_NONE              0
+#define ATTR_SPECIAL           (1 << 0)
 
 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
+static struct obstack *hash_obstack = &obstack1;
+static struct obstack *temp_obstack = &obstack2;
 
 /* enough space to reserve for printing out ints */
 #define MAX_DIGITS (HOST_BITS_PER_INT * 3 / 10 + 3)
@@ -139,7 +136,7 @@ struct insn_def
   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 insn_index;              /* Expression number in file, for errors.  */
   int lineno;                  /* Line number.  */
   int num_alternatives;                /* Number of alternatives.  */
   int vec_idx;                 /* Index of attribute vector in `def'.  */
@@ -152,9 +149,7 @@ 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 */
-  int lineno;                  /* Line number.  */
+  struct insn_def *def;                /* Instruction definition.  */
 };
 
 /* Each value of an attribute (either constant or computed) is assigned a
@@ -176,26 +171,12 @@ struct attr_desc
 {
   char *name;                  /* Name of attribute.  */
   struct attr_desc *next;      /* Next attribute.  */
+  struct attr_value *first_value; /* First value of this attribute.  */
+  struct attr_value *default_val; /* Default value for this attribute.  */
+  int lineno : 24;             /* Line number.  */
   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
-
-/* A range of values.  */
-
-struct range
-{
-  int min;
-  int max;
 };
 
 /* Structure for each DEFINE_DELAY.  */
@@ -208,41 +189,12 @@ struct delay_desc
   int lineno;                  /* Line number.  */
 };
 
-/* Record information about each DEFINE_FUNCTION_UNIT.  */
-
-struct function_unit_op
-{
-  rtx condexp;                 /* Expression TRUE for applicable insn.  */
-  struct function_unit_op *next; /* Next operation for this function unit.  */
-  int num;                     /* Ordinal for this operation type in unit.  */
-  int ready;                   /* Cost until data is ready.  */
-  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
-   DEFINE_FUNCTION_UNIT.  */
-
-struct function_unit
+struct attr_value_list
 {
-  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.  */
-  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.  */
-  int needs_blockage_function; /* Nonzero if a blockage function required.  */
-  int needs_range_function;    /* Nonzero if blockage range function needed.  */
-  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.  */
+  struct attr_value *av;
+  struct insn_ent *ie;
+  struct attr_desc *attr;
+  struct attr_value_list *next;
 };
 
 /* Listheads of above structures.  */
@@ -252,50 +204,7 @@ struct function_unit
 static struct attr_desc *attrs[MAX_ATTRS_INDEX];
 static struct insn_def *defs;
 static struct delay_desc *delays;
-static struct function_unit *units;
-
-/* An expression where all the unknown terms are EQ_ATTR tests can be
-   rearranged into a COND provided we can enumerate all possible
-   combinations of the unknown values.  The set of combinations become the
-   tests of the COND; the value of the expression given that combination is
-   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
-   attributes.  We record each of these as a struct dimension.
-
-   The algorithm relies on sharing EQ_ATTR nodes: if two nodes in an
-   expression are the same, the will also have the same address.  We find
-   all the EQ_ATTR nodes by marking them ATTR_EQ_ATTR_P.  This bit later
-   represents the value of an EQ_ATTR node, so once all nodes are marked,
-   they are also given an initial value of FALSE.
-
-   We then separate the set of EQ_ATTR nodes into dimensions for each
-   attribute and put them on the VALUES list.  Terms are added as needed by
-   `add_values_to_cover' so that all possible values of the attribute are
-   tested.
-
-   Each dimension also has a current value.  This is the node that is
-   currently considered to be TRUE.  If this is one of the nodes added by
-   `add_values_to_cover', all the EQ_ATTR tests in the original expression
-   will be FALSE.  Otherwise, only the CURRENT_VALUE will be true.
-
-   NUM_VALUES is simply the length of the VALUES list and is there for
-   convenience.
-
-   Once the dimensions are created, the algorithm enumerates all possible
-   values and computes the current value of the given expression.  */
-
-struct dimension
-{
-  struct attr_desc *attr;      /* Attribute for this dimension.  */
-  rtx values;                  /* List of attribute values used.  */
-  rtx current_value;           /* Position in the list for the TRUE value.  */
-  int num_values;              /* Length of the values list.  */
-};
+struct attr_value_list **insn_code_values;
 
 /* Other variables.  */
 
@@ -308,15 +217,8 @@ static int address_used;
 static int length_used;
 static int num_delays;
 static int have_annul_true, have_annul_false;
-static int num_units, num_unit_opclasses;
 static int num_insn_ents;
 
-int num_dfa_decls;
-
-/* Used as operand to `operate_exp':  */
-
-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.  */
 
 static int *insn_n_alternatives;
@@ -326,29 +228,17 @@ static int *insn_n_alternatives;
 
 static int *insn_alternatives;
 
-/* If nonzero, assume that the `alternative' attr has this value.
-   This is the hashed, unique string for the numeral
-   whose value is chosen alternative.  */
-
-static const char *current_alternative_string;
-
 /* Used to simplify expressions.  */
 
 static rtx true_rtx, false_rtx;
 
 /* Used to reduce calls to `strcmp' */
 
-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;
+static const char *alternative_name;
+static const char *length_str;
+static const char *delay_type_str;
+static const char *delay_1_0_str;
+static const char *num_delay_slots_str;
 
 /* Simplify an expression.  Only call the routine if there is something to
    simplify.  */
@@ -356,119 +246,50 @@ int optimize = 0;
   (ATTR_IND_SIMPLIFIED_P (EXP) || ATTR_CURR_SIMPLIFIED_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)                              \
-  if (current_alternative_string                               \
-      && GET_CODE ((EXP)) == EQ_ATTR                           \
-      && XSTR ((EXP), 0) == alternative_name)                  \
-    (EXP) = (XSTR ((EXP), 1) == current_alternative_string     \
-           ? true_rtx : false_rtx);
-
-/* These are referenced by rtlanal.c and hence need to be defined somewhere.
-   They won't actually be used.  */
-
-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 rtx attr_rtx_1          PARAMS ((enum rtx_code, va_list));
-static char *attr_string        PARAMS ((const char *, 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 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  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 rtx simplify_test_exp_in_temp 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 char *next_comma_elt    PARAMS ((const char **));
-static struct attr_desc *find_attr PARAMS ((const char *, int));
-static struct attr_value *find_most_used  PARAMS ((struct attr_desc *));
-static rtx find_single_value   PARAMS ((struct attr_desc *));
-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
+#define DEF_ATTR_STRING(S) (attr_string ((S), strlen (S)))
+
+/* Forward declarations of functions used before their definitions, only.  */
+static char *attr_string           (const char *, int);
+static char *attr_printf           (unsigned int, const char *, ...)
+  ATTRIBUTE_PRINTF_2;
+static rtx make_numeric_value      (int);
+static struct attr_desc *find_attr (const char **, int);
+static rtx mk_attr_alt             (int);
+static char *next_comma_elt       (const char **);
+static rtx insert_right_side      (enum rtx_code, rtx, rtx, int, int);
+static rtx copy_boolean                   (rtx);
+static int compares_alternatives_p (rtx);
+static void make_internal_attr     (const char *, rtx, int);
+static void insert_insn_ent        (struct attr_value *, struct insn_ent *);
+static void walk_attr_value       (rtx);
+static int max_attr_value         (rtx, int*);
+static int min_attr_value         (rtx, int*);
+static int or_attr_value          (rtx, int*);
+static rtx simplify_test_exp      (rtx, int, int);
+static rtx simplify_test_exp_in_temp (rtx, int, int);
+static rtx copy_rtx_unchanging    (rtx);
+static bool attr_alt_subset_p      (rtx, rtx);
+static bool attr_alt_subset_of_compl_p (rtx, rtx);
+static void clear_struct_flag      (rtx);
+static void write_attr_valueq     (struct attr_desc *, const char *);
+static struct attr_value *find_most_used  (struct attr_desc *);
+static void write_attr_set        (struct attr_desc *, int, rtx,
+                                   const char *, const char *, rtx,
+                                   int, int);
+static void write_attr_case       (struct attr_desc *, struct attr_value *,
+                                   int, const char *, const char *, int, rtx);
+static void write_attr_value      (struct attr_desc *, rtx);
+static void write_upcase          (const char *);
+static void write_indent          (int);
+static rtx identity_fn            (rtx);
+static rtx zero_fn                (rtx);
+static rtx one_fn                 (rtx);
+static rtx max_fn                 (rtx);
+static rtx min_fn                 (rtx);
+
+#define oballoc(T) XOBNEW (hash_obstack, T)
+#define oballocvec(T, N) XOBNEWVEC (hash_obstack, T, (N))
+
 /* Hash table for sharing RTL and strings.  */
 
 /* Each hash table slot is a bucket containing a chain of these structures.
@@ -494,23 +315,20 @@ struct attr_hash
    for each.  */
 
 #define RTL_HASH_SIZE 4093
-struct attr_hash *attr_hash_table[RTL_HASH_SIZE];
+static 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) ((long) (RTL) & 0777777)
+#define RTL_HASH(RTL) ((intptr_t) (RTL) & 0777777)
 
 /* Add an entry to the hash table for RTL with hash code HASHCODE.  */
 
 static void
-attr_hash_add_rtx (hashcode, rtl)
-     int hashcode;
-     rtx rtl;
+attr_hash_add_rtx (int hashcode, rtx rtl)
 {
   struct attr_hash *h;
 
-  h = (struct attr_hash *) obstack_alloc (hash_obstack,
-                                         sizeof (struct attr_hash));
+  h = XOBNEW (hash_obstack, struct attr_hash);
   h->hashcode = hashcode;
   h->u.rtl = rtl;
   h->next = attr_hash_table[hashcode % RTL_HASH_SIZE];
@@ -520,14 +338,11 @@ attr_hash_add_rtx (hashcode, rtl)
 /* Add an entry to the hash table for STRING with hash code HASHCODE.  */
 
 static void
-attr_hash_add_string (hashcode, str)
-     int hashcode;
-     char *str;
+attr_hash_add_string (int hashcode, char *str)
 {
   struct attr_hash *h;
 
-  h = (struct attr_hash *) obstack_alloc (hash_obstack,
-                                         sizeof (struct attr_hash));
+  h = XOBNEW (hash_obstack, struct attr_hash);
   h->hashcode = -hashcode;
   h->u.str = str;
   h->next = attr_hash_table[hashcode % RTL_HASH_SIZE];
@@ -540,14 +355,12 @@ attr_hash_add_string (hashcode, str)
    In some cases we cannot uniquify; then we return an ordinary
    impermanent rtx with ATTR_PERMANENT_P clear.
 
-   Args are like gen_rtx, but without the mode:
+   Args are as follows:
 
    rtx attr_rtx (code, [element1, ..., elementn])  */
 
 static rtx
-attr_rtx_1 (code, p)
-     enum rtx_code code;
-     va_list p;
+attr_rtx_1 (enum rtx_code code, va_list p)
 {
   rtx rt_val = NULL_RTX;/* RTX to return to caller...          */
   int hashcode;
@@ -558,7 +371,7 @@ attr_rtx_1 (code, p)
      Use that entry if one is found; otherwise create a new RTL and add it
      to the table.  */
 
-  if (GET_RTX_CLASS (code) == '1')
+  if (GET_RTX_CLASS (code) == RTX_UNARY)
     {
       rtx arg0 = va_arg (p, rtx);
 
@@ -584,9 +397,10 @@ attr_rtx_1 (code, p)
          XEXP (rt_val, 0) = arg0;
        }
     }
-  else if (GET_RTX_CLASS (code) == 'c'
-          || GET_RTX_CLASS (code) == '2'
-          || GET_RTX_CLASS (code) == '<')
+  else if (GET_RTX_CLASS (code) == RTX_BIN_ARITH
+          || GET_RTX_CLASS (code) == RTX_COMM_ARITH
+          || GET_RTX_CLASS (code) == RTX_COMPARE
+          || GET_RTX_CLASS (code) == RTX_COMM_COMPARE)
     {
       rtx arg0 = va_arg (p, rtx);
       rtx arg1 = va_arg (p, rtx);
@@ -621,8 +435,7 @@ attr_rtx_1 (code, p)
     {
       char *arg0 = va_arg (p, char *);
 
-      if (code == SYMBOL_REF)
-       arg0 = attr_string (arg0, strlen (arg0));
+      arg0 = DEF_ATTR_STRING (arg0);
 
       hashcode = ((HOST_WIDE_INT) code + RTL_HASH (arg0));
       for (h = attr_hash_table[hashcode % RTL_HASH_SIZE]; h; h = h->next)
@@ -708,7 +521,7 @@ attr_rtx_1 (code, p)
              break;
 
            default:
-             abort ();
+             gcc_unreachable ();
            }
        }
       return rt_val;
@@ -721,14 +534,14 @@ attr_rtx_1 (code, p)
 }
 
 static rtx
-attr_rtx VPARAMS ((enum rtx_code code, ...))
+attr_rtx (enum rtx_code code, ...)
 {
   rtx result;
-  
-  VA_OPEN (p, code);
-  VA_FIXEDARG (p, enum rtx_code, code);
+  va_list p;
+
+  va_start (p, code);
   result = attr_rtx_1 (code, p);
-  VA_CLOSE (p);
+  va_end (p);
   return result;
 }
 
@@ -737,35 +550,30 @@ attr_rtx VPARAMS ((enum rtx_code code, ...))
 
    rtx attr_printf (len, format, [arg1, ..., argn])  */
 
-char *
-attr_printf VPARAMS ((unsigned int len, const char *fmt, ...))
+static char *
+attr_printf (unsigned int len, const char *fmt, ...)
 {
   char str[256];
+  va_list p;
 
-  VA_OPEN (p, fmt);
-  VA_FIXEDARG (p, unsigned int, len);
-  VA_FIXEDARG (p, const char *, fmt);
-  
-  if (len > sizeof str - 1) /* Leave room for \0.  */
-    abort ();
+  va_start (p, fmt);
+
+  gcc_assert (len < sizeof str); /* Leave room for \0.  */
 
   vsprintf (str, fmt, p);
-  VA_CLOSE (p);
+  va_end (p);
 
-  return attr_string (str, strlen (str));
+  return DEF_ATTR_STRING (str);
 }
 
 static rtx
-attr_eq (name, value)
-     const char *name, *value;
+attr_eq (const char *name, const char *value)
 {
-  return attr_rtx (EQ_ATTR, attr_string (name, strlen (name)),
-                  attr_string (value, strlen (value)));
+  return attr_rtx (EQ_ATTR, DEF_ATTR_STRING (name), DEF_ATTR_STRING (value));
 }
 
 static const char *
-attr_numeral (n)
-     int n;
+attr_numeral (int n)
 {
   return XSTR (make_numeric_value (n), 0);
 }
@@ -774,9 +582,7 @@ attr_numeral (n)
    to be null terminated) with LEN bytes.  */
 
 static char *
-attr_string (str, len)
-     const char *str;
-     int len;
+attr_string (const char *str, int len)
 {
   struct attr_hash *h;
   int hashcode;
@@ -785,7 +591,7 @@ attr_string (str, len)
 
   /* Compute the hash code.  */
   hashcode = (len + 1) * 613 + (unsigned) str[0];
-  for (i = 1; i <= len; i += 2)
+  for (i = 1; i < len; i += 2)
     hashcode = ((hashcode * 613) + (unsigned) str[i]);
   if (hashcode < 0)
     hashcode = -hashcode;
@@ -797,7 +603,7 @@ attr_string (str, len)
       return h->u.str;                 /* <-- return if found.  */
 
   /* Not found; create a permanent copy and add it to the hash table.  */
-  new_str = (char *) obstack_alloc (hash_obstack, len + 1);
+  new_str = XOBNEWVAR (hash_obstack, char, len + 1);
   memcpy (new_str, str, len);
   new_str[len] = '\0';
   attr_hash_add_string (hashcode, new_str);
@@ -810,20 +616,18 @@ attr_string (str, len)
    then they can't be equal unless they are the same object.  */
 
 static int
-attr_equal_p (x, y)
-     rtx x, y;
+attr_equal_p (rtx x, rtx y)
 {
   return (x == y || (! (ATTR_PERMANENT_P (x) && ATTR_PERMANENT_P (y))
                     && rtx_equal_p (x, y)));
 }
-\f
+
 /* Copy an attribute value expression,
    descending to all depths, but not copying any
    permanent hashed subexpressions.  */
 
 static rtx
-attr_copy_rtx (orig)
-     rtx orig;
+attr_copy_rtx (rtx orig)
 {
   rtx copy;
   int i, j;
@@ -839,7 +643,6 @@ attr_copy_rtx (orig)
   switch (code)
     {
     case REG:
-    case QUEUED:
     case CONST_INT:
     case CONST_DOUBLE:
     case CONST_VECTOR:
@@ -858,7 +661,6 @@ attr_copy_rtx (orig)
   ATTR_IND_SIMPLIFIED_P (copy) = ATTR_IND_SIMPLIFIED_P (orig);
   ATTR_CURR_SIMPLIFIED_P (copy) = ATTR_CURR_SIMPLIFIED_P (orig);
   ATTR_PERMANENT_P (copy) = ATTR_PERMANENT_P (orig);
-  ATTR_EQ_ATTR_P (copy) = ATTR_EQ_ATTR_P (orig);
 
   format_ptr = GET_RTX_FORMAT (GET_CODE (copy));
 
@@ -898,12 +700,12 @@ attr_copy_rtx (orig)
          break;
 
        default:
-         abort ();
+         gcc_unreachable ();
        }
     }
   return copy;
 }
-\f
+
 /* Given a test expression for an attribute, ensure it is validly formed.
    IS_CONST indicates whether the expression is constant for each compiler
    run (a constant expression may not test any particular insn).
@@ -918,11 +720,8 @@ attr_copy_rtx (orig)
 
    Return the new expression, if any.  */
 
-rtx
-check_attr_test (exp, is_const, lineno)
-     rtx exp;
-     int is_const;
-     int lineno;
+static rtx
+check_attr_test (rtx exp, int is_const, int lineno)
 {
   struct attr_desc *attr;
   struct attr_value *av;
@@ -941,16 +740,11 @@ check_attr_test (exp, is_const, lineno)
 
       else if (n_comma_elts (XSTR (exp, 1)) == 1)
        {
-         attr = find_attr (XSTR (exp, 0), 0);
+         attr = find_attr (&XSTR (exp, 0), 0);
          if (attr == NULL)
            {
              if (! strcmp (XSTR (exp, 0), "alternative"))
-               {
-                 XSTR (exp, 0) = alternative_name;
-                 /* This can't be simplified any further.  */
-                 ATTR_IND_SIMPLIFIED_P (exp) = 1;
-                 return exp;
-               }
+               return mk_attr_alt (1 << atoi (XSTR (exp, 1)));
              else
                fatal ("unknown attribute `%s' in EQ_ATTR", XSTR (exp, 0));
            }
@@ -990,16 +784,29 @@ check_attr_test (exp, is_const, lineno)
        }
       else
        {
-         /* Make an IOR tree of the possible values.  */
-         orexp = false_rtx;
-         name_ptr = XSTR (exp, 1);
-         while ((p = next_comma_elt (&name_ptr)) != NULL)
+         if (! strcmp (XSTR (exp, 0), "alternative"))
            {
-             newexp = attr_eq (XSTR (exp, 0), p);
-             orexp = insert_right_side (IOR, orexp, newexp, -2, -2);
+             int set = 0;
+
+             name_ptr = XSTR (exp, 1);
+             while ((p = next_comma_elt (&name_ptr)) != NULL)
+               set |= 1 << atoi (p);
+
+             return mk_attr_alt (set);
            }
+         else
+           {
+             /* Make an IOR tree of the possible values.  */
+             orexp = false_rtx;
+             name_ptr = XSTR (exp, 1);
+             while ((p = next_comma_elt (&name_ptr)) != NULL)
+               {
+                 newexp = attr_eq (XSTR (exp, 0), p);
+                 orexp = insert_right_side (IOR, orexp, newexp, -2, -2);
+               }
 
-         return check_attr_test (orexp, is_const, lineno);
+             return check_attr_test (orexp, is_const, lineno);
+           }
        }
       break;
 
@@ -1023,7 +830,6 @@ check_attr_test (exp, is_const, lineno)
       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",
@@ -1060,7 +866,7 @@ check_attr_test (exp, is_const, lineno)
 
   return exp;
 }
-\f
+
 /* Given an expression, ensure that it is validly formed and that all named
    attribute values are valid for the given attribute.  Issue a fatal error
    if not.  If no attribute is specified, assume a numeric attribute.
@@ -1068,9 +874,7 @@ check_attr_test (exp, is_const, lineno)
    Return a perhaps modified replacement expression for the value.  */
 
 static rtx
-check_attr_value (exp, attr)
-     rtx exp;
-     struct attr_desc *attr;
+check_attr_value (rtx exp, struct attr_desc *attr)
 {
   struct attr_value *av;
   const char *p;
@@ -1088,7 +892,7 @@ check_attr_value (exp, attr)
          break;
        }
 
-      if (INTVAL (exp) < 0 && ! attr->negative_ok)
+      if (INTVAL (exp) < 0)
        {
          message_with_line (attr->lineno,
                             "negative numeric value specified for attribute %s",
@@ -1105,8 +909,6 @@ check_attr_value (exp, attr)
       if (attr == 0 || attr->is_numeric)
        {
          p = XSTR (exp, 0);
-         if (attr && attr->negative_ok && *p == '-')
-           p++;
          for (; *p; p++)
            if (! ISDIGIT (*p))
              {
@@ -1154,7 +956,7 @@ check_attr_value (exp, attr)
          have_error = 1;
          break;
        }
-      /* FALLTHRU */
+      /* Fall through.  */
 
     case IOR:
     case AND:
@@ -1163,6 +965,11 @@ check_attr_value (exp, attr)
       break;
 
     case FFS:
+    case CLZ:
+    case CTZ:
+    case POPCOUNT:
+    case PARITY:
+    case BSWAP:
       XEXP (exp, 0) = check_attr_value (XEXP (exp, 0), attr);
       break;
 
@@ -1189,7 +996,7 @@ check_attr_value (exp, attr)
 
     case ATTR:
       {
-       struct attr_desc *attr2 = find_attr (XSTR (exp, 0), 0);
+       struct attr_desc *attr2 = find_attr (&XSTR (exp, 0), 0);
        if (attr2 == NULL)
          {
            message_with_line (attr ? attr->lineno : 0,
@@ -1205,8 +1012,7 @@ check_attr_value (exp, attr)
            have_error = 1;
          }
        else if (attr
-                && (attr->is_numeric != attr2->is_numeric
-                    || (! attr->negative_ok && attr2->negative_ok)))
+                && attr->is_numeric != attr2->is_numeric)
          {
            message_with_line (attr->lineno,
                "numeric attribute mismatch calling `%s' from `%s'",
@@ -1232,14 +1038,12 @@ check_attr_value (exp, attr)
 
   return exp;
 }
-\f
+
 /* Given an SET_ATTR_ALTERNATIVE expression, convert to the canonical SET.
-   It becomes a COND with each test being (eq_attr "alternative "n") */
+   It becomes a COND with each test being (eq_attr "alternative" "n") */
 
 static rtx
-convert_set_attr_alternative (exp, id)
-     rtx exp;
-     struct insn_def *id;
+convert_set_attr_alternative (rtx exp, struct insn_def *id)
 {
   int num_alt = id->num_alternatives;
   rtx condexp;
@@ -1271,14 +1075,12 @@ convert_set_attr_alternative (exp, id)
 
   return attr_rtx (SET, attr_rtx (ATTR, XSTR (exp, 0)), condexp);
 }
-\f
+
 /* Given a SET_ATTR, convert to the appropriate SET.  If a comma-separated
    list of values is given, convert to SET_ATTR_ALTERNATIVE first.  */
 
 static rtx
-convert_set_attr (exp, id)
-     rtx exp;
-     struct insn_def *id;
+convert_set_attr (rtx exp, struct insn_def *id)
 {
   rtx newexp;
   const char *name_ptr;
@@ -1304,13 +1106,13 @@ convert_set_attr (exp, id)
 
   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.  */
 
 static void
-check_defs ()
+check_defs (void)
 {
   struct insn_def *id;
   struct attr_desc *attr;
@@ -1353,7 +1155,7 @@ check_defs ()
          if (value == NULL_RTX)
            continue;
 
-         if ((attr = find_attr (XSTR (XEXP (value, 0), 0), 0)) == NULL)
+         if ((attr = find_attr (&XSTR (XEXP (value, 0), 0), 0)) == NULL)
            {
              message_with_line (id->lineno, "unknown attribute %s",
                                 XSTR (XEXP (value, 0), 0));
@@ -1366,68 +1168,14 @@ check_defs ()
        }
     }
 }
-\f
-#if 0
-/* Given a constant SYMBOL_REF expression, convert to a COND that
-   explicitly tests each enumerated value.  */
-
-static rtx
-convert_const_symbol_ref (exp, attr)
-     rtx exp;
-     struct attr_desc *attr;
-{
-  rtx condexp;
-  struct attr_value *av;
-  int i;
-  int num_alt = 0;
-
-  for (av = attr->first_value; av; av = av->next)
-    num_alt++;
-
-  /* Make a COND with all tests but the last, and in the original order.
-     Select the last value via the default.  Note that the attr values
-     are constructed in reverse order.  */
-
-  condexp = rtx_alloc (COND);
-  XVEC (condexp, 0) = rtvec_alloc ((num_alt - 1) * 2);
-  av = attr->first_value;
-  XEXP (condexp, 1) = av->value;
 
-  for (i = num_alt - 2; av = av->next, i >= 0; i--)
-    {
-      char *p, *string;
-      rtx value;
-
-      string = p = (char *) oballoc (2
-                                    + strlen (attr->name)
-                                    + strlen (XSTR (av->value, 0)));
-      strcpy (p, attr->name);
-      strcat (p, "_");
-      strcat (p, XSTR (av->value, 0));
-      for (; *p != '\0'; p++)
-       *p = TOUPPER (*p);
-
-      value = attr_rtx (SYMBOL_REF, string);
-      ATTR_IND_SIMPLIFIED_P (value) = 1;
-
-      XVECEXP (condexp, 0, 2 * i) = attr_rtx (EQ, exp, value);
-
-      XVECEXP (condexp, 0, 2 * i + 1) = av->value;
-    }
-
-  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
    program.  Also, replace an attribute value of "*" with the default attribute
    value.  */
 
 static rtx
-make_canonical (attr, exp)
-     struct attr_desc *attr;
-     rtx exp;
+make_canonical (struct attr_desc *attr, rtx exp)
 {
   int i;
   rtx newexp;
@@ -1445,6 +1193,8 @@ make_canonical (attr, exp)
            fatal ("(attr_value \"*\") used in invalid context");
          exp = attr->default_val->value;
        }
+      else
+       XSTR (exp, 0) = DEF_ATTR_STRING (XSTR (exp, 0));
 
       break;
 
@@ -1455,21 +1205,8 @@ make_canonical (attr, exp)
         This makes the COND something that won't be considered an arbitrary
         expression by walk_attr_value.  */
       ATTR_IND_SIMPLIFIED_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);
-      ATTR_IND_SIMPLIFIED_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 marked as
-        unchanging.  */
-      goto cond;
-#else
       exp = check_attr_value (exp, attr);
       break;
-#endif
 
     case IF_THEN_ELSE:
       newexp = rtx_alloc (COND);
@@ -1513,15 +1250,25 @@ make_canonical (attr, exp)
 }
 
 static rtx
-copy_boolean (exp)
-     rtx exp;
+copy_boolean (rtx exp)
 {
   if (GET_CODE (exp) == AND || GET_CODE (exp) == IOR)
     return attr_rtx (GET_CODE (exp), copy_boolean (XEXP (exp, 0)),
                     copy_boolean (XEXP (exp, 1)));
+  if (GET_CODE (exp) == MATCH_OPERAND)
+    {
+      XSTR (exp, 1) = DEF_ATTR_STRING (XSTR (exp, 1));
+      XSTR (exp, 2) = DEF_ATTR_STRING (XSTR (exp, 2));
+    }
+  else if (GET_CODE (exp) == EQ_ATTR)
+    {
+      XSTR (exp, 0) = DEF_ATTR_STRING (XSTR (exp, 0));
+      XSTR (exp, 1) = DEF_ATTR_STRING (XSTR (exp, 1));
+    }
+
   return exp;
 }
-\f
+
 /* Given a value and an attribute description, return a `struct attr_value *'
    that represents that value.  This is either an existing structure, if the
    value has been previously encountered, or a newly-created structure.
@@ -1532,10 +1279,7 @@ copy_boolean (exp)
    alternatives.  */
 
 static struct attr_value *
-get_attr_value (value, attr, insn_code)
-     rtx value;
-     struct attr_desc *attr;
-     int insn_code;
+get_attr_value (rtx value, struct attr_desc *attr, int insn_code)
 {
   struct attr_value *av;
   int num_alt = 0;
@@ -1552,10 +1296,10 @@ get_attr_value (value, attr, insn_code)
   for (av = attr->first_value; av; av = av->next)
     if (rtx_equal_p (value, av->value)
        && (num_alt == 0 || av->first_insn == NULL
-           || insn_alternatives[av->first_insn->insn_code]))
+           || insn_alternatives[av->first_insn->def->insn_code]))
       return av;
 
-  av = (struct attr_value *) oballoc (sizeof (struct attr_value));
+  av = oballoc (struct attr_value);
   av->value = value;
   av->next = attr->first_value;
   attr->first_value = av;
@@ -1565,7 +1309,7 @@ get_attr_value (value, attr, insn_code)
 
   return av;
 }
-\f
+
 /* After all DEFINE_DELAYs have been read in, create internal attributes
    to generate the required routines.
 
@@ -1581,7 +1325,7 @@ get_attr_value (value, attr, insn_code)
    information needed to handle delay slots.  */
 
 static void
-expand_delays ()
+expand_delays (void)
 {
   struct delay_desc *delay;
   rtx condexp;
@@ -1602,7 +1346,7 @@ expand_delays ()
        = make_numeric_value (XVECLEN (delay->def, 1) / 3);
     }
 
-  make_internal_attr ("*num_delay_slots", condexp, 0);
+  make_internal_attr (num_delay_slots_str, condexp, ATTR_NONE);
 
   /* If more than one delay type, do the same for computing the delay type.  */
   if (num_delays > 1)
@@ -1617,7 +1361,7 @@ expand_delays ()
          XVECEXP (condexp, 0, i + 1) = make_numeric_value (delay->num);
        }
 
-      make_internal_attr ("*delay_type", condexp, 1);
+      make_internal_attr (delay_type_str, condexp, ATTR_SPECIAL);
     }
 
   /* For each delay possibility and delay slot, compute an eligibility
@@ -1635,7 +1379,7 @@ expand_delays ()
 
          p = attr_printf (sizeof "*delay__" + MAX_DIGITS * 2,
                           "*delay_%d_%d", delay->num, i / 3);
-         make_internal_attr (p, newexp, 1);
+         make_internal_attr (p, newexp, ATTR_SPECIAL);
 
          if (have_annul_true)
            {
@@ -1646,7 +1390,7 @@ expand_delays ()
                                 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);
+             make_internal_attr (p, newexp, ATTR_SPECIAL);
            }
 
          if (have_annul_false)
@@ -1658,1328 +1402,789 @@ expand_delays ()
                                 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);
+             make_internal_attr (p, newexp, ATTR_SPECIAL);
            }
        }
     }
 }
-\f
-/* This function is given a left and right side expression and an operator.
-   Each side is a conditional expression, each alternative of which has a
-   numerical value.  The function returns another conditional expression
-   which, for every possible set of condition values, returns a value that is
-   the operator applied to the values of the two sides.
 
-   Since this is called early, it must also support IF_THEN_ELSE.  */
+/* Once all attributes and insns have been read and checked, we construct for
+   each attribute value a list of all the insns that have that value for
+   the attribute.  */
 
-static rtx
-operate_exp (op, left, right)
-     enum operator op;
-     rtx left, right;
+static void
+fill_attr (struct attr_desc *attr)
 {
-  int left_value, right_value;
-  rtx newexp;
+  struct attr_value *av;
+  struct insn_ent *ie;
+  struct insn_def *id;
   int i;
+  rtx value;
 
-  /* If left is a string, apply operator to it and the right side.  */
-  if (GET_CODE (left) == CONST_STRING)
-    {
-      /* If right is also a string, just perform the operation.  */
-      if (GET_CODE (right) == CONST_STRING)
-       {
-         left_value = atoi (XSTR (left, 0));
-         right_value = atoi (XSTR (right, 0));
-         switch (op)
-           {
-           case PLUS_OP:
-             i = left_value + right_value;
-             break;
-
-           case MINUS_OP:
-             i = left_value - right_value;
-             break;
+  /* Don't fill constant attributes.  The value is independent of
+     any particular insn.  */
+  if (attr->is_const)
+    return;
 
-           case POS_MINUS_OP:  /* The positive part of LEFT - RIGHT.  */
-             if (left_value > right_value)
-               i = left_value - right_value;
-             else
-               i = 0;
-             break;
+  for (id = defs; id; id = id->next)
+    {
+      /* If no value is specified for this insn for this attribute, use the
+        default.  */
+      value = NULL;
+      if (XVEC (id->def, id->vec_idx))
+       for (i = 0; i < XVECLEN (id->def, id->vec_idx); i++)
+         if (! strcmp_check (XSTR (XEXP (XVECEXP (id->def, id->vec_idx, i), 0), 0),
+                             attr->name))
+           value = XEXP (XVECEXP (id->def, id->vec_idx, i), 1);
 
-           case OR_OP:
-           case ORX_OP:
-             i = left_value | right_value;
-             break;
+      if (value == NULL)
+       av = attr->default_val;
+      else
+       av = get_attr_value (value, attr, id->insn_code);
 
-           case EQ_OP:
-             i = left_value == right_value;
-             break;
+      ie = oballoc (struct insn_ent);
+      ie->def = id;
+      insert_insn_ent (av, ie);
+    }
+}
 
-           case RANGE_OP:
-             i = (left_value << (HOST_BITS_PER_INT / 2)) | right_value;
-             break;
+/* Given an expression EXP, see if it is a COND or IF_THEN_ELSE that has a
+   test that checks relative positions of insns (uses MATCH_DUP or PC).
+   If so, replace it with what is obtained by passing the expression to
+   ADDRESS_FN.  If not but it is a COND or IF_THEN_ELSE, call this routine
+   recursively on each value (including the default value).  Otherwise,
+   return the value returned by NO_ADDRESS_FN applied to EXP.  */
 
-           case MAX_OP:
-             if (left_value > right_value)
-               i = left_value;
-             else
-               i = right_value;
-             break;
+static rtx
+substitute_address (rtx exp, rtx (*no_address_fn) (rtx),
+                   rtx (*address_fn) (rtx))
+{
+  int i;
+  rtx newexp;
 
-           case MIN_OP:
-             if (left_value < right_value)
-               i = left_value;
-             else
-               i = right_value;
-             break;
+  if (GET_CODE (exp) == COND)
+    {
+      /* See if any tests use addresses.  */
+      address_used = 0;
+      for (i = 0; i < XVECLEN (exp, 0); i += 2)
+       walk_attr_value (XVECEXP (exp, 0, i));
 
-           default:
-             abort ();
-           }
+      if (address_used)
+       return (*address_fn) (exp);
 
-         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)
+      /* Make a new copy of this COND, replacing each element.  */
+      newexp = rtx_alloc (COND);
+      XVEC (newexp, 0) = rtvec_alloc (XVECLEN (exp, 0));
+      for (i = 0; i < XVECLEN (exp, 0); i += 2)
        {
-         /* Apply recursively to all values within.  */
-         rtx newleft = operate_exp (op, left, XEXP (right, 1));
-         rtx newright = operate_exp (op, left, XEXP (right, 2));
-         if (rtx_equal_p (newleft, newright))
-           return newleft;
-         return attr_rtx (IF_THEN_ELSE, XEXP (right, 0), newleft, newright);
+         XVECEXP (newexp, 0, i) = XVECEXP (exp, 0, i);
+         XVECEXP (newexp, 0, i + 1)
+           = substitute_address (XVECEXP (exp, 0, i + 1),
+                                 no_address_fn, address_fn);
        }
-      else if (GET_CODE (right) == COND)
-       {
-         int allsame = 1;
-         rtx defval;
-
-         newexp = rtx_alloc (COND);
-         XVEC (newexp, 0) = rtvec_alloc (XVECLEN (right, 0));
-         defval = XEXP (newexp, 1) = operate_exp (op, left, XEXP (right, 1));
-
-         for (i = 0; i < XVECLEN (right, 0); i += 2)
-           {
-             XVECEXP (newexp, 0, i) = XVECEXP (right, 0, i);
-             XVECEXP (newexp, 0, i + 1)
-               = operate_exp (op, left, XVECEXP (right, 0, i + 1));
-             if (! rtx_equal_p (XVECEXP (newexp, 0, i + 1),
-                                defval))
-               allsame = 0;
-           }
 
-         /* If the resulting cond is trivial (all alternatives
-            give the same value), optimize it away.  */
-         if (allsame)
-           return operate_exp (op, left, XEXP (right, 1));
+      XEXP (newexp, 1) = substitute_address (XEXP (exp, 1),
+                                            no_address_fn, address_fn);
 
-         return newexp;
-       }
-      else
-       fatal ("badly formed attribute value");
+      return newexp;
     }
 
-  /* 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)
+  else if (GET_CODE (exp) == IF_THEN_ELSE)
     {
-      return attr_rtx (IOR, left, right);
-    }
+      address_used = 0;
+      walk_attr_value (XEXP (exp, 0));
+      if (address_used)
+       return (*address_fn) (exp);
 
-  /* Otherwise, do recursion the other way.  */
-  else if (GET_CODE (left) == IF_THEN_ELSE)
-    {
-      rtx newleft = operate_exp (op, XEXP (left, 1), right);
-      rtx newright = operate_exp (op, XEXP (left, 2), right);
-      if (rtx_equal_p (newleft, newright))
-       return newleft;
-      return attr_rtx (IF_THEN_ELSE, XEXP (left, 0), newleft, newright);
+      return attr_rtx (IF_THEN_ELSE,
+                      substitute_address (XEXP (exp, 0),
+                                          no_address_fn, address_fn),
+                      substitute_address (XEXP (exp, 1),
+                                          no_address_fn, address_fn),
+                      substitute_address (XEXP (exp, 2),
+                                          no_address_fn, address_fn));
     }
-  else if (GET_CODE (left) == COND)
-    {
-      int allsame = 1;
-      rtx defval;
-
-      newexp = rtx_alloc (COND);
-      XVEC (newexp, 0) = rtvec_alloc (XVECLEN (left, 0));
-      defval = XEXP (newexp, 1) = operate_exp (op, XEXP (left, 1), right);
-
-      for (i = 0; i < XVECLEN (left, 0); i += 2)
-       {
-         XVECEXP (newexp, 0, i) = XVECEXP (left, 0, i);
-         XVECEXP (newexp, 0, i + 1)
-           = operate_exp (op, XVECEXP (left, 0, i + 1), right);
-         if (! rtx_equal_p (XVECEXP (newexp, 0, i + 1),
-                            defval))
-           allsame = 0;
-       }
-
-      /* If the cond is trivial (all alternatives give the same value),
-        optimize it away.  */
-      if (allsame)
-       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))
-       return left;
+  return (*no_address_fn) (exp);
+}
 
-      return newexp;
-    }
+/* Make new attributes from the `length' attribute.  The following are made,
+   each corresponding to a function called from `shorten_branches' or
+   `get_attr_length':
 
-  else
-    fatal ("badly formed attribute value");
-  /* NOTREACHED */
-  return NULL;
-}
-\f
-/* Once all attributes and DEFINE_FUNCTION_UNITs have been read, we
-   construct a number of attributes.
+   *insn_default_length                This is the length of the insn to be returned
+                               by `get_attr_length' before `shorten_branches'
+                               has been called.  In each case where the length
+                               depends on relative addresses, the largest
+                               possible is used.  This routine is also used
+                               to compute the initial size of the insn.
 
-   The first produces a function `function_units_used' which is given an
-   insn and produces an encoding showing which function units are required
-   for the execution of that insn.  If the value is non-negative, the insn
-   uses that unit; otherwise, the value is a one's compliment mask of units
-   used.
+   *insn_variable_length_p     This returns 1 if the insn's length depends
+                               on relative addresses, zero otherwise.
 
-   The second produces a function `result_ready_cost' which is used to
-   determine the time that the result of an insn will be ready and hence
-   a worst-case schedule.
-
-   Both of these produce quite complex expressions which are then set as the
-   default value of internal attributes.  Normal attribute simplification
-   should produce reasonable expressions.
-
-   For each unit, a `<name>_unit_ready_cost' function will take an
-   insn and give the delay until that unit will be ready with the result
-   and a `<name>_unit_conflict_cost' function is given an insn already
-   executing on the unit and a candidate to execute and will give the
-   cost from the time the executing insn started until the candidate
-   can start (ignore limitations on the number of simultaneous insns).
-
-   For each unit, a `<name>_unit_blockage' function is given an insn
-   already executing on the unit and a candidate to execute and will
-   give the delay incurred due to function unit conflicts.  The range of
-   blockage cost values for a given executing insn is given by the
-   `<name>_unit_blockage_range' function.  These values are encoded in
-   an int where the upper half gives the minimum value and the lower
-   half gives the maximum value.  */
+   *insn_current_length                This is only called when it is known that the
+                               insn has a variable length and returns the
+                               current length, based on relative addresses.
+  */
 
 static void
-expand_units ()
+make_length_attrs (void)
 {
-  struct function_unit *unit, **unit_num;
-  struct function_unit_op *op, **op_array, ***unit_ops;
-  rtx unitsmask;
-  rtx readycost;
-  rtx newexp;
-  const char *str;
-  int i, j, u, num, nvalues;
+  static const char *new_names[] =
+    {
+      "*insn_default_length",
+      "*insn_min_length",
+      "*insn_variable_length_p",
+      "*insn_current_length"
+    };
+  static rtx (*const no_address_fn[]) (rtx)
+    = {identity_fn,identity_fn, zero_fn, zero_fn};
+  static rtx (*const address_fn[]) (rtx)
+    = {max_fn, min_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;
 
-  /* Rebuild the condition for the unit to share the RTL expressions.
-     Sharing is required by simplify_by_exploding.  Build the issue delay
-     expressions.  Validate the expressions we were given for the conditions
-     and conflict vector.  Then make attributes for use in the conflict
-     function.  */
+  /* See if length attribute is defined.  If so, it must be numeric.  Make
+     it special so we don't output anything for it.  */
+  length_attr = find_attr (&length_str, 0);
+  if (length_attr == 0)
+    return;
 
-  for (unit = units; unit; unit = unit->next)
-    {
-      unit->condexp = check_attr_test (unit->condexp, 0, unit->first_lineno);
+  if (! length_attr->is_numeric)
+    fatal ("length attribute must be numeric");
 
-      for (op = unit->ops; op; op = op->next)
-       {
-         rtx issue_delay = make_numeric_value (op->issue_delay);
-         rtx issue_exp = issue_delay;
-
-         /* Build, validate, and simplify the issue delay expression.  */
-         if (op->conflict_exp != true_rtx)
-           issue_exp = attr_rtx (IF_THEN_ELSE, op->conflict_exp,
-                                 issue_exp, make_numeric_value (0));
-         issue_exp = check_attr_value (make_canonical (NULL_ATTR,
-                                                       issue_exp),
-                                       NULL_ATTR);
-         issue_exp = simplify_knowing (issue_exp, unit->condexp);
-         op->issue_exp = issue_exp;
-
-         /* Make an attribute for use in the conflict function if needed.  */
-         unit->needs_conflict_function = (unit->issue_delay.min
-                                          != unit->issue_delay.max);
-         if (unit->needs_conflict_function)
-           {
-             str = attr_printf ((strlen (unit->name) + sizeof "*_cost_"
-                                 + MAX_DIGITS),
-                                "*%s_cost_%d", unit->name, op->num);
-             make_internal_attr (str, issue_exp, 1);
-           }
-
-         /* Validate the condition.  */
-         op->condexp = check_attr_test (op->condexp, 0, op->lineno);
-       }
-    }
-
-  /* Compute the mask of function units used.  Initially, the unitsmask is
-     zero.   Set up a conditional to compute each unit's contribution.  */
-  unitsmask = make_numeric_value (0);
-  newexp = rtx_alloc (IF_THEN_ELSE);
-  XEXP (newexp, 2) = make_numeric_value (0);
-
-  /* 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
+  length_attr->is_const = 0;
+  length_attr->is_special = 1;
 
-  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
+  /* Make each new attribute, in turn.  */
+  for (i = 0; i < ARRAY_SIZE (new_names); i++)
     {
-      /* 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));
-       }
+      make_internal_attr (new_names[i],
+                         substitute_address (length_attr->default_val->value,
+                                             no_address_fn[i], address_fn[i]),
+                         ATTR_NONE);
+      new_attr = find_attr (&new_names[i], 0);
+      for (av = length_attr->first_value; av; av = av->next)
+       for (ie = av->first_insn; ie; ie = ie->next)
+         {
+           new_av = get_attr_value (substitute_address (av->value,
+                                                        no_address_fn[i],
+                                                        address_fn[i]),
+                                    new_attr, ie->def->insn_code);
+           new_ie = oballoc (struct insn_ent);
+           new_ie->def = ie->def;
+           insert_insn_ent (new_av, new_ie);
+         }
     }
+}
 
-  /* Simplify the unit mask expression, encode it, and make an attribute
-     for the function_units_used function.  */
-  unitsmask = simplify_by_exploding (unitsmask);
-
-  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);
-    }
+/* Utility functions called from above routine.  */
 
-  make_internal_attr ("*function_units_used", unitsmask, 10);
+static rtx
+identity_fn (rtx exp)
+{
+  return exp;
+}
 
-  /* 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.  */
-  unit_ops = (struct function_unit_op ***)
-    xmalloc ((num_units + 1) * sizeof (struct function_unit_op **));
-  unit_num = (struct function_unit **)
-    xmalloc ((num_units + 1) * sizeof (struct function_unit *));
+static rtx
+zero_fn (rtx exp ATTRIBUTE_UNUSED)
+{
+  return make_numeric_value (0);
+}
 
-  unit_num[num_units] = unit = (struct function_unit *)
-    xmalloc (sizeof (struct function_unit));
-  unit->num = num_units;
-  unit->num_opclasses = 0;
+static rtx
+one_fn (rtx exp ATTRIBUTE_UNUSED)
+{
+  return make_numeric_value (1);
+}
 
-  for (unit = units; unit; unit = unit->next)
-    {
-      unit_num[num_units]->num_opclasses += unit->num_opclasses;
-      unit_num[unit->num] = unit;
-      unit_ops[unit->num] = op_array = (struct function_unit_op **)
-       xmalloc (unit->num_opclasses * sizeof (struct function_unit_op *));
+static rtx
+max_fn (rtx exp)
+{
+  int unknown;
+  return make_numeric_value (max_attr_value (exp, &unknown));
+}
 
-      for (op = unit->ops; op; op = op->next)
-       op_array[op->num] = op;
-    }
+static rtx
+min_fn (rtx exp)
+{
+  int unknown;
+  return make_numeric_value (min_attr_value (exp, &unknown));
+}
 
-  /* Compose the array of ops for the extra unit.  */
-  unit_ops[num_units] = op_array = (struct function_unit_op **)
-    xmalloc (unit_num[num_units]->num_opclasses
-           * sizeof (struct function_unit_op *));
+static void
+write_length_unit_log (void)
+{
+  struct attr_desc *length_attr = find_attr (&length_str, 0);
+  struct attr_value *av;
+  struct insn_ent *ie;
+  unsigned int length_unit_log, length_or;
+  int unknown = 0;
 
-  for (unit = units, i = 0; unit; i += unit->num_opclasses, unit = unit->next)
-    memcpy (&op_array[i], unit_ops[unit->num],
-           unit->num_opclasses * sizeof (struct function_unit_op *));
+  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);
 
-  /* Compute the ready cost function for each unit by computing the
-     condition for each non-default value.  */
-  for (u = 0; u <= num_units; u++)
+  if (unknown)
+    length_unit_log = 0;
+  else
     {
-      rtx orexp;
-      int value;
-
-      unit = unit_num[u];
-      op_array = unit_ops[unit->num];
-      num = unit->num_opclasses;
-
-      /* 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)
-           {
-             op = op_array[j];
-             op_array[j] = op_array[j - 1];
-             op_array[j - 1] = op;
-           }
-
-      /* Determine how many distinct non-default ready cost values there
-        are.  We use a default ready cost value of 1.  */
-      nvalues = 0; value = 1;
-      for (i = num - 1; i >= 0; i--)
-       if (op_array[i]->ready > value)
-         {
-           value = op_array[i]->ready;
-           nvalues++;
-         }
-
-      if (nvalues == 0)
-       readycost = make_numeric_value (1);
-      else
-       {
-         /* Construct the ready cost expression as a COND of each value from
-            the largest to the smallest.  */
-         readycost = rtx_alloc (COND);
-         XVEC (readycost, 0) = rtvec_alloc (nvalues * 2);
-         XEXP (readycost, 1) = make_numeric_value (1);
-
-         nvalues = 0;
-         orexp = false_rtx;
-         value = op_array[0]->ready;
-         for (i = 0; i < num; i++)
-           {
-             op = op_array[i];
-             if (op->ready <= 1)
-               break;
-             else if (op->ready == value)
-               orexp = insert_right_side (IOR, orexp, op->condexp, -2, -2);
-             else
-               {
-                 XVECEXP (readycost, 0, nvalues * 2) = orexp;
-                 XVECEXP (readycost, 0, nvalues * 2 + 1)
-                   = make_numeric_value (value);
-                 nvalues++;
-                 value = op->ready;
-                 orexp = op->condexp;
-               }
-           }
-         XVECEXP (readycost, 0, nvalues * 2) = orexp;
-         XVECEXP (readycost, 0, nvalues * 2 + 1) = make_numeric_value (value);
-       }
-
-      if (u < num_units)
-       {
-         rtx max_blockage = 0, min_blockage = 0;
-
-         /* Simplify the readycost expression by only considering insns
-            that use the unit.  */
-         readycost = simplify_knowing (readycost, unit->condexp);
+      length_or = ~length_or;
+      for (length_unit_log = 0; length_or & 1; length_or >>= 1)
+       length_unit_log++;
+    }
+  printf ("EXPORTED_CONST int length_unit_log = %u;\n", length_unit_log);
+}
 
-         /* Determine the blockage cost the executing insn (E) given
-            the candidate insn (C).  This is the maximum of the issue
-            delay, the pipeline delay, and the simultaneity constraint.
-            Each function_unit_op represents the characteristics of the
-            candidate insn, so in the expressions below, C is a known
-            term and E is an unknown term.
+/* Take a COND expression and see if any of the conditions in it can be
+   simplified.  If any are known true or known false for the particular insn
+   code, the COND can be further simplified.
 
-            We compute the blockage cost for each E for every possible C.
-            Thus OP represents E, and READYCOST is a list of values for
-            every possible C.
+   Also call ourselves on any COND operations that are values of this COND.
 
-            The issue delay function for C is op->issue_exp and is used to
-            write the `<name>_unit_conflict_cost' function.  Symbolicly
-            this is "ISSUE-DELAY (E,C)".
+   We do not modify EXP; rather, we make and return a new rtx.  */
 
-            The pipeline delay results form the FIFO constraint on the
-            function unit and is "READY-COST (E) + 1 - READY-COST (C)".
+static rtx
+simplify_cond (rtx exp, int insn_code, int insn_index)
+{
+  int i, j;
+  /* We store the desired contents here,
+     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 = XNEWVEC (rtx, len);
+  int allsame = 1;
+  rtx ret;
 
-            The simultaneity constraint is based on how long it takes to
-            fill the unit given the minimum issue delay.  FILL-TIME is the
-            constant "MIN (ISSUE-DELAY (*,*)) * (SIMULTANEITY - 1)", and
-            the simultaneity constraint is "READY-COST (E) - FILL-TIME"
-            if SIMULTANEITY is non-zero and zero otherwise.
+  /* This lets us free all storage allocated below, if appropriate.  */
+  obstack_finish (rtl_obstack);
 
-            Thus, BLOCKAGE (E,C) when SIMULTANEITY is zero is
+  memcpy (tests, XVEC (exp, 0)->elem, len * sizeof (rtx));
 
-                MAX (ISSUE-DELAY (E,C),
-                     READY-COST (E) - (READY-COST (C) - 1))
+  /* See if default value needs simplification.  */
+  if (GET_CODE (defval) == COND)
+    new_defval = simplify_cond (defval, insn_code, insn_index);
 
-            and otherwise
+  /* Simplify the subexpressions, and see what tests we can get rid of.  */
 
-                MAX (ISSUE-DELAY (E,C),
-                     READY-COST (E) - (READY-COST (C) - 1),
-                     READY-COST (E) - FILL-TIME)
+  for (i = 0; i < len; i += 2)
+    {
+      rtx newtest, newval;
 
-            The `<name>_unit_blockage' function is computed by determining
-            this value for each candidate insn.  As these values are
-            computed, we also compute the upper and lower bounds for
-            BLOCKAGE (E,*).  These are combined to form the function
-            `<name>_unit_blockage_range'.  Finally, the maximum blockage
-            cost, MAX (BLOCKAGE (*,*)), is computed.  */
+      /* Simplify this test.  */
+      newtest = simplify_test_exp_in_temp (tests[i], insn_code, insn_index);
+      tests[i] = newtest;
 
-         for (op = unit->ops; op; op = op->next)
-           {
-             rtx blockage = op->issue_exp;
-             blockage = simplify_knowing (blockage, unit->condexp);
+      newval = tests[i + 1];
+      /* See if this value may need simplification.  */
+      if (GET_CODE (newval) == COND)
+       newval = simplify_cond (newval, insn_code, insn_index);
 
-             /* Add this op's contribution to MAX (BLOCKAGE (E,*)) and
-                MIN (BLOCKAGE (E,*)).  */
-             if (max_blockage == 0)
-               max_blockage = min_blockage = blockage;
-             else
-               {
-                 max_blockage
-                   = simplify_knowing (operate_exp (MAX_OP, max_blockage,
-                                                    blockage),
-                                       unit->condexp);
-                 min_blockage
-                   = simplify_knowing (operate_exp (MIN_OP, min_blockage,
-                                                    blockage),
-                                       unit->condexp);
-               }
+      /* Look for ways to delete or combine this test.  */
+      if (newtest == true_rtx)
+       {
+         /* If test is true, make this value the default
+            and discard this + any following tests.  */
+         len = i;
+         defval = tests[i + 1];
+         new_defval = newval;
+       }
 
-             /* Make an attribute for use in the blockage function.  */
-             str = attr_printf ((strlen (unit->name) + sizeof "*_block_"
-                                 + MAX_DIGITS),
-                                "*%s_block_%d", unit->name, op->num);
-             make_internal_attr (str, blockage, 1);
-           }
+      else if (newtest == false_rtx)
+       {
+         /* If test is false, discard it and its value.  */
+         for (j = i; j < len - 2; j++)
+           tests[j] = tests[j + 2];
+         i -= 2;
+         len -= 2;
+       }
 
-         /* Record MAX (BLOCKAGE (*,*)).  */
-         {
-           int unknown;
-           unit->max_blockage = max_attr_value (max_blockage, &unknown);
-         }
+      else if (i > 0 && attr_equal_p (newval, tests[i - 1]))
+       {
+         /* If this value and the value for the prev test are the same,
+            merge the tests.  */
 
-         /* See if the upper and lower bounds of BLOCKAGE (E,*) are the
-            same.  If so, the blockage function carries no additional
-            information and is not written.  */
-         newexp = operate_exp (EQ_OP, max_blockage, min_blockage);
-         newexp = simplify_knowing (newexp, unit->condexp);
-         unit->needs_blockage_function
-           = (GET_CODE (newexp) != CONST_STRING
-              || atoi (XSTR (newexp, 0)) != 1);
-
-         /* If the all values of BLOCKAGE (E,C) have the same value,
-            neither blockage function is written.  */
-         unit->needs_range_function
-           = (unit->needs_blockage_function
-              || GET_CODE (max_blockage) != CONST_STRING);
-
-         if (unit->needs_range_function)
-           {
-             /* Compute the blockage range function and make an attribute
-                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, 20);
-           }
+         tests[i - 2]
+           = insert_right_side (IOR, tests[i - 2], newtest,
+                                insn_code, insn_index);
 
-         str = attr_printf (strlen (unit->name) + sizeof "*_unit_ready_cost",
-                            "*%s_unit_ready_cost", unit->name);
+         /* Delete this test/value.  */
+         for (j = i; j < len - 2; j++)
+           tests[j] = tests[j + 2];
+         len -= 2;
+         i -= 2;
        }
-      else
-       str = "*result_ready_cost";
 
-      /* Make an attribute for the ready_cost function.  Simplifying
-        further with simplify_by_exploding doesn't win.  */
-      make_internal_attr (str, readycost, 0);
+      else
+       tests[i + 1] = newval;
     }
 
-  /* For each unit that requires a conflict cost function, make an attribute
-     that maps insns to the operation number.  */
-  for (unit = units; unit; unit = unit->next)
-    {
-      rtx caseexp;
-
-      if (! unit->needs_conflict_function
-         && ! unit->needs_blockage_function)
-       continue;
+  /* If the last test in a COND has the same value
+     as the default value, that test isn't needed.  */
 
-      caseexp = rtx_alloc (COND);
-      XVEC (caseexp, 0) = rtvec_alloc ((unit->num_opclasses - 1) * 2);
+  while (len > 0 && attr_equal_p (tests[len - 1], new_defval))
+    len -= 2;
 
-      for (op = unit->ops; op; op = op->next)
+  /* See if we changed anything.  */
+  if (len != XVECLEN (exp, 0) || new_defval != XEXP (exp, 1))
+    allsame = 0;
+  else
+    for (i = 0; i < len; i++)
+      if (! attr_equal_p (tests[i], XVECEXP (exp, 0, i)))
        {
-         /* Make our adjustment to the COND being computed.  If we are the
-            last operation class, place our values into the default of the
-            COND.  */
-         if (op->num == unit->num_opclasses - 1)
-           {
-             XEXP (caseexp, 1) = make_numeric_value (op->num);
-           }
-         else
-           {
-             XVECEXP (caseexp, 0, op->num * 2) = op->condexp;
-             XVECEXP (caseexp, 0, op->num * 2 + 1)
-               = make_numeric_value (op->num);
-           }
+         allsame = 0;
+         break;
        }
 
-      /* Simplifying caseexp with simplify_by_exploding doesn't win.  */
-      str = attr_printf (strlen (unit->name) + sizeof "*_cases",
-                        "*%s_cases", unit->name);
-      make_internal_attr (str, caseexp, 1);
+  if (len == 0)
+    {
+      if (GET_CODE (defval) == COND)
+       ret = simplify_cond (defval, insn_code, insn_index);
+      else
+       ret = defval;
+    }
+  else if (allsame)
+    ret = exp;
+  else
+    {
+      rtx newexp = rtx_alloc (COND);
+
+      XVEC (newexp, 0) = rtvec_alloc (len);
+      memcpy (XVEC (newexp, 0)->elem, tests, len * sizeof (rtx));
+      XEXP (newexp, 1) = new_defval;
+      ret = newexp;
     }
+  free (tests);
+  return ret;
 }
 
-/* Simplify EXP given KNOWN_TRUE.  */
+/* Remove an insn entry from an attribute value.  */
 
-static rtx
-simplify_knowing (exp, known_true)
-     rtx exp, known_true;
+static void
+remove_insn_ent (struct attr_value *av, struct insn_ent *ie)
 {
-  if (GET_CODE (exp) != CONST_STRING)
+  struct insn_ent *previe;
+
+  if (av->first_insn == ie)
+    av->first_insn = ie->next;
+  else
     {
-      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);
-       }
+      for (previe = av->first_insn; previe->next != ie; previe = previe->next)
+       ;
+      previe->next = ie->next;
     }
-  return exp;
+
+  av->num_insns--;
+  if (ie->def->insn_code == -1)
+    av->has_asm_insn = 0;
+
+  num_insn_ents--;
 }
 
-/* Translate the CONST_STRING expressions in X to change the encoding of
-   value.  On input, the value is a bitmask with a one bit for each unit
-   used; on output, the value is the unit number (zero based) if one
-   and only one unit is used or the one's compliment of the bitmask.  */
+/* Insert an insn entry in an attribute value list.  */
 
-static rtx
-encode_units_mask (x)
-     rtx x;
+static void
+insert_insn_ent (struct attr_value *av, struct insn_ent *ie)
 {
-  int i;
-  int j;
-  enum rtx_code code;
-  const char *fmt;
+  ie->next = av->first_insn;
+  av->first_insn = ie;
+  av->num_insns++;
+  if (ie->def->insn_code == -1)
+    av->has_asm_insn = 1;
 
-  code = GET_CODE (x);
+  num_insn_ents++;
+}
 
-  switch (code)
-    {
-    case CONST_STRING:
-      i = atoi (XSTR (x, 0));
-      if (i < 0)
-       /* 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++)
-         ;
-      else
-       j = ~i;
-      return attr_rtx (CONST_STRING, attr_printf (MAX_DIGITS, "%d", j));
+/* This is a utility routine to take an expression that is a tree of either
+   AND or IOR expressions and insert a new term.  The new term will be
+   inserted at the right side of the first node whose code does not match
+   the root.  A new node will be created with the root's code.  Its left
+   side will be the old right side and its right side will be the new
+   term.
 
-    case REG:
-    case QUEUED:
-    case CONST_INT:
-    case CONST_DOUBLE:
-    case CONST_VECTOR:
-    case SYMBOL_REF:
-    case CODE_LABEL:
-    case PC:
-    case CC0:
-    case EQ_ATTR:
-      return x;
-
-    default:
-      break;
-    }
-
-  /* Compare the elements.  If any pair of corresponding elements
-     fail to match, return 0 for the whole things.  */
-
-  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++)
-           XVECEXP (x, i, j) = encode_units_mask (XVECEXP (x, i, j));
-         break;
-
-       case 'e':
-         XEXP (x, i) = encode_units_mask (XEXP (x, i));
-         break;
-       }
-    }
-  return x;
-}
-\f
-/* Once all attributes and insns have been read and checked, we construct for
-   each attribute value a list of all the insns that have that value for
-   the attribute.  */
+   If the `term' is itself a tree, all its leaves will be inserted.  */
 
-static void
-fill_attr (attr)
-     struct attr_desc *attr;
+static rtx
+insert_right_side (enum rtx_code code, rtx exp, rtx term, int insn_code, int insn_index)
 {
-  struct attr_value *av;
-  struct insn_ent *ie;
-  struct insn_def *id;
-  int i;
-  rtx value;
+  rtx newexp;
 
-  /* Don't fill constant attributes.  The value is independent of
-     any particular insn.  */
-  if (attr->is_const)
-    return;
+  /* Avoid consing in some special cases.  */
+  if (code == AND && term == true_rtx)
+    return exp;
+  if (code == AND && term == false_rtx)
+    return false_rtx;
+  if (code == AND && exp == true_rtx)
+    return term;
+  if (code == AND && exp == false_rtx)
+    return false_rtx;
+  if (code == IOR && term == true_rtx)
+    return true_rtx;
+  if (code == IOR && term == false_rtx)
+    return exp;
+  if (code == IOR && exp == true_rtx)
+    return true_rtx;
+  if (code == IOR && exp == false_rtx)
+    return term;
+  if (attr_equal_p (exp, term))
+    return exp;
 
-  for (id = defs; id; id = id->next)
+  if (GET_CODE (term) == code)
     {
-      /* If no value is specified for this insn for this attribute, use the
-        default.  */
-      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),
-                       attr->name))
-           value = XEXP (XVECEXP (id->def, id->vec_idx, i), 1);
-
-      if (value == NULL)
-       av = attr->default_val;
-      else
-       av = get_attr_value (value, attr, id->insn_code);
+      exp = insert_right_side (code, exp, XEXP (term, 0),
+                              insn_code, insn_index);
+      exp = insert_right_side (code, exp, XEXP (term, 1),
+                              insn_code, insn_index);
 
-      ie = (struct insn_ent *) oballoc (sizeof (struct insn_ent));
-      ie->insn_code = id->insn_code;
-      ie->insn_index = id->insn_code;
-      insert_insn_ent (av, ie);
+      return exp;
     }
-}
-\f
-/* Given an expression EXP, see if it is a COND or IF_THEN_ELSE that has a
-   test that checks relative positions of insns (uses MATCH_DUP or PC).
-   If so, replace it with what is obtained by passing the expression to
-   ADDRESS_FN.  If not but it is a COND or IF_THEN_ELSE, call this routine
-   recursively on each value (including the default value).  Otherwise,
-   return the value returned by NO_ADDRESS_FN applied to EXP.  */
-
-static rtx
-substitute_address (exp, no_address_fn, address_fn)
-     rtx exp;
-     rtx (*no_address_fn) PARAMS ((rtx));
-     rtx (*address_fn) PARAMS ((rtx));
-{
-  int i;
-  rtx newexp;
 
-  if (GET_CODE (exp) == COND)
+  if (GET_CODE (exp) == code)
     {
-      /* See if any tests use addresses.  */
-      address_used = 0;
-      for (i = 0; i < XVECLEN (exp, 0); i += 2)
-       walk_attr_value (XVECEXP (exp, 0, i));
-
-      if (address_used)
-       return (*address_fn) (exp);
-
-      /* Make a new copy of this COND, replacing each element.  */
-      newexp = rtx_alloc (COND);
-      XVEC (newexp, 0) = rtvec_alloc (XVECLEN (exp, 0));
-      for (i = 0; i < XVECLEN (exp, 0); i += 2)
-       {
-         XVECEXP (newexp, 0, i) = XVECEXP (exp, 0, i);
-         XVECEXP (newexp, 0, i + 1)
-           = substitute_address (XVECEXP (exp, 0, i + 1),
-                                 no_address_fn, address_fn);
-       }
-
-      XEXP (newexp, 1) = substitute_address (XEXP (exp, 1),
-                                            no_address_fn, address_fn);
-
-      return newexp;
+      rtx new_rtx = insert_right_side (code, XEXP (exp, 1),
+                                      term, insn_code, insn_index);
+      if (new_rtx != XEXP (exp, 1))
+       /* Make a copy of this expression and call recursively.  */
+       newexp = attr_rtx (code, XEXP (exp, 0), new_rtx);
+      else
+       newexp = exp;
     }
-
-  else if (GET_CODE (exp) == IF_THEN_ELSE)
+  else
     {
-      address_used = 0;
-      walk_attr_value (XEXP (exp, 0));
-      if (address_used)
-       return (*address_fn) (exp);
-
-      return attr_rtx (IF_THEN_ELSE,
-                      substitute_address (XEXP (exp, 0),
-                                          no_address_fn, address_fn),
-                      substitute_address (XEXP (exp, 1),
-                                          no_address_fn, address_fn),
-                      substitute_address (XEXP (exp, 2),
-                                          no_address_fn, address_fn));
+      /* Insert the new term.  */
+      newexp = attr_rtx (code, exp, term);
     }
 
-  return (*no_address_fn) (exp);
+  return simplify_test_exp_in_temp (newexp, insn_code, insn_index);
 }
-\f
-/* Make new attributes from the `length' attribute.  The following are made,
-   each corresponding to a function called from `shorten_branches' or
-   `get_attr_length':
-
-   *insn_default_length                This is the length of the insn to be returned
-                               by `get_attr_length' before `shorten_branches'
-                               has been called.  In each case where the length
-                               depends on relative addresses, the largest
-                               possible is used.  This routine is also used
-                               to compute the initial size of the insn.
 
-   *insn_variable_length_p     This returns 1 if the insn's length depends
-                               on relative addresses, zero otherwise.
+/* If we have an expression which AND's a bunch of
+       (not (eq_attrq "alternative" "n"))
+   terms, we may have covered all or all but one of the possible alternatives.
+   If so, we can optimize.  Similarly for IOR's of EQ_ATTR.
 
-   *insn_current_length                This is only called when it is known that the
-                               insn has a variable length and returns the
-                               current length, based on relative addresses.
-  */
+   This routine is passed an expression and either AND or IOR.  It returns a
+   bitmask indicating which alternatives are mentioned within EXP.  */
 
-static void
-make_length_attrs ()
+static int
+compute_alternative_mask (rtx exp, enum rtx_code code)
 {
-  static const char *const new_names[] = {"*insn_default_length",
-                                     "*insn_variable_length_p",
-                                     "*insn_current_length"};
-  static rtx (*const no_address_fn[]) PARAMS ((rtx)) = {identity_fn, zero_fn, zero_fn};
-  static rtx (*const 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;
-
-  /* See if length attribute is defined.  If so, it must be numeric.  Make
-     it special so we don't output anything for it.  */
-  length_attr = find_attr ("length", 0);
-  if (length_attr == 0)
-    return;
+  const char *string;
+  if (GET_CODE (exp) == code)
+    return compute_alternative_mask (XEXP (exp, 0), code)
+          | compute_alternative_mask (XEXP (exp, 1), code);
 
-  if (! length_attr->is_numeric)
-    fatal ("length attribute must be numeric");
+  else if (code == AND && GET_CODE (exp) == NOT
+          && GET_CODE (XEXP (exp, 0)) == EQ_ATTR
+          && XSTR (XEXP (exp, 0), 0) == alternative_name)
+    string = XSTR (XEXP (exp, 0), 1);
 
-  length_attr->is_const = 0;
-  length_attr->is_special = 1;
+  else if (code == IOR && GET_CODE (exp) == EQ_ATTR
+          && XSTR (exp, 0) == alternative_name)
+    string = XSTR (exp, 1);
 
-  /* Make each new attribute, in turn.  */
-  for (i = 0; i < ARRAY_SIZE (new_names); i++)
+  else if (GET_CODE (exp) == EQ_ATTR_ALT)
     {
-      make_internal_attr (new_names[i],
-                         substitute_address (length_attr->default_val->value,
-                                             no_address_fn[i], address_fn[i]),
-                         0);
-      new_attr = find_attr (new_names[i], 0);
-      for (av = length_attr->first_value; av; av = av->next)
-       for (ie = av->first_insn; ie; ie = ie->next)
-         {
-           new_av = get_attr_value (substitute_address (av->value,
-                                                        no_address_fn[i],
-                                                        address_fn[i]),
-                                    new_attr, ie->insn_code);
-           new_ie = (struct insn_ent *) oballoc (sizeof (struct insn_ent));
-           new_ie->insn_code = ie->insn_code;
-           new_ie->insn_index = ie->insn_index;
-           insert_insn_ent (new_av, new_ie);
-         }
+      if (code == AND && XINT (exp, 1))
+       return XINT (exp, 0);
+
+      if (code == IOR && !XINT (exp, 1))
+       return XINT (exp, 0);
+
+      return 0;
     }
+  else
+    return 0;
+
+  if (string[1] == 0)
+    return 1 << (string[0] - '0');
+  return 1 << atoi (string);
 }
 
-/* Utility functions called from above routine.  */
+/* Given I, a single-bit mask, return RTX to compare the `alternative'
+   attribute with the value represented by that bit.  */
 
 static rtx
-identity_fn (exp)
-     rtx exp;
+make_alternative_compare (int mask)
 {
-  return exp;
+  return mk_attr_alt (mask);
 }
 
-static rtx
-zero_fn (exp)
-     rtx exp ATTRIBUTE_UNUSED;
-{
-  return make_numeric_value (0);
-}
+/* If we are processing an (eq_attr "attr" "value") test, we find the value
+   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.
 
-static rtx
-one_fn (exp)
-     rtx exp ATTRIBUTE_UNUSED;
-{
-  return make_numeric_value (1);
-}
+   EXP is the EQ_ATTR expression and VALUE is the value of that attribute
+   for the insn corresponding to INSN_CODE and INSN_INDEX.  */
 
 static rtx
-max_fn (exp)
-     rtx exp;
+evaluate_eq_attr (rtx exp, rtx value, int insn_code, int insn_index)
 {
-  int unknown;
-  return make_numeric_value (max_attr_value (exp, &unknown));
-}
+  rtx orexp, andexp;
+  rtx right;
+  rtx newexp;
+  int i;
 
-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;
+  switch (GET_CODE (value))
+    {
+    case CONST_STRING:
+      if (! strcmp_check (XSTR (value, 0), XSTR (exp, 1)))
+       newexp = true_rtx;
+      else
+       newexp = false_rtx;
+      break;
+      
+    case SYMBOL_REF:
+      {
+       char *p;
+       char string[256];
+       
+       gcc_assert (GET_CODE (exp) == EQ_ATTR);
+       gcc_assert (strlen (XSTR (exp, 0)) + strlen (XSTR (exp, 1)) + 2
+                   <= 256);
+       
+       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,
+                                    DEF_ATTR_STRING (string)));
+       break;
+      }
 
-  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);
+    case COND:
+      /* We construct an IOR of all the cases for which the
+        requested attribute value is present.  Since we start with
+        FALSE, if it is not present, 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.
+         
+        For each possible COND value, call ourselves recursively.
+         
+        The extra TRUE and FALSE expressions will be eliminated by another
+        call to the simplification routine.  */
 
-  if (unknown)
-    length_unit_log = 0;
-  else
+      orexp = false_rtx;
+      andexp = true_rtx;
+
+      for (i = 0; i < XVECLEN (value, 0); i += 2)
+       {
+         rtx this_cond = simplify_test_exp_in_temp (XVECEXP (value, 0, i),
+                                                   insn_code, insn_index);
+
+         right = insert_right_side (AND, andexp, this_cond,
+                                    insn_code, insn_index);
+         right = insert_right_side (AND, right,
+                                    evaluate_eq_attr (exp,
+                                                      XVECEXP (value, 0,
+                                                               i + 1),
+                                                      insn_code, insn_index),
+                                    insn_code, insn_index);
+         orexp = insert_right_side (IOR, orexp, right,
+                                    insn_code, insn_index);
+
+         /* Add this condition into the AND expression.  */
+         newexp = attr_rtx (NOT, this_cond);
+         andexp = insert_right_side (AND, andexp, newexp,
+                                     insn_code, insn_index);
+       }
+
+      /* Handle the default case.  */
+      right = insert_right_side (AND, andexp,
+                                evaluate_eq_attr (exp, XEXP (value, 1),
+                                                  insn_code, insn_index),
+                                insn_code, insn_index);
+      newexp = insert_right_side (IOR, orexp, right, insn_code, insn_index);
+      break;
+
+    default:
+      gcc_unreachable ();
+    }
+
+  /* If uses an address, must return original expression.  But set the
+     ATTR_IND_SIMPLIFIED_P bit so we don't try to simplify it again.  */
+
+  address_used = 0;
+  walk_attr_value (newexp);
+
+  if (address_used)
     {
-      length_or = ~length_or;
-      for (length_unit_log = 0; length_or & 1; length_or >>= 1)
-       length_unit_log++;
+      if (! ATTR_IND_SIMPLIFIED_P (exp))
+       return copy_rtx_unchanging (exp);
+      return exp;
     }
-  printf ("int length_unit_log = %u;\n", length_unit_log);
+  else
+    return newexp;
 }
-\f
-/* Take a COND expression and see if any of the conditions in it can be
-   simplified.  If any are known true or known false for the particular insn
-   code, the COND can be further simplified.
 
-   Also call ourselves on any COND operations that are values of this COND.
+/* This routine is called when an AND of a term with a tree of AND's is
+   encountered.  If the term or its complement is present in the tree, it
+   can be replaced with TRUE or FALSE, respectively.
 
-   We do not modify EXP; rather, we make and return a new rtx.  */
+   Note that (eq_attr "att" "v1") and (eq_attr "att" "v2") cannot both
+   be true and hence are complementary.
+
+   There is one special case:  If we see
+       (and (not (eq_attr "att" "v1"))
+            (eq_attr "att" "v2"))
+   this can be replaced by (eq_attr "att" "v2").  To do this we need to
+   replace the term, not anything in the AND tree.  So we pass a pointer to
+   the term.  */
 
 static rtx
-simplify_cond (exp, insn_code, insn_index)
-     rtx exp;
-     int insn_code, insn_index;
+simplify_and_tree (rtx exp, rtx *pterm, int insn_code, int insn_index)
 {
-  int i, j;
-  /* We store the desired contents here,
-     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 *) xmalloc (len * sizeof (rtx));
-  int allsame = 1;
-  char *first_spacer;
-  rtx ret;
-
-  /* This lets us free all storage allocated below, if appropriate.  */
-  first_spacer = (char *) obstack_finish (rtl_obstack);
-
-  memcpy (tests, XVEC (exp, 0)->elem, len * sizeof (rtx));
+  rtx left, right;
+  rtx newexp;
+  rtx temp;
+  int left_eliminates_term, right_eliminates_term;
 
-  /* See if default value needs simplification.  */
-  if (GET_CODE (defval) == COND)
-    new_defval = simplify_cond (defval, insn_code, insn_index);
+  if (GET_CODE (exp) == AND)
+    {
+      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))
+       {
+         newexp = attr_rtx (AND, left, right);
 
-  /* Simplify the subexpressions, and see what tests we can get rid of.  */
+         exp = simplify_test_exp_in_temp (newexp, insn_code, insn_index);
+       }
+    }
 
-  for (i = 0; i < len; i += 2)
+  else if (GET_CODE (exp) == IOR)
     {
-      rtx newtest, newval;
+      /* 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_eliminates_term = (temp == true_rtx);
 
-      /* Simplify this test.  */
-      newtest = simplify_test_exp_in_temp (tests[i], insn_code, insn_index);
-      tests[i] = newtest;
+      temp = *pterm;
+      right = simplify_and_tree (XEXP (exp, 1), &temp, insn_code, insn_index);
+      right_eliminates_term = (temp == true_rtx);
 
-      newval = tests[i + 1];
-      /* See if this value may need simplification.  */
-      if (GET_CODE (newval) == COND)
-       newval = simplify_cond (newval, insn_code, insn_index);
+      if (left_eliminates_term && right_eliminates_term)
+       *pterm = true_rtx;
 
-      /* Look for ways to delete or combine this test.  */
-      if (newtest == true_rtx)
+      if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
        {
-         /* If test is true, make this value the default
-            and discard this + any following tests.  */
-         len = i;
-         defval = tests[i + 1];
-         new_defval = newval;
-       }
+         newexp = attr_rtx (IOR, left, right);
 
-      else if (newtest == false_rtx)
-       {
-         /* If test is false, discard it and its value.  */
-         for (j = i; j < len - 2; j++)
-           tests[j] = tests[j + 2];
-         len -= 2;
+         exp = simplify_test_exp_in_temp (newexp, insn_code, insn_index);
        }
+    }
 
-      else if (i > 0 && attr_equal_p (newval, tests[i - 1]))
-       {
-         /* If this value and the value for the prev test are the same,
-            merge the tests.  */
+  /* Check for simplifications.  Do some extra checking here since this
+     routine is called so many times.  */
 
-         tests[i - 2]
-           = insert_right_side (IOR, tests[i - 2], newtest,
-                                insn_code, insn_index);
+  if (exp == *pterm)
+    return true_rtx;
 
-         /* Delete this test/value.  */
-         for (j = i; j < len - 2; j++)
-           tests[j] = tests[j + 2];
-         len -= 2;
-       }
+  else if (GET_CODE (exp) == NOT && XEXP (exp, 0) == *pterm)
+    return false_rtx;
 
-      else
-       tests[i + 1] = newval;
-    }
+  else if (GET_CODE (*pterm) == NOT && exp == XEXP (*pterm, 0))
+    return false_rtx;
 
-  /* If the last test in a COND has the same value
-     as the default value, that test isn't needed.  */
+  else if (GET_CODE (exp) == EQ_ATTR_ALT && GET_CODE (*pterm) == EQ_ATTR_ALT)
+    {
+      if (attr_alt_subset_p (*pterm, exp))
+       return true_rtx;
 
-  while (len > 0 && attr_equal_p (tests[len - 1], new_defval))
-    len -= 2;
+      if (attr_alt_subset_of_compl_p (*pterm, exp))
+       return false_rtx;
 
-  /* See if we changed anything.  */
-  if (len != XVECLEN (exp, 0) || new_defval != XEXP (exp, 1))
-    allsame = 0;
-  else
-    for (i = 0; i < len; i++)
-      if (! attr_equal_p (tests[i], XVECEXP (exp, 0, i)))
-       {
-         allsame = 0;
-         break;
-       }
+      if (attr_alt_subset_p (exp, *pterm))
+       *pterm = true_rtx;
+       
+      return exp;
+    }
 
-  if (len == 0)
+  else if (GET_CODE (exp) == EQ_ATTR && GET_CODE (*pterm) == EQ_ATTR)
     {
-      if (GET_CODE (defval) == COND)
-       ret = simplify_cond (defval, insn_code, insn_index);
+      if (XSTR (exp, 0) != XSTR (*pterm, 0))
+       return exp;
+
+      if (! strcmp_check (XSTR (exp, 1), XSTR (*pterm, 1)))
+       return true_rtx;
       else
-       ret = defval;
+       return false_rtx;
     }
-  else if (allsame)
-    ret = exp;
-  else
+
+  else if (GET_CODE (*pterm) == EQ_ATTR && GET_CODE (exp) == NOT
+          && GET_CODE (XEXP (exp, 0)) == EQ_ATTR)
     {
-      rtx newexp = rtx_alloc (COND);
+      if (XSTR (*pterm, 0) != XSTR (XEXP (exp, 0), 0))
+       return exp;
 
-      XVEC (newexp, 0) = rtvec_alloc (len);
-      memcpy (XVEC (newexp, 0)->elem, tests, len * sizeof (rtx));
-      XEXP (newexp, 1) = new_defval;
-      ret = newexp;
+      if (! strcmp_check (XSTR (*pterm, 1), XSTR (XEXP (exp, 0), 1)))
+       return false_rtx;
+      else
+       return true_rtx;
     }
-  free (tests);
-  return ret;
-}
-\f
-/* Remove an insn entry from an attribute value.  */
 
-static void
-remove_insn_ent (av, ie)
-     struct attr_value *av;
-     struct insn_ent *ie;
-{
-  struct insn_ent *previe;
+  else if (GET_CODE (exp) == EQ_ATTR && GET_CODE (*pterm) == NOT
+          && GET_CODE (XEXP (*pterm, 0)) == EQ_ATTR)
+    {
+      if (XSTR (exp, 0) != XSTR (XEXP (*pterm, 0), 0))
+       return exp;
 
-  if (av->first_insn == ie)
-    av->first_insn = ie->next;
-  else
+      if (! strcmp_check (XSTR (exp, 1), XSTR (XEXP (*pterm, 0), 1)))
+       return false_rtx;
+      else
+       *pterm = true_rtx;
+    }
+
+  else if (GET_CODE (exp) == NOT && GET_CODE (*pterm) == NOT)
     {
-      for (previe = av->first_insn; previe->next != ie; previe = previe->next)
-       ;
-      previe->next = ie->next;
+      if (attr_equal_p (XEXP (exp, 0), XEXP (*pterm, 0)))
+       return true_rtx;
     }
 
-  av->num_insns--;
-  if (ie->insn_code == -1)
-    av->has_asm_insn = 0;
+  else if (GET_CODE (exp) == NOT)
+    {
+      if (attr_equal_p (XEXP (exp, 0), *pterm))
+       return false_rtx;
+    }
 
-  num_insn_ents--;
-}
-
-/* Insert an insn entry in an attribute value list.  */
-
-static void
-insert_insn_ent (av, ie)
-     struct attr_value *av;
-     struct insn_ent *ie;
-{
-  ie->next = av->first_insn;
-  av->first_insn = 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
-   AND or IOR expressions and insert a new term.  The new term will be
-   inserted at the right side of the first node whose code does not match
-   the root.  A new node will be created with the root's code.  Its left
-   side will be the old right side and its right side will be the new
-   term.
-
-   If the `term' is itself a tree, all its leaves will be inserted.  */
-
-static rtx
-insert_right_side (code, exp, term, insn_code, insn_index)
-     enum rtx_code code;
-     rtx exp;
-     rtx term;
-     int insn_code, insn_index;
-{
-  rtx newexp;
-
-  /* Avoid consing in some special cases.  */
-  if (code == AND && term == true_rtx)
-    return exp;
-  if (code == AND && term == false_rtx)
-    return false_rtx;
-  if (code == AND && exp == true_rtx)
-    return term;
-  if (code == AND && exp == false_rtx)
-    return false_rtx;
-  if (code == IOR && term == true_rtx)
-    return true_rtx;
-  if (code == IOR && term == false_rtx)
-    return exp;
-  if (code == IOR && exp == true_rtx)
-    return true_rtx;
-  if (code == IOR && exp == false_rtx)
-    return term;
-  if (attr_equal_p (exp, term))
-    return exp;
-
-  if (GET_CODE (term) == code)
-    {
-      exp = insert_right_side (code, exp, XEXP (term, 0),
-                              insn_code, insn_index);
-      exp = insert_right_side (code, exp, XEXP (term, 1),
-                              insn_code, insn_index);
-
-      return exp;
-    }
-
-  if (GET_CODE (exp) == code)
-    {
-      rtx new = insert_right_side (code, XEXP (exp, 1),
-                                  term, insn_code, insn_index);
-      if (new != XEXP (exp, 1))
-       /* Make a copy of this expression and call recursively.  */
-       newexp = attr_rtx (code, XEXP (exp, 0), new);
-      else
-       newexp = exp;
-    }
-  else
-    {
-      /* Insert the new term.  */
-      newexp = attr_rtx (code, exp, term);
-    }
-
-  return simplify_test_exp_in_temp (newexp, insn_code, insn_index);
-}
-\f
-/* If we have an expression which AND's a bunch of
-       (not (eq_attrq "alternative" "n"))
-   terms, we may have covered all or all but one of the possible alternatives.
-   If so, we can optimize.  Similarly for IOR's of EQ_ATTR.
-
-   This routine is passed an expression and either AND or IOR.  It returns a
-   bitmask indicating which alternatives are mentioned within EXP.  */
-
-static int
-compute_alternative_mask (exp, code)
-     rtx exp;
-     enum rtx_code code;
-{
-  const char *string;
-  if (GET_CODE (exp) == code)
-    return compute_alternative_mask (XEXP (exp, 0), code)
-          | compute_alternative_mask (XEXP (exp, 1), code);
-
-  else if (code == AND && GET_CODE (exp) == NOT
-          && GET_CODE (XEXP (exp, 0)) == EQ_ATTR
-          && XSTR (XEXP (exp, 0), 0) == alternative_name)
-    string = XSTR (XEXP (exp, 0), 1);
-
-  else if (code == IOR && GET_CODE (exp) == EQ_ATTR
-          && XSTR (exp, 0) == alternative_name)
-    string = XSTR (exp, 1);
-
-  else
-    return 0;
-
-  if (string[1] == 0)
-    return 1 << (string[0] - '0');
-  return 1 << atoi (string);
-}
-
-/* Given I, a single-bit mask, return RTX to compare the `alternative'
-   attribute with the value represented by that bit.  */
-
-static rtx
-make_alternative_compare (mask)
-     int mask;
-{
-  rtx newexp;
-  int i;
-
-  /* Find the bit.  */
-  for (i = 0; (mask & (1 << i)) == 0; i++)
-    ;
-
-  newexp = attr_rtx (EQ_ATTR, alternative_name, attr_numeral (i));
-  ATTR_IND_SIMPLIFIED_P (newexp) = 1;
-
-  return newexp;
-}
-\f
-/* If we are processing an (eq_attr "attr" "value") test, we find the value
-   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.
-
-   EXP is the EQ_ATTR expression and VALUE is the value of that attribute
-   for the insn corresponding to INSN_CODE and INSN_INDEX.  */
-
-static rtx
-evaluate_eq_attr (exp, value, insn_code, insn_index)
-     rtx exp;
-     rtx value;
-     int insn_code, insn_index;
-{
-  rtx orexp, andexp;
-  rtx right;
-  rtx newexp;
-  int i;
-
-  if (GET_CODE (value) == CONST_STRING)
-    {
-      if (! strcmp (XSTR (value, 0), XSTR (exp, 1)))
-       newexp = true_rtx;
-      else
-       newexp = false_rtx;
-    }
-  else if (GET_CODE (value) == SYMBOL_REF)
-    {
-      char *p;
-      char string[256];
-
-      if (GET_CODE (exp) != EQ_ATTR)
-       abort ();
-
-      if (strlen (XSTR (exp, 0)) + strlen (XSTR (exp, 1)) + 2 > 256)
-       abort ();
-
-      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)
+  else if (GET_CODE (*pterm) == NOT)
     {
-      /* We construct an IOR of all the cases for which the requested attribute
-        value is present.  Since we start with FALSE, if it is not present,
-        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.
-
-        For each possible COND value, call ourselves recursively.
-
-        The extra TRUE and FALSE expressions will be eliminated by another
-        call to the simplification routine.  */
-
-      orexp = false_rtx;
-      andexp = true_rtx;
-
-      if (current_alternative_string)
-       clear_struct_flag (value);
-
-      for (i = 0; i < XVECLEN (value, 0); i += 2)
-       {
-         rtx this = simplify_test_exp_in_temp (XVECEXP (value, 0, i),
-                                               insn_code, insn_index);
-
-         SIMPLIFY_ALTERNATIVE (this);
-
-         right = insert_right_side (AND, andexp, this,
-                                    insn_code, insn_index);
-         right = insert_right_side (AND, right,
-                                    evaluate_eq_attr (exp,
-                                                      XVECEXP (value, 0,
-                                                               i + 1),
-                                                      insn_code, insn_index),
-                                    insn_code, insn_index);
-         orexp = insert_right_side (IOR, orexp, right,
-                                    insn_code, insn_index);
-
-         /* Add this condition into the AND expression.  */
-         newexp = attr_rtx (NOT, this);
-         andexp = insert_right_side (AND, andexp, newexp,
-                                     insn_code, insn_index);
-       }
-
-      /* Handle the default case.  */
-      right = insert_right_side (AND, andexp,
-                                evaluate_eq_attr (exp, XEXP (value, 1),
-                                                  insn_code, insn_index),
-                                insn_code, insn_index);
-      newexp = insert_right_side (IOR, orexp, right, insn_code, insn_index);
+      if (attr_equal_p (XEXP (*pterm, 0), exp))
+       return false_rtx;
     }
-  else
-    abort ();
-
-  /* If uses an address, must return original expression.  But set the
-     ATTR_IND_SIMPLIFIED_P bit so we don't try to simplify it again.  */
 
-  address_used = 0;
-  walk_attr_value (newexp);
+  else if (attr_equal_p (exp, *pterm))
+    return true_rtx;
 
-  if (address_used)
-    {
-      /* This had `&& current_alternative_string', which seems to be wrong.  */
-      if (! ATTR_IND_SIMPLIFIED_P (exp))
-       return copy_rtx_unchanging (exp);
-      return exp;
-    }
-  else
-    return newexp;
+  return exp;
 }
-\f
-/* This routine is called when an AND of a term with a tree of AND's is
-   encountered.  If the term or its complement is present in the tree, it
-   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.
 
-   There is one special case:  If we see
-       (and (not (eq_attr "att" "v1"))
-            (eq_attr "att" "v2"))
-   this can be replaced by (eq_attr "att" "v2").  To do this we need to
-   replace the term, not anything in the AND tree.  So we pass a pointer to
-   the term.  */
+/* Similar to `simplify_and_tree', but for IOR trees.  */
 
 static rtx
-simplify_and_tree (exp, pterm, insn_code, insn_index)
-     rtx exp;
-     rtx *pterm;
-     int insn_code, insn_index;
+simplify_or_tree (rtx exp, rtx *pterm, int insn_code, int insn_index)
 {
   rtx left, right;
   rtx newexp;
   rtx temp;
   int left_eliminates_term, right_eliminates_term;
 
-  if (GET_CODE (exp) == AND)
+  if (GET_CODE (exp) == IOR)
     {
-      left  = simplify_and_tree (XEXP (exp, 0), pterm, insn_code, insn_index);
-      right = simplify_and_tree (XEXP (exp, 1), 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))
        {
          newexp = attr_rtx (GET_CODE (exp), left, right);
@@ -2988,20 +2193,20 @@ simplify_and_tree (exp, pterm, insn_code, insn_index)
        }
     }
 
-  else if (GET_CODE (exp) == IOR)
+  else if (GET_CODE (exp) == AND)
     {
-      /* 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.  */
+      /* 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_and_tree (XEXP (exp, 0), &temp, insn_code, insn_index);
-      left_eliminates_term = (temp == true_rtx);
+      left = simplify_or_tree (XEXP (exp, 0), &temp, insn_code, insn_index);
+      left_eliminates_term = (temp == false_rtx);
 
       temp = *pterm;
-      right = simplify_and_tree (XEXP (exp, 1), &temp, insn_code, insn_index);
-      right_eliminates_term = (temp == true_rtx);
+      right = simplify_or_tree (XEXP (exp, 1), &temp, insn_code, insn_index);
+      right_eliminates_term = (temp == false_rtx);
 
       if (left_eliminates_term && right_eliminates_term)
-       *pterm = true_rtx;
+       *pterm = false_rtx;
 
       if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
        {
@@ -3011,151 +2216,32 @@ simplify_and_tree (exp, pterm, insn_code, insn_index)
        }
     }
 
-  /* Check for simplifications.  Do some extra checking here since this
-     routine is called so many times.  */
+  if (attr_equal_p (exp, *pterm))
+    return false_rtx;
 
-  if (exp == *pterm)
+  else if (GET_CODE (exp) == NOT && attr_equal_p (XEXP (exp, 0), *pterm))
     return true_rtx;
 
-  else if (GET_CODE (exp) == NOT && XEXP (exp, 0) == *pterm)
-    return false_rtx;
+  else if (GET_CODE (*pterm) == NOT && attr_equal_p (XEXP (*pterm, 0), exp))
+    return true_rtx;
 
-  else if (GET_CODE (*pterm) == NOT && exp == XEXP (*pterm, 0))
+  else if (GET_CODE (*pterm) == EQ_ATTR && GET_CODE (exp) == NOT
+          && GET_CODE (XEXP (exp, 0)) == EQ_ATTR
+          && XSTR (*pterm, 0) == XSTR (XEXP (exp, 0), 0))
+    *pterm = false_rtx;
+
+  else if (GET_CODE (exp) == EQ_ATTR && GET_CODE (*pterm) == NOT
+          && GET_CODE (XEXP (*pterm, 0)) == EQ_ATTR
+          && XSTR (exp, 0) == XSTR (XEXP (*pterm, 0), 0))
     return false_rtx;
 
-  else if (GET_CODE (exp) == EQ_ATTR && GET_CODE (*pterm) == EQ_ATTR)
-    {
-      if (XSTR (exp, 0) != XSTR (*pterm, 0))
-       return exp;
-
-      if (! strcmp (XSTR (exp, 1), XSTR (*pterm, 1)))
-       return true_rtx;
-      else
-       return false_rtx;
-    }
-
-  else if (GET_CODE (*pterm) == EQ_ATTR && GET_CODE (exp) == NOT
-          && GET_CODE (XEXP (exp, 0)) == EQ_ATTR)
-    {
-      if (XSTR (*pterm, 0) != XSTR (XEXP (exp, 0), 0))
-       return exp;
-
-      if (! strcmp (XSTR (*pterm, 1), XSTR (XEXP (exp, 0), 1)))
-       return false_rtx;
-      else
-       return true_rtx;
-    }
-
-  else if (GET_CODE (exp) == EQ_ATTR && GET_CODE (*pterm) == NOT
-          && GET_CODE (XEXP (*pterm, 0)) == EQ_ATTR)
-    {
-      if (XSTR (exp, 0) != XSTR (XEXP (*pterm, 0), 0))
-       return exp;
-
-      if (! strcmp (XSTR (exp, 1), XSTR (XEXP (*pterm, 0), 1)))
-       return false_rtx;
-      else
-       *pterm = true_rtx;
-    }
-
-  else if (GET_CODE (exp) == NOT && GET_CODE (*pterm) == NOT)
-    {
-      if (attr_equal_p (XEXP (exp, 0), XEXP (*pterm, 0)))
-       return true_rtx;
-    }
-
-  else if (GET_CODE (exp) == NOT)
-    {
-      if (attr_equal_p (XEXP (exp, 0), *pterm))
-       return false_rtx;
-    }
-
-  else if (GET_CODE (*pterm) == NOT)
-    {
-      if (attr_equal_p (XEXP (*pterm, 0), exp))
-       return false_rtx;
-    }
-
-  else if (attr_equal_p (exp, *pterm))
-    return true_rtx;
-
   return exp;
 }
-\f
-/* Similar to `simplify_and_tree', but for IOR trees.  */
-
-static rtx
-simplify_or_tree (exp, pterm, insn_code, insn_index)
-     rtx exp;
-     rtx *pterm;
-     int insn_code, insn_index;
-{
-  rtx left, right;
-  rtx newexp;
-  rtx temp;
-  int left_eliminates_term, right_eliminates_term;
-
-  if (GET_CODE (exp) == IOR)
-    {
-      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))
-       {
-         newexp = attr_rtx (GET_CODE (exp), left, right);
-
-         exp = simplify_test_exp_in_temp (newexp, insn_code, insn_index);
-       }
-    }
-
-  else if (GET_CODE (exp) == AND)
-    {
-      /* 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_eliminates_term = (temp == false_rtx);
-
-      temp = *pterm;
-      right = simplify_or_tree (XEXP (exp, 1), &temp, insn_code, insn_index);
-      right_eliminates_term = (temp == false_rtx);
-
-      if (left_eliminates_term && right_eliminates_term)
-       *pterm = false_rtx;
-
-      if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
-       {
-         newexp = attr_rtx (GET_CODE (exp), left, right);
-
-         exp = simplify_test_exp_in_temp (newexp, insn_code, insn_index);
-       }
-    }
 
-  if (attr_equal_p (exp, *pterm))
-    return false_rtx;
-
-  else if (GET_CODE (exp) == NOT && attr_equal_p (XEXP (exp, 0), *pterm))
-    return true_rtx;
-
-  else if (GET_CODE (*pterm) == NOT && attr_equal_p (XEXP (*pterm, 0), exp))
-    return true_rtx;
-
-  else if (GET_CODE (*pterm) == EQ_ATTR && GET_CODE (exp) == NOT
-          && GET_CODE (XEXP (exp, 0)) == EQ_ATTR
-          && XSTR (*pterm, 0) == XSTR (XEXP (exp, 0), 0))
-    *pterm = false_rtx;
-
-  else if (GET_CODE (exp) == EQ_ATTR && GET_CODE (*pterm) == NOT
-          && GET_CODE (XEXP (*pterm, 0)) == EQ_ATTR
-          && XSTR (exp, 0) == XSTR (XEXP (*pterm, 0), 0))
-    return false_rtx;
-
-  return exp;
-}
 /* Compute approximate cost of the expression.  Used to decide whether
    expression is cheap enough for inline.  */
 static int
-attr_rtx_cost (x)
-     rtx x;
+attr_rtx_cost (rtx x)
 {
   int cost = 0;
   enum rtx_code code;
@@ -3169,9 +2255,13 @@ attr_rtx_cost (x)
        return 10;
       else
        return 0;
+
+    case EQ_ATTR_ALT:
+      return 0;
+
     case EQ_ATTR:
       /* Alternatives don't result into function call.  */
-      if (!strcmp (XSTR (x, 0), "alternative"))
+      if (!strcmp_check (XSTR (x, 0), alternative_name))
        return 0;
       else
        return 5;
@@ -3198,16 +2288,13 @@ attr_rtx_cost (x)
     }
   return cost;
 }
-\f
 
 /* Simplify test expression and use temporary obstack in order to avoid
-   memory bloat.  Use ATTR_IND_SIMPLIFIED to avoid unnecesary simplifications
-   and avoid unnecesary copying if possible.  */
+   memory bloat.  Use ATTR_IND_SIMPLIFIED to avoid unnecessary simplifications
+   and avoid unnecessary copying if possible.  */
 
 static rtx
-simplify_test_exp_in_temp (exp, insn_code, insn_index)
-  rtx exp;
-  int insn_code, insn_index;
+simplify_test_exp_in_temp (rtx exp, int insn_code, int insn_index)
 {
   rtx x;
   struct obstack *old;
@@ -3222,955 +2309,582 @@ simplify_test_exp_in_temp (exp, insn_code, insn_index)
   return attr_copy_rtx (x);
 }
 
-/* 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
-   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!  */
-
-static rtx
-simplify_test_exp (exp, insn_code, insn_index)
-     rtx exp;
-     int insn_code, insn_index;
-{
-  rtx left, right;
-  struct attr_desc *attr;
-  struct attr_value *av;
-  struct insn_ent *ie;
-  int i;
-  rtx newexp = exp;
-
-  /* Don't re-simplify something we already simplified.  */
-  if (ATTR_IND_SIMPLIFIED_P (exp) || ATTR_CURR_SIMPLIFIED_P (exp))
-    return exp;
-
-  switch (GET_CODE (exp))
-    {
-    case AND:
-      left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
-      SIMPLIFY_ALTERNATIVE (left);
-      if (left == false_rtx)
-       return false_rtx;
-      right = SIMPLIFY_TEST_EXP (XEXP (exp, 1), insn_code, insn_index);
-      SIMPLIFY_ALTERNATIVE (right);
-      if (left == 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
-        yield simplifications.  */
-      if ((GET_CODE (left) == IOR || GET_CODE (right) == IOR)
-         && compute_alternative_mask (left, IOR)
-         && compute_alternative_mask (right, IOR))
-       {
-         if (GET_CODE (left) == IOR)
-           {
-             rtx tem = left;
-             left = right;
-             right = tem;
-           }
-
-         newexp = attr_rtx (IOR,
-                            attr_rtx (AND, left, XEXP (right, 0)),
-                            attr_rtx (AND, left, XEXP (right, 1)));
-
-         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-       }
-
-      /* Try with the term on both sides.  */
-      right = simplify_and_tree (right, &left, insn_code, insn_index);
-      if (left == XEXP (exp, 0) && right == XEXP (exp, 1))
-       left = simplify_and_tree (left, &right, insn_code, insn_index);
-
-      if (left == false_rtx || right == false_rtx)
-       return false_rtx;
-      else if (left == true_rtx)
-       {
-         return right;
-       }
-      else if (right == true_rtx)
-       {
-         return left;
-       }
-      /* See if all or all but one of the insn's alternatives are specified
-        in this tree.  Optimize if so.  */
-
-      else if (insn_code >= 0
-              && (GET_CODE (left) == AND
-                  || (GET_CODE (left) == NOT
-                      && GET_CODE (XEXP (left, 0)) == EQ_ATTR
-                      && XSTR (XEXP (left, 0), 0) == alternative_name)
-                  || GET_CODE (right) == AND
-                  || (GET_CODE (right) == NOT
-                      && GET_CODE (XEXP (right, 0)) == EQ_ATTR
-                      && XSTR (XEXP (right, 0), 0) == alternative_name)))
-       {
-         i = compute_alternative_mask (exp, AND);
-         if (i & ~insn_alternatives[insn_code])
-           fatal ("invalid alternative specified for pattern number %d",
-                  insn_index);
-
-         /* If all alternatives are excluded, this is false.  */
-         i ^= insn_alternatives[insn_code];
-         if (i == 0)
-           return false_rtx;
-         else if ((i & (i - 1)) == 0 && insn_alternatives[insn_code] > 1)
-           {
-             /* If just one excluded, AND a comparison with that one to the
-                front of the tree.  The others will be eliminated by
-                optimization.  We do not want to do this if the insn has one
-                alternative and we have tested none of them!  */
-             left = make_alternative_compare (i);
-             right = simplify_and_tree (exp, &left, insn_code, insn_index);
-             newexp = attr_rtx (AND, left, right);
-
-             return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-           }
-       }
+/* Returns true if S1 is a subset of S2.  */
 
-      if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
-       {
-         newexp = attr_rtx (AND, left, right);
-         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-       }
-      break;
-
-    case IOR:
-      left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
-      SIMPLIFY_ALTERNATIVE (left);
-      if (left == true_rtx)
-       return true_rtx;
-      right = SIMPLIFY_TEST_EXP (XEXP (exp, 1), insn_code, insn_index);
-      SIMPLIFY_ALTERNATIVE (right);
-      if (right == 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)
-       return true_rtx;
-      else if (left == false_rtx)
-       {
-         return right;
-       }
-      else if (right == false_rtx)
-       {
-         return left;
-       }
-
-      /* Test for simple cases where the distributive law is useful.  I.e.,
-           convert (ior (and (x) (y))
-                        (and (x) (z)))
-           to      (and (x)
-                        (ior (y) (z)))
-       */
-
-      else if (GET_CODE (left) == AND && GET_CODE (right) == AND
-              && attr_equal_p (XEXP (left, 0), XEXP (right, 0)))
-       {
-         newexp = attr_rtx (IOR, XEXP (left, 1), XEXP (right, 1));
-
-         left = XEXP (left, 0);
-         right = newexp;
-         newexp = attr_rtx (AND, left, right);
-         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-       }
-
-      /* See if all or all but one of the insn's alternatives are specified
-        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)))
-       {
-         i = compute_alternative_mask (exp, IOR);
-         if (i & ~insn_alternatives[insn_code])
-           fatal ("invalid alternative specified for pattern number %d",
-                  insn_index);
-
-         /* If all alternatives are included, this is true.  */
-         i ^= insn_alternatives[insn_code];
-         if (i == 0)
-           return true_rtx;
-         else if ((i & (i - 1)) == 0 && insn_alternatives[insn_code] > 1)
-           {
-             /* If just one excluded, IOR a comparison with that one to the
-                front of the tree.  The others will be eliminated by
-                optimization.  We do not want to do this if the insn has one
-                alternative and we have tested none of them!  */
-             left = make_alternative_compare (i);
-             right = simplify_and_tree (exp, &left, insn_code, insn_index);
-             newexp = attr_rtx (IOR, attr_rtx (NOT, left), right);
-
-             return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-           }
-       }
-
-      if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
-       {
-         newexp = attr_rtx (IOR, left, right);
-         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-       }
-      break;
-
-    case NOT:
-      if (GET_CODE (XEXP (exp, 0)) == NOT)
-       {
-         left = SIMPLIFY_TEST_EXP (XEXP (XEXP (exp, 0), 0),
-                                   insn_code, insn_index);
-         SIMPLIFY_ALTERNATIVE (left);
-         return left;
-       }
-
-      left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
-      SIMPLIFY_ALTERNATIVE (left);
-      if (GET_CODE (left) == NOT)
-       return XEXP (left, 0);
-
-      if (left == false_rtx)
-       return true_rtx;
-      else if (left == true_rtx)
-       return false_rtx;
-
-      /* Try to apply De`Morgan's laws.  */
-      else if (GET_CODE (left) == IOR)
-       {
-         newexp = attr_rtx (AND,
-                            attr_rtx (NOT, XEXP (left, 0)),
-                            attr_rtx (NOT, XEXP (left, 1)));
-
-         newexp = SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-       }
-      else if (GET_CODE (left) == AND)
-       {
-         newexp = attr_rtx (IOR,
-                            attr_rtx (NOT, XEXP (left, 0)),
-                            attr_rtx (NOT, XEXP (left, 1)));
-
-         newexp = SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
-       }
-      else if (left != XEXP (exp, 0))
-       {
-         newexp = attr_rtx (NOT, left);
-       }
-      break;
-
-    case EQ_ATTR:
-      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.  */
-      if (XSTR (exp, 0) != alternative_name
-         && (attr = find_attr (XSTR (exp, 0), 0)) != NULL)
-       for (av = attr->first_value; av; av = av->next)
-         for (ie = av->first_insn; ie; ie = ie->next)
-           if (ie->insn_code == insn_code)
-             {
-               rtx x;
-               x = evaluate_eq_attr (exp, av->value, insn_code, insn_index);
-               x = SIMPLIFY_TEST_EXP (x, insn_code, insn_index);
-               if (attr_rtx_cost(x) < 20)
-                 return x;
-             }
-      break;
-
-    default:
-      break;
-    }
-
-  /* We have already simplified this expression.  Simplifying it again
-     won't buy anything unless we weren't given a valid insn code
-     to process (i.e., we are canonicalizing something.).  */
-  if (insn_code != -2 /* Seems wrong: && current_alternative_string.  */
-      && ! ATTR_IND_SIMPLIFIED_P (newexp))
-    return copy_rtx_unchanging (newexp);
-
-  return newexp;
-}
-\f
-/* Optimize the attribute lists by seeing if we can determine conditional
-   values from the known values of other attributes.  This will save subroutine
-   calls during the compilation.  */
-
-static void
-optimize_attrs ()
+static bool
+attr_alt_subset_p (rtx s1, rtx s2)
 {
-  struct attr_desc *attr;
-  struct attr_value *av;
-  struct insn_ent *ie;
-  rtx newexp;
-  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 **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;
+  switch ((XINT (s1, 1) << 1) | XINT (s2, 1))
+    {
+    case (0 << 1) | 0:
+      return !(XINT (s1, 0) &~ XINT (s2, 0));
 
-  /* Make 2 extra elements, for "code" values -2 and -1.  */
-  insn_code_values
-    = (struct attr_value_list **) xmalloc ((insn_code_number + 2)
-                                         * sizeof (struct attr_value_list *));
-  memset ((char *) insn_code_values, 0,
-        (insn_code_number + 2) * sizeof (struct attr_value_list *));
+    case (0 << 1) | 1:
+      return !(XINT (s1, 0) & XINT (s2, 0));
 
-  /* Offset the table address so we can index by -2 or -1.  */
-  insn_code_values += 2;
+    case (1 << 1) | 0:
+      return false;
 
-  iv = ivbuf = ((struct attr_value_list *)
-               xmalloc (num_insn_ents * sizeof (struct attr_value_list)));
+    case (1 << 1) | 1:
+      return !(XINT (s2, 0) &~ XINT (s1, 0));
 
-  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->attr = attr;
-           iv->av = av;
-           iv->ie = ie;
-           iv->next = insn_code_values[ie->insn_code];
-           insn_code_values[ie->insn_code] = iv;
-           iv++;
-         }
+    default:
+      gcc_unreachable ();
+    }
+}
 
-  /* Sanity check on num_insn_ents.  */
-  if (iv != ivbuf + num_insn_ents)
-    abort ();
+/* Returns true if S1 is a subset of complement of S2.  */
 
-  /* Process one insn code at a time.  */
-  for (i = -2; i < insn_code_number; i++)
+static bool
+attr_alt_subset_of_compl_p (rtx s1, rtx s2)
+{
+  switch ((XINT (s1, 1) << 1) | XINT (s2, 1))
     {
-      /* Clear the ATTR_CURR_SIMPLIFIED_P flag everywhere relevant.
-        We use it to mean "already simplified for this insn".  */
-      for (iv = insn_code_values[i]; iv; iv = iv->next)
-       clear_struct_flag (iv->av->value);
+    case (0 << 1) | 0:
+      return !(XINT (s1, 0) & XINT (s2, 0));
 
-      for (iv = insn_code_values[i]; iv; iv = iv->next)
-       {
-         struct obstack *old = rtl_obstack;
+    case (0 << 1) | 1:
+      return !(XINT (s1, 0) & ~XINT (s2, 0));
 
-         attr = iv->attr;
-         av = iv->av;
-         ie = iv->ie;
-         if (GET_CODE (av->value) != COND)
-           continue;
+    case (1 << 1) | 0:
+      return !(XINT (s2, 0) &~ XINT (s1, 0));
 
-         rtl_obstack = temp_obstack;
-#if 0 /* This was intended as a speed up, but it was slower.  */
-         if (insn_n_alternatives[ie->insn_code] > 6
-             && count_sub_rtxs (av->value, 200) >= 200)
-           newexp = simplify_by_alternatives (av->value, ie->insn_code,
-                                              ie->insn_index);
-         else
-#endif
-         newexp = av->value;
-         while (GET_CODE (newexp) == COND)
-           {
-             rtx newexp2 = simplify_cond (newexp, ie->insn_code,
-                                          ie->insn_index);
-             if (newexp2 == newexp)
-               break;
-             newexp = newexp2;
-           }
+    case (1 << 1) | 1:
+      return false;
 
-         rtl_obstack = old;
-         if (newexp != av->value)
-           {
-             newexp = attr_copy_rtx (newexp);
-             remove_insn_ent (av, ie);
-             av = get_attr_value (newexp, attr, ie->insn_code);
-             iv->av = av;
-             insert_insn_ent (av, ie);
-           }
-       }
+    default:
+      gcc_unreachable ();
     }
-
-  free (ivbuf);
-  free (insn_code_values - 2);
 }
 
-#if 0
+/* Return EQ_ATTR_ALT expression representing intersection of S1 and S2.  */
+
 static rtx
-simplify_by_alternatives (exp, insn_code, insn_index)
-     rtx exp;
-     int insn_code, insn_index;
+attr_alt_intersection (rtx s1, rtx s2)
 {
-  int i;
-  int len = insn_n_alternatives[insn_code];
-  rtx newexp = rtx_alloc (COND);
-  rtx ultimate;
-
-  XVEC (newexp, 0) = rtvec_alloc (len * 2);
+  rtx result = rtx_alloc (EQ_ATTR_ALT);
 
-  /* It will not matter what value we use as the default value
-     of the new COND, since that default will never be used.
-     Choose something of the right type.  */
-  for (ultimate = exp; GET_CODE (ultimate) == COND;)
-    ultimate = XEXP (ultimate, 1);
-  XEXP (newexp, 1) = ultimate;
-
-  for (i = 0; i < insn_n_alternatives[insn_code]; i++)
+  switch ((XINT (s1, 1) << 1) | XINT (s2, 1))
     {
-      current_alternative_string = attr_numeral (i);
-      XVECEXP (newexp, 0, i * 2) = make_alternative_compare (1 << i);
-      XVECEXP (newexp, 0, i * 2 + 1)
-       = simplify_cond (exp, insn_code, insn_index);
+    case (0 << 1) | 0:
+      XINT (result, 0) = XINT (s1, 0) & XINT (s2, 0);
+      break;
+    case (0 << 1) | 1:
+      XINT (result, 0) = XINT (s1, 0) & ~XINT (s2, 0);
+      break;
+    case (1 << 1) | 0:
+      XINT (result, 0) = XINT (s2, 0) & ~XINT (s1, 0);
+      break;
+    case (1 << 1) | 1:
+      XINT (result, 0) = XINT (s1, 0) | XINT (s2, 0);
+      break;
+    default:
+      gcc_unreachable ();
     }
+  XINT (result, 1) = XINT (s1, 1) & XINT (s2, 1);
 
-  current_alternative_string = 0;
-  return simplify_cond (newexp, insn_code, insn_index);
+  return result;
 }
-#endif
-\f
-/* If EXP is a suitable expression, reorganize it by constructing an
-   equivalent expression that is a COND with the tests being all combinations
-   of attribute values and the values being simple constants.  */
+
+/* Return EQ_ATTR_ALT expression representing union of S1 and S2.  */
 
 static rtx
-simplify_by_exploding (exp)
-     rtx exp;
+attr_alt_union (rtx s1, rtx s2)
 {
-  rtx list = 0, link, condexp, defval = NULL_RTX;
-  struct dimension *space;
-  rtx *condtest, *condval;
-  int i, j, total, ndim = 0;
-  int most_tests, num_marks, new_marks;
-  rtx ret;
+  rtx result = rtx_alloc (EQ_ATTR_ALT);
 
-  /* Locate all the EQ_ATTR expressions.  */
-  if (! find_and_mark_used_attributes (exp, &list, &ndim) || ndim == 0)
+  switch ((XINT (s1, 1) << 1) | XINT (s2, 1))
     {
-      unmark_used_attributes (list, 0, 0);
-      return exp;
+    case (0 << 1) | 0:
+      XINT (result, 0) = XINT (s1, 0) | XINT (s2, 0);
+      break;
+    case (0 << 1) | 1:
+      XINT (result, 0) = XINT (s2, 0) & ~XINT (s1, 0);
+      break;
+    case (1 << 1) | 0:
+      XINT (result, 0) = XINT (s1, 0) & ~XINT (s2, 0);
+      break;
+    case (1 << 1) | 1:
+      XINT (result, 0) = XINT (s1, 0) & XINT (s2, 0);
+      break;
+    default:
+      gcc_unreachable ();
     }
 
-  /* Create an attribute space from the list of used attributes.  For each
-     dimension in the attribute space, record the attribute, list of values
-     used, and number of values used.  Add members to the list of values to
-     cover the domain of the attribute.  This makes the expanded COND form
-     order independent.  */
-
-  space = (struct dimension *) xmalloc (ndim * sizeof (struct dimension));
-
-  total = 1;
-  for (ndim = 0; list; ndim++)
-    {
-      /* Pull the first attribute value from the list and record that
-        attribute as another dimension in the attribute space.  */
-      const char *name = XSTR (XEXP (list, 0), 0);
-      rtx *prev;
-
-      if ((space[ndim].attr = find_attr (name, 0)) == 0
-         || space[ndim].attr->is_numeric)
-       {
-         unmark_used_attributes (list, space, ndim);
-         return exp;
-       }
+  XINT (result, 1) = XINT (s1, 1) | XINT (s2, 1);
+  return result;
+}
 
-      /* Add all remaining attribute values that refer to this attribute.  */
-      space[ndim].num_values = 0;
-      space[ndim].values = 0;
-      prev = &list;
-      for (link = list; link; link = *prev)
-       if (! strcmp (XSTR (XEXP (link, 0), 0), name))
-         {
-           space[ndim].num_values++;
-           *prev = XEXP (link, 1);
-           XEXP (link, 1) = space[ndim].values;
-           space[ndim].values = link;
-         }
-       else
-         prev = &XEXP (link, 1);
+/* Return EQ_ATTR_ALT expression representing complement of S.  */
 
-      /* Add sufficient members to the list of values to make the list
-        mutually exclusive and record the total size of the attribute
-        space.  */
-      total *= add_values_to_cover (&space[ndim]);
-    }
+static rtx
+attr_alt_complement (rtx s)
+{
+  rtx result = rtx_alloc (EQ_ATTR_ALT);
 
-  /* Sort the attribute space so that the attributes go from non-constant
-     to constant and from most values to least values.  */
-  for (i = 0; i < ndim; i++)
-    for (j = ndim - 1; j > i; j--)
-      if ((space[j-1].attr->is_const && !space[j].attr->is_const)
-         || space[j-1].num_values < space[j].num_values)
-       {
-         struct dimension tmp;
-         tmp = space[j];
-         space[j] = space[j - 1];
-         space[j - 1] = tmp;
-       }
+  XINT (result, 0) = XINT (s, 0);
+  XINT (result, 1) = 1 - XINT (s, 1);
 
-  /* Establish the initial current value.  */
-  for (i = 0; i < ndim; i++)
-    space[i].current_value = space[i].values;
+  return result;
+}
 
-  condtest = (rtx *) xmalloc (total * sizeof (rtx));
-  condval = (rtx *) xmalloc (total * sizeof (rtx));
+/* Return EQ_ATTR_ALT expression representing set containing elements set
+   in E.  */
 
-  /* Expand the tests and values by iterating over all values in the
-     attribute space.  */
-  for (i = 0;; i++)
-    {
-      condtest[i] = test_for_current_value (space, ndim);
-      condval[i] = simplify_with_current_value (exp, space, ndim);
-      if (! increment_current_value (space, ndim))
-       break;
-    }
-  if (i != total - 1)
-    abort ();
-
-  /* We are now finished with the original expression.  */
-  unmark_used_attributes (0, space, ndim);
-  free (space);
-
-  /* Find the most used constant value and make that the default.  */
-  most_tests = -1;
-  for (i = num_marks = 0; i < total; i++)
-    if (GET_CODE (condval[i]) == CONST_STRING
-       && ! ATTR_EQ_ATTR_P (condval[i]))
-      {
-       /* Mark the unmarked constant value and count how many are marked.  */
-       ATTR_EQ_ATTR_P (condval[i]) = 1;
-       for (j = new_marks = 0; j < total; j++)
-         if (GET_CODE (condval[j]) == CONST_STRING
-             && ATTR_EQ_ATTR_P (condval[j]))
-           new_marks++;
-       if (new_marks - num_marks > most_tests)
-         {
-           most_tests = new_marks - num_marks;
-           defval = condval[i];
-         }
-       num_marks = new_marks;
-      }
-  /* Clear all the marks.  */
-  for (i = 0; i < total; i++)
-    ATTR_EQ_ATTR_P (condval[i]) = 0;
+static rtx
+mk_attr_alt (int e)
+{
+  rtx result = rtx_alloc (EQ_ATTR_ALT);
 
-  /* Give up if nothing is constant.  */
-  if (num_marks == 0)
-    ret = exp;
-  
-  /* If all values are the default, use that.  */
-  else if (total == most_tests)
-    ret = defval;
+  XINT (result, 0) = e;
+  XINT (result, 1) = 0;
 
-  /* Make a COND with the most common constant value the default.  (A more
-     complex method where tests with the same value were combined didn't
-     seem to improve things.)  */
-  else
-    {
-      condexp = rtx_alloc (COND);
-      XVEC (condexp, 0) = rtvec_alloc ((total - most_tests) * 2);
-      XEXP (condexp, 1) = defval;
-      for (i = j = 0; i < total; i++)
-       if (condval[i] != defval)
-         {
-           XVECEXP (condexp, 0, 2 * j) = condtest[i];
-           XVECEXP (condexp, 0, 2 * j + 1) = condval[i];
-           j++;
-         }
-      ret = condexp;
-    }
-  free (condtest);
-  free (condval);
-  return ret;
+  return result;
 }
 
-/* Set the ATTR_EQ_ATTR_P flag for all EQ_ATTR expressions in EXP and
-   verify that EXP can be simplified to a constant term if all the EQ_ATTR
-   tests have known value.  */
+/* 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.
 
-static int
-find_and_mark_used_attributes (exp, terms, nterms)
-     rtx exp, *terms;
-     int *nterms;
+   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!  */
+
+static rtx
+simplify_test_exp (rtx exp, int insn_code, int insn_index)
 {
+  rtx left, right;
+  struct attr_desc *attr;
+  struct attr_value *av;
+  struct insn_ent *ie;
+  struct attr_value_list *iv;
   int i;
+  rtx newexp = exp;
+  bool left_alt, right_alt;
+
+  /* Don't re-simplify something we already simplified.  */
+  if (ATTR_IND_SIMPLIFIED_P (exp) || ATTR_CURR_SIMPLIFIED_P (exp))
+    return exp;
 
   switch (GET_CODE (exp))
     {
-    case EQ_ATTR:
-      if (! ATTR_EQ_ATTR_P (exp))
+    case AND:
+      left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
+      if (left == false_rtx)
+       return false_rtx;
+      right = SIMPLIFY_TEST_EXP (XEXP (exp, 1), insn_code, insn_index);
+      if (right == false_rtx)
+       return false_rtx;
+
+      if (GET_CODE (left) == EQ_ATTR_ALT
+         && GET_CODE (right) == EQ_ATTR_ALT)
        {
-         rtx link = rtx_alloc (EXPR_LIST);
-         XEXP (link, 0) = exp;
-         XEXP (link, 1) = *terms;
-         *terms = link;
-         *nterms += 1;
-         ATTR_EQ_ATTR_P (exp) = 1;
+         exp = attr_alt_intersection (left, right);
+         return simplify_test_exp (exp, insn_code, insn_index);
        }
-      return 1;
 
-    case CONST_STRING:
-    case CONST_INT:
-      return 1;
+      /* If either side is an IOR and we have (eq_attr "alternative" ..")
+        present on both sides, apply the distributive law since this will
+        yield simplifications.  */
+      if ((GET_CODE (left) == IOR || GET_CODE (right) == IOR)
+         && compute_alternative_mask (left, IOR)
+         && compute_alternative_mask (right, IOR))
+       {
+         if (GET_CODE (left) == IOR)
+           {
+             rtx tem = left;
+             left = right;
+             right = tem;
+           }
 
-    case IF_THEN_ELSE:
-      if (! find_and_mark_used_attributes (XEXP (exp, 2), terms, nterms))
-       return 0;
-    case IOR:
-    case AND:
-      if (! find_and_mark_used_attributes (XEXP (exp, 1), terms, nterms))
-       return 0;
-    case NOT:
-      if (! find_and_mark_used_attributes (XEXP (exp, 0), terms, nterms))
-       return 0;
-      return 1;
+         newexp = attr_rtx (IOR,
+                            attr_rtx (AND, left, XEXP (right, 0)),
+                            attr_rtx (AND, left, XEXP (right, 1)));
 
-    case COND:
-      for (i = 0; i < XVECLEN (exp, 0); i++)
-       if (! find_and_mark_used_attributes (XVECEXP (exp, 0, i), terms, nterms))
-         return 0;
-      if (! find_and_mark_used_attributes (XEXP (exp, 1), terms, nterms))
-       return 0;
-      return 1;
+         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+       }
 
-    default:
-      return 0;
-    }
-}
+      /* Try with the term on both sides.  */
+      right = simplify_and_tree (right, &left, insn_code, insn_index);
+      if (left == XEXP (exp, 0) && right == XEXP (exp, 1))
+       left = simplify_and_tree (left, &right, insn_code, insn_index);
 
-/* Clear the ATTR_EQ_ATTR_P flag in all EQ_ATTR expressions on LIST and
-   in the values of the NDIM-dimensional attribute space SPACE.  */
+      if (left == false_rtx || right == false_rtx)
+       return false_rtx;
+      else if (left == true_rtx)
+       {
+         return right;
+       }
+      else if (right == true_rtx)
+       {
+         return left;
+       }
+      /* See if all or all but one of the insn's alternatives are specified
+        in this tree.  Optimize if so.  */
 
-static void
-unmark_used_attributes (list, space, ndim)
-     rtx list;
-     struct dimension *space;
-     int ndim;
-{
-  rtx link, exp;
-  int i;
+      if (GET_CODE (left) == NOT)
+       left_alt = (GET_CODE (XEXP (left, 0)) == EQ_ATTR
+                   && XSTR (XEXP (left, 0), 0) == alternative_name);
+      else
+       left_alt = (GET_CODE (left) == EQ_ATTR_ALT
+                   && XINT (left, 1));
 
-  for (i = 0; i < ndim; i++)
-    unmark_used_attributes (space[i].values, 0, 0);
+      if (GET_CODE (right) == NOT)
+       right_alt = (GET_CODE (XEXP (right, 0)) == EQ_ATTR
+                    && XSTR (XEXP (right, 0), 0) == alternative_name);
+      else
+       right_alt = (GET_CODE (right) == EQ_ATTR_ALT
+                    && XINT (right, 1));
+
+      if (insn_code >= 0
+         && (GET_CODE (left) == AND
+             || left_alt
+             || GET_CODE (right) == AND
+             || right_alt))
+       {
+         i = compute_alternative_mask (exp, AND);
+         if (i & ~insn_alternatives[insn_code])
+           fatal ("invalid alternative specified for pattern number %d",
+                  insn_index);
 
-  for (link = list; link; link = XEXP (link, 1))
-    {
-      exp = XEXP (link, 0);
-      if (GET_CODE (exp) == EQ_ATTR)
-       ATTR_EQ_ATTR_P (exp) = 0;
-    }
-}
+         /* If all alternatives are excluded, this is false.  */
+         i ^= insn_alternatives[insn_code];
+         if (i == 0)
+           return false_rtx;
+         else if ((i & (i - 1)) == 0 && insn_alternatives[insn_code] > 1)
+           {
+             /* If just one excluded, AND a comparison with that one to the
+                front of the tree.  The others will be eliminated by
+                optimization.  We do not want to do this if the insn has one
+                alternative and we have tested none of them!  */
+             left = make_alternative_compare (i);
+             right = simplify_and_tree (exp, &left, insn_code, insn_index);
+             newexp = attr_rtx (AND, left, right);
 
-/* Update the attribute dimension DIM so that all values of the attribute
-   are tested.  Return the updated number of values.  */
+             return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+           }
+       }
 
-static int
-add_values_to_cover (dim)
-     struct dimension *dim;
-{
-  struct attr_value *av;
-  rtx exp, link, *prev;
-  int nalt = 0;
-
-  for (av = dim->attr->first_value; av; av = av->next)
-    if (GET_CODE (av->value) == CONST_STRING)
-      nalt++;
-
-  if (nalt < dim->num_values)
-    abort ();
-  else if (nalt == dim->num_values)
-    /* OK.  */
-    ;
-  else if (nalt * 2 < dim->num_values * 3)
-    {
-      /* Most all the values of the attribute are used, so add all the unused
-        values.  */
-      prev = &dim->values;
-      for (link = dim->values; link; link = *prev)
-       prev = &XEXP (link, 1);
-
-      for (av = dim->attr->first_value; av; av = av->next)
-       if (GET_CODE (av->value) == CONST_STRING)
-         {
-           exp = attr_eq (dim->attr->name, XSTR (av->value, 0));
-           if (ATTR_EQ_ATTR_P (exp))
-             continue;
+      if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
+       {
+         newexp = attr_rtx (AND, left, right);
+         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+       }
+      break;
 
-           link = rtx_alloc (EXPR_LIST);
-           XEXP (link, 0) = exp;
-           XEXP (link, 1) = 0;
-           *prev = link;
-           prev = &XEXP (link, 1);
-         }
-      dim->num_values = nalt;
-    }
-  else
-    {
-      rtx orexp = false_rtx;
+    case IOR:
+      left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
+      if (left == true_rtx)
+       return true_rtx;
+      right = SIMPLIFY_TEST_EXP (XEXP (exp, 1), insn_code, insn_index);
+      if (right == true_rtx)
+       return true_rtx;
+
+      if (GET_CODE (left) == EQ_ATTR_ALT
+         && GET_CODE (right) == EQ_ATTR_ALT)
+       {
+         exp = attr_alt_union (left, right);
+         return simplify_test_exp (exp, insn_code, insn_index);
+       }
+
+      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);
 
-      /* Very few values are used, so compute a mutually exclusive
-        expression.  (We could do this for numeric values if that becomes
-        important.)  */
-      prev = &dim->values;
-      for (link = dim->values; link; link = *prev)
+      if (right == true_rtx || left == true_rtx)
+       return true_rtx;
+      else if (left == false_rtx)
+       {
+         return right;
+       }
+      else if (right == false_rtx)
        {
-         orexp = insert_right_side (IOR, orexp, XEXP (link, 0), -2, -2);
-         prev = &XEXP (link, 1);
+         return left;
        }
-      link = rtx_alloc (EXPR_LIST);
-      XEXP (link, 0) = attr_rtx (NOT, orexp);
-      XEXP (link, 1) = 0;
-      *prev = link;
-      dim->num_values++;
-    }
-  return dim->num_values;
-}
-
-/* Increment the current value for the NDIM-dimensional attribute space SPACE
-   and return FALSE if the increment overflowed.  */
-
-static int
-increment_current_value (space, ndim)
-     struct dimension *space;
-     int ndim;
-{
-  int i;
-
-  for (i = ndim - 1; i >= 0; i--)
-    {
-      if ((space[i].current_value = XEXP (space[i].current_value, 1)) == 0)
-       space[i].current_value = space[i].values;
-      else
-       return 1;
-    }
-  return 0;
-}
 
-/* Construct an expression corresponding to the current value for the
-   NDIM-dimensional attribute space SPACE.  */
+      /* Test for simple cases where the distributive law is useful.  I.e.,
+           convert (ior (and (x) (y))
+                        (and (x) (z)))
+           to      (and (x)
+                        (ior (y) (z)))
+       */
 
-static rtx
-test_for_current_value (space, ndim)
-     struct dimension *space;
-     int ndim;
-{
-  int i;
-  rtx exp = true_rtx;
+      else if (GET_CODE (left) == AND && GET_CODE (right) == AND
+              && attr_equal_p (XEXP (left, 0), XEXP (right, 0)))
+       {
+         newexp = attr_rtx (IOR, XEXP (left, 1), XEXP (right, 1));
 
-  for (i = 0; i < ndim; i++)
-    exp = insert_right_side (AND, exp, XEXP (space[i].current_value, 0),
-                            -2, -2);
+         left = XEXP (left, 0);
+         right = newexp;
+         newexp = attr_rtx (AND, left, right);
+         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+       }
 
-  return exp;
-}
+      /* See if all or all but one of the insn's alternatives are specified
+        in this tree.  Optimize if so.  */
 
-/* Given the current value of the NDIM-dimensional attribute space SPACE,
-   set the corresponding EQ_ATTR expressions to that value and reduce
-   the expression EXP as much as possible.  On input [and output], all
-   known EQ_ATTR expressions are set to FALSE.  */
+      else if (insn_code >= 0
+              && (GET_CODE (left) == IOR
+                  || (GET_CODE (left) == EQ_ATTR_ALT
+                      && !XINT (left, 1))
+                  || (GET_CODE (left) == EQ_ATTR
+                      && XSTR (left, 0) == alternative_name)
+                  || GET_CODE (right) == IOR
+                  || (GET_CODE (right) == EQ_ATTR_ALT
+                      && !XINT (right, 1))
+                  || (GET_CODE (right) == EQ_ATTR
+                      && XSTR (right, 0) == alternative_name)))
+       {
+         i = compute_alternative_mask (exp, IOR);
+         if (i & ~insn_alternatives[insn_code])
+           fatal ("invalid alternative specified for pattern number %d",
+                  insn_index);
 
-static rtx
-simplify_with_current_value (exp, space, ndim)
-     rtx exp;
-     struct dimension *space;
-     int ndim;
-{
-  int i;
-  rtx x;
+         /* If all alternatives are included, this is true.  */
+         i ^= insn_alternatives[insn_code];
+         if (i == 0)
+           return true_rtx;
+         else if ((i & (i - 1)) == 0 && insn_alternatives[insn_code] > 1)
+           {
+             /* If just one excluded, IOR a comparison with that one to the
+                front of the tree.  The others will be eliminated by
+                optimization.  We do not want to do this if the insn has one
+                alternative and we have tested none of them!  */
+             left = make_alternative_compare (i);
+             right = simplify_and_tree (exp, &left, insn_code, insn_index);
+             newexp = attr_rtx (IOR, attr_rtx (NOT, left), right);
 
-  /* Mark each current value as TRUE.  */
-  for (i = 0; i < ndim; i++)
-    {
-      x = XEXP (space[i].current_value, 0);
-      if (GET_CODE (x) == EQ_ATTR)
-       ATTR_EQ_ATTR_P (x) = 0;
-    }
+             return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+           }
+       }
 
-  exp = simplify_with_current_value_aux (exp);
+      if (left != XEXP (exp, 0) || right != XEXP (exp, 1))
+       {
+         newexp = attr_rtx (IOR, left, right);
+         return SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+       }
+      break;
 
-  /* Change each current value back to FALSE.  */
-  for (i = 0; i < ndim; i++)
-    {
-      x = XEXP (space[i].current_value, 0);
-      if (GET_CODE (x) == EQ_ATTR)
-       ATTR_EQ_ATTR_P (x) = 1;
-    }
+    case NOT:
+      if (GET_CODE (XEXP (exp, 0)) == NOT)
+       {
+         left = SIMPLIFY_TEST_EXP (XEXP (XEXP (exp, 0), 0),
+                                   insn_code, insn_index);
+         return left;
+       }
 
-  return exp;
-}
+      left = SIMPLIFY_TEST_EXP (XEXP (exp, 0), insn_code, insn_index);
+      if (GET_CODE (left) == NOT)
+       return XEXP (left, 0);
 
-/* Reduce the expression EXP based on the ATTR_EQ_ATTR_P settings of
-   all EQ_ATTR expressions.  */
+      if (left == false_rtx)
+       return true_rtx;
+      if (left == true_rtx)
+       return false_rtx;
 
-static rtx
-simplify_with_current_value_aux (exp)
-     rtx exp;
-{
-  int i;
-  rtx cond;
+      if (GET_CODE (left) == EQ_ATTR_ALT)
+       {
+         exp = attr_alt_complement (left);
+         return simplify_test_exp (exp, insn_code, insn_index);
+       }
 
-  switch (GET_CODE (exp))
-    {
-    case EQ_ATTR:
-      if (ATTR_EQ_ATTR_P (exp))
-       return false_rtx;
-      else
-       return true_rtx;
-    case CONST_STRING:
-    case CONST_INT:
-      return exp;
+      /* Try to apply De`Morgan's laws.  */
+      if (GET_CODE (left) == IOR)
+       {
+         newexp = attr_rtx (AND,
+                            attr_rtx (NOT, XEXP (left, 0)),
+                            attr_rtx (NOT, XEXP (left, 1)));
 
-    case IF_THEN_ELSE:
-      cond = simplify_with_current_value_aux (XEXP (exp, 0));
-      if (cond == true_rtx)
-       return simplify_with_current_value_aux (XEXP (exp, 1));
-      else if (cond == false_rtx)
-       return simplify_with_current_value_aux (XEXP (exp, 2));
-      else
-       return attr_rtx (IF_THEN_ELSE, cond,
-                        simplify_with_current_value_aux (XEXP (exp, 1)),
-                        simplify_with_current_value_aux (XEXP (exp, 2)));
+         newexp = SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+       }
+      else if (GET_CODE (left) == AND)
+       {
+         newexp = attr_rtx (IOR,
+                            attr_rtx (NOT, XEXP (left, 0)),
+                            attr_rtx (NOT, XEXP (left, 1)));
 
-    case IOR:
-      cond = simplify_with_current_value_aux (XEXP (exp, 1));
-      if (cond == true_rtx)
-       return cond;
-      else if (cond == false_rtx)
-       return simplify_with_current_value_aux (XEXP (exp, 0));
-      else
-       return attr_rtx (IOR, cond,
-                        simplify_with_current_value_aux (XEXP (exp, 0)));
+         newexp = SIMPLIFY_TEST_EXP (newexp, insn_code, insn_index);
+       }
+      else if (left != XEXP (exp, 0))
+       {
+         newexp = attr_rtx (NOT, left);
+       }
+      break;
 
-    case AND:
-      cond = simplify_with_current_value_aux (XEXP (exp, 1));
-      if (cond == true_rtx)
-       return simplify_with_current_value_aux (XEXP (exp, 0));
-      else if (cond == false_rtx)
-       return cond;
-      else
-       return attr_rtx (AND, cond,
-                        simplify_with_current_value_aux (XEXP (exp, 0)));
+    case EQ_ATTR_ALT:
+      if (!XINT (exp, 0))
+       return XINT (exp, 1) ? true_rtx : false_rtx;
+      break;
 
-    case NOT:
-      cond = simplify_with_current_value_aux (XEXP (exp, 0));
-      if (cond == true_rtx)
-       return false_rtx;
-      else if (cond == false_rtx)
-       return true_rtx;
-      else
-       return attr_rtx (NOT, cond);
+    case EQ_ATTR:
+      if (XSTR (exp, 0) == alternative_name)
+       {
+         newexp = mk_attr_alt (1 << atoi (XSTR (exp, 1)));
+         break;
+       }
 
-    case COND:
-      for (i = 0; i < XVECLEN (exp, 0); i += 2)
+      /* 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.  */
+      if (insn_code >= 0
+         && (attr = find_attr (&XSTR (exp, 0), 0)) != NULL)
        {
-         cond = simplify_with_current_value_aux (XVECEXP (exp, 0, i));
-         if (cond == true_rtx)
-           return simplify_with_current_value_aux (XVECEXP (exp, 0, i + 1));
-         else if (cond == false_rtx)
-           continue;
+         rtx x;
+
+         av = NULL;
+         if (insn_code_values)
+           {
+             for (iv = insn_code_values[insn_code]; iv; iv = iv->next)
+               if (iv->attr == attr)
+                 {
+                   av = iv->av;
+                   break;
+                 }
+           }
          else
-           abort (); /* With all EQ_ATTR's of known value, a case should
-                        have been selected.  */
+           {
+             for (av = attr->first_value; av; av = av->next)
+               for (ie = av->first_insn; ie; ie = ie->next)
+                 if (ie->def->insn_code == insn_code)
+                   goto got_av;
+           }
+
+         if (av)
+           {
+           got_av:
+             x = evaluate_eq_attr (exp, av->value, insn_code, insn_index);
+             x = SIMPLIFY_TEST_EXP (x, insn_code, insn_index);
+             if (attr_rtx_cost(x) < 20)
+               return x;
+           }
        }
-      return simplify_with_current_value_aux (XEXP (exp, 1));
+      break;
 
     default:
-      abort ();
+      break;
     }
+
+  /* We have already simplified this expression.  Simplifying it again
+     won't buy anything unless we weren't given a valid insn code
+     to process (i.e., we are canonicalizing something.).  */
+  if (insn_code != -2
+      && ! ATTR_IND_SIMPLIFIED_P (newexp))
+    return copy_rtx_unchanging (newexp);
+
+  return newexp;
 }
-\f
-/* Clear the ATTR_CURR_SIMPLIFIED_P flag in EXP and its subexpressions.  */
+
+/* Optimize the attribute lists by seeing if we can determine conditional
+   values from the known values of other attributes.  This will save subroutine
+   calls during the compilation.  */
 
 static void
-clear_struct_flag (x)
-     rtx x;
+optimize_attrs (void)
 {
+  struct attr_desc *attr;
+  struct attr_value *av;
+  struct insn_ent *ie;
+  rtx newexp;
   int i;
-  int j;
-  enum rtx_code code;
-  const char *fmt;
+  struct attr_value_list *ivbuf;
+  struct attr_value_list *iv;
 
-  ATTR_CURR_SIMPLIFIED_P (x) = 0;
-  if (ATTR_IND_SIMPLIFIED_P (x))
+  /* 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;
 
-  code = GET_CODE (x);
+  /* Make 2 extra elements, for "code" values -2 and -1.  */
+  insn_code_values = XCNEWVEC (struct attr_value_list *, insn_code_number + 2);
 
-  switch (code)
-    {
-    case REG:
-    case QUEUED:
-    case CONST_INT:
-    case CONST_DOUBLE:
-    case CONST_VECTOR:
-    case SYMBOL_REF:
-    case CODE_LABEL:
-    case PC:
-    case CC0:
-    case EQ_ATTR:
-    case ATTR_FLAG:
-      return;
+  /* Offset the table address so we can index by -2 or -1.  */
+  insn_code_values += 2;
 
-    default:
-      break;
-    }
+  iv = ivbuf = XNEWVEC (struct attr_value_list, num_insn_ents);
 
-  /* Compare the elements.  If any pair of corresponding elements
-     fail to match, return 0 for the whole things.  */
+  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->attr = attr;
+           iv->av = av;
+           iv->ie = ie;
+           iv->next = insn_code_values[ie->def->insn_code];
+           insn_code_values[ie->def->insn_code] = iv;
+           iv++;
+         }
 
-  fmt = GET_RTX_FORMAT (code);
-  for (i = GET_RTX_LENGTH (code) - 1; i >= 0; i--)
+  /* Sanity check on num_insn_ents.  */
+  gcc_assert (iv == ivbuf + num_insn_ents);
+
+  /* Process one insn code at a time.  */
+  for (i = -2; i < insn_code_number; i++)
     {
-      switch (fmt[i])
+      /* Clear the ATTR_CURR_SIMPLIFIED_P flag everywhere relevant.
+        We use it to mean "already simplified for this insn".  */
+      for (iv = insn_code_values[i]; iv; iv = iv->next)
+       clear_struct_flag (iv->av->value);
+
+      for (iv = insn_code_values[i]; iv; iv = iv->next)
        {
-       case 'V':
-       case 'E':
-         for (j = 0; j < XVECLEN (x, i); j++)
-           clear_struct_flag (XVECEXP (x, i, j));
-         break;
+         struct obstack *old = rtl_obstack;
+
+         attr = iv->attr;
+         av = iv->av;
+         ie = iv->ie;
+         if (GET_CODE (av->value) != COND)
+           continue;
+
+         rtl_obstack = temp_obstack;
+         newexp = av->value;
+         while (GET_CODE (newexp) == COND)
+           {
+             rtx newexp2 = simplify_cond (newexp, ie->def->insn_code,
+                                          ie->def->insn_index);
+             if (newexp2 == newexp)
+               break;
+             newexp = newexp2;
+           }
 
-       case 'e':
-         clear_struct_flag (XEXP (x, i));
-         break;
+         rtl_obstack = old;
+         if (newexp != av->value)
+           {
+             newexp = attr_copy_rtx (newexp);
+             remove_insn_ent (av, ie);
+             av = get_attr_value (newexp, attr, ie->def->insn_code);
+             iv->av = av;
+             insert_insn_ent (av, ie);
+           }
        }
     }
+
+  free (ivbuf);
+  free (insn_code_values - 2);
+  insn_code_values = NULL;
 }
 
-/* Return the number of RTX objects making up the expression X.
-   But if we count more than MAX objects, stop counting.  */
+/* Clear the ATTR_CURR_SIMPLIFIED_P flag in EXP and its subexpressions.  */
 
-static int
-count_sub_rtxs (x, max)
-     rtx x;
-     int max;
+static void
+clear_struct_flag (rtx x)
 {
   int i;
   int j;
   enum rtx_code code;
   const char *fmt;
-  int total = 0;
+
+  ATTR_CURR_SIMPLIFIED_P (x) = 0;
+  if (ATTR_IND_SIMPLIFIED_P (x))
+    return;
 
   code = GET_CODE (x);
 
   switch (code)
     {
     case REG:
-    case QUEUED:
     case CONST_INT:
     case CONST_DOUBLE:
     case CONST_VECTOR:
@@ -4180,7 +2894,7 @@ count_sub_rtxs (x, max)
     case CC0:
     case EQ_ATTR:
     case ATTR_FLAG:
-      return 1;
+      return;
 
     default:
       break;
@@ -4192,32 +2906,25 @@ count_sub_rtxs (x, max)
   fmt = GET_RTX_FORMAT (code);
   for (i = GET_RTX_LENGTH (code) - 1; i >= 0; i--)
     {
-      if (total >= max)
-       return total;
-
       switch (fmt[i])
        {
        case 'V':
        case 'E':
          for (j = 0; j < XVECLEN (x, i); j++)
-           total += count_sub_rtxs (XVECEXP (x, i, j), max);
+           clear_struct_flag (XVECEXP (x, i, j));
          break;
 
        case 'e':
-         total += count_sub_rtxs (XEXP (x, i), max);
+         clear_struct_flag (XEXP (x, i));
          break;
        }
     }
-  return total;
-
 }
-\f
+
 /* Create table entries for DEFINE_ATTR.  */
 
 static void
-gen_attr (exp, lineno)
-     rtx exp;
-     int lineno;
+gen_attr (rtx exp, int lineno)
 {
   struct attr_desc *attr;
   struct attr_value *av;
@@ -4226,7 +2933,7 @@ gen_attr (exp, lineno)
 
   /* 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);
+  attr = find_attr (&XSTR (exp, 0), 1);
   if (attr->default_val)
     {
       message_with_line (lineno, "duplicate definition for attribute %s",
@@ -4244,7 +2951,7 @@ gen_attr (exp, lineno)
       name_ptr = XSTR (exp, 1);
       while ((p = next_comma_elt (&name_ptr)) != NULL)
        {
-         av = (struct attr_value *) oballoc (sizeof (struct attr_value));
+         av = oballoc (struct attr_value);
          av->value = attr_rtx (CONST_STRING, p);
          av->next = attr->first_value;
          attr->first_value = av;
@@ -4268,7 +2975,7 @@ gen_attr (exp, lineno)
       XEXP (exp, 2) = XEXP (XEXP (exp, 2), 0);
     }
 
-  if (! strcmp (attr->name, "length") && ! attr->is_numeric)
+  if (! strcmp_check (attr->name, length_str) && ! attr->is_numeric)
     {
       message_with_line (lineno,
                         "`length' attribute must take numeric values");
@@ -4279,14 +2986,13 @@ gen_attr (exp, lineno)
   XEXP (exp, 2) = check_attr_value (XEXP (exp, 2), attr);
   attr->default_val = get_attr_value (XEXP (exp, 2), attr, -2);
 }
-\f
+
 /* Given a pattern for DEFINE_PEEPHOLE or DEFINE_INSN, return the number of
    alternatives in the constraints.  Assume all MATCH_OPERANDs have the same
    number of alternatives as this should be checked elsewhere.  */
 
 static int
-count_alternatives (exp)
-     rtx exp;
+count_alternatives (rtx exp)
 {
   int i, j, n;
   const char *fmt;
@@ -4318,13 +3024,12 @@ count_alternatives (exp)
 
   return 0;
 }
-\f
-/* Returns non-zero if the given expression contains an EQ_ATTR with the
+
+/* Returns nonzero if the given expression contains an EQ_ATTR with the
    `alternative' attribute.  */
 
 static int
-compares_alternatives_p (exp)
-     rtx exp;
+compares_alternatives_p (rtx exp)
 {
   int i, j;
   const char *fmt;
@@ -4351,50 +3056,15 @@ compares_alternatives_p (exp)
 
   return 0;
 }
-\f
-/* Returns non-zero is INNER is contained in EXP.  */
-
-static int
-contained_in_p (inner, exp)
-     rtx inner;
-     rtx exp;
-{
-  int i, j;
-  const char *fmt;
-
-  if (rtx_equal_p (inner, exp))
-    return 1;
-
-  for (i = 0, fmt = GET_RTX_FORMAT (GET_CODE (exp));
-       i < GET_RTX_LENGTH (GET_CODE (exp)); i++)
-    switch (*fmt++)
-      {
-      case 'e':
-      case 'u':
-       if (contained_in_p (inner, XEXP (exp, i)))
-         return 1;
-       break;
-
-      case 'E':
-       for (j = 0; j < XVECLEN (exp, i); j++)
-         if (contained_in_p (inner, XVECEXP (exp, i, j)))
-           return 1;
-       break;
-      }
 
-  return 0;
-}
-\f
 /* Process DEFINE_PEEPHOLE, DEFINE_INSN, and DEFINE_ASM_ATTRIBUTES.  */
 
 static void
-gen_insn (exp, lineno)
-     rtx exp;
-     int lineno;
+gen_insn (rtx exp, int lineno)
 {
   struct insn_def *id;
 
-  id = (struct insn_def *) oballoc (sizeof (struct insn_def));
+  id = oballoc (struct insn_def);
   id->next = defs;
   defs = id;
   id->def = exp;
@@ -4429,17 +3099,15 @@ gen_insn (exp, lineno)
       break;
 
     default:
-      abort ();
+      gcc_unreachable ();
     }
 }
-\f
+
 /* Process a DEFINE_DELAY.  Validate the vector length, check if annul
    true or annul false is specified, and make a `struct delay_desc'.  */
 
 static void
-gen_delay (def, lineno)
-     rtx def;
-     int lineno;
+gen_delay (rtx def, int lineno)
 {
   struct delay_desc *delay;
   int i;
@@ -4460,117 +3128,24 @@ gen_delay (def, lineno)
        have_annul_false = 1;
     }
 
-  delay = (struct delay_desc *) oballoc (sizeof (struct delay_desc));
+  delay = oballoc (struct delay_desc);
   delay->def = def;
   delay->num = ++num_delays;
   delay->next = delays;
   delay->lineno = lineno;
   delays = delay;
 }
-\f
-/* 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, lineno)
-     rtx def;
-     int lineno;
-{
-  struct function_unit *unit;
-  struct function_unit_op *op;
-  const char *name = XSTR (def, 0);
-  int multiplicity = XINT (def, 1);
-  int simultaneity = XINT (def, 2);
-  rtx condexp = XEXP (def, 3);
-  int ready_cost = MAX (XINT (def, 4), 1);
-  int issue_delay = MAX (XINT (def, 5), 1);
-
-  /* See if we have already seen this function unit.  If so, check that
-     the multiplicity and simultaneity values are the same.  If not, make
-     a structure for this function unit.  */
-  for (unit = units; unit; unit = unit->next)
-    if (! strcmp (unit->name, name))
-      {
-       if (unit->multiplicity != multiplicity
-           || unit->simultaneity != simultaneity)
-         {
-           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;
-      }
-
-  if (unit == 0)
-    {
-      unit = (struct function_unit *) oballoc (sizeof (struct function_unit));
-      unit->name = name;
-      unit->multiplicity = multiplicity;
-      unit->simultaneity = simultaneity;
-      unit->issue_delay.min = unit->issue_delay.max = issue_delay;
-      unit->num = num_units++;
-      unit->num_opclasses = 0;
-      unit->condexp = false_rtx;
-      unit->ops = 0;
-      unit->next = units;
-      unit->first_lineno = lineno;
-      units = unit;
-    }
-
-  /* Make a new operation class structure entry and initialize it.  */
-  op = (struct function_unit_op *) oballoc (sizeof (struct function_unit_op));
-  op->condexp = condexp;
-  op->num = unit->num_opclasses++;
-  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.  */
-  if (XVEC (def, 6))
-    {
-      /* Compute the IOR of all the specified expressions.  */
-      rtx orexp = false_rtx;
-      int i;
-
-      for (i = 0; i < XVECLEN (def, 6); i++)
-       orexp = insert_right_side (IOR, orexp, XVECEXP (def, 6, i), -2, -2);
-
-      op->conflict_exp = orexp;
-      extend_range (&unit->issue_delay, 1, issue_delay);
-    }
-  else
-    {
-      op->conflict_exp = true_rtx;
-      extend_range (&unit->issue_delay, issue_delay, issue_delay);
-    }
-
-  /* Merge our conditional into that of the function unit so we can determine
-     which insns are used by the function unit.  */
-  unit->condexp = insert_right_side (IOR, unit->condexp, op->condexp, -2, -2);
-}
-\f
 /* 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 first bit of FLAGS will be non-zero in that case.
+   The first bit of FLAGS will be nonzero 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, flags)
-     rtx exp;
-     int flags;
+write_test_expr (rtx exp, int flags)
 {
   int comparison_operator = 0;
   RTX_CODE code;
@@ -4584,9 +3159,14 @@ write_test_expr (exp, flags)
   switch (code)
     {
     /* Binary operators.  */
+    case GEU: case GTU:
+    case LEU: case LTU:
+      printf ("(unsigned) ");
+      /* Fall through.  */
+
     case EQ: case NE:
-    case GE: case GT: case GEU: case GTU:
-    case LE: case LT: case LEU: case LTU:
+    case GE: case GT:
+    case LE: case LT:
       comparison_operator = 1;
 
     case PLUS:   case MINUS:  case MULT:     case DIV:      case MOD:
@@ -4663,7 +3243,7 @@ write_test_expr (exp, flags)
          printf (" >> ");
          break;
        default:
-         abort ();
+         gcc_unreachable ();
        }
 
       write_test_expr (XEXP (exp, 1), flags | comparison_operator);
@@ -4697,12 +3277,58 @@ write_test_expr (exp, flags)
          printf ("-");
          break;
        default:
-         abort ();
+         gcc_unreachable ();
        }
 
       write_test_expr (XEXP (exp, 0), flags);
       break;
 
+    case EQ_ATTR_ALT:
+       {
+         int set = XINT (exp, 0), bit = 0;
+
+         if (flags & 1)
+           fatal ("EQ_ATTR_ALT not valid inside comparison");
+
+         if (!set)
+           fatal ("Empty EQ_ATTR_ALT should be optimized out");
+
+         if (!(set & (set - 1)))
+           {
+             if (!(set & 0xffff))
+               {
+                 bit += 16;
+                 set >>= 16;
+               }
+             if (!(set & 0xff))
+               {
+                 bit += 8;
+                 set >>= 8;
+               }
+             if (!(set & 0xf))
+               {
+                 bit += 4;
+                 set >>= 4;
+               }
+             if (!(set & 0x3))
+               {
+                 bit += 2;
+                 set >>= 2;
+               }
+             if (!(set & 1))
+               bit++;
+
+             printf ("which_alternative %s= %d",
+                     XINT (exp, 1) ? "!" : "=", bit);
+           }
+         else
+           {
+             printf ("%s((1 << which_alternative) & 0x%x)",
+                     XINT (exp, 1) ? "!" : "", set);
+           }
+       }
+      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.  */
@@ -4716,9 +3342,8 @@ write_test_expr (exp, flags)
          break;
        }
 
-      attr = find_attr (XSTR (exp, 0), 0);
-      if (! attr)
-       abort ();
+      attr = find_attr (&XSTR (exp, 0), 0);
+      gcc_assert (attr);
 
       /* Now is the time to expand the value of a constant attribute.  */
       if (attr->is_const)
@@ -4762,10 +3387,6 @@ write_test_expr (exp, flags)
                XSTR (exp, 1), XINT (exp, 0), GET_MODE_NAME (GET_MODE (exp)));
       break;
 
-    case MATCH_INSN:
-      printf ("%s (insn)", XSTR (exp, 0));
-      break;
-
     /* Constant integer.  */
     case CONST_INT:
       printf (HOST_WIDE_INT_PRINT_DEC, XWINT (exp, 0));
@@ -4773,7 +3394,7 @@ write_test_expr (exp, flags)
 
     /* A random C expression.  */
     case SYMBOL_REF:
-      printf ("%s", XSTR (exp, 0));
+      print_c_condition (XSTR (exp, 0));
       break;
 
     /* The address of the branch target.  */
@@ -4810,14 +3431,12 @@ write_test_expr (exp, flags)
 
   printf (")");
 }
-\f
+
 /* Given an attribute value, return the maximum CONST_STRING argument
    encountered.  Set *UNKNOWNP and return INT_MAX if the value is unknown.  */
 
 static int
-max_attr_value (exp, unknownp)
-     rtx exp;
-     int *unknownp;
+max_attr_value (rtx exp, int *unknownp)
 {
   int current_max;
   int i, n;
@@ -4854,14 +3473,53 @@ max_attr_value (exp, unknownp)
   return current_max;
 }
 
+/* Given an attribute value, return the minimum CONST_STRING argument
+   encountered.  Set *UNKNOWNP and return 0 if the value is unknown.  */
+
+static int
+min_attr_value (rtx exp, int *unknownp)
+{
+  int current_min;
+  int i, n;
+
+  switch (GET_CODE (exp))
+    {
+    case CONST_STRING:
+      current_min = atoi (XSTR (exp, 0));
+      break;
+
+    case COND:
+      current_min = min_attr_value (XEXP (exp, 1), unknownp);
+      for (i = 0; i < XVECLEN (exp, 0); i += 2)
+       {
+         n = min_attr_value (XVECEXP (exp, 0, i + 1), unknownp);
+         if (n < current_min)
+           current_min = n;
+       }
+      break;
+
+    case IF_THEN_ELSE:
+      current_min = min_attr_value (XEXP (exp, 1), unknownp);
+      n = min_attr_value (XEXP (exp, 2), unknownp);
+      if (n < current_min)
+       current_min = n;
+      break;
+
+    default:
+      *unknownp = 1;
+      current_min = INT_MAX;
+      break;
+    }
+
+  return current_min;
+}
+
 /* 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;
+or_attr_value (rtx exp, int *unknownp)
 {
   int current_or;
   int i;
@@ -4891,7 +3549,7 @@ or_attr_value (exp, unknownp)
 
   return current_or;
 }
-\f
+
 /* Scan an attribute value, possibly a conditional, and record what actions
    will be required to do any conditional tests in it.
 
@@ -4903,8 +3561,7 @@ or_attr_value (exp, unknownp)
  */
 
 static void
-walk_attr_value (exp)
-     rtx exp;
+walk_attr_value (rtx exp)
 {
   int i, j;
   const char *fmt;
@@ -4928,10 +3585,14 @@ walk_attr_value (exp)
       must_extract = 1;
       return;
 
+    case EQ_ATTR_ALT:
+      must_extract = must_constrain = 1;
+      break;
+
     case EQ_ATTR:
       if (XSTR (exp, 0) == alternative_name)
        must_extract = must_constrain = 1;
-      else if (strcmp (XSTR (exp, 0), "length") == 0)
+      else if (strcmp_check (XSTR (exp, 0), length_str) == 0)
        length_used = 1;
       return;
 
@@ -4966,12 +3627,11 @@ walk_attr_value (exp)
        break;
       }
 }
-\f
+
 /* Write out a function to obtain the attribute for a given INSN.  */
 
 static void
-write_attr_get (attr)
-     struct attr_desc *attr;
+write_attr_get (struct attr_desc *attr)
 {
   struct attr_value *av, *common_av;
 
@@ -4979,96 +3639,56 @@ write_attr_get (attr)
      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)
     printf ("enum attr_%s\n", attr->name);
-  else if (attr->unsigned_p)
-    printf ("unsigned int\n");
   else
     printf ("int\n");
 
   /* 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 (insn)\n", &attr->name[1]);
+    printf ("%s (rtx insn ATTRIBUTE_UNUSED)\n", &attr->name[1]);
   else if (attr->is_const == 0)
-    printf ("get_attr_%s (insn)\n", attr->name);
+    printf ("get_attr_%s (rtx insn ATTRIBUTE_UNUSED)\n", attr->name);
   else
     {
-      printf ("get_attr_%s ()\n", attr->name);
+      printf ("get_attr_%s (void)\n", attr->name);
       printf ("{\n");
 
       for (av = attr->first_value; av; av = av->next)
-       if (av->num_insns != 0)
+       if (av->num_insns == 1)
+         write_attr_set (attr, 2, av->value, "return", ";",
+                         true_rtx, av->first_insn->def->insn_code,
+                         av->first_insn->def->insn_index);
+       else if (av->num_insns != 0)
          write_attr_set (attr, 2, av->value, "return", ";",
-                         true_rtx, av->first_insn->insn_code,
-                         av->first_insn->insn_index);
+                         true_rtx, -2, 0);
 
       printf ("}\n\n");
       return;
     }
 
-  printf ("     rtx insn;\n");
   printf ("{\n");
+  printf ("  switch (recog_memoized (insn))\n");
+  printf ("    {\n");
 
-  if (GET_CODE (common_av->value) == FFS)
-    {
-      rtx p = XEXP (common_av->value, 0);
-
-      /* 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);
+  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");
-    }
+  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
    that as the condition or are in an `else' clause) and an expression,
    replace any known true terms with TRUE.  Use `simplify_and_tree' to do
    the bulk of the work.  */
 
 static rtx
-eliminate_known_true (known_true, exp, insn_code, insn_index)
-     rtx known_true;
-     rtx exp;
-     int insn_code, insn_index;
+eliminate_known_true (rtx known_true, rtx exp, int insn_code, int insn_index)
 {
   rtx term;
 
@@ -5089,22 +3709,16 @@ eliminate_known_true (known_true, exp, insn_code, insn_index)
 
   return exp;
 }
-\f
+
 /* Write out a series of tests and assignment statements to perform tests and
    sets of an attribute value.  We are passed an indentation amount and prefix
    and suffix strings to write around each attribute value (e.g., "return"
    and ";").  */
 
 static void
-write_attr_set (attr, indent, value, prefix, suffix, known_true,
-               insn_code, insn_index)
-     struct attr_desc *attr;
-     int indent;
-     rtx value;
-     const char *prefix;
-     const char *suffix;
-     rtx known_true;
-     int insn_code, insn_index;
+write_attr_set (struct attr_desc *attr, int indent, rtx value,
+               const char *prefix, const char *suffix, rtx known_true,
+               int insn_code, int insn_index)
 {
   if (GET_CODE (value) == COND)
     {
@@ -5187,21 +3801,33 @@ write_attr_set (attr, indent, value, prefix, suffix, known_true,
       printf ("%s\n", suffix);
     }
 }
-\f
-/* Write out the computation for one attribute value.  */
+
+/* Write a series of case statements for every instruction in list IE.
+   INDENT is the amount of indentation to write before each case.  */
 
 static void
-write_attr_case (attr, av, write_case_lines, prefix, suffix, indent,
-                known_true)
-     struct attr_desc *attr;
-     struct attr_value *av;
-     int write_case_lines;
-     const char *prefix, *suffix;
-     int indent;
-     rtx known_true;
+write_insn_cases (struct insn_ent *ie, int indent)
 {
-  struct insn_ent *ie;
+  for (; ie != 0; ie = ie->next)
+    if (ie->def->insn_code != -1)
+      {
+       write_indent (indent);
+       if (GET_CODE (ie->def->def) == DEFINE_PEEPHOLE)
+         printf ("case %d:  /* define_peephole, line %d */\n",
+                 ie->def->insn_code, ie->def->lineno);
+       else
+         printf ("case %d:  /* %s */\n",
+                 ie->def->insn_code, XSTR (ie->def->def, 0));
+      }
+}
+
+/* Write out the computation for one attribute value.  */
 
+static void
+write_attr_case (struct attr_desc *attr, struct attr_value *av,
+                int write_case_lines, const char *prefix, const char *suffix,
+                int indent, rtx known_true)
+{
   if (av->num_insns == 0)
     return;
 
@@ -5215,158 +3841,51 @@ write_attr_case (attr, av, write_case_lines, prefix, suffix, indent,
       printf ("    && asm_noperands (PATTERN (insn)) < 0)\n");
       write_indent (indent + 2);
       printf ("  fatal_insn_not_found (insn);\n");
-    }
-
-  if (write_case_lines)
-    {
-      for (ie = av->first_insn; ie; ie = ie->next)
-       if (ie->insn_code != -1)
-         {
-           write_indent (indent);
-           printf ("case %d:\n", ie->insn_code);
-         }
-    }
-  else
-    {
-      write_indent (indent);
-      printf ("default:\n");
-    }
-
-  /* See what we have to do to output this value.  */
-  must_extract = must_constrain = address_used = 0;
-  walk_attr_value (av->value);
-
-  if (must_constrain)
-    {
-      write_indent (indent + 2);
-      printf ("extract_constrain_insn_cached (insn);\n");
-    }
-  else if (must_extract)
-    {
-      write_indent (indent + 2);
-      printf ("extract_insn_cached (insn);\n");
-    }
-
-  write_attr_set (attr, indent + 2, av->value, prefix, suffix,
-                 known_true, av->first_insn->insn_code,
-                 av->first_insn->insn_index);
-
-  if (strncmp (prefix, "return", 6))
-    {
-      write_indent (indent + 2);
-      printf ("break;\n");
-    }
-  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 ("  enum attr_%s ", attr->name);
-      else if (attr->unsigned_p)
-       printf ("  unsigned int ");
-      else
-       printf ("  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.  */
+  if (write_case_lines)
+    write_insn_cases (av->first_insn, indent);
+  else
+    {
+      write_indent (indent);
+      printf ("default:\n");
+    }
 
-static void
-write_toplevel_expr (p)
-     rtx p;
-{
-  struct attr_desc *attr;
-  int i;
+  /* See what we have to do to output this value.  */
+  must_extract = must_constrain = address_used = 0;
+  walk_attr_value (av->value);
 
-  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);
+  if (must_constrain)
+    {
+      write_indent (indent + 2);
+      printf ("extract_constrain_insn_cached (insn);\n");
+    }
+  else if (must_extract)
+    {
+      write_indent (indent + 2);
+      printf ("extract_insn_cached (insn);\n");
+    }
 
-  printf ("  unsigned long accum = 0;\n\n");
+  if (av->num_insns == 1)
+    write_attr_set (attr, indent + 2, av->value, prefix, suffix,
+                   known_true, av->first_insn->def->insn_code,
+                   av->first_insn->def->insn_index);
+  else
+    write_attr_set (attr, indent + 2, av->value, prefix, suffix,
+                   known_true, -2, 0);
 
-  while (GET_CODE (p) == IOR)
+  if (strncmp (prefix, "return", 6))
     {
-      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");
+      write_indent (indent + 2);
+      printf ("break;\n");
     }
-  printf ("  accum |= ");
-  write_test_expr (p, 3);
-  printf (";\n");
+  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);
-}
+/* Utilities to write in various forms.  */
 
 static void
-write_attr_valueq (attr, s)
-     struct attr_desc *attr;
-     const char *s;
+write_attr_valueq (struct attr_desc *attr, const char *s)
 {
   if (attr->is_numeric)
     {
@@ -5374,32 +3893,7 @@ write_attr_valueq (attr, 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)
+      if (num > 9 || num < 0)
        printf (" /* 0x%x */", num);
     }
   else
@@ -5411,9 +3905,7 @@ write_attr_valueq (attr, s)
 }
 
 static void
-write_attr_value (attr, value)
-     struct attr_desc *attr;
-     rtx value;
+write_attr_value (struct attr_desc *attr, rtx value)
 {
   int op;
 
@@ -5428,12 +3920,12 @@ write_attr_value (attr, value)
       break;
 
     case SYMBOL_REF:
-      fputs (XSTR (value, 0), stdout);
+      print_c_condition (XSTR (value, 0));
       break;
 
     case ATTR:
       {
-       struct attr_desc *attr2 = find_attr (XSTR (value, 0), 0);
+       struct attr_desc *attr2 = find_attr (&XSTR (value, 0), 0);
        printf ("get_attr_%s (%s)", attr2->name,
                (attr2->is_const ? "" : "insn"));
       }
@@ -5464,13 +3956,12 @@ write_attr_value (attr, value)
       break;
 
     default:
-      abort ();
+      gcc_unreachable ();
     }
 }
 
 static void
-write_upcase (str)
-     const char *str;
+write_upcase (const char *str)
 {
   while (*str)
     {
@@ -5481,8 +3972,7 @@ write_upcase (str)
 }
 
 static void
-write_indent (indent)
-     int indent;
+write_indent (int indent)
 {
   for (; indent > 8; indent -= 8)
     printf ("\t");
@@ -5490,9 +3980,9 @@ write_indent (indent)
   for (; indent; indent--)
     printf (" ");
 }
-\f
+
 /* Write a subroutine that is given an insn that requires a delay slot, a
-   delay slot ordinal, and a candidate insn.  It returns non-zero if the
+   delay slot ordinal, and a candidate insn.  It returns nonzero if the
    candidate can be placed in the specified delay slot of the insn.
 
    We can write as many as three subroutines.  `eligible_for_delay'
@@ -5504,12 +3994,12 @@ write_indent (indent)
    or "annul_false").  */
 
 static void
-write_eligible_delay (kind)
-     const char *kind;
+write_eligible_delay (const char *kind)
 {
   struct delay_desc *delay;
   int max_slots;
   char str[50];
+  const char *pstr;
   struct attr_desc *attr;
   struct attr_value *av, *common_av;
   int i;
@@ -5525,26 +4015,25 @@ write_eligible_delay (kind)
   /* Write function prelude.  */
 
   printf ("int\n");
-  printf ("eligible_for_%s (delay_insn, slot, candidate_insn, flags)\n",
+  printf ("eligible_for_%s (rtx delay_insn ATTRIBUTE_UNUSED, int slot, rtx candidate_insn, int flags ATTRIBUTE_UNUSED)\n",
          kind);
-  printf ("     rtx delay_insn ATTRIBUTE_UNUSED;\n");
-  printf ("     int slot;\n");
-  printf ("     rtx candidate_insn;\n");
-  printf ("     int flags ATTRIBUTE_UNUSED;\n");
   printf ("{\n");
   printf ("  rtx insn;\n");
   printf ("\n");
-  printf ("  if (slot >= %d)\n", max_slots);
-  printf ("    abort ();\n");
+  printf ("  gcc_assert (slot < %d);\n", max_slots);
+  printf ("\n");
+  /* Allow dbr_schedule to pass labels, etc.  This can happen if try_split
+     converts a compound instruction into a loop.  */
+  printf ("  if (!INSN_P (candidate_insn))\n");
+  printf ("    return 0;\n");
   printf ("\n");
 
   /* If more than one delay type, find out which type the delay insn is.  */
 
   if (num_delays > 1)
     {
-      attr = find_attr ("*delay_type", 0);
-      if (! attr)
-       abort ();
+      attr = find_attr (&delay_type_str, 0);
+      gcc_assert (attr);
       common_av = find_most_used (attr);
 
       printf ("  insn = delay_insn;\n");
@@ -5560,8 +4049,7 @@ write_eligible_delay (kind)
       printf ("    }\n\n");
 
       /* Ensure matched.  Otherwise, shouldn't have been called.  */
-      printf ("  if (slot < %d)\n", max_slots);
-      printf ("    abort ();\n\n");
+      printf ("  gcc_assert (slot >= %d);\n\n", max_slots);
     }
 
   /* If just one type of delay slot, write simple switch.  */
@@ -5571,9 +4059,8 @@ write_eligible_delay (kind)
       printf ("  switch (recog_memoized (insn))\n");
       printf ("    {\n");
 
-      attr = find_attr ("*delay_1_0", 0);
-      if (! attr)
-       abort ();
+      attr = find_attr (&delay_1_0_str, 0);
+      gcc_assert (attr);
       common_av = find_most_used (attr);
 
       for (av = attr->first_value; av; av = av->next)
@@ -5601,9 +4088,9 @@ write_eligible_delay (kind)
            printf ("\t{\n");
 
            sprintf (str, "*%s_%d_%d", kind, delay->num, i / 3);
-           attr = find_attr (str, 0);
-           if (! attr)
-             abort ();
+           pstr = str;
+           attr = find_attr (&pstr, 0);
+           gcc_assert (attr);
            common_av = find_most_used (attr);
 
            for (av = attr->first_value; av; av = av->next)
@@ -5615,181 +4102,13 @@ write_eligible_delay (kind)
          }
 
       printf ("    default:\n");
-      printf ("      abort ();\n");
+      printf ("      gcc_unreachable ();\n");
       printf ("    }\n");
     }
 
   printf ("}\n\n");
 }
-\f
-/* Write routines to compute conflict cost for function units.  Then write a
-   table describing the available function units.  */
-
-static void
-write_function_unit_info ()
-{
-  struct function_unit *unit;
-  int i;
-
-  /* Write out conflict routines for function units.  Don't bother writing
-     one if there is only one issue delay value.  */
-
-  for (unit = units; unit; unit = unit->next)
-    {
-      if (unit->needs_blockage_function)
-       write_complex_function (unit, "blockage", "block");
-
-      /* If the minimum and maximum conflict costs are the same, there
-        is only one value, so we don't need a function.  */
-      if (! unit->needs_conflict_function)
-       {
-         unit->default_cost = make_numeric_value (unit->issue_delay.max);
-         continue;
-       }
-
-      /* The function first computes the case from the candidate insn.  */
-      unit->default_cost = make_numeric_value (0);
-      write_complex_function (unit, "conflict_cost", "cost");
-    }
-
-  /* Now that all functions have been written, write the table describing
-     the function units.   The name is included for documentation purposes
-     only.  */
-
-  printf ("const struct function_unit_desc function_units[] = {\n");
-
-  /* Write out the descriptions in numeric order, but don't force that order
-     on the list.  Doing so increases the runtime of genattrtab.c.  */
-  for (i = 0; i < num_units; i++)
-    {
-      for (unit = units; unit; unit = unit->next)
-       if (unit->num == i)
-         break;
-
-      printf ("  {\"%s\", %d, %d, %d, %s, %d, %s_unit_ready_cost, ",
-             unit->name, 1 << unit->num, unit->multiplicity,
-             unit->simultaneity, XSTR (unit->default_cost, 0),
-             unit->issue_delay.max, unit->name);
-
-      if (unit->needs_conflict_function)
-       printf ("%s_unit_conflict_cost, ", unit->name);
-      else
-       printf ("0, ");
-
-      printf ("%d, ", unit->max_blockage);
-
-      if (unit->needs_range_function)
-       printf ("%s_unit_blockage_range, ", unit->name);
-      else
-       printf ("0, ");
-
-      if (unit->needs_blockage_function)
-       printf ("%s_unit_blockage", unit->name);
-      else
-       printf ("0");
-
-      printf ("}, \n");
-    }
-
-  if (num_units == 0)
-    printf ("{\"dummy\", 0, 0, 0, 0, 0, 0, 0, 0, 0, 0} /* a dummy element */");
-  printf ("};\n\n");
-}
-
-static void
-write_complex_function (unit, name, connection)
-     struct function_unit *unit;
-     const char *name, *connection;
-{
-  struct attr_desc *case_attr, *attr;
-  struct attr_value *av, *common_av;
-  rtx value;
-  char str[256];
-  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 ("     rtx executing_insn;\n");
-  printf ("     rtx candidate_insn;\n");
-  printf ("{\n");
-  printf ("  rtx insn;\n");
-  printf ("  int casenum;\n\n");
-  printf ("  insn = executing_insn;\n");
-  printf ("  switch (recog_memoized (insn))\n");
-  printf ("    {\n");
-
-  /* Write the `switch' statement to get the case value.  */
-  if (strlen (unit->name) + sizeof "*_cases" > 256)
-    abort ();
-  sprintf (str, "*%s_cases", unit->name);
-  case_attr = find_attr (str, 0);
-  if (! case_attr)
-    abort ();
-  common_av = find_most_used (case_attr);
-
-  for (av = case_attr->first_value; av; av = av->next)
-    if (av != common_av)
-      write_attr_case (case_attr, av, 1,
-                      "casenum =", ";", 4, unit->condexp);
-
-  write_attr_case (case_attr, common_av, 0,
-                  "casenum =", ";", 4, unit->condexp);
-  printf ("    }\n\n");
-
-  /* Now write an outer switch statement on each case.  Then write
-     the tests on the executing function within each.  */
-  printf ("  insn = candidate_insn;\n");
-  printf ("  switch (casenum)\n");
-  printf ("    {\n");
-
-  for (i = 0; i < unit->num_opclasses; i++)
-    {
-      /* Ensure using this case.  */
-      using_case = 0;
-      for (av = case_attr->first_value; av; av = av->next)
-       if (av->num_insns
-           && contained_in_p (make_numeric_value (i), av->value))
-         using_case = 1;
-
-      if (! using_case)
-       continue;
-
-      printf ("    case %d:\n", i);
-      sprintf (str, "*%s_%s_%d", unit->name, connection, i);
-      attr = find_attr (str, 0);
-      if (! attr)
-       abort ();
 
-      /* If single value, just write it.  */
-      value = find_single_value (attr);
-      if (value)
-       write_attr_set (attr, 6, value, "return", ";\n", true_rtx, -2, -2);
-      else
-       {
-         common_av = find_most_used (attr);
-         printf ("      switch (recog_memoized (insn))\n");
-         printf ("\t{\n");
-
-         for (av = attr->first_value; av; av = av->next)
-           if (av != common_av)
-             write_attr_case (attr, av, 1,
-                              "return", ";", 8, unit->condexp);
-
-         write_attr_case (attr, common_av, 0,
-                          "return", ";", 8, unit->condexp);
-         printf ("      }\n\n");
-       }
-    }
-
-  /* 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
 /* This page contains miscellaneous utility routines.  */
 
 /* Given a pointer to a (char *), return a malloc'ed string containing the
@@ -5797,8 +4116,7 @@ write_complex_function (unit, name, connection)
    scanned, or the end-of-string.  Return NULL if at end of string.  */
 
 static char *
-next_comma_elt (pstr)
-     const char **pstr;
+next_comma_elt (const char **pstr)
 {
   const char *start;
 
@@ -5811,15 +4129,15 @@ next_comma_elt (pstr)
 }
 
 /* Return a `struct attr_desc' pointer for a given named attribute.  If CREATE
-   is non-zero, build a new attribute, if one does not exist.  */
+   is nonzero, build a new attribute, if one does not exist.  *NAME_P is
+   replaced by a pointer to a canonical copy of the string.  */
 
 static struct attr_desc *
-find_attr (name, create)
-     const char *name;
-     int create;
+find_attr (const char **name_p, int create)
 {
   struct attr_desc *attr;
   int index;
+  const char *name = *name_p;
 
   /* Before we resort to using `strcmp', see if the string address matches
      anywhere.  In most cases, it should have been canonicalized to do so.  */
@@ -5834,51 +4152,46 @@ find_attr (name, create)
   /* Otherwise, do it the slow way.  */
   for (attr = attrs[index]; attr; attr = attr->next)
     if (name[0] == attr->name[0] && ! strcmp (name, attr->name))
-      return attr;
+      {
+       *name_p = attr->name;
+       return attr;
+      }
 
   if (! create)
     return NULL;
 
-  attr = (struct attr_desc *) oballoc (sizeof (struct attr_desc));
-  attr->name = attr_string (name, strlen (name));
+  attr = oballoc (struct attr_desc);
+  attr->name = DEF_ATTR_STRING (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->is_numeric = attr->is_const = attr->is_special = 0;
   attr->next = attrs[index];
   attrs[index] = attr;
 
+  *name_p = attr->name;
+
   return attr;
 }
 
 /* Create internal attribute with the given default value.  */
 
-void
-make_internal_attr (name, value, special)
-     const char *name;
-     rtx value;
-     int special;
+static void
+make_internal_attr (const char *name, rtx value, int special)
 {
   struct attr_desc *attr;
 
-  attr = find_attr (name, 1);
-  if (attr->default_val)
-    abort ();
+  attr = find_attr (&name, 1);
+  gcc_assert (!attr->default_val);
 
   attr->is_numeric = 1;
   attr->is_const = 0;
-  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->is_special = (special & ATTR_SPECIAL) != 0;
   attr->default_val = get_attr_value (value, attr, -2);
 }
 
 /* Find the most used value of an attribute.  */
 
 static struct attr_value *
-find_most_used (attr)
-     struct attr_desc *attr;
+find_most_used (struct attr_desc *attr)
 {
   struct attr_value *av;
   struct attr_value *most_used;
@@ -5894,41 +4207,16 @@ find_most_used (attr)
   return most_used;
 }
 
-/* If an attribute only has a single value used, return it.  Otherwise
-   return NULL.  */
-
-static rtx
-find_single_value (attr)
-     struct attr_desc *attr;
-{
-  struct attr_value *av;
-  rtx unique_value;
-
-  unique_value = NULL;
-  for (av = attr->first_value; av; av = av->next)
-    if (av->num_insns)
-      {
-       if (unique_value)
-         return NULL;
-       else
-         unique_value = av->value;
-      }
-
-  return unique_value;
-}
-
 /* Return (attr_value "n") */
 
-rtx
-make_numeric_value (n)
-     int n;
+static rtx
+make_numeric_value (int n)
 {
   static rtx int_values[20];
   rtx exp;
   char *p;
 
-  if (n < 0)
-    abort ();
+  gcc_assert (n >= 0);
 
   if (n < 20 && int_values[n])
     return int_values[n];
@@ -5941,72 +4229,29 @@ make_numeric_value (n)
 
   return exp;
 }
-\f
-static void
-extend_range (range, min, max)
-     struct range *range;
-     int min;
-     int max;
-{
-  if (range->min > min)
-    range->min = min;
-  if (range->max < max)
-    range->max = max;
-}
 
 static rtx
-copy_rtx_unchanging (orig)
-     rtx orig;
+copy_rtx_unchanging (rtx orig)
 {
-#if 0
-  rtx copy;
-  RTX_CODE code;
-#endif
-
   if (ATTR_IND_SIMPLIFIED_P (orig) || ATTR_CURR_SIMPLIFIED_P (orig))
     return orig;
 
   ATTR_CURR_SIMPLIFIED_P (orig) = 1;
   return orig;
-
-#if 0
-  code = GET_CODE (orig);
-  switch (code)
-    {
-    case CONST_INT:
-    case CONST_DOUBLE:
-    case SYMBOL_REF:
-    case CODE_LABEL:
-      return orig;
-
-    default:
-      break;
-    }
-
-  copy = rtx_alloc (code);
-  PUT_MODE (copy, GET_MODE (orig));
-  ATTR_IND_SIMPLIFIED_P (copy) = 1;
-
-  memcpy (&XEXP (copy, 0), &XEXP (orig, 0),
-         GET_RTX_LENGTH (GET_CODE (copy)) * sizeof (rtx));
-  return copy;
-#endif
 }
 
 /* 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.  */
 
 static void
-write_const_num_delay_slots ()
+write_const_num_delay_slots (void)
 {
-  struct attr_desc *attr = find_attr ("*num_delay_slots", 0);
+  struct attr_desc *attr = find_attr (&num_delay_slots_str, 0);
   struct attr_value *av;
-  struct insn_ent *ie;
 
   if (attr)
     {
-      printf ("int\nconst_num_delay_slots (insn)\n");
-      printf ("     rtx insn;\n");
+      printf ("int\nconst_num_delay_slots (rtx insn)\n");
       printf ("{\n");
       printf ("  switch (recog_memoized (insn))\n");
       printf ("    {\n");
@@ -6016,12 +4261,7 @@ write_const_num_delay_slots ()
          length_used = 0;
          walk_attr_value (av->value);
          if (length_used)
-           {
-             for (ie = av->first_insn; ie; ie = ie->next)
-               if (ie->insn_code != -1)
-                 printf ("    case %d:\n", ie->insn_code);
-             printf ("      return 0;\n");
-           }
+           write_insn_cases (av->first_insn, 4);
        }
 
       printf ("    default:\n");
@@ -6030,12 +4270,173 @@ write_const_num_delay_slots ()
     }
 }
 \f
-extern int main PARAMS ((int, char **));
+/* Synthetic attributes used by insn-automata.c and the scheduler.
+   These are primarily concerned with (define_insn_reservation)
+   patterns.  */
+
+struct insn_reserv
+{
+  struct insn_reserv *next;
+
+  const char *name;
+  int default_latency;
+  rtx condexp;
+
+  /* Sequence number of this insn.  */
+  int insn_num;
+
+  /* Whether a (define_bypass) construct names this insn in its
+     output list.  */
+  bool bypassed;
+};
+
+static struct insn_reserv *all_insn_reservs = 0;
+static struct insn_reserv **last_insn_reserv_p = &all_insn_reservs;
+static size_t n_insn_reservs;
+
+/* Store information from a DEFINE_INSN_RESERVATION for future
+   attribute generation.  */
+static void
+gen_insn_reserv (rtx def)
+{
+  struct insn_reserv *decl = oballoc (struct insn_reserv);
+
+  decl->name            = DEF_ATTR_STRING (XSTR (def, 0));
+  decl->default_latency = XINT (def, 1);
+  decl->condexp         = check_attr_test (XEXP (def, 2), 0, 0);
+  decl->insn_num        = n_insn_reservs;
+  decl->bypassed       = false;
+  decl->next            = 0;
+  
+  *last_insn_reserv_p = decl;
+  last_insn_reserv_p  = &decl->next;
+  n_insn_reservs++;
+}
+
+/* Store information from a DEFINE_BYPASS for future attribute
+   generation.  The only thing we care about is the list of output
+   insns, which will later be used to tag reservation structures with
+   a 'bypassed' bit.  */
+
+struct bypass_list
+{
+  struct bypass_list *next;
+  const char *insn;
+};
+
+static struct bypass_list *all_bypasses;
+static size_t n_bypasses;
+
+static void
+gen_bypass_1 (const char *s, size_t len)
+{
+  struct bypass_list *b;
+
+  if (len == 0)
+    return;
+
+  s = attr_string (s, len);
+  for (b = all_bypasses; b; b = b->next)
+    if (s == b->insn)
+      return;  /* already got that one */
+
+  b = oballoc (struct bypass_list);
+  b->insn = s;
+  b->next = all_bypasses;
+  all_bypasses = b;
+  n_bypasses++;
+}
+
+static void
+gen_bypass (rtx def)
+{
+  const char *p, *base;
+
+  for (p = base = XSTR (def, 1); *p; p++)
+    if (*p == ',')
+      {
+       gen_bypass_1 (base, p - base);
+       do
+         p++;
+       while (ISSPACE (*p));
+       base = p;
+      }
+  gen_bypass_1 (base, p - base);
+}
+
+/* Find and mark all of the bypassed insns.  */
+static void
+process_bypasses (void)
+{
+  struct bypass_list *b;
+  struct insn_reserv *r;
+
+  /* The reservation list is likely to be much longer than the bypass
+     list.  */
+  for (r = all_insn_reservs; r; r = r->next)
+    for (b = all_bypasses; b; b = b->next)
+      if (r->name == b->insn)
+       r->bypassed = true;
+}
+
+/* Create all of the attributes that describe automaton properties.  */
+static void
+make_automaton_attrs (void)
+{
+  int i;
+  struct insn_reserv *decl;
+  rtx code_exp, lats_exp, byps_exp;
+
+  if (n_insn_reservs == 0)
+    return;
+
+  code_exp = rtx_alloc (COND);
+  lats_exp = rtx_alloc (COND);
+  
+  XVEC (code_exp, 0) = rtvec_alloc (n_insn_reservs * 2);
+  XVEC (lats_exp, 0) = rtvec_alloc (n_insn_reservs * 2);
+
+  XEXP (code_exp, 1) = make_numeric_value (n_insn_reservs + 1);
+  XEXP (lats_exp, 1) = make_numeric_value (0);
+
+  for (decl = all_insn_reservs, i = 0;
+       decl;
+       decl = decl->next, i += 2)
+    {
+      XVECEXP (code_exp, 0, i)   = decl->condexp;
+      XVECEXP (lats_exp, 0, i)   = decl->condexp;
+      
+      XVECEXP (code_exp, 0, i+1) = make_numeric_value (decl->insn_num);
+      XVECEXP (lats_exp, 0, i+1) = make_numeric_value (decl->default_latency);
+    }
+
+  if (n_bypasses == 0)
+    byps_exp = make_numeric_value (0);
+  else
+    {
+      process_bypasses ();
+
+      byps_exp = rtx_alloc (COND);
+      XVEC (byps_exp, 0) = rtvec_alloc (n_bypasses * 2);
+      XEXP (byps_exp, 1) = make_numeric_value (0);
+      for (decl = all_insn_reservs, i = 0;
+          decl;
+          decl = decl->next)
+       if (decl->bypassed)
+         {
+           XVECEXP (byps_exp, 0, i)   = decl->condexp;
+           XVECEXP (byps_exp, 0, i+1) = make_numeric_value (1);
+           i += 2;
+         }
+    }
+
+  make_internal_attr ("*internal_dfa_insn_code", code_exp, ATTR_NONE);
+  make_internal_attr ("*insn_default_latency",   lats_exp, ATTR_NONE);
+  make_internal_attr ("*bypass_p",               byps_exp, ATTR_NONE);
+}
 
 int
-main (argc, argv)
-     int argc;
-     char **argv;
+main (int argc, char **argv)
 {
   rtx desc;
   struct attr_desc *attr;
@@ -6045,9 +4446,6 @@ main (argc, argv)
 
   progname = "genattrtab";
 
-  if (argc <= 1)
-    fatal ("no input file name");
-
   if (init_md_reader_args (argc, argv) != SUCCESS_EXIT_CODE)
     return (FATAL_EXIT_CODE);
 
@@ -6062,14 +4460,17 @@ main (argc, argv)
   ATTR_IND_SIMPLIFIED_P (true_rtx) = ATTR_IND_SIMPLIFIED_P (false_rtx) = 1;
   ATTR_PERMANENT_P (true_rtx) = ATTR_PERMANENT_P (false_rtx) = 1;
 
-  alternative_name = attr_string ("alternative", strlen ("alternative"));
+  alternative_name = DEF_ATTR_STRING ("alternative");
+  length_str = DEF_ATTR_STRING ("length");
+  delay_type_str = DEF_ATTR_STRING ("*delay_type");
+  delay_1_0_str = DEF_ATTR_STRING ("*delay_1_0");
+  num_delay_slots_str = DEF_ATTR_STRING ("*num_delay_slots");
 
   printf ("/* Generated automatically by the program `genattrtab'\n\
 from the machine description file `md'.  */\n\n");
 
   /* Read the machine description.  */
 
-  initiate_automaton_gen (argc, argv);
   while (1)
     {
       int lineno;
@@ -6094,49 +4495,13 @@ from the machine description file `md'.  */\n\n");
          gen_delay (desc, lineno);
          break;
 
-       case DEFINE_FUNCTION_UNIT:
-         gen_unit (desc, lineno);
+       case DEFINE_INSN_RESERVATION:
+         gen_insn_reserv (desc);
          break;
 
-       case DEFINE_CPU_UNIT:
-         gen_cpu_unit (desc);
-         break;
-         
-       case DEFINE_QUERY_CPU_UNIT:
-         gen_query_cpu_unit (desc);
-         break;
-         
        case DEFINE_BYPASS:
          gen_bypass (desc);
          break;
-         
-       case EXCLUSION_SET:
-         gen_excl_set (desc);
-         break;
-         
-       case PRESENCE_SET:
-         gen_presence_set (desc);
-         break;
-         
-       case ABSENCE_SET:
-         gen_absence_set (desc);
-         break;
-         
-       case DEFINE_AUTOMATON:
-         gen_automaton (desc);
-         break;
-         
-       case AUTOMATA_OPTION:
-         gen_automata_option (desc);
-         break;
-         
-       case DEFINE_RESERVATION:
-         gen_reserv (desc);
-         break;
-         
-       case DEFINE_INSN_RESERVATION:
-         gen_insn_reserv (desc);
-         break;
 
        default:
          break;
@@ -6162,25 +4527,18 @@ from the machine description file `md'.  */\n\n");
   if (num_delays)
     expand_delays ();
 
-  if (num_units || num_dfa_decls)
-    {
-      /* Expand DEFINE_FUNCTION_UNIT information into new attributes.  */
-      expand_units ();
-      /* Build DFA, output some functions and expand DFA information
-        into new attributes.  */
-      expand_automata ();
-    }
-
   printf ("#include \"config.h\"\n");
   printf ("#include \"system.h\"\n");
+  printf ("#include \"coretypes.h\"\n");
+  printf ("#include \"tm.h\"\n");
   printf ("#include \"rtl.h\"\n");
+  printf ("#include \"insn-attr.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 ("#include \"toplev.h\"\n");
   printf ("#include \"flags.h\"\n");
   printf ("#include \"function.h\"\n");
@@ -6188,17 +4546,20 @@ from the machine description file `md'.  */\n\n");
   printf ("#define operands recog_data.operand\n\n");
 
   /* Make `insn_alternatives'.  */
-  insn_alternatives = (int *) oballoc (insn_code_number * sizeof (int));
+  insn_alternatives = oballocvec (int, insn_code_number);
   for (id = defs; id; id = id->next)
     if (id->insn_code >= 0)
       insn_alternatives[id->insn_code] = (1 << id->num_alternatives) - 1;
 
   /* Make `insn_n_alternatives'.  */
-  insn_n_alternatives = (int *) oballoc (insn_code_number * sizeof (int));
+  insn_n_alternatives = oballocvec (int, insn_code_number);
   for (id = defs; id; id = id->next)
     if (id->insn_code >= 0)
       insn_n_alternatives[id->insn_code] = id->num_alternatives;
 
+  /* Construct extra attributes for automata.  */
+  make_automaton_attrs ();
+
   /* Prepare to write out attribute subroutines by checking everything stored
      away and building the attribute cases.  */
 
@@ -6223,8 +4584,7 @@ from the machine description file `md'.  */\n\n");
   optimize_attrs ();
 
   /* Now write out all the `gen_attr_...' routines.  Do these before the
-     special routines (specifically before write_function_unit_info), so
-     that they get defined before they are used.  */
+     special routines so that they get defined before they are used.  */
 
   for (i = 0; i < MAX_ATTRS_INDEX; i++)
     for (attr = attrs[i]; attr; attr = attr->next)
@@ -6245,16 +4605,7 @@ from the machine description file `md'.  */\n\n");
        write_eligible_delay ("annul_false");
     }
 
-  if (num_units || num_dfa_decls)
-    {
-      /* Write out information about function units.  */
-      write_function_unit_info ();
-      /* Output code for pipeline hazards recognition based on DFA
-        (deterministic finite state automata. */
-      write_automata ();
-    }
-
-  /* Write out constant delay slot info */
+  /* Write out constant delay slot info.  */
   write_const_num_delay_slots ();
 
   write_length_unit_log ();
@@ -6262,11 +4613,3 @@ from the machine description file `md'.  */\n\n");
   fflush (stdout);
   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;
-}