Gate the displaying of non-debug sections in separate debuginfo files.
[deliverable/binutils-gdb.git] / gdb / eval.c
index 14a3e05ade223385989446c663cd0a620058ad7b..7ba3ee59522fc4d627f85315eba77835547a6b5b 100644 (file)
@@ -1,6 +1,6 @@
 /* Evaluate expressions for GDB.
 
-   Copyright (C) 1986-2017 Free Software Foundation, Inc.
+   Copyright (C) 1986-2021 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -26,7 +26,6 @@
 #include "frame.h"
 #include "gdbthread.h"
 #include "language.h"          /* For CAST_IS_CONVERSION.  */
-#include "f-lang.h"            /* For array bound stuff.  */
 #include "cp-abi.h"
 #include "infcall.h"
 #include "objc-lang.h"
@@ -42,9 +41,6 @@
 #include "typeprint.h"
 #include <ctype.h>
 
-/* This is defined in valops.c */
-extern int overload_resolution;
-
 /* Prototypes for local functions.  */
 
 static struct value *evaluate_subexp_for_sizeof (struct expression *, int *,
@@ -61,37 +57,12 @@ static struct value *evaluate_struct_tuple (struct value *,
                                            struct expression *, int *,
                                            enum noside, int);
 
-static LONGEST init_array_element (struct value *, struct value *,
-                                  struct expression *, int *, enum noside,
-                                  LONGEST, LONGEST);
-
 struct value *
 evaluate_subexp (struct type *expect_type, struct expression *exp,
                 int *pos, enum noside noside)
 {
-  struct cleanup *cleanups;
-  struct value *retval;
-  int cleanup_temps = 0;
-
-  if (*pos == 0 && target_has_execution
-      && exp->language_defn->la_language == language_cplus
-      && !thread_stack_temporaries_enabled_p (inferior_ptid))
-    {
-      cleanups = enable_thread_stack_temporaries (inferior_ptid);
-      cleanup_temps = 1;
-    }
-
-  retval = (*exp->language_defn->la_exp_desc->evaluate_exp)
-    (expect_type, exp, pos, noside);
-
-  if (cleanup_temps)
-    {
-      if (value_in_thread_stack_temporaries (retval, inferior_ptid))
-       retval = value_non_lval (retval);
-      do_cleanups (cleanups);
-    }
-
-  return retval;
+  return ((*exp->language_defn->expression_ops ()->evaluate_exp)
+         (expect_type, exp, pos, noside));
 }
 \f
 /* Parse the string EXP as a C expression, evaluate it,
@@ -130,22 +101,39 @@ parse_and_eval (const char *exp)
 struct value *
 parse_to_comma_and_eval (const char **expp)
 {
-  expression_up expr = parse_exp_1 (expp, 0, (struct block *) 0, 1);
+  expression_up expr = parse_exp_1 (expp, 0, nullptr, 1);
 
   return evaluate_expression (expr.get ());
 }
 \f
-/* Evaluate an expression in internal prefix form
-   such as is constructed by parse.y.
 
-   See expression.h for info on the format of an expression.  */
+/* See expression.h.  */
 
 struct value *
-evaluate_expression (struct expression *exp)
+expression::evaluate (struct type *expect_type, enum noside noside)
 {
-  int pc = 0;
+  gdb::optional<enable_thread_stack_temporaries> stack_temporaries;
+  if (target_has_execution ()
+      && language_defn->la_language == language_cplus
+      && !thread_stack_temporaries_enabled_p (inferior_thread ()))
+    stack_temporaries.emplace (inferior_thread ());
 
-  return evaluate_subexp (NULL_TYPE, exp, &pc, EVAL_NORMAL);
+  int pos = 0;
+  struct value *retval = evaluate_subexp (expect_type, this, &pos, noside);
+
+  if (stack_temporaries.has_value ()
+      && value_in_thread_stack_temporaries (retval, inferior_thread ()))
+    retval = value_non_lval (retval);
+
+  return retval;
+}
+
+/* See value.h.  */
+
+struct value *
+evaluate_expression (struct expression *exp, struct type *expect_type)
+{
+  return exp->evaluate (expect_type, EVAL_NORMAL);
 }
 
 /* Evaluate an expression, avoiding all memory references
@@ -154,9 +142,7 @@ evaluate_expression (struct expression *exp)
 struct value *
 evaluate_type (struct expression *exp)
 {
-  int pc = 0;
-
-  return evaluate_subexp (NULL_TYPE, exp, &pc, EVAL_AVOID_SIDE_EFFECTS);
+  return exp->evaluate (nullptr, EVAL_AVOID_SIDE_EFFECTS);
 }
 
 /* Evaluate a subexpression, avoiding all memory references and
@@ -165,7 +151,7 @@ evaluate_type (struct expression *exp)
 struct value *
 evaluate_subexpression_type (struct expression *exp, int subexp)
 {
-  return evaluate_subexp (NULL_TYPE, exp, &subexp, EVAL_AVOID_SIDE_EFFECTS);
+  return evaluate_subexp (nullptr, exp, &subexp, EVAL_AVOID_SIDE_EFFECTS);
 }
 
 /* Find the current value of a watchpoint on EXP.  Return the value in
@@ -186,15 +172,15 @@ evaluate_subexpression_type (struct expression *exp, int subexp)
    set to any referenced values.  *VALP will never be a lazy value.
    This is the value which we store in struct breakpoint.
 
-   If VAL_CHAIN is non-NULL, *VAL_CHAIN will be released from the
-   value chain.  The caller must free the values individually.  If
-   VAL_CHAIN is NULL, all generated values will be left on the value
-   chain.  */
+   If VAL_CHAIN is non-NULL, the values put into *VAL_CHAIN will be
+   released from the value chain.  If VAL_CHAIN is NULL, all generated
+   values will be left on the value chain.  */
 
 void
 fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
-                   struct value **resultp, struct value **val_chain,
-                   int preserve_errors)
+                   struct value **resultp,
+                   std::vector<value_ref_ptr> *val_chain,
+                   bool preserve_errors)
 {
   struct value *mark, *new_mark, *result;
 
@@ -202,17 +188,17 @@ fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
   if (resultp)
     *resultp = NULL;
   if (val_chain)
-    *val_chain = NULL;
+    val_chain->clear ();
 
   /* Evaluate the expression.  */
   mark = value_mark ();
   result = NULL;
 
-  TRY
+  try
     {
-      result = evaluate_subexp (NULL_TYPE, exp, pc, EVAL_NORMAL);
+      result = evaluate_subexp (nullptr, exp, pc, EVAL_NORMAL);
     }
-  CATCH (ex, RETURN_MASK_ALL)
+  catch (const gdb_exception &ex)
     {
       /* Ignore memory errors if we want watchpoints pointing at
         inaccessible memory to still be created; otherwise, throw the
@@ -222,12 +208,12 @@ fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
        case MEMORY_ERROR:
          if (!preserve_errors)
            break;
+         /* Fall through.  */
        default:
-         throw_exception (ex);
+         throw;
          break;
        }
     }
-  END_CATCH
 
   new_mark = value_mark ();
   if (mark == new_mark)
@@ -244,15 +230,14 @@ fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
       else
        {
 
-         TRY
+         try
            {
              value_fetch_lazy (result);
              *valp = result;
            }
-         CATCH (except, RETURN_MASK_ERROR)
+         catch (const gdb_exception_error &except)
            {
            }
-         END_CATCH
        }
     }
 
@@ -260,8 +245,7 @@ fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
     {
       /* Return the chain of intermediate values.  We use this to
         decide which addresses to watch.  */
-      *val_chain = new_mark;
-      value_release_to_mark (mark);
+      *val_chain = value_release_to_mark (mark);
     }
 }
 
@@ -272,7 +256,7 @@ fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
    subexpression of the left-hand-side of the dereference.  This is
    used when completing field names.  */
 
