OSDN Git Service

2005-06-28 Thomas Koenig <Thomas.Koenig@online.de>
[pf3gnuchains/gcc-fork.git] / gcc / read-rtl.c
index fe0c7a4..2835c26 100644 (file)
@@ -1,6 +1,6 @@
 /* RTL reader for GCC.
    Copyright (C) 1987, 1988, 1991, 1994, 1997, 1998, 1999, 2000, 2001, 2002,
-   2003, 2004
+   2003, 2004, 2005
    Free Software Foundation, Inc.
 
 This file is part of GCC.
@@ -17,10 +17,14 @@ 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.  */
+Software Foundation, 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301, USA.  */
 
 #include "bconfig.h"
+
+/* Disable rtl checking; it conflicts with the macro handling.  */
+#undef ENABLE_RTL_CHECKING
+
 #include "system.h"
 #include "coretypes.h"
 #include "tm.h"
@@ -75,6 +79,27 @@ struct macro_group {
   void (*apply_macro) (rtx, int);
 };
 
+/* Associates PTR (which can be a string, etc.) with the file location
+   specified by FILENAME and LINENO.  */
+struct ptr_loc {
+  const void *ptr;
+  const char *filename;
+  int lineno;
+};
+
+/* A structure used to pass data from read_rtx to apply_macro_traverse
+   via htab_traverse.  */
+struct macro_traverse_data {
+  /* Instruction queue.  */
+  rtx queue;
+  /* Attributes seen for modes.  */
+  struct map_value *mode_maps;
+  /* Input file.  */
+  FILE *infile;
+  /* The last unknown attribute used as a mode.  */
+  const char *unknown_mode_attr;
+};
+
 /* If CODE is the number of a code macro, return a real rtx code that
    has the same format.  Return CODE otherwise.  */
 #define BELLWETHER_CODE(CODE) \
@@ -90,7 +115,8 @@ static int find_code (const char *, FILE *);
 static bool uses_code_macro_p (rtx, int);
 static void apply_code_macro (rtx, int);
 static const char *apply_macro_to_string (const char *, struct mapping *, int);
-static rtx apply_macro_to_rtx (rtx, struct mapping *, int);
+static rtx apply_macro_to_rtx (rtx, struct mapping *, int,
+                              struct map_value *, FILE *, const char **);
 static bool uses_macro_p (rtx, struct mapping *);
 static const char *add_condition_to_string (const char *, const char *);
 static void add_condition_to_rtx (rtx, const char *);
@@ -101,6 +127,10 @@ static struct map_value **add_map_value (struct map_value **,
                                         int, const char *);
 static void initialize_macros (void);
 static void read_name (char *, FILE *);
+static hashval_t leading_ptr_hash (const void *);
+static int leading_ptr_eq_p (const void *, const void *);
+static void set_rtx_ptr_loc (const void *, const char *, int);
+static const struct ptr_loc *get_rtx_ptr_loc (const void *);
 static char *read_string (FILE *, int);
 static char *read_quoted_string (FILE *);
 static char *read_braced_string (FILE *);
@@ -112,7 +142,7 @@ static void validate_const_int (FILE *, const char *);
 static int find_macro (struct macro_group *, const char *, FILE *);
 static struct mapping *read_mapping (struct macro_group *, htab_t, FILE *);
 static void check_code_macro (struct mapping *, FILE *);
-static rtx read_rtx_1 (FILE *);
+static rtx read_rtx_1 (FILE *, struct map_value **);
 
 /* The mode and code macro structures.  */
 static struct macro_group modes, codes;
@@ -123,12 +153,28 @@ static enum rtx_code *bellwether_codes;
 /* Obstack used for allocating RTL strings.  */
 static struct obstack string_obstack;
 
+/* A table of ptr_locs, hashed on the PTR field.  */
+static htab_t ptr_locs;
+
+/* An obstack for the above.  Plain xmalloc is a bit heavyweight for a
+   small structure like ptr_loc.  */
+static struct obstack ptr_loc_obstack;
+
+/* A hash table of triples (A, B, C), where each of A, B and C is a condition
+   and A is equivalent to "B && C".  This is used to keep track of the source
+   of conditions that are made up of separate rtx strings (such as the split
+   condition of a define_insn_and_split).  */
+static htab_t joined_conditions;
+
+/* An obstack for allocating joined_conditions entries.  */
+static struct obstack joined_conditions_obstack;
+
 /* Subroutines of read_rtx.  */
 
 /* The current line number for the file.  */
 int read_rtx_lineno = 1;
 
-/* The filename for aborting with file and line.  */
+/* The filename for error reporting.  */
 const char *read_rtx_filename = "<unknown>";
 
 static void
@@ -197,7 +243,7 @@ uses_mode_macro_p (rtx x, int mode)
 static void
 apply_mode_macro (rtx x, int mode)
 {
-  PUT_MODE (x, mode);
+  PUT_MODE (x, (enum machine_mode) mode);
 }
 
 /* Implementations of the macro_group callbacks for codes.  */
@@ -223,7 +269,106 @@ uses_code_macro_p (rtx x, int code)
 static void
 apply_code_macro (rtx x, int code)
 {
-  PUT_CODE (x, code);
+  PUT_CODE (x, (enum rtx_code) code);
+}
+
+/* Map a code or mode attribute string P to the underlying string for
+   MACRO and VALUE.  */
+
+static struct map_value *
+map_attr_string (const char *p, struct mapping *macro, int value)
+{
+  const char *attr;
+  struct mapping *m;
+  struct map_value *v;
+
+  /* If there's a "macro:" prefix, check whether the macro name matches.
+     Set ATTR to the start of the attribute name.  */
+  attr = strchr (p, ':');
+  if (attr == 0)
+    attr = p;
+  else
+    {
+      if (strncmp (p, macro->name, attr - p) != 0
+         || macro->name[attr - p] != 0)
+       return 0;
+      attr++;
+    }
+
+  /* Find the attribute specification.  */
+  m = (struct mapping *) htab_find (macro->group->attrs, &attr);
+  if (m == 0)
+    return 0;
+
+  /* Find the attribute value for VALUE.  */
+  for (v = m->values; v != 0; v = v->next)
+    if (v->number == value)
+      break;
+
+  return v;
+}
+
+/* Given an attribute string used as a machine mode, return an index
+   to store in the machine mode to be translated by
+   apply_macro_to_rtx.  */
+
+static unsigned int
+mode_attr_index (struct map_value **mode_maps, const char *string)
+{
+  char *p;
+  struct map_value *mv;
+
+  /* Copy the attribute string into permanent storage, without the
+     angle brackets around it.  */
+  obstack_grow0 (&string_obstack, string + 1, strlen (string) - 2);
+  p = XOBFINISH (&string_obstack, char *);
+
+  mv = XNEW (struct map_value);
+  mv->number = *mode_maps == 0 ? 0 : (*mode_maps)->number + 1;
+  mv->string = p;
+  mv->next = *mode_maps;
+  *mode_maps = mv;
+
+  /* We return a code which we can map back into this string: the
+     number of machine modes + the number of mode macros + the index
+     we just used.  */
+  return MAX_MACHINE_MODE + htab_elements (modes.macros) + mv->number;
+}
+
+/* Apply MODE_MAPS to the top level of X, expanding cases where an
+   attribute is used for a mode.  MACRO is the current macro we are
+   expanding, and VALUE is the value to which we are expanding it.
+   INFILE is used for error messages.  This sets *UNKNOWN to true if
+   we find a mode attribute which has not yet been defined, and does
+   not change it otherwise.  */
+
+static void
+apply_mode_maps (rtx x, struct map_value *mode_maps, struct mapping *macro,
+                int value, FILE *infile, const char **unknown)
+{
+  unsigned int offset;
+  int indx;
+  struct map_value *pm;
+
+  offset = MAX_MACHINE_MODE + htab_elements (modes.macros);
+  if (GET_MODE (x) < offset)
+    return;
+
+  indx = GET_MODE (x) - offset;
+  for (pm = mode_maps; pm; pm = pm->next)
+    {
+      if (pm->number == indx)
+       {
+         struct map_value *v;
+
+         v = map_attr_string (pm->string, macro, value);
+         if (v)
+           PUT_MODE (x, (enum machine_mode) find_mode (v->string, infile));
+         else
+           *unknown = pm->string;
+         return;
+       }
+    }
 }
 
 /* Given that MACRO is being expanded as VALUE, apply the appropriate
@@ -233,42 +378,20 @@ apply_code_macro (rtx x, int code)
 static const char *
 apply_macro_to_string (const char *string, struct mapping *macro, int value)
 {
-  char *base, *copy, *p, *attr, *start, *end;
-  struct mapping *m;
+  char *base, *copy, *p, *start, *end;
   struct map_value *v;
 
   if (string == 0)
     return string;
 
   base = p = copy = ASTRDUP (string);
-  while ((start = index (p, '<')) && (end = index (start, '>')))
+  while ((start = strchr (p, '<')) && (end = strchr (start, '>')))
     {
       p = start + 1;
 
-      /* If there's a "macro:" prefix, check whether the macro name matches.
-        Set ATTR to the start of the attribute name.  */
-      attr = index (p, ':');
-      if (attr == 0 || attr > end)
-       attr = p;
-      else
-       {
-         if (strncmp (p, macro->name, attr - p) != 0
-             || macro->name[attr - p] != 0)
-           continue;
-         attr++;
-       }
-
-      /* Find the attribute specification.  */
       *end = 0;