-char *
+const char *
 extract_field_op (struct expression *exp, int *subexp)
 {
   int tem;
@@ -307,14 +291,13 @@ evaluate_struct_tuple (struct value *struct_val,
 
       fieldno++;
       /* Skip static fields.  */
-      while (fieldno < TYPE_NFIELDS (struct_type)
-            && field_is_static (&TYPE_FIELD (struct_type,
-                                             fieldno)))
+      while (fieldno < struct_type->num_fields ()
+            && field_is_static (&struct_type->field (fieldno)))
        fieldno++;
-      if (fieldno >= TYPE_NFIELDS (struct_type))
+      if (fieldno >= struct_type->num_fields ())
        error (_("too many initializers"));
-      field_type = TYPE_FIELD_TYPE (struct_type, fieldno);
-      if (TYPE_CODE (field_type) == TYPE_CODE_UNION
+      field_type = struct_type->field (fieldno).type ();
+      if (field_type->code () == TYPE_CODE_UNION
          && TYPE_FIELD_NAME (struct_type, fieldno)[0] == '0')
        error (_("don't know which variant you want to set"));
 
@@ -327,7 +310,7 @@ evaluate_struct_tuple (struct value *struct_val,
         subfieldno is the index of the actual real (named inner) field
         in substruct_type.  */
 
-      field_type = TYPE_FIELD_TYPE (struct_type, fieldno);
+      field_type = struct_type->field (fieldno).type ();
       if (val == 0)
        val = evaluate_subexp (field_type, exp, pos, noside);
 
@@ -351,65 +334,6 @@ evaluate_struct_tuple (struct value *struct_val,
   return struct_val;
 }
 
-/* Recursive helper function for setting elements of array tuples.
-   The target is ARRAY (which has bounds LOW_BOUND to HIGH_BOUND); the
-   element value is ELEMENT; EXP, POS and NOSIDE are as usual.
-   Evaluates index expresions and sets the specified element(s) of
-   ARRAY to ELEMENT.  Returns last index value.  */
-
-static LONGEST
-init_array_element (struct value *array, struct value *element,
-                   struct expression *exp, int *pos,
-                   enum noside noside, LONGEST low_bound, LONGEST high_bound)
-{
-  LONGEST index;
-  int element_size = TYPE_LENGTH (value_type (element));
-
-  if (exp->elts[*pos].opcode == BINOP_COMMA)
-    {
-      (*pos)++;
-      init_array_element (array, element, exp, pos, noside,
-                         low_bound, high_bound);
-      return init_array_element (array, element,
-                                exp, pos, noside, low_bound, high_bound);
-    }
-  else
-    {
-      index = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
-      if (index < low_bound || index > high_bound)
-       error (_("tuple index out of range"));
-      memcpy (value_contents_raw (array) + (index - low_bound) * element_size,
-             value_contents (element), element_size);
-    }
-  return index;
-}
-
-static struct value *
-value_f90_subarray (struct value *array,
-                   struct expression *exp, int *pos, enum noside noside)
-{
-  int pc = (*pos) + 1;
-  LONGEST low_bound, high_bound;
-  struct type *range = check_typedef (TYPE_INDEX_TYPE (value_type (array)));
-  enum range_type range_type
-    = (enum range_type) longest_to_int (exp->elts[pc].longconst);
-  *pos += 3;
-
-  if (range_type == LOW_BOUND_DEFAULT || range_type == BOTH_BOUND_DEFAULT)
-    low_bound = TYPE_LOW_BOUND (range);
-  else
-    low_bound = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
-
-  if (range_type == HIGH_BOUND_DEFAULT || range_type == BOTH_BOUND_DEFAULT)
-    high_bound = TYPE_HIGH_BOUND (range);
-  else
-    high_bound = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
-
-  return value_slice (array, low_bound, high_bound - low_bound + 1);
-}
-
-
 /* Promote value ARG1 as appropriate before performing a unary operation
    on this argument.
    If the result is not appropriate for any particular language then it
@@ -430,7 +354,7 @@ unop_promote (const struct language_defn *language, struct gdbarch *gdbarch,
        {
        default:
          /* Perform integral promotion for ANSI C/C++.
-            If not appropropriate for any particular language
+            If not appropriate for any particular language
             it needs to modify this function.  */
          {
            struct type *builtin_int = builtin_type (gdbarch)->builtin_int;
@@ -462,21 +386,24 @@ binop_promote (const struct language_defn *language, struct gdbarch *gdbarch,
   type1 = check_typedef (value_type (*arg1));
   type2 = check_typedef (value_type (*arg2));
 
-  if ((TYPE_CODE (type1) != TYPE_CODE_FLT
-       && TYPE_CODE (type1) != TYPE_CODE_DECFLOAT
+  if ((type1->code () != TYPE_CODE_FLT
+       && type1->code () != TYPE_CODE_DECFLOAT
        && !is_integral_type (type1))
-      || (TYPE_CODE (type2) != TYPE_CODE_FLT
-         && TYPE_CODE (type2) != TYPE_CODE_DECFLOAT
+      || (type2->code () != TYPE_CODE_FLT
+         && type2->code () != TYPE_CODE_DECFLOAT
          && !is_integral_type (type2)))
     return;
 
-  if (TYPE_CODE (type1) == TYPE_CODE_DECFLOAT
-      || TYPE_CODE (type2) == TYPE_CODE_DECFLOAT)
+  if (is_fixed_point_type (type1) || is_fixed_point_type (type2))
+        return;
+
+  if (type1->code () == TYPE_CODE_DECFLOAT
+      || type2->code () == TYPE_CODE_DECFLOAT)
     {
       /* No promotion required.  */
     }
-  else if (TYPE_CODE (type1) == TYPE_CODE_FLT
-          || TYPE_CODE (type2) == TYPE_CODE_FLT)
+  else if (type1->code () == TYPE_CODE_FLT
+          || type2->code () == TYPE_CODE_FLT)
     {
       switch (language->la_language)
        {
@@ -501,8 +428,8 @@ binop_promote (const struct language_defn *language, struct gdbarch *gdbarch,
          break;
        }
     }
-  else if (TYPE_CODE (type1) == TYPE_CODE_BOOL
-          && TYPE_CODE (type2) == TYPE_CODE_BOOL)
+  else if (type1->code () == TYPE_CODE_BOOL
+          && type2->code () == TYPE_CODE_BOOL)
     {
       /* No promotion required.  */
     }
@@ -513,13 +440,13 @@ binop_promote (const struct language_defn *language, struct gdbarch *gdbarch,
       const struct builtin_type *builtin = builtin_type (gdbarch);
       unsigned int promoted_len1 = TYPE_LENGTH (type1);
       unsigned int promoted_len2 = TYPE_LENGTH (type2);
-      int is_unsigned1 = TYPE_UNSIGNED (type1);
-      int is_unsigned2 = TYPE_UNSIGNED (type2);
+      int is_unsigned1 = type1->is_unsigned ();
+      int is_unsigned2 = type2->is_unsigned ();
       unsigned int result_len;
       int unsigned_operation;
 
       /* Determine type length and signedness after promotion for
-         both operands.  */
+        both operands.  */
       if (promoted_len1 < TYPE_LENGTH (builtin->builtin_int))
        {
          is_unsigned1 = 0;
@@ -574,20 +501,20 @@ binop_promote (const struct language_defn *language, struct gdbarch *gdbarch,
          break;
        case language_opencl:
          if (result_len <= TYPE_LENGTH (lookup_signed_typename
-                                        (language, gdbarch, "int")))
+                                        (language, "int")))
            {
              promoted_type =
                (unsigned_operation
-                ? lookup_unsigned_typename (language, gdbarch, "int")
-                : lookup_signed_typename (language, gdbarch, "int"));
+                ? lookup_unsigned_typename (language, "int")
+                : lookup_signed_typename (language, "int"));
            }
          else if (result_len <= TYPE_LENGTH (lookup_signed_typename
-                                             (language, gdbarch, "long")))
+                                             (language, "long")))
            {
              promoted_type =
                (unsigned_operation
-                ? lookup_unsigned_typename (language, gdbarch, "long")
-                : lookup_signed_typename (language, gdbarch,"long"));
+                ? lookup_unsigned_typename (language, "long")
+                : lookup_signed_typename (language,"long"));
            }
          break;
        default:
@@ -628,14 +555,14 @@ ptrmath_type_p (const struct language_defn *lang, struct type *type)
   if (TYPE_IS_REFERENCE (type))
     type = TYPE_TARGET_TYPE (type);
 
-  switch (TYPE_CODE (type))
+  switch (type->code ())
     {
     case TYPE_CODE_PTR:
     case TYPE_CODE_FUNC:
       return 1;
 
     case TYPE_CODE_ARRAY:
-      return TYPE_VECTOR (type) ? 0 : lang->c_style_arrays;
+      return type->is_vector () ? 0 : lang->c_style_arrays_p ();
 
     default:
       return 0;
@@ -670,37 +597,41 @@ fake_method::fake_method (type_instance_flags flags,
 
   TYPE_MAIN_TYPE (type) = &m_main_type;
   TYPE_LENGTH (type) = 1;
-  TYPE_CODE (type) = TYPE_CODE_METHOD;
+  type->set_code (TYPE_CODE_METHOD);
   TYPE_CHAIN (type) = type;
-  TYPE_INSTANCE_FLAGS (type) = flags;
+  type->set_instance_flags (flags);
   if (num_types > 0)
     {
       if (param_types[num_types - 1] == NULL)
        {
          --num_types;
-         TYPE_VARARGS (type) = 1;
+         type->set_has_varargs (true);
        }
-      else if (TYPE_CODE (check_typedef (param_types[num_types - 1]))
+      else if (check_typedef (param_types[num_types - 1])->code ()
               == TYPE_CODE_VOID)
        {
          --num_types;
          /* Caller should have ensured this.  */
          gdb_assert (num_types == 0);
-         TYPE_PROTOTYPED (type) = 1;
+         type->set_is_prototyped (true);
        }
     }
 
-  TYPE_NFIELDS (type) = num_types;
-  TYPE_FIELDS (type) = (struct field *)
-    TYPE_ZALLOC (type, sizeof (struct field) * num_types);
+  /* We don't use TYPE_ZALLOC here to allocate space as TYPE is owned by
+     neither an objfile nor a gdbarch.  As a result we must manually
+     allocate memory for auxiliary fields, and free the memory ourselves
+     when we are done with it.  */
+  type->set_num_fields (num_types);
+  type->set_fields
+    ((struct field *) xzalloc (sizeof (struct field) * num_types));
 
   while (num_types-- > 0)
-    TYPE_FIELD_TYPE (type, num_types) = param_types[num_types];
+    type->field (num_types).set_type (param_types[num_types]);
 }
 
 fake_method::~fake_method ()
 {
-  xfree (TYPE_FIELDS (&m_type));
+  xfree (m_type.fields ());
 }
 
 /* Helper for evaluating an OP_VAR_VALUE.  */
@@ -719,19 +650,18 @@ evaluate_var_value (enum noside noside, const block *blk, symbol *var)
 
   struct value *ret = NULL;
 
-  TRY
+  try
     {
       ret = value_of_variable (var, blk);
     }
 
-  CATCH (except, RETURN_MASK_ERROR)
+  catch (const gdb_exception_error &except)
     {
       if (noside != EVAL_AVOID_SIDE_EFFECTS)
-       throw_exception (except);
+       throw;
 
       ret = value_zero (SYMBOL_TYPE (var), not_lval);
     }
-  END_CATCH
 
   return ret;
 }
@@ -742,17 +672,13 @@ value *
 evaluate_var_msym_value (enum noside noside,
                         struct objfile *objfile, minimal_symbol *msymbol)
 {
-  if (noside == EVAL_AVOID_SIDE_EFFECTS)
-    {
-      type *the_type = find_minsym_type_and_address (msymbol, objfile, NULL);
-      return value_zero (the_type, not_lval);
-    }
+  CORE_ADDR address;
+  type *the_type = find_minsym_type_and_address (msymbol, objfile, &address);
+
+  if (noside == EVAL_AVOID_SIDE_EFFECTS && !the_type->is_gnu_ifunc ())
+    return value_zero (the_type, not_lval);
   else
-    {
-      CORE_ADDR address;
-      type *the_type = find_minsym_type_and_address (msymbol, objfile, &address);
-      return value_at_lazy (the_type, address);
-    }
+    return value_at_lazy (the_type, address);
 }
 
 /* Helper for returning a value when handling EVAL_SKIP.  */
@@ -763,19 +689,16 @@ eval_skip_value (expression *exp)
   return value_from_longest (builtin_type (exp->gdbarch)->builtin_int, 1);
 }
 
-/* Evaluate a function call.  The function to be called is in
-   ARGVEC[0] and the arguments passed to the function are in
-   ARGVEC[1..NARGS].  FUNCTION_NAME is the name of the function, if
-   known.  DEFAULT_RETURN_TYPE is used as the function's return type
-   if the return type is unknown.  */
+/* See expression.h.  */
 
-static value *
-eval_call (expression *exp, enum noside noside,
-          int nargs, value **argvec,
-          const char *function_name,
-          type *default_return_type)
+value *
+evaluate_subexp_do_call (expression *exp, enum noside noside,
+                        value *callee,
+                        gdb::array_view<value *> argvec,
+                        const char *function_name,
+                        type *default_return_type)
 {
-  if (argvec[0] == NULL)
+  if (callee == NULL)
     error (_("Cannot evaluate function -- may be inlined"));
   if (noside == EVAL_AVOID_SIDE_EFFECTS)
     {
@@ -783,9 +706,9 @@ eval_call (expression *exp, enum noside noside,
         call an error.  This can happen if somebody tries to turn
         a variable into a function call.  */
 
-      type *ftype = value_type (argvec[0]);
+      type *ftype = value_type (callee);
 
-      if (TYPE_CODE (ftype) == TYPE_CODE_INTERNAL_FUNCTION)
+      if (ftype->code () == TYPE_CODE_INTERNAL_FUNCTION)
        {
          /* We don't know anything about what the internal
             function might return, but we have to return
@@ -793,18 +716,26 @@ eval_call (expression *exp, enum noside noside,
          return value_zero (builtin_type (exp->gdbarch)->builtin_int,
                             not_lval);
        }
-      else if (TYPE_CODE (ftype) == TYPE_CODE_XMETHOD)
+      else if (ftype->code () == TYPE_CODE_XMETHOD)
        {
-         type *return_type
-           = result_type_of_xmethod (argvec[0], nargs, argvec + 1);
+         type *return_type = result_type_of_xmethod (callee, argvec);
 
          if (return_type == NULL)
            error (_("Xmethod is missing return type."));
          return value_zero (return_type, not_lval);
        }
-      else if (TYPE_CODE (ftype) == TYPE_CODE_FUNC
-              || TYPE_CODE (ftype) == TYPE_CODE_METHOD)
+      else if (ftype->code () == TYPE_CODE_FUNC
+              || ftype->code () == TYPE_CODE_METHOD)
        {
+         if (ftype->is_gnu_ifunc ())
+           {
+             CORE_ADDR address = value_address (callee);
+             type *resolved_type = find_gnu_ifunc_target_type (address);
+
+             if (resolved_type != NULL)
+               ftype = resolved_type;
+           }
+
          type *return_type = TYPE_TARGET_TYPE (ftype);
 
          if (return_type == NULL)
@@ -819,16 +750,15 @@ eval_call (expression *exp, enum noside noside,
        error (_("Expression of type other than "
                 "\"Function returning ...\" used as function"));
     }
-  switch (TYPE_CODE (value_type (argvec[0])))
+  switch (value_type (callee)->code ())
     {
     case TYPE_CODE_INTERNAL_FUNCTION:
       return call_internal_function (exp->gdbarch, exp->language_defn,
-                                    argvec[0], nargs, argvec + 1);
+                                    callee, argvec.size (), argvec.data ());
     case TYPE_CODE_XMETHOD:
-      return call_xmethod (argvec[0], nargs, argvec + 1);
+      return call_xmethod (callee, argvec);
     default:
-      return call_function_by_hand (argvec[0], default_return_type,
-                                   nargs, argvec + 1);
+      return call_function_by_hand (callee, default_return_type, argvec);
     }
 }
 
@@ -867,7 +797,7 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
        }
       else
        {
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         arg2 = evaluate_subexp (nullptr, exp, pos, noside);
        }
 
       /* If the function is a virtual function, then the aggregate
@@ -875,13 +805,13 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
         the vtable.  Otherwise, it is just along for the ride: call
         the function directly.  */
 
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
 
       type *a1_type = check_typedef (value_type (arg1));
       if (noside == EVAL_SKIP)
        tem = 1;  /* Set it to the right arg index so that all
                     arguments can also be skipped.  */
-      else if (TYPE_CODE (a1_type) == TYPE_CODE_METHODPTR)
+      else if (a1_type->code () == TYPE_CODE_METHODPTR)
        {
          if (noside == EVAL_AVOID_SIDE_EFFECTS)
            arg1 = value_zero (TYPE_TARGET_TYPE (a1_type), not_lval);
@@ -893,7 +823,7 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
          tem = 2;
          argvec[1] = arg2;
        }
-      else if (TYPE_CODE (a1_type) == TYPE_CODE_MEMBERPTR)
+      else if (a1_type->code () == TYPE_CODE_MEMBERPTR)
        {
          struct type *type_ptr
            = lookup_pointer_type (TYPE_SELF_TYPE (a1_type));
@@ -944,7 +874,7 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
        }
       else
        {
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         arg2 = evaluate_subexp (nullptr, exp, pos, noside);
 
          /* Check to see if the operator '->' has been overloaded.
             If the operator has been overloaded replace arg2 with the
@@ -953,19 +883,18 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
          while (unop_user_defined_p (op, arg2))
            {
              struct value *value = NULL;
-             TRY
+             try
                {
                  value = value_x_unop (arg2, op, noside);
                }
 
-             CATCH (except, RETURN_MASK_ERROR)
+             catch (const gdb_exception_error &except)
                {
                  if (except.error == NOT_FOUND_ERROR)
                    break;
                  else
-                   throw_exception (except);
+                   throw;
                }
-             END_CATCH
 
                arg2 = value;
            }
@@ -990,23 +919,23 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
 
       function = NULL;
       function_name = NULL;
-      if (TYPE_CODE (type) == TYPE_CODE_NAMESPACE)
+      if (type->code () == TYPE_CODE_NAMESPACE)
        {
-         function = cp_lookup_symbol_namespace (TYPE_TAG_NAME (type),
+         function = cp_lookup_symbol_namespace (type->name (),
                                                 name,
                                                 get_selected_block (0),
                                                 VAR_DOMAIN).symbol;
          if (function == NULL)
            error (_("No symbol \"%s\" in namespace \"%s\"."),
-                  name, TYPE_TAG_NAME (type));
+                  name, type->name ());
 
          tem = 1;
          /* arg2 is left as NULL on purpose.  */
        }
       else
        {
-         gdb_assert (TYPE_CODE (type) == TYPE_CODE_STRUCT
-                     || TYPE_CODE (type) == TYPE_CODE_UNION);
+         gdb_assert (type->code () == TYPE_CODE_STRUCT
+                     || type->code () == TYPE_CODE_UNION);
          function_name = name;
 
          /* We need a properly typed value for method lookup.  For
@@ -1046,25 +975,24 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
        {
          if (op == OP_VAR_MSYM_VALUE)
            {
-             symbol *sym = exp->elts[*pos + 2].symbol;
-             var_func_name = SYMBOL_PRINT_NAME (sym);
+             minimal_symbol *msym = exp->elts[*pos + 2].msymbol;
+             var_func_name = msym->print_name ();
            }
          else if (op == OP_VAR_VALUE)
            {
-             minimal_symbol *msym = exp->elts[*pos + 2].msymbol;
-             var_func_name = MSYMBOL_PRINT_NAME (msym);
+             symbol *sym = exp->elts[*pos + 2].symbol;
+             var_func_name = sym->print_name ();
            }
 
          argvec[0] = evaluate_subexp_with_coercion (exp, pos, noside);
          type *type = value_type (argvec[0]);
-         if (type && TYPE_CODE (type) == TYPE_CODE_PTR)
+         if (type && type->code () == TYPE_CODE_PTR)
            type = TYPE_TARGET_TYPE (type);
-         if (type && TYPE_CODE (type) == TYPE_CODE_FUNC)
+         if (type && type->code () == TYPE_CODE_FUNC)
            {
-             for (; tem <= nargs && tem <= TYPE_NFIELDS (type); tem++)
+             for (; tem <= nargs && tem <= type->num_fields (); tem++)
                {
-                 argvec[tem] = evaluate_subexp (TYPE_FIELD_TYPE (type,
-                                                                 tem - 1),
+                 argvec[tem] = evaluate_subexp (type->field (tem - 1).type (),
                                                 exp, pos, noside);
                }
            }
@@ -1098,7 +1026,8 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
       func_name = (char *) alloca (name_len + 1);
       strcpy (func_name, &exp->elts[string_pc + 1].string);
 
-      find_overload_match (&argvec[1], nargs, func_name,
+      find_overload_match (gdb::make_array_view (&argvec[1], nargs),
+                          func_name,
                           NON_METHOD, /* not method */
                           NULL, NULL, /* pass NULL symbol since
                                          symbol is unknown */
@@ -1134,7 +1063,8 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
             evaluation.  */
          struct value *valp = NULL;
 
-         (void) find_overload_match (&argvec[1], nargs, tstr,
+         (void) find_overload_match (gdb::make_array_view (&argvec[1], nargs),
+                                     tstr,
                                      METHOD, /* method */
                                      &arg2,  /* the object */
                                      NULL, &valp, NULL,
@@ -1205,7 +1135,7 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
          if (op == OP_VAR_VALUE)
            function = exp->elts[save_pos1+2].symbol;
 
-         (void) find_overload_match (&argvec[1], nargs,
+         (void) find_overload_match (gdb::make_array_view (&argvec[1], nargs),
                                      NULL,        /* no need for name */
                                      NON_METHOD,  /* not method */
                                      NULL, function, /* the function */
@@ -1233,7 +1163,23 @@ evaluate_funcall (type *expect_type, expression *exp, int *pos,
       /* Nothing to be done; argvec already correctly set up.  */
     }
 
-  return eval_call (exp, noside, nargs, argvec, var_func_name, expect_type);
+  return evaluate_subexp_do_call (exp, noside, argvec[0],
+                                 gdb::make_array_view (argvec + 1, nargs),
+                                 var_func_name, expect_type);
+}
+
+/* Return true if type is integral or reference to integral */
+
+static bool
+is_integral_or_integral_reference (struct type *type)
+{
+  if (is_integral_type (type))
+    return true;
+
+  type = check_typedef (type);
+  return (type != nullptr
+         && TYPE_IS_REFERENCE (type)
+         && is_integral_type (TYPE_TARGET_TYPE (type)));
 }
 
 struct value *
@@ -1250,7 +1196,6 @@ evaluate_subexp_standard (struct type *expect_type,
   struct type *type;
   int nargs;
   struct value **argvec;
-  int code;
   int ix;
   long mem_offset;
   struct type **arg_types;
@@ -1284,16 +1229,19 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case OP_ADL_FUNC:
     case OP_VAR_VALUE:
-      (*pos) += 3;
-      if (noside == EVAL_SKIP)
-       return eval_skip_value (exp);
-
       {
+       (*pos) += 3;
        symbol *var = exp->elts[pc + 2].symbol;
-       if (TYPE_CODE (SYMBOL_TYPE (var)) == TYPE_CODE_ERROR)
-         error_unknown_type (SYMBOL_PRINT_NAME (var));
-
-       return evaluate_var_value (noside, exp->elts[pc + 1].block, var);
+       if (SYMBOL_TYPE (var)->code () == TYPE_CODE_ERROR)
+         error_unknown_type (var->print_name ());
+       if (noside != EVAL_SKIP)
+           return evaluate_var_value (noside, exp->elts[pc + 1].block, var);
+       else
+         {
+           /* Return a dummy value of the correct type when skipping, so
+              that parent functions know what is to be skipped.  */
+           return allocate_value (SYMBOL_TYPE (var));
+         }
       }
 
     case OP_VAR_MSYM_VALUE:
@@ -1306,9 +1254,9 @@ evaluate_subexp_standard (struct type *expect_type,
                                              msymbol);
 
        type = value_type (val);
-       if (TYPE_CODE (type) == TYPE_CODE_ERROR
+       if (type->code () == TYPE_CODE_ERROR
            && (noside != EVAL_AVOID_SIDE_EFFECTS || pc != 0))
-         error_unknown_type (MSYMBOL_PRINT_NAME (msymbol));
+         error_unknown_type (msymbol->print_name ());
        return val;
       }
 
@@ -1327,7 +1275,7 @@ evaluate_subexp_standard (struct type *expect_type,
        if (SYMBOL_COMPUTED_OPS (sym) == NULL
            || SYMBOL_COMPUTED_OPS (sym)->read_variable_at_entry == NULL)
          error (_("Symbol \"%s\" does not have any specific entry value"),
-                SYMBOL_PRINT_NAME (sym));
+                sym->print_name ());
 
        frame = get_selected_frame (NULL);
        return SYMBOL_COMPUTED_OPS (sym)->read_variable_at_entry (sym, frame);
@@ -1371,14 +1319,13 @@ evaluate_subexp_standard (struct type *expect_type,
        if (regno == -1)
          error (_("Register $%s not available."), name);
 
-        /* In EVAL_AVOID_SIDE_EFFECTS mode, we only need to return
-           a value with the appropriate register type.  Unfortunately,
-           we don't have easy access to the type of user registers.
-           So for these registers, we fetch the register value regardless
-           of the evaluation mode.  */
+       /* In EVAL_AVOID_SIDE_EFFECTS mode, we only need to return
+          a value with the appropriate register type.  Unfortunately,
+          we don't have easy access to the type of user registers.
+          So for these registers, we fetch the register value regardless
+          of the evaluation mode.  */
        if (noside == EVAL_AVOID_SIDE_EFFECTS
-           && regno < gdbarch_num_regs (exp->gdbarch)
-                       + gdbarch_num_pseudo_regs (exp->gdbarch))
+           && regno < gdbarch_num_cooked_regs (exp->gdbarch))
          val = value_zero (register_type (exp->gdbarch, regno), not_lval);
        else
          val = value_of_register (regno, get_selected_frame (NULL));
@@ -1418,10 +1365,10 @@ evaluate_subexp_standard (struct type *expect_type,
       tem2 = longest_to_int (exp->elts[pc + 1].longconst);
       tem3 = longest_to_int (exp->elts[pc + 2].longconst);
       nargs = tem3 - tem2 + 1;
-      type = expect_type ? check_typedef (expect_type) : NULL_TYPE;
+      type = expect_type ? check_typedef (expect_type) : nullptr;
 
-      if (expect_type != NULL_TYPE && noside != EVAL_SKIP
-         && TYPE_CODE (type) == TYPE_CODE_STRUCT)
+      if (expect_type != nullptr && noside != EVAL_SKIP
+         && type->code () == TYPE_CODE_STRUCT)
        {
          struct value *rec = allocate_value (expect_type);
 
@@ -1429,16 +1376,16 @@ evaluate_subexp_standard (struct type *expect_type,
          return evaluate_struct_tuple (rec, exp, pos, noside, nargs);
        }
 
-      if (expect_type != NULL_TYPE && noside != EVAL_SKIP
-         && TYPE_CODE (type) == TYPE_CODE_ARRAY)
+      if (expect_type != nullptr && noside != EVAL_SKIP
+         && type->code () == TYPE_CODE_ARRAY)
        {
-         struct type *range_type = TYPE_INDEX_TYPE (type);
+         struct type *range_type = type->index_type ();
          struct type *element_type = TYPE_TARGET_TYPE (type);
          struct value *array = allocate_value (expect_type);
          int element_size = TYPE_LENGTH (check_typedef (element_type));
          LONGEST low_bound, high_bound, index;
 
-         if (get_discrete_bounds (range_type, &low_bound, &high_bound) < 0)
+         if (!get_discrete_bounds (range_type, &low_bound, &high_bound))
            {
              low_bound = 0;
              high_bound = (TYPE_LENGTH (type) / element_size) - 1;
@@ -1448,50 +1395,37 @@ evaluate_subexp_standard (struct type *expect_type,
          for (tem = nargs; --nargs >= 0;)
            {
              struct value *element;
-             int index_pc = 0;
 
              element = evaluate_subexp (element_type, exp, pos, noside);
              if (value_type (element) != element_type)
                element = value_cast (element_type, element);
-             if (index_pc)
-               {
-                 int continue_pc = *pos;
-
-                 *pos = index_pc;
-                 index = init_array_element (array, element, exp, pos, noside,
-                                             low_bound, high_bound);
-                 *pos = continue_pc;
-               }
-             else
-               {
-                 if (index > high_bound)
-                   /* To avoid memory corruption.  */
-                   error (_("Too many array elements"));
-                 memcpy (value_contents_raw (array)
-                         + (index - low_bound) * element_size,
-                         value_contents (element),
-                         element_size);
-               }
+             if (index > high_bound)
+               /* To avoid memory corruption.  */
+               error (_("Too many array elements"));
+             memcpy (value_contents_raw (array)
+                     + (index - low_bound) * element_size,
+                     value_contents (element),
+                     element_size);
              index++;
            }
          return array;
        }
 
-      if (expect_type != NULL_TYPE && noside != EVAL_SKIP
-         && TYPE_CODE (type) == TYPE_CODE_SET)
+      if (expect_type != nullptr && noside != EVAL_SKIP
+         && type->code () == TYPE_CODE_SET)
        {
          struct value *set = allocate_value (expect_type);
          gdb_byte *valaddr = value_contents_raw (set);
-         struct type *element_type = TYPE_INDEX_TYPE (type);
+         struct type *element_type = type->index_type ();
          struct type *check_type = element_type;
          LONGEST low_bound, high_bound;
 
          /* Get targettype of elementtype.  */
-         while (TYPE_CODE (check_type) == TYPE_CODE_RANGE
-                || TYPE_CODE (check_type) == TYPE_CODE_TYPEDEF)
+         while (check_type->code () == TYPE_CODE_RANGE
+                || check_type->code () == TYPE_CODE_TYPEDEF)
            check_type = TYPE_TARGET_TYPE (check_type);
 
-         if (get_discrete_bounds (element_type, &low_bound, &high_bound) < 0)
+         if (!get_discrete_bounds (element_type, &low_bound, &high_bound))
            error (_("(power)set type with unknown size"));
          memset (valaddr, '\0', TYPE_LENGTH (type));
          for (tem = 0; tem < nargs; tem++)
@@ -1505,19 +1439,19 @@ evaluate_subexp_standard (struct type *expect_type,
              range_low = range_high = value_as_long (elem_val);
 
              /* Check types of elements to avoid mixture of elements from
-                different types. Also check if type of element is "compatible"
-                with element type of powerset.  */
-             if (TYPE_CODE (range_low_type) == TYPE_CODE_RANGE)
+                different types. Also check if type of element is "compatible"
+                with element type of powerset.  */
+             if (range_low_type->code () == TYPE_CODE_RANGE)
                range_low_type = TYPE_TARGET_TYPE (range_low_type);
-             if (TYPE_CODE (range_high_type) == TYPE_CODE_RANGE)
+             if (range_high_type->code () == TYPE_CODE_RANGE)
                range_high_type = TYPE_TARGET_TYPE (range_high_type);
-             if ((TYPE_CODE (range_low_type) != TYPE_CODE (range_high_type))
-                 || (TYPE_CODE (range_low_type) == TYPE_CODE_ENUM
+             if ((range_low_type->code () != range_high_type->code ())
+                 || (range_low_type->code () == TYPE_CODE_ENUM
                      && (range_low_type != range_high_type)))
                /* different element modes.  */
                error (_("POWERSET tuple elements of different mode"));
-             if ((TYPE_CODE (check_type) != TYPE_CODE (range_low_type))
-                 || (TYPE_CODE (check_type) == TYPE_CODE_ENUM
+             if ((check_type->code () != range_low_type->code ())
+                 || (check_type->code () == TYPE_CODE_ENUM
                      && range_low_type != check_type))
                error (_("incompatible POWERSET tuple elements"));
              if (range_low > range_high)
@@ -1533,7 +1467,7 @@ evaluate_subexp_standard (struct type *expect_type,
                {
                  int bit_index = (unsigned) range_low % TARGET_CHAR_BIT;
 
-                 if (gdbarch_bits_big_endian (exp->gdbarch))
+                 if (gdbarch_byte_order (exp->gdbarch) == BFD_ENDIAN_BIG)
                    bit_index = TARGET_CHAR_BIT - 1 - bit_index;
                  valaddr[(unsigned) range_low / TARGET_CHAR_BIT]
                    |= 1 << bit_index;
@@ -1555,11 +1489,10 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case TERNOP_SLICE:
       {
-       struct value *array = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+       struct value *array = evaluate_subexp (nullptr, exp, pos, noside);
        int lowbound
-         = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
-       int upper
-         = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
+         = value_as_long (evaluate_subexp (nullptr, exp, pos, noside));
+       int upper = value_as_long (evaluate_subexp (nullptr, exp, pos, noside));
 
        if (noside == EVAL_SKIP)
          return eval_skip_value (exp);
@@ -1568,16 +1501,16 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case TERNOP_COND:
       /* Skip third and second args to evaluate the first one.  */
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (value_logical_not (arg1))
        {
-         evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
-         return evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         evaluate_subexp (nullptr, exp, pos, EVAL_SKIP);
+         return evaluate_subexp (nullptr, exp, pos, noside);
        }
       else
        {
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-         evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
+         arg2 = evaluate_subexp (nullptr, exp, pos, noside);
+         evaluate_subexp (nullptr, exp, pos, EVAL_SKIP);
          return arg2;
        }
 
@@ -1656,7 +1589,7 @@ evaluate_subexp_standard (struct type *expect_type,
           only).  */
        if (gnu_runtime)
          {
-           struct type *type = selector_type;
+           type = selector_type;
 
            type = lookup_function_type (type);
            type = lookup_pointer_type (type);
@@ -1711,12 +1644,12 @@ evaluate_subexp_standard (struct type *expect_type,
        argvec[3] = value_from_longest (long_type, selector);
        argvec[4] = 0;
 
-       ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
+       ret = call_function_by_hand (argvec[0], NULL, {argvec + 1, 3});
        if (gnu_runtime)
          {
            /* Function objc_msg_lookup returns a pointer.  */
            argvec[0] = ret;
-           ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
+           ret = call_function_by_hand (argvec[0], NULL, {argvec + 1, 3});
          }
        if (value_as_long (ret) == 0)
          error (_("Target does not respond to this message selector."));
@@ -1733,11 +1666,11 @@ evaluate_subexp_standard (struct type *expect_type,
        argvec[3] = value_from_longest (long_type, selector);
        argvec[4] = 0;
 
-       ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
+       ret = call_function_by_hand (argvec[0], NULL, {argvec + 1, 3});
        if (gnu_runtime)
          {
            argvec[0] = ret;
-           ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
+           ret = call_function_by_hand (argvec[0], NULL, {argvec + 1, 3});
          }
 
        /* ret should now be the selector.  */
@@ -1750,7 +1683,7 @@ evaluate_subexp_standard (struct type *expect_type,
            /* The address might point to a function descriptor;
               resolve it to the actual code address instead.  */
            addr = gdbarch_convert_from_func_ptr_addr (exp->gdbarch, addr,
-                                                      &current_target);
+                                                      current_top_target ());
 
            /* Is it a high_level symbol?  */
            sym = find_pc_function (addr);
@@ -1759,7 +1692,7 @@ evaluate_subexp_standard (struct type *expect_type,
          }
 
        /* If we found a method with symbol information, check to see
-           if it returns a struct.  Otherwise assume it doesn't.  */
+          if it returns a struct.  Otherwise assume it doesn't.  */
 
        if (method)
          {
@@ -1771,9 +1704,9 @@ evaluate_subexp_standard (struct type *expect_type,
            block_for_pc (funaddr);
 
            val_type = check_typedef (val_type);
-         
-           if ((val_type == NULL) 
-               || (TYPE_CODE(val_type) == TYPE_CODE_ERROR))
+
+           if ((val_type == NULL)
+               || (val_type->code () == TYPE_CODE_ERROR))
              {
                if (expect_type != NULL)
                  val_type = expect_type;
@@ -1787,7 +1720,7 @@ evaluate_subexp_standard (struct type *expect_type,
            struct_return = using_struct_return (exp->gdbarch, NULL,
                                                 check_typedef (expect_type));
          }
-       
+
        /* Found a function symbol.  Now we will substitute its
           value in place of the message dispatcher (obj_msgSend),
           so that we call the method directly instead of thru
@@ -1803,7 +1736,7 @@ evaluate_subexp_standard (struct type *expect_type,
        
        if (method)
          {
-           if (TYPE_CODE (value_type (method)) != TYPE_CODE_FUNC)
+           if (value_type (method)->code () != TYPE_CODE_FUNC)
              error (_("method address has symbol information "
                       "with non-function type; skipping"));
 
@@ -1843,18 +1776,18 @@ evaluate_subexp_standard (struct type *expect_type,
               it's opinion (ie. through "whatis"), it won't offer
               it.  */
 
-           struct type *type = value_type (called_method);
+           struct type *callee_type = value_type (called_method);
 
-           if (type && TYPE_CODE (type) == TYPE_CODE_PTR)
-             type = TYPE_TARGET_TYPE (type);
-           type = TYPE_TARGET_TYPE (type);
+           if (callee_type && callee_type->code () == TYPE_CODE_PTR)
+             callee_type = TYPE_TARGET_TYPE (callee_type);
+           callee_type = TYPE_TARGET_TYPE (callee_type);
 
-           if (type)
+           if (callee_type)
            {
-             if ((TYPE_CODE (type) == TYPE_CODE_ERROR) && expect_type)
+             if ((callee_type->code () == TYPE_CODE_ERROR) && expect_type)
                return allocate_value (expect_type);
              else
-               return allocate_value (type);
+               return allocate_value (callee_type);
            }
            else
              error (_("Expression of type other than "
@@ -1873,108 +1806,36 @@ evaluate_subexp_standard (struct type *expect_type,
          argvec[tem + 3] = evaluate_subexp_with_coercion (exp, pos, noside);
        argvec[tem + 3] = 0;
 
+       auto call_args = gdb::make_array_view (argvec + 1, nargs + 2);
+
        if (gnu_runtime && (method != NULL))
          {
            /* Function objc_msg_lookup returns a pointer.  */
            deprecated_set_value_type (argvec[0],
                                       lookup_pointer_type (lookup_function_type (value_type (argvec[0]))));
-           argvec[0]
-             = call_function_by_hand (argvec[0], NULL, nargs + 2, argvec + 1);
+           argvec[0] = call_function_by_hand (argvec[0], NULL, call_args);
          }
 
-       ret = call_function_by_hand (argvec[0], NULL, nargs + 2, argvec + 1);
-       return ret;
+       return call_function_by_hand (argvec[0], NULL, call_args);
       }
       break;
 
     case OP_FUNCALL:
       return evaluate_funcall (expect_type, exp, pos, noside);
 
-    case OP_F77_UNDETERMINED_ARGLIST:
-
-      /* Remember that in F77, functions, substring ops and 
-         array subscript operations cannot be disambiguated 
-         at parse time.  We have made all array subscript operations, 
-         substring operations as well as function calls  come here 
-         and we now have to discover what the heck this thing actually was.
-         If it is a function, we process just as if we got an OP_FUNCALL.  */
-
-      nargs = longest_to_int (exp->elts[pc + 1].longconst);
-      (*pos) += 2;
-
-      /* First determine the type code we are dealing with.  */
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      type = check_typedef (value_type (arg1));
-      code = TYPE_CODE (type);
-
-      if (code == TYPE_CODE_PTR)
-       {
-         /* Fortran always passes variable to subroutines as pointer.
-            So we need to look into its target type to see if it is
-            array, string or function.  If it is, we need to switch
-            to the target value the original one points to.  */ 
-         struct type *target_type = check_typedef (TYPE_TARGET_TYPE (type));
-
-         if (TYPE_CODE (target_type) == TYPE_CODE_ARRAY
-             || TYPE_CODE (target_type) == TYPE_CODE_STRING
-             || TYPE_CODE (target_type) == TYPE_CODE_FUNC)
-           {
-             arg1 = value_ind (arg1);
-             type = check_typedef (value_type (arg1));
-             code = TYPE_CODE (type);
-           }
-       } 
-
-      switch (code)
-       {
-       case TYPE_CODE_ARRAY:
-         if (exp->elts[*pos].opcode == OP_RANGE)
-           return value_f90_subarray (arg1, exp, pos, noside);
-         else
-           goto multi_f77_subscript;
-
-       case TYPE_CODE_STRING:
-         if (exp->elts[*pos].opcode == OP_RANGE)
-           return value_f90_subarray (arg1, exp, pos, noside);
-         else
-           {
-             arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
-             return value_subscript (arg1, value_as_long (arg2));
-           }
-
-       case TYPE_CODE_PTR:
-       case TYPE_CODE_FUNC:
-         /* It's a function call.  */
-         /* Allocate arg vector, including space for the function to be
-            called in argvec[0] and a terminating NULL.  */
-         argvec = (struct value **)
-           alloca (sizeof (struct value *) * (nargs + 2));
-         argvec[0] = arg1;
-         tem = 1;
-         for (; tem <= nargs; tem++)
-           argvec[tem] = evaluate_subexp_with_coercion (exp, pos, noside);
-         argvec[tem] = 0;      /* signal end of arglist */
-         if (noside == EVAL_SKIP)
-           return eval_skip_value (exp);
-         return eval_call (exp, noside, nargs, argvec, NULL, expect_type);
-
-       default:
-         error (_("Cannot perform substring on this type"));
-       }
-
     case OP_COMPLEX:
       /* We have a complex number, There should be 2 floating 
-         point numbers that compose it.  */
+        point numbers that compose it.  */
       (*pos) += 2;
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
+      arg2 = evaluate_subexp (nullptr, exp, pos, noside);
 
       return value_literal_complex (arg1, arg2, exp->elts[pc + 1].type);
 
     case STRUCTOP_STRUCT:
       tem = longest_to_int (exp->elts[pc + 1].longconst);
       (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       arg3 = value_struct_elt (&arg1, NULL, &exp->elts[pc + 2].string,
@@ -1986,28 +1847,27 @@ evaluate_subexp_standard (struct type *expect_type,
     case STRUCTOP_PTR:
       tem = longest_to_int (exp->elts[pc + 1].longconst);
       (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
 
       /* Check to see if operator '->' has been overloaded.  If so replace
-         arg1 with the value returned by evaluating operator->().  */
+        arg1 with the value returned by evaluating operator->().  */
       while (unop_user_defined_p (op, arg1))
        {
          struct value *value = NULL;
-         TRY
+         try
            {
              value = value_x_unop (arg1, op, noside);
            }
 
-         CATCH (except, RETURN_MASK_ERROR)
+         catch (const gdb_exception_error &except)
            {
              if (except.error == NOT_FOUND_ERROR)
                break;
              else
-               throw_exception (except);
+               throw;
            }
-         END_CATCH
 
          arg1 = value;
        }
@@ -2016,21 +1876,21 @@ evaluate_subexp_standard (struct type *expect_type,
         with rtti type in order to continue on with successful
         lookup of member / method only available in the rtti type.  */
       {
-        struct type *type = value_type (arg1);
-        struct type *real_type;
-        int full, using_enc;
-        LONGEST top;
+       struct type *arg_type = value_type (arg1);
+       struct type *real_type;
+       int full, using_enc;
+       LONGEST top;
        struct value_print_options opts;
 
        get_user_print_options (&opts);
-        if (opts.objectprint && TYPE_TARGET_TYPE(type)
-            && (TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_STRUCT))
-          {
-            real_type = value_rtti_indirect_type (arg1, &full, &top,
+       if (opts.objectprint && TYPE_TARGET_TYPE (arg_type)
+           && (TYPE_TARGET_TYPE (arg_type)->code () == TYPE_CODE_STRUCT))
+         {
+           real_type = value_rtti_indirect_type (arg1, &full, &top,
                                                  &using_enc);
-            if (real_type)
-                arg1 = value_cast (real_type, arg1);
-          }
+           if (real_type)
+               arg1 = value_cast (real_type, arg1);
+         }
       }
 
       arg3 = value_struct_elt (&arg1, NULL, &exp->elts[pc + 2].string,
@@ -2044,15 +1904,15 @@ evaluate_subexp_standard (struct type *expect_type,
       if (op == STRUCTOP_MEMBER)
        arg1 = evaluate_subexp_for_address (exp, pos, noside);
       else
-       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+       arg1 = evaluate_subexp (nullptr, exp, pos, noside);
 
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg2 = evaluate_subexp (nullptr, exp, pos, noside);
 
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
 
       type = check_typedef (value_type (arg2));
-      switch (TYPE_CODE (type))
+      switch (type->code ())
        {
        case TYPE_CODE_METHODPTR:
          if (noside == EVAL_AVOID_SIDE_EFFECTS)
@@ -2060,7 +1920,7 @@ evaluate_subexp_standard (struct type *expect_type,
          else
            {
              arg2 = cplus_method_ptr_to_value (&arg1, arg2);
-             gdb_assert (TYPE_CODE (value_type (arg2)) == TYPE_CODE_PTR);
+             gdb_assert (value_type (arg2)->code () == TYPE_CODE_PTR);
              return value_ind (arg2);
            }
 
@@ -2089,9 +1949,10 @@ evaluate_subexp_standard (struct type *expect_type,
        for (ix = 0; ix < nargs; ++ix)
          arg_types[ix] = exp->elts[pc + 2 + ix + 1].type;
 
-       fake_method expect_type (flags, nargs, arg_types);
+       fake_method fake_expect_type (flags, nargs, arg_types);
        *(pos) += 4 + nargs;
-       return evaluate_subexp_standard (expect_type.type (), exp, pos, noside);
+       return evaluate_subexp_standard (fake_expect_type.type (), exp, pos,
+                                        noside);
       }
 
     case BINOP_CONCAT:
@@ -2105,8 +1966,16 @@ evaluate_subexp_standard (struct type *expect_type,
        return value_concat (arg1, arg2);
 
     case BINOP_ASSIGN:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
+      /* Special-case assignments where the left-hand-side is a
+        convenience variable -- in these, don't bother setting an
+        expected type.  This avoids a weird case where re-assigning a
+        string or array to an internal variable could error with "Too
+        many array elements".  */
+      arg2 = evaluate_subexp (VALUE_LVAL (arg1) == lval_internalvar
+                               ? nullptr
+                               : value_type (arg1),
+                             exp, pos, noside);
 
       if (noside == EVAL_SKIP || noside == EVAL_AVOID_SIDE_EFFECTS)
        return arg1;
@@ -2117,7 +1986,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_ASSIGN_MODIFY:
       (*pos) += 2;
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP || noside == EVAL_AVOID_SIDE_EFFECTS)
        return arg1;
@@ -2156,10 +2025,10 @@ evaluate_subexp_standard (struct type *expect_type,
       if (binop_user_defined_p (op, arg1, arg2))
        return value_x_binop (arg1, arg2, op, OP_NULL, noside);
       else if (ptrmath_type_p (exp->language_defn, value_type (arg1))
-              && is_integral_type (value_type (arg2)))
+              && is_integral_or_integral_reference (value_type (arg2)))
        return value_ptradd (arg1, value_as_long (arg2));
       else if (ptrmath_type_p (exp->language_defn, value_type (arg2))
-              && is_integral_type (value_type (arg1)))
+              && is_integral_or_integral_reference (value_type (arg1)))
        return value_ptradd (arg2, value_as_long (arg1));
       else
        {
@@ -2182,7 +2051,7 @@ evaluate_subexp_standard (struct type *expect_type,
          return value_from_longest (type, value_ptrdiff (arg1, arg2));
        }
       else if (ptrmath_type_p (exp->language_defn, value_type (arg1))
-              && is_integral_type (value_type (arg2)))
+              && is_integral_or_integral_reference (value_type (arg2)))
        return value_ptradd (arg1, - value_as_long (arg2));
       else
        {
@@ -2201,8 +2070,8 @@ evaluate_subexp_standard (struct type *expect_type,
     case BINOP_BITWISE_AND:
     case BINOP_BITWISE_IOR:
     case BINOP_BITWISE_XOR:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
+      arg2 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
@@ -2222,12 +2091,11 @@ evaluate_subexp_standard (struct type *expect_type,
                  || op == BINOP_MOD)
              && value_logical_not (arg2))
            {
-             struct value *v_one, *retval;
+             struct value *v_one;
 
              v_one = value_one (value_type (arg2));
              binop_promote (exp->language_defn, exp->gdbarch, &arg1, &v_one);
-             retval = value_binop (arg1, v_one, op);
-             return retval;
+             return value_binop (arg1, v_one, op);
            }
          else
            {
@@ -2244,8 +2112,8 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_SUBSCRIPT:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
+      arg2 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
@@ -2258,12 +2126,12 @@ evaluate_subexp_standard (struct type *expect_type,
 
          arg1 = coerce_ref (arg1);
          type = check_typedef (value_type (arg1));
-         if (TYPE_CODE (type) != TYPE_CODE_ARRAY
-             && TYPE_CODE (type) != TYPE_CODE_PTR)
+         if (type->code () != TYPE_CODE_ARRAY
+             && type->code () != TYPE_CODE_PTR)
            {
-             if (TYPE_NAME (type))
+             if (type->name ())
                error (_("cannot subscript something of type `%s'"),
-                      TYPE_NAME (type));
+                      type->name ());
              else
                error (_("cannot subscript requested type"));
            }
@@ -2277,36 +2145,14 @@ evaluate_subexp_standard (struct type *expect_type,
       (*pos) += 2;
       nargs = longest_to_int (exp->elts[pc + 1].longconst);
       arg1 = evaluate_subexp_with_coercion (exp, pos, noside);
-      while (nargs-- > 0)
+      argvec = XALLOCAVEC (struct value *, nargs);
+      for (ix = 0; ix < nargs; ++ix)
+       argvec[ix] = evaluate_subexp_with_coercion (exp, pos, noside);
+      if (noside == EVAL_SKIP)
+       return arg1;
+      for (ix = 0; ix < nargs; ++ix)
        {
-         arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
-         /* FIXME:  EVAL_SKIP handling may not be correct.  */
-         if (noside == EVAL_SKIP)
-           {
-             if (nargs > 0)
-               continue;
-             return eval_skip_value (exp);
-           }
-         /* FIXME:  EVAL_AVOID_SIDE_EFFECTS handling may not be correct.  */
-         if (noside == EVAL_AVOID_SIDE_EFFECTS)
-           {
-             /* If the user attempts to subscript something that has no target
-                type (like a plain int variable for example), then report this
-                as an error.  */
-
-             type = TYPE_TARGET_TYPE (check_typedef (value_type (arg1)));
-             if (type != NULL)
-               {
-                 arg1 = value_zero (type, VALUE_LVAL (arg1));
-                 noside = EVAL_SKIP;
-                 continue;
-               }
-             else
-               {
-                 error (_("cannot subscript something of type `%s'"),
-                        TYPE_NAME (value_type (arg1)));
-               }
-           }
+         arg2 = argvec[ix];
 
          if (binop_user_defined_p (op, arg1, arg2))
            {
@@ -2317,7 +2163,7 @@ evaluate_subexp_standard (struct type *expect_type,
              arg1 = coerce_ref (arg1);
              type = check_typedef (value_type (arg1));
 
-             switch (TYPE_CODE (type))
+             switch (type->code ())
                {
                case TYPE_CODE_PTR:
                case TYPE_CODE_ARRAY:
@@ -2326,9 +2172,9 @@ evaluate_subexp_standard (struct type *expect_type,
                  break;
 
                default:
-                 if (TYPE_NAME (type))
+                 if (type->name ())
                    error (_("cannot subscript something of type `%s'"),
-                          TYPE_NAME (type));
+                          type->name ());
                  else
                    error (_("cannot subscript requested type"));
                }
@@ -2336,105 +2182,62 @@ evaluate_subexp_standard (struct type *expect_type,
        }
       return (arg1);
 
-    multi_f77_subscript:
-      {
-       LONGEST subscript_array[MAX_FORTRAN_DIMS];
-       int ndimensions = 1, i;
-       struct value *array = arg1;
-
-       if (nargs > MAX_FORTRAN_DIMS)
-         error (_("Too many subscripts for F77 (%d Max)"), MAX_FORTRAN_DIMS);
-
-       ndimensions = calc_f77_array_dims (type);
-
-       if (nargs != ndimensions)
-         error (_("Wrong number of subscripts"));
-
-       gdb_assert (nargs > 0);
-
-       /* Now that we know we have a legal array subscript expression 
-          let us actually find out where this element exists in the array.  */
-
-       /* Take array indices left to right.  */
-       for (i = 0; i < nargs; i++)
-         {
-           /* Evaluate each subscript; it must be a legal integer in F77.  */
-           arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
-
-           /* Fill in the subscript array.  */
-
-           subscript_array[i] = value_as_long (arg2);
-         }
-
-       /* Internal type of array is arranged right to left.  */
-       for (i = nargs; i > 0; i--)
-         {
-           struct type *array_type = check_typedef (value_type (array));
-           LONGEST index = subscript_array[i - 1];
-
-           array = value_subscripted_rvalue (array, index,
-                                             f77_get_lowerbound (array_type));
-         }
-
-       return array;
-      }
-
     case BINOP_LOGICAL_AND:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        {
-         evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         evaluate_subexp (nullptr, exp, pos, noside);
          return eval_skip_value (exp);
        }
 
       oldpos = *pos;
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
+      arg2 = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       *pos = oldpos;
 
       if (binop_user_defined_p (op, arg1, arg2))
        {
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         arg2 = evaluate_subexp (nullptr, exp, pos, noside);
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
        }
       else
        {
          tem = value_logical_not (arg1);
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos,
-                                 (tem ? EVAL_SKIP : noside));
+         arg2
+           = evaluate_subexp (nullptr, exp, pos, (tem ? EVAL_SKIP : noside));
          type = language_bool_type (exp->language_defn, exp->gdbarch);
          return value_from_longest (type,
                             (LONGEST) (!tem && !value_logical_not (arg2)));
        }
 
     case BINOP_LOGICAL_OR:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        {
-         evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         evaluate_subexp (nullptr, exp, pos, noside);
          return eval_skip_value (exp);
        }
 
       oldpos = *pos;
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
+      arg2 = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       *pos = oldpos;
 
       if (binop_user_defined_p (op, arg1, arg2))
        {
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         arg2 = evaluate_subexp (nullptr, exp, pos, noside);
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
        }
       else
        {
          tem = value_logical_not (arg1);
-         arg2 = evaluate_subexp (NULL_TYPE, exp, pos,
-                                 (!tem ? EVAL_SKIP : noside));
+         arg2
+           = evaluate_subexp (nullptr, exp, pos, (!tem ? EVAL_SKIP : noside));
          type = language_bool_type (exp->language_defn, exp->gdbarch);
          return value_from_longest (type,
                             (LONGEST) (!tem || !value_logical_not (arg2)));
        }
 
     case BINOP_EQUAL:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
@@ -2451,7 +2254,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_NOTEQUAL:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
@@ -2468,7 +2271,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_LESS:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
@@ -2485,7 +2288,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_GTR:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
@@ -2502,7 +2305,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_GEQ:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
@@ -2519,7 +2322,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_LEQ:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
@@ -2536,13 +2339,13 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case BINOP_REPEAT:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
+      arg2 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       type = check_typedef (value_type (arg2));
-      if (TYPE_CODE (type) != TYPE_CODE_INT
-          && TYPE_CODE (type) != TYPE_CODE_ENUM)
+      if (type->code () != TYPE_CODE_INT
+         && type->code () != TYPE_CODE_ENUM)
        error (_("Non-integral right operand for \"@\" operator."));
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
@@ -2553,11 +2356,11 @@ evaluate_subexp_standard (struct type *expect_type,
        return value_repeat (arg1, longest_to_int (value_as_long (arg2)));
 
     case BINOP_COMMA:
-      evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      return evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      evaluate_subexp (nullptr, exp, pos, noside);
+      return evaluate_subexp (nullptr, exp, pos, noside);
 
     case UNOP_PLUS:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
@@ -2569,7 +2372,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
       
     case UNOP_NEG:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
@@ -2583,7 +2386,7 @@ evaluate_subexp_standard (struct type *expect_type,
     case UNOP_COMPLEMENT:
       /* C++: check for and handle destructor names.  */
 
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       if (unop_user_defined_p (UNOP_COMPLEMENT, arg1))
@@ -2595,7 +2398,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case UNOP_LOGICAL_NOT:
-      arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
@@ -2607,12 +2410,12 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case UNOP_IND:
-      if (expect_type && TYPE_CODE (expect_type) == TYPE_CODE_PTR)
+      if (expect_type && expect_type->code () == TYPE_CODE_PTR)
        expect_type = TYPE_TARGET_TYPE (check_typedef (expect_type));
       arg1 = evaluate_subexp (expect_type, exp, pos, noside);
       type = check_typedef (value_type (arg1));
-      if (TYPE_CODE (type) == TYPE_CODE_METHODPTR
-         || TYPE_CODE (type) == TYPE_CODE_MEMBERPTR)
+      if (type->code () == TYPE_CODE_METHODPTR
+         || type->code () == TYPE_CODE_MEMBERPTR)
        error (_("Attempt to dereference pointer "
                 "to member without an object"));
       if (noside == EVAL_SKIP)
@@ -2622,26 +2425,36 @@ evaluate_subexp_standard (struct type *expect_type,
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
          type = check_typedef (value_type (arg1));
-         if (TYPE_CODE (type) == TYPE_CODE_PTR
-             || TYPE_IS_REFERENCE (type)
-         /* In C you can dereference an array to get the 1st elt.  */
-             || TYPE_CODE (type) == TYPE_CODE_ARRAY
-           )
-           return value_zero (TYPE_TARGET_TYPE (type),
-                              lval_memory);
-         else if (TYPE_CODE (type) == TYPE_CODE_INT)
-           /* GDB allows dereferencing an int.  */
-           return value_zero (builtin_type (exp->gdbarch)->builtin_int,
-                              lval_memory);
-         else
-           error (_("Attempt to take contents of a non-pointer value."));
+
+         /* If the type pointed to is dynamic then in order to resolve the
+            dynamic properties we must actually dereference the pointer.
+            There is a risk that this dereference will have side-effects
+            in the inferior, but being able to print accurate type
+            information seems worth the risk. */
+         if ((type->code () != TYPE_CODE_PTR
+              && !TYPE_IS_REFERENCE (type))
+             || !is_dynamic_type (TYPE_TARGET_TYPE (type)))
+           {
+             if (type->code () == TYPE_CODE_PTR
+                 || TYPE_IS_REFERENCE (type)
+                 /* In C you can dereference an array to get the 1st elt.  */
+                 || type->code () == TYPE_CODE_ARRAY)
+               return value_zero (TYPE_TARGET_TYPE (type),
+                                  lval_memory);
+             else if (type->code () == TYPE_CODE_INT)
+               /* GDB allows dereferencing an int.  */
+               return value_zero (builtin_type (exp->gdbarch)->builtin_int,
+                                  lval_memory);
+             else
+               error (_("Attempt to take contents of a non-pointer value."));
+           }
        }
 
       /* Allow * on an integer so we can cast it to whatever we want.
         This returns an int, which seems like the most C-like thing to
         do.  "long long" variables are rare enough that
         BUILTIN_TYPE_LONGEST would seem to be a mistake.  */
-      if (TYPE_CODE (type) == TYPE_CODE_INT)
+      if (type->code () == TYPE_CODE_INT)
        return value_at_lazy (builtin_type (exp->gdbarch)->builtin_int,
                              (CORE_ADDR) value_as_address (arg1));
       return value_ind (arg1);
@@ -2651,25 +2464,32 @@ evaluate_subexp_standard (struct type *expect_type,
 
       if (noside == EVAL_SKIP)
        {
-         evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
+         evaluate_subexp (nullptr, exp, pos, EVAL_SKIP);
          return eval_skip_value (exp);
        }
       else
-       {
-         struct value *retvalp = evaluate_subexp_for_address (exp, pos,
-                                                              noside);
-
-         return retvalp;
-       }
+       return evaluate_subexp_for_address (exp, pos, noside);
 
     case UNOP_SIZEOF:
       if (noside == EVAL_SKIP)
        {
-         evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
+         evaluate_subexp (nullptr, exp, pos, EVAL_SKIP);
          return eval_skip_value (exp);
        }
       return evaluate_subexp_for_sizeof (exp, pos, noside);
 
+    case UNOP_ALIGNOF:
+      {
+       type = value_type (
+         evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS));
+       /* FIXME: This should be size_t.  */
+       struct type *size_type = builtin_type (exp->gdbarch)->builtin_int;
+       ULONGEST align = type_align (type);
+       if (align == 0)
+         error (_("could not determine alignment of type"));
+       return value_from_longest (size_type, align);
+      }
+
     case UNOP_CAST:
       (*pos) += 2;
       type = exp->elts[pc + 1].type;
@@ -2826,20 +2646,20 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case OP_TYPE:
       /* The value is not supposed to be used.  This is here to make it
-         easier to accommodate expressions that contain types.  */
+        easier to accommodate expressions that contain types.  */
       (*pos) += 2;
       if (noside == EVAL_SKIP)
        return eval_skip_value (exp);
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
        return allocate_value (exp->elts[pc + 1].type);
       else
-        error (_("Attempt to use a type name as an expression"));
+       error (_("Attempt to use a type name as an expression"));
 
     case OP_TYPEOF:
     case OP_DECLTYPE:
       if (noside == EVAL_SKIP)
        {
-         evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
+         evaluate_subexp (nullptr, exp, pos, EVAL_SKIP);
          return eval_skip_value (exp);
        }
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
@@ -2847,8 +2667,7 @@ evaluate_subexp_standard (struct type *expect_type,
          enum exp_opcode sub_op = exp->elts[*pos].opcode;
          struct value *result;
 
-         result = evaluate_subexp (NULL_TYPE, exp, pos,
-                                   EVAL_AVOID_SIDE_EFFECTS);
+         result = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
 
          /* 'decltype' has special semantics for lvalues.  */
          if (op == OP_DECLTYPE
@@ -2860,7 +2679,7 @@ evaluate_subexp_standard (struct type *expect_type,
                  || sub_op == STRUCTOP_PTR
                  || sub_op == OP_SCOPE))
            {
-             struct type *type = value_type (result);
+             type = value_type (result);
 
              if (!TYPE_IS_REFERENCE (type))
                {
@@ -2872,7 +2691,7 @@ evaluate_subexp_standard (struct type *expect_type,
          return result;
        }
       else
-        error (_("Attempt to use a type as an expression"));
+       error (_("Attempt to use a type as an expression"));
 
     case OP_TYPEID:
       {
@@ -2880,10 +2699,9 @@ evaluate_subexp_standard (struct type *expect_type,
        enum exp_opcode sub_op = exp->elts[*pos].opcode;
 
        if (sub_op == OP_TYPE || sub_op == OP_DECLTYPE || sub_op == OP_TYPEOF)
-         result = evaluate_subexp (NULL_TYPE, exp, pos,
-                                   EVAL_AVOID_SIDE_EFFECTS);
+         result = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
        else
-         result = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+         result = evaluate_subexp (nullptr, exp, pos, noside);
 
        if (noside != EVAL_NORMAL)
          return allocate_value (cplus_typeid_type (exp->gdbarch));
@@ -2893,13 +2711,13 @@ evaluate_subexp_standard (struct type *expect_type,
 
     default:
       /* Removing this case and compiling with gcc -Wall reveals that
-         a lot of cases are hitting this case.  Some of these should
-         probably be removed from expression.h; others are legitimate
-         expressions which are (apparently) not fully implemented.
+        a lot of cases are hitting this case.  Some of these should
+        probably be removed from expression.h; others are legitimate
+        expressions which are (apparently) not fully implemented.
 
-         If there are any cases landing here which mean a user error,
-         then they should be separate cases, with more descriptive
-         error messages.  */
+        If there are any cases landing here which mean a user error,
+        then they should be separate cases, with more descriptive
+        error messages.  */
 
       error (_("GDB does not (yet) know how to "
               "evaluate that kind of expression"));
@@ -2932,7 +2750,7 @@ evaluate_subexp_for_address (struct expression *exp, int *pos,
     {
     case UNOP_IND:
       (*pos)++;
-      x = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      x = evaluate_subexp (nullptr, exp, pos, noside);
 
       /* We can't optimize out "&*" if there's a user-defined operator*.  */
       if (unop_user_defined_p (op, x))
@@ -2946,17 +2764,17 @@ evaluate_subexp_for_address (struct expression *exp, int *pos,
     case UNOP_MEMVAL:
       (*pos) += 3;
       return value_cast (lookup_pointer_type (exp->elts[pc + 1].type),
-                        evaluate_subexp (NULL_TYPE, exp, pos, noside));
+                        evaluate_subexp (nullptr, exp, pos, noside));
 
     case UNOP_MEMVAL_TYPE:
       {
        struct type *type;
 
        (*pos) += 1;
-       x = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
+       x = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
        type = value_type (x);
        return value_cast (lookup_pointer_type (type),
-                          evaluate_subexp (NULL_TYPE, exp, pos, noside));
+                          evaluate_subexp (nullptr, exp, pos, noside));
       }
 
     case OP_VAR_VALUE:
@@ -3013,7 +2831,7 @@ evaluate_subexp_for_address (struct expression *exp, int *pos,
 
     default:
     default_case:
-      x = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      x = evaluate_subexp (nullptr, exp, pos, noside);
     default_case_after_eval:
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
@@ -3062,8 +2880,8 @@ evaluate_subexp_with_coercion (struct expression *exp,
     case OP_VAR_VALUE:
       var = exp->elts[pc + 2].symbol;
       type = check_typedef (SYMBOL_TYPE (var));
-      if (TYPE_CODE (type) == TYPE_CODE_ARRAY
-         && !TYPE_VECTOR (type)
+      if (type->code () == TYPE_CODE_ARRAY
+         && !type->is_vector ()
          && CAST_IS_CONVERSION (exp->language_defn))
        {
          (*pos) += 4;
@@ -3074,7 +2892,7 @@ evaluate_subexp_with_coercion (struct expression *exp,
       /* FALLTHROUGH */
 
     default:
-      return evaluate_subexp (NULL_TYPE, exp, pos, noside);
+      return evaluate_subexp (nullptr, exp, pos, noside);
     }
 }
 
@@ -3101,16 +2919,16 @@ evaluate_subexp_for_sizeof (struct expression *exp, int *pos,
   switch (op)
     {
       /* This case is handled specially
-         so that we avoid creating a value for the result type.
-         If the result type is very big, it's desirable not to
-         create a value unnecessarily.  */
+        so that we avoid creating a value for the result type.
+        If the result type is very big, it's desirable not to
+        create a value unnecessarily.  */
     case UNOP_IND:
       (*pos)++;
-      val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
+      val = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       type = check_typedef (value_type (val));
-      if (TYPE_CODE (type) != TYPE_CODE_PTR
+      if (type->code () != TYPE_CODE_PTR
          && !TYPE_IS_REFERENCE (type)
-         && TYPE_CODE (type) != TYPE_CODE_ARRAY)
+         && type->code () != TYPE_CODE_ARRAY)
        error (_("Attempt to take contents of a non-pointer value."));
       type = TYPE_TARGET_TYPE (type);
       if (is_dynamic_type (type))
@@ -3132,8 +2950,16 @@ evaluate_subexp_for_sizeof (struct expression *exp, int *pos,
       type = SYMBOL_TYPE (exp->elts[pc + 2].symbol);
       if (is_dynamic_type (type))
        {
-         val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_NORMAL);
+         val = evaluate_subexp (nullptr, exp, pos, EVAL_NORMAL);
          type = value_type (val);
+         if (type->code () == TYPE_CODE_ARRAY)
+           {
+             if (type_not_allocated (type) || type_not_associated (type))
+               return value_zero (size_type, not_lval);
+             else if (is_dynamic_type (type->index_type ())
+                      && type->bounds ()->high.kind () == PROP_UNDEFINED)
+               return allocate_optimized_out_value (size_type);
+           }
        }
       else
        (*pos) += 4;
@@ -3144,13 +2970,13 @@ evaluate_subexp_for_sizeof (struct expression *exp, int *pos,
        (*pos) += 4;
 
        minimal_symbol *msymbol = exp->elts[pc + 2].msymbol;
-       value *val = evaluate_var_msym_value (noside,
-                                             exp->elts[pc + 1].objfile,
-                                             msymbol);
+       value *mval = evaluate_var_msym_value (noside,
+                                              exp->elts[pc + 1].objfile,
+                                              msymbol);
 
-       type = value_type (val);
-       if (TYPE_CODE (type) == TYPE_CODE_ERROR)
-         error_unknown_type (MSYMBOL_PRINT_NAME (msymbol));
+       type = value_type (mval);
+       if (type->code () == TYPE_CODE_ERROR)
+         error_unknown_type (msymbol->print_name ());
 
        return value_from_longest (size_type, TYPE_LENGTH (type));
       }
@@ -3158,26 +2984,26 @@ evaluate_subexp_for_sizeof (struct expression *exp, int *pos,
 
       /* Deal with the special case if NOSIDE is EVAL_NORMAL and the resulting
         type of the subscript is a variable length array type. In this case we
-        must re-evaluate the right hand side of the subcription to allow
+        must re-evaluate the right hand side of the subscription to allow
         side-effects. */
     case BINOP_SUBSCRIPT:
       if (noside == EVAL_NORMAL)
        {
-         int pc = (*pos) + 1;
+         int npc = (*pos) + 1;
 
-         val = evaluate_subexp (NULL_TYPE, exp, &pc, EVAL_AVOID_SIDE_EFFECTS);
+         val = evaluate_subexp (nullptr, exp, &npc, EVAL_AVOID_SIDE_EFFECTS);
          type = check_typedef (value_type (val));
-         if (TYPE_CODE (type) == TYPE_CODE_ARRAY)
+         if (type->code () == TYPE_CODE_ARRAY)
            {
              type = check_typedef (TYPE_TARGET_TYPE (type));
-             if (TYPE_CODE (type) == TYPE_CODE_ARRAY)
+             if (type->code () == TYPE_CODE_ARRAY)
                {
-                 type = TYPE_INDEX_TYPE (type);
+                 type = type->index_type ();
                  /* Only re-evaluate the right hand side if the resulting type
                     is a variable length type.  */
-                 if (TYPE_RANGE_DATA (type)->flag_bound_evaluated)
+                 if (type->bounds ()->flag_bound_evaluated)
                    {
-                     val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_NORMAL);
+                     val = evaluate_subexp (nullptr, exp, pos, EVAL_NORMAL);
                      return value_from_longest
                        (size_type, (LONGEST) TYPE_LENGTH (value_type (val)));
                    }
@@ -3188,7 +3014,7 @@ evaluate_subexp_for_sizeof (struct expression *exp, int *pos,
       /* Fall through.  */
 
     default:
-      val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
+      val = evaluate_subexp (nullptr, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       type = value_type (val);
       break;
     }
@@ -3261,7 +3087,7 @@ evaluate_subexp_for_cast (expression *exp, int *pos,
 /* Parse a type expression in the string [P..P+LENGTH).  */
 
 struct type *
-parse_and_eval_type (char *p, int length)
+parse_and_eval_type (const char *p, int length)
 {
   char *tmp = (char *) alloca (length + 4);
 
@@ -3271,26 +3097,7 @@ parse_and_eval_type (char *p, int length)
   tmp[length + 2] = '0';
   tmp[length + 3] = '\0';
   expression_up expr = parse_expression (tmp);
-  if (expr->elts[0].opcode != UNOP_CAST)
+  if (expr->first_opcode () != UNOP_CAST)
     error (_("Internal error in eval_type."));
   return expr->elts[1].type;
 }
-
-int
-calc_f77_array_dims (struct type *array_type)
-{
-  int ndimen = 1;
-  struct type *tmp_type;
-
-  if ((TYPE_CODE (array_type) != TYPE_CODE_ARRAY))
-    error (_("Can't get dimensions for a non-array type"));
-
-  tmp_type = array_type;
-
-  while ((tmp_type = TYPE_TARGET_TYPE (tmp_type)))
-    {
-      if (TYPE_CODE (tmp_type) == TYPE_CODE_ARRAY)
-       ++ndimen;
-    }
-  return ndimen;
-}
This page took 0.05922 seconds and 4 git commands to generate.