-      m = (struct mapping *) htab_find (macro->group->attrs, &attr);
+      v = map_attr_string (p, macro, value);
       *end = '>';
-      if (m == 0)
-       continue;
-
-      /* Find the attribute value for VALUE.  */
-      for (v = m->values; v != 0; v = v->next)
-       if (v->number == value)
-         break;
       if (v == 0)
        continue;
 
@@ -281,16 +404,23 @@ apply_macro_to_string (const char *string, struct mapping *macro, int value)
   if (base != copy)
     {
       obstack_grow (&string_obstack, base, strlen (base) + 1);
-      return (char *) obstack_finish (&string_obstack);
+      copy = XOBFINISH (&string_obstack, char *);
+      copy_rtx_ptr_loc (copy, string);
+      return copy;
     }
   return string;
 }
 
 /* Return a copy of ORIGINAL in which all uses of MACRO have been
-   replaced by VALUE.  */
+   replaced by VALUE.  MODE_MAPS holds information about attribute
+   strings used for modes.  INFILE is used for error messages.  This
+   sets *UNKNOWN_MODE_ATTR to the value of an unknown mode attribute,
+   and does not change it otherwise.  */
 
 static rtx
-apply_macro_to_rtx (rtx original, struct mapping *macro, int value)
+apply_macro_to_rtx (rtx original, struct mapping *macro, int value,
+                   struct map_value *mode_maps, FILE *infile,
+                   const char **unknown_mode_attr)
 {
   struct macro_group *group;
   const char *format_ptr;
@@ -311,19 +441,27 @@ apply_macro_to_rtx (rtx original, struct mapping *macro, int value)
   if (group->uses_macro_p (x, macro->index + group->num_builtins))
     group->apply_macro (x, value);
 
+  if (mode_maps)
+    apply_mode_maps (x, mode_maps, macro, value, infile, unknown_mode_attr);
+
   /* Change each string and recursively change each rtx.  */
   format_ptr = GET_RTX_FORMAT (bellwether_code);
   for (i = 0; format_ptr[i] != 0; i++)
     switch (format_ptr[i])
       {
-      case 'S':
       case 'T':
+       XTMPL (x, i) = apply_macro_to_string (XTMPL (x, i), macro, value);
+       break;
+
+      case 'S':
       case 's':
        XSTR (x, i) = apply_macro_to_string (XSTR (x, i), macro, value);
        break;
 
       case 'e':
-       XEXP (x, i) = apply_macro_to_rtx (XEXP (x, i), macro, value);
+       XEXP (x, i) = apply_macro_to_rtx (XEXP (x, i), macro, value,
+                                         mode_maps, infile,
+                                         unknown_mode_attr);
        break;
 
       case 'V':
@@ -333,7 +471,9 @@ apply_macro_to_rtx (rtx original, struct mapping *macro, int value)
            XVEC (x, i) = rtvec_alloc (XVECLEN (original, i));
            for (j = 0; j < XVECLEN (x, i); j++)
              XVECEXP (x, i, j) = apply_macro_to_rtx (XVECEXP (original, i, j),
-                                                     macro, value);
+                                                     macro, value, mode_maps,
+                                                     infile,
+                                                     unknown_mode_attr);
          }
        break;
 
@@ -389,16 +529,9 @@ uses_macro_p (rtx x, struct mapping *macro)
 static const char *
 add_condition_to_string (const char *original, const char *extra)
 {
-  char *result;
-
-  if (original == 0 || original[0] == 0)
-    return extra;
-
-  if ((original[0] == '&' && original[1] == '&') || extra[0] == 0)
+  if (original != 0 && original[0] == '&' && original[1] == '&')
     return original;
-
-  asprintf (&result, "(%s) && (%s)", original, extra);
-  return result;
+  return join_c_conditions (original, extra);
 }
 
 /* Like add_condition, but applied to all conditions in rtx X.  */
@@ -437,18 +570,29 @@ add_condition_to_rtx (rtx x, const char *extra)
 static int
 apply_macro_traverse (void **slot, void *data)
 {
+  struct macro_traverse_data *mtd = (struct macro_traverse_data *) data;
   struct mapping *macro;
   struct map_value *v;
   rtx elem, new_elem, original, x;
 
   macro = (struct mapping *) *slot;
-  for (elem = (rtx) data; elem != 0; elem = XEXP (elem, 1))
+  for (elem = mtd->queue; elem != 0; elem = XEXP (elem, 1))
     if (uses_macro_p (XEXP (elem, 0), macro))
       {
+       /* For each macro we expand, we set UNKNOWN_MODE_ATTR to NULL.
+          If apply_macro_rtx finds an unknown attribute for a mode,
+          it will set it to the attribute.  We want to know whether
+          the attribute is unknown after we have expanded all
+          possible macros, so setting it to NULL here gives us the
+          right result when the hash table traversal is complete.  */
+       mtd->unknown_mode_attr = NULL;
+
        original = XEXP (elem, 0);
        for (v = macro->values; v != 0; v = v->next)
          {
-           x = apply_macro_to_rtx (original, macro, v->number);
+           x = apply_macro_to_rtx (original, macro, v->number,
+                                   mtd->mode_maps, mtd->infile,
+                                   &mtd->unknown_mode_attr);
            add_condition_to_rtx (x, v->string);
            if (v != macro->values)
              {
@@ -561,6 +705,116 @@ initialize_macros (void)
     }
 }
 
+/* Return a hash value for the pointer pointed to by DEF.  */
+
+static hashval_t
+leading_ptr_hash (const void *def)
+{
+  return htab_hash_pointer (*(const void *const *) def);
+}
+
+/* Return true if DEF1 and DEF2 are pointers to the same pointer.  */
+
+static int
+leading_ptr_eq_p (const void *def1, const void *def2)
+{
+  return *(const void *const *) def1 == *(const void *const *) def2;
+}
+
+/* Associate PTR with the file position given by FILENAME and LINENO.  */
+
+static void
+set_rtx_ptr_loc (const void *ptr, const char *filename, int lineno)
+{
+  struct ptr_loc *loc;
+
+  loc = (struct ptr_loc *) obstack_alloc (&ptr_loc_obstack,
+                                         sizeof (struct ptr_loc));
+  loc->ptr = ptr;
+  loc->filename = filename;
+  loc->lineno = lineno;
+  *htab_find_slot (ptr_locs, loc, INSERT) = loc;
+}
+
+/* Return the position associated with pointer PTR.  Return null if no
+   position was set.  */
+
+static const struct ptr_loc *
+get_rtx_ptr_loc (const void *ptr)
+{
+  return (const struct ptr_loc *) htab_find (ptr_locs, &ptr);
+}
+
+/* Associate NEW_PTR with the same file position as OLD_PTR.  */
+
+void
+copy_rtx_ptr_loc (const void *new_ptr, const void *old_ptr)
+{
+  const struct ptr_loc *loc = get_rtx_ptr_loc (old_ptr);
+  if (loc != 0)
+    set_rtx_ptr_loc (new_ptr, loc->filename, loc->lineno);
+}
+
+/* If PTR is associated with a known file position, print a #line
+   directive for it.  */
+
+void
+print_rtx_ptr_loc (const void *ptr)
+{
+  const struct ptr_loc *loc = get_rtx_ptr_loc (ptr);
+  if (loc != 0)
+    printf ("#line %d \"%s\"\n", loc->lineno, loc->filename);
+}
+
+/* Return a condition that satisfies both COND1 and COND2.  Either string
+   may be null or empty.  */
+
+const char *
+join_c_conditions (const char *cond1, const char *cond2)
+{
+  char *result;
+  const void **entry;
+
+  if (cond1 == 0 || cond1[0] == 0)
+    return cond2;
+
+  if (cond2 == 0 || cond2[0] == 0)
+    return cond1;
+
+  result = concat ("(", cond1, ") && (", cond2, ")", NULL);
+  obstack_ptr_grow (&joined_conditions_obstack, result);
+  obstack_ptr_grow (&joined_conditions_obstack, cond1);
+  obstack_ptr_grow (&joined_conditions_obstack, cond2);
+  entry = XOBFINISH (&joined_conditions_obstack, const void **);
+  *htab_find_slot (joined_conditions, entry, INSERT) = entry;
+  return result;
+}
+
+/* Print condition COND, wrapped in brackets.  If COND was created by
+   join_c_conditions, recursively invoke this function for the original
+   conditions and join the result with "&&".  Otherwise print a #line
+   directive for COND if its original file position is known.  */
+
+void
+print_c_condition (const char *cond)
+{
+  const char **halves = (const char **) htab_find (joined_conditions, &cond);
+  if (halves != 0)
+    {
+      printf ("(");
+      print_c_condition (halves[1]);
+      printf (" && ");
+      print_c_condition (halves[2]);
+      printf (")");
+    }
+  else
+    {
+      putc ('\n', stdout);
+      print_rtx_ptr_loc (cond);
+      printf ("(%s)", cond);
+    }
+}
+
 /* Read chars from INFILE until a non-whitespace char
    and return that.  Comments, both Lisp style and C style,
    are treated as whitespace.
@@ -744,7 +998,7 @@ read_quoted_string (FILE *infile)
     }
 
   obstack_1grow (&string_obstack, 0);
-  return (char *) obstack_finish (&string_obstack);
+  return XOBFINISH (&string_obstack, char *);
 }
 
 /* Read a braced string (a la Tcl) onto the string obstack.  Caller
@@ -782,7 +1036,7 @@ read_braced_string (FILE *infile)
     }
 
   obstack_1grow (&string_obstack, 0);
-  return (char *) obstack_finish (&string_obstack);
+  return XOBFINISH (&string_obstack, char *);
 }
 
 /* Read some kind of string constant.  This is the high-level routine
@@ -794,7 +1048,7 @@ read_string (FILE *infile, int star_if_braced)
 {
   char *stringbuf;
   int saw_paren = 0;
-  int c;
+  int c, old_lineno;
 
   c = read_skip_spaces (infile);
   if (c == '(')
@@ -803,6 +1057,7 @@ read_string (FILE *infile, int star_if_braced)
       c = read_skip_spaces (infile);
     }
 
+  old_lineno = read_rtx_lineno;
   if (c == '"')
     stringbuf = read_quoted_string (infile);
   else if (c == '{')
@@ -821,6 +1076,7 @@ read_string (FILE *infile, int star_if_braced)
        fatal_expected_char (infile, ')', c);
     }
 
+  set_rtx_ptr_loc (stringbuf, read_rtx_filename, old_lineno);
   return stringbuf;
 }
 \f
@@ -1055,7 +1311,7 @@ check_code_macro (struct mapping *macro, FILE *infile)
   struct map_value *v;
   enum rtx_code bellwether;
 
-  bellwether = macro->values->number;
+  bellwether = (enum rtx_code) macro->values->number;
   for (v = macro->values->next; v != 0; v = v->next)
     if (strcmp (GET_RTX_FORMAT (bellwether), GET_RTX_FORMAT (v->number)) != 0)
       fatal_with_file_and_line (infile, "code macro `%s' combines "
@@ -1066,16 +1322,20 @@ check_code_macro (struct mapping *macro, FILE *infile)
   bellwether_codes[macro->index] = bellwether;
 }
 
-/* Read an rtx in printed representation from INFILE
-   and return an actual rtx in core constructed accordingly.
+/* Read an rtx in printed representation from INFILE and store its
+   core representation in *X.  Also store the line number of the
+   opening '(' in *LINENO.  Return true on success or false if the
+   end of file has been reached.
+
    read_rtx is not used in the compiler proper, but rather in
    the utilities gen*.c that construct C code from machine descriptions.  */
 
-rtx
-read_rtx (FILE *infile)
+bool
+read_rtx (FILE *infile, rtx *x, int *lineno)
 {
   static rtx queue_head, queue_next;
-  rtx return_rtx;
+  static int queue_lineno;
+  int c;
 
   /* Do one-time initialization.  */
   if (queue_head == 0)
@@ -1083,30 +1343,53 @@ read_rtx (FILE *infile)
       initialize_macros ();
       obstack_init (&string_obstack);
       queue_head = rtx_alloc (EXPR_LIST);
+      ptr_locs = htab_create (161, leading_ptr_hash, leading_ptr_eq_p, 0);
+      obstack_init (&ptr_loc_obstack);
+      joined_conditions = htab_create (161, leading_ptr_hash,
+                                      leading_ptr_eq_p, 0);
+      obstack_init (&joined_conditions_obstack);
     }
 
   if (queue_next == 0)
     {
-      queue_next = queue_head;
+      struct map_value *mode_maps;
+      struct macro_traverse_data mtd;
 
-      XEXP (queue_next, 0) = read_rtx_1 (infile);
+      c = read_skip_spaces (infile);
+      if (c == EOF)
+       return false;
+      ungetc (c, infile);
+
+      queue_next = queue_head;
+      queue_lineno = read_rtx_lineno;
+      mode_maps = 0;
+      XEXP (queue_next, 0) = read_rtx_1 (infile, &mode_maps);
       XEXP (queue_next, 1) = 0;
 
-      htab_traverse (modes.macros, apply_macro_traverse, queue_next);
-      htab_traverse (codes.macros, apply_macro_traverse, queue_next);
+      mtd.queue = queue_next;
+      mtd.mode_maps = mode_maps;
+      mtd.infile = infile;
+      mtd.unknown_mode_attr = mode_maps ? mode_maps->string : NULL;
+      htab_traverse (modes.macros, apply_macro_traverse, &mtd);
+      htab_traverse (codes.macros, apply_macro_traverse, &mtd);
+      if (mtd.unknown_mode_attr)
+       fatal_with_file_and_line (infile,
+                                 "undefined attribute '%s' used for mode",
+                                 mtd.unknown_mode_attr);
     }
 
-  return_rtx = XEXP (queue_next, 0);
+  *x = XEXP (queue_next, 0);
+  *lineno = queue_lineno;
   queue_next = XEXP (queue_next, 1);
 
-  return return_rtx;
+  return true;
 }
 
 /* Subroutine of read_rtx that reads one construct from INFILE but
    doesn't apply any macros.  */
 
 static rtx
-read_rtx_1 (FILE *infile)
+read_rtx_1 (FILE *infile, struct map_value **mode_maps)
 {
   int i;
   RTX_CODE real_code, bellwether_code;
@@ -1166,7 +1449,7 @@ read_rtx_1 (FILE *infile)
       check_code_macro (read_mapping (&codes, codes.macros, infile), infile);
       goto again;
     }
-  real_code = find_macro (&codes, tmp_char, infile);
+  real_code = (enum rtx_code) find_macro (&codes, tmp_char, infile);
   bellwether_code = BELLWETHER_CODE (real_code);
 
   /* If we end up with an insn expression then we free this space below.  */
@@ -1180,8 +1463,16 @@ read_rtx_1 (FILE *infile)
   i = read_skip_spaces (infile);
   if (i == ':')
     {
+      unsigned int mode;
+
       read_name (tmp_char, infile);
-      PUT_MODE (return_rtx, find_macro (&modes, tmp_char, infile));
+      if (tmp_char[0] != '<' || tmp_char[strlen (tmp_char) - 1] != '>')
+       mode = find_macro (&modes, tmp_char, infile);
+      else
+       mode = mode_attr_index (mode_maps, tmp_char);
+      PUT_MODE (return_rtx, (enum machine_mode) mode);
+      if (GET_MODE (return_rtx) != mode)
+       fatal_with_file_and_line (infile, "mode too large");
     }
   else
     ungetc (i, infile);
@@ -1196,7 +1487,7 @@ read_rtx_1 (FILE *infile)
 
       case 'e':
       case 'u':
-       XEXP (return_rtx, i) = read_rtx_1 (infile);
+       XEXP (return_rtx, i) = read_rtx_1 (infile, mode_maps);
        break;
 
       case 'V':
@@ -1228,7 +1519,7 @@ read_rtx_1 (FILE *infile)
            {
              ungetc (c, infile);
              list_counter++;
-             obstack_ptr_grow (&vector_stack, read_rtx_1 (infile));
+             obstack_ptr_grow (&vector_stack, read_rtx_1 (infile, mode_maps));
            }
          if (list_counter > 0)
            {
@@ -1236,6 +1527,9 @@ read_rtx_1 (FILE *infile)
              memcpy (&return_vec->elem[0], obstack_finish (&vector_stack),
                      list_counter * sizeof (rtx));
            }
+         else if (format_ptr[i] == 'E')
+           fatal_with_file_and_line (infile,
+                                     "vector must have at least one element");
          XVEC (return_rtx, i) = return_vec;
          obstack_free (&vector_stack, NULL);
          /* close bracket gotten */
@@ -1287,7 +1581,7 @@ read_rtx_1 (FILE *infile)
              obstack_grow (&string_obstack, fn, strlen (fn));
              sprintf (line_name, ":%d", read_rtx_lineno);
              obstack_grow (&string_obstack, line_name, strlen (line_name)+1);
-             stringbuf = (char *) obstack_finish (&string_obstack);
+             stringbuf = XOBFINISH (&string_obstack, char *);
            }
 
          if (star_if_braced)
@@ -1327,11 +1621,7 @@ read_rtx_1 (FILE *infile)
        break;
 
       default:
-       fprintf (stderr,
-                "switch format wrong in rtl.read_rtx(). format was: %c.\n",
-                format_ptr[i]);
-       fprintf (stderr, "\tfile position: %ld\n", ftell (infile));
-       abort ();
+       gcc_unreachable ();
       }
 
   c = read_skip_spaces (infile);