]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/eval.c
gdb: Fix sizeof for dynamic types other than arrays
[thirdparty/binutils-gdb.git] / gdb / eval.c
index a91ba2271abd589541a6d5910ff4dfe73ace7d86..2e08e9355f5e1ba8bf0ec9818e2291e23676100c 100644 (file)
@@ -1,6 +1,6 @@
 /* Evaluate expressions for GDB.
 
-   Copyright (C) 1986-2013 Free Software Foundation, Inc.
+   Copyright (C) 1986-2018 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
-#include "gdb_string.h"
 #include "symtab.h"
 #include "gdbtypes.h"
 #include "value.h"
 #include "expression.h"
 #include "target.h"
 #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 "parser-defs.h"
 #include "cp-support.h"
 #include "ui-out.h"
-#include "exceptions.h"
 #include "regcache.h"
 #include "user-regs.h"
 #include "valprint.h"
 #include "gdb_obstack.h"
 #include "objfiles.h"
-#include "python/python.h"
-
-#include "gdb_assert.h"
-
+#include "typeprint.h"
 #include <ctype.h>
 
 /* This is defined in valops.c */
@@ -51,11 +47,16 @@ extern int overload_resolution;
 
 /* Prototypes for local functions.  */
 
-static struct value *evaluate_subexp_for_sizeof (struct expression *, int *);
+static struct value *evaluate_subexp_for_sizeof (struct expression *, int *,
+                                                enum noside);
 
 static struct value *evaluate_subexp_for_address (struct expression *,
                                                  int *, enum noside);
 
+static value *evaluate_subexp_for_cast (expression *exp, int *pos,
+                                       enum noside noside,
+                                       struct type *type);
+
 static struct value *evaluate_struct_tuple (struct value *,
                                            struct expression *, int *,
                                            enum noside, int);
@@ -68,8 +69,22 @@ struct value *
 evaluate_subexp (struct type *expect_type, struct expression *exp,
                 int *pos, enum noside noside)
 {
-  return (*exp->language_defn->la_exp_desc->evaluate_exp) 
+  struct value *retval;
+
+  gdb::optional<enable_thread_stack_temporaries> stack_temporaries;
+  if (*pos == 0 && target_has_execution
+      && exp->language_defn->la_language == language_cplus
+      && !thread_stack_temporaries_enabled_p (inferior_thread ()))
+    stack_temporaries.emplace (inferior_thread ());
+
+  retval = (*exp->language_defn->la_exp_desc->evaluate_exp)
     (expect_type, exp, pos, noside);
+
+  if (stack_temporaries.has_value ()
+      && value_in_thread_stack_temporaries (retval, inferior_thread ()))
+    retval = value_non_lval (retval);
+
+  return retval;
 }
 \f
 /* Parse the string EXP as a C expression, evaluate it,
@@ -78,42 +93,27 @@ evaluate_subexp (struct type *expect_type, struct expression *exp,
 CORE_ADDR
 parse_and_eval_address (const char *exp)
 {
-  struct expression *expr = parse_expression (exp);
-  CORE_ADDR addr;
-  struct cleanup *old_chain =
-    make_cleanup (free_current_contents, &expr);
-
-  addr = value_as_address (evaluate_expression (expr));
-  do_cleanups (old_chain);
-  return addr;
+  expression_up expr = parse_expression (exp);
+
+  return value_as_address (evaluate_expression (expr.get ()));
 }
 
 /* Like parse_and_eval_address, but treats the value of the expression
    as an integer, not an address, returns a LONGEST, not a CORE_ADDR.  */
 LONGEST
-parse_and_eval_long (char *exp)
+parse_and_eval_long (const char *exp)
 {
-  struct expression *expr = parse_expression (exp);
-  LONGEST retval;
-  struct cleanup *old_chain =
-    make_cleanup (free_current_contents, &expr);
-
-  retval = value_as_long (evaluate_expression (expr));
-  do_cleanups (old_chain);
-  return (retval);
+  expression_up expr = parse_expression (exp);
+
+  return value_as_long (evaluate_expression (expr.get ()));
 }
 
 struct value *
 parse_and_eval (const char *exp)
 {
-  struct expression *expr = parse_expression (exp);
-  struct value *val;
-  struct cleanup *old_chain =
-    make_cleanup (free_current_contents, &expr);
+  expression_up expr = parse_expression (exp);
 
-  val = evaluate_expression (expr);
-  do_cleanups (old_chain);
-  return val;
+  return evaluate_expression (expr.get ());
 }
 
 /* Parse up to a comma (or to a closeparen)
@@ -123,14 +123,9 @@ parse_and_eval (const char *exp)
 struct value *
 parse_to_comma_and_eval (const char **expp)
 {
-  struct expression *expr = parse_exp_1 (expp, 0, (struct block *) 0, 1);
-  struct value *val;
-  struct cleanup *old_chain =
-    make_cleanup (free_current_contents, &expr);
+  expression_up expr = parse_exp_1 (expp, 0, (struct block *) 0, 1);
 
-  val = evaluate_expression (expr);
-  do_cleanups (old_chain);
-  return val;
+  return evaluate_expression (expr.get ());
 }
 \f
 /* Evaluate an expression in internal prefix form
@@ -171,10 +166,12 @@ evaluate_subexpression_type (struct expression *exp, int subexp)
    in *VAL_CHAIN.  RESULTP and VAL_CHAIN may be NULL if the caller does
    not need them.
 
-   If a memory error occurs while evaluating the expression, *RESULTP will
-   be set to NULL.  *RESULTP may be a lazy value, if the result could
-   not be read from memory.  It is used to determine whether a value
-   is user-specified (we should watch the whole value) or intermediate
+   If PRESERVE_ERRORS is true, then exceptions are passed through.
+   Otherwise, if PRESERVE_ERRORS is false, then if a memory error
+   occurs while evaluating the expression, *RESULTP will be set to
+   NULL.  *RESULTP may be a lazy value, if the result could not be
+   read from memory.  It is used to determine whether a value is
+   user-specified (we should watch the whole value) or intermediate
    (we should watch only the bit used to locate the final value).
 
    If the final value, or any intermediate value, could not be read
@@ -182,46 +179,49 @@ 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)
+                   struct value **resultp,
+                   std::vector<value_ref_ptr> *val_chain,
+                   int preserve_errors)
 {
   struct value *mark, *new_mark, *result;
-  volatile struct gdb_exception ex;
 
   *valp = NULL;
   if (resultp)
     *resultp = NULL;
   if (val_chain)
-    *val_chain = NULL;
+    val_chain->clear ();
 
   /* Evaluate the expression.  */
   mark = value_mark ();
   result = NULL;
 
-  TRY_CATCH (ex, RETURN_MASK_ALL)
+  TRY
     {
       result = evaluate_subexp (NULL_TYPE, exp, pc, EVAL_NORMAL);
     }
-  if (ex.reason < 0)
+  CATCH (ex, RETURN_MASK_ALL)
     {
-      /* Ignore memory errors, we want watchpoints pointing at
+      /* Ignore memory errors if we want watchpoints pointing at
         inaccessible memory to still be created; otherwise, throw the
         error to some higher catcher.  */
       switch (ex.error)
        {
        case MEMORY_ERROR:
-         break;
+         if (!preserve_errors)
+           break;
+         /* Fall through.  */
        default:
          throw_exception (ex);
          break;
        }
     }
+  END_CATCH
 
   new_mark = value_mark ();
   if (mark == new_mark)
@@ -237,13 +237,16 @@ fetch_subexp_value (struct expression *exp, int *pc, struct value **valp,
        *valp = result;
       else
        {
-         volatile struct gdb_exception except;
 
-         TRY_CATCH (except, RETURN_MASK_ERROR)
+         TRY
            {
              value_fetch_lazy (result);
              *valp = result;
            }
+         CATCH (except, RETURN_MASK_ERROR)
+           {
+           }
+         END_CATCH
        }
     }
 
@@ -251,8 +254,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);
     }
 }
 
@@ -263,7 +265,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;
@@ -342,12 +344,11 @@ evaluate_struct_tuple (struct value *struct_val,
   return struct_val;
 }
 
-/* Recursive helper function for setting elements of array tuples for
-   (the deleted) Chill.  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.  */
+/* 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,
@@ -365,22 +366,6 @@ init_array_element (struct value *array, struct value *element,
       return init_array_element (array, element,
                                 exp, pos, noside, low_bound, high_bound);
     }
-  else if (exp->elts[*pos].opcode == BINOP_RANGE)
-    {
-      LONGEST low, high;
-
-      (*pos)++;
-      low = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
-      high = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
-      if (low < low_bound || high > high_bound)
-       error (_("tuple range index out of range"));
-      for (index = low; index <= high; index++)
-       {
-         memcpy (value_contents_raw (array)
-                 + (index - low_bound) * element_size,
-                 value_contents (element), element_size);
-       }
-    }
   else
     {
       index = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
@@ -399,7 +384,8 @@ value_f90_subarray (struct value *array,
   int pc = (*pos) + 1;
   LONGEST low_bound, high_bound;
   struct type *range = check_typedef (TYPE_INDEX_TYPE (value_type (array)));
-  enum f90_range_type range_type = longest_to_int (exp->elts[pc].longconst);
+  enum range_type range_type
+    = (enum range_type) longest_to_int (exp->elts[pc].longconst);
  
   *pos += 3;
 
@@ -632,7 +618,7 @@ static int
 ptrmath_type_p (const struct language_defn *lang, struct type *type)
 {
   type = check_typedef (type);
-  if (TYPE_CODE (type) == TYPE_CODE_REF)
+  if (TYPE_IS_REFERENCE (type))
     type = TYPE_TARGET_TYPE (type);
 
   switch (TYPE_CODE (type))
@@ -649,19 +635,37 @@ ptrmath_type_p (const struct language_defn *lang, struct type *type)
     }
 }
 
-/* Constructs a fake method with the given parameter types.
-   This function is used by the parser to construct an "expected"
-   type for method overload resolution.  */
+/* Represents a fake method with the given parameter types.  This is
+   used by the parser to construct a temporary "expected" type for
+   method overload resolution.  FLAGS is used as instance flags of the
+   new type, in order to be able to make the new type represent a
+   const/volatile overload.  */
+
+class fake_method
+{
+public:
+  fake_method (type_instance_flags flags,
+              int num_types, struct type **param_types);
+  ~fake_method ();
+
+  /* The constructed type.  */
+  struct type *type () { return &m_type; }
+
+private:
+  struct type m_type {};
+  main_type m_main_type {};
+};
 
-static struct type *
-make_params (int num_types, struct type **param_types)
+fake_method::fake_method (type_instance_flags flags,
+                         int num_types, struct type **param_types)
 {
-  struct type *type = XZALLOC (struct type);
-  TYPE_MAIN_TYPE (type) = XZALLOC (struct main_type);
+  struct type *type = &m_type;
+
+  TYPE_MAIN_TYPE (type) = &m_main_type;
   TYPE_LENGTH (type) = 1;
   TYPE_CODE (type) = TYPE_CODE_METHOD;
-  TYPE_VPTR_FIELDNO (type) = -1;
   TYPE_CHAIN (type) = type;
+  TYPE_INSTANCE_FLAGS (type) = flags;
   if (num_types > 0)
     {
       if (param_types[num_types - 1] == NULL)
@@ -685,8 +689,549 @@ make_params (int num_types, struct type **param_types)
 
   while (num_types-- > 0)
     TYPE_FIELD_TYPE (type, num_types) = param_types[num_types];
+}
+
+fake_method::~fake_method ()
+{
+  xfree (TYPE_FIELDS (&m_type));
+}
+
+/* Helper for evaluating an OP_VAR_VALUE.  */
+
+value *
+evaluate_var_value (enum noside noside, const block *blk, symbol *var)
+{
+  /* JYG: We used to just return value_zero of the symbol type if
+     we're asked to avoid side effects.  Otherwise we return
+     value_of_variable (...).  However I'm not sure if
+     value_of_variable () has any side effect.  We need a full value
+     object returned here for whatis_exp () to call evaluate_type ()
+     and then pass the full value to value_rtti_target_type () if we
+     are dealing with a pointer or reference to a base class and print
+     object is on.  */
+
+  struct value *ret = NULL;
+
+  TRY
+    {
+      ret = value_of_variable (var, blk);
+    }
+
+  CATCH (except, RETURN_MASK_ERROR)
+    {
+      if (noside != EVAL_AVOID_SIDE_EFFECTS)
+       throw_exception (except);
+
+      ret = value_zero (SYMBOL_TYPE (var), not_lval);
+    }
+  END_CATCH
+
+  return ret;
+}
+
+/* Helper for evaluating an OP_VAR_MSYM_VALUE.  */
+
+value *
+evaluate_var_msym_value (enum noside noside,
+                        struct objfile *objfile, minimal_symbol *msymbol)
+{
+  CORE_ADDR address;
+  type *the_type = find_minsym_type_and_address (msymbol, objfile, &address);
+
+  if (noside == EVAL_AVOID_SIDE_EFFECTS && !TYPE_GNU_IFUNC (the_type))
+    return value_zero (the_type, not_lval);
+  else
+    return value_at_lazy (the_type, address);
+}
+
+/* Helper for returning a value when handling EVAL_SKIP.  */
+
+value *
+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.  */
+
+static value *
+eval_call (expression *exp, enum noside noside,
+          int nargs, value **argvec,
+          const char *function_name,
+          type *default_return_type)
+{
+  if (argvec[0] == NULL)
+    error (_("Cannot evaluate function -- may be inlined"));
+  if (noside == EVAL_AVOID_SIDE_EFFECTS)
+    {
+      /* If the return type doesn't look like a function type,
+        call an error.  This can happen if somebody tries to turn
+        a variable into a function call.  */
+
+      type *ftype = value_type (argvec[0]);
+
+      if (TYPE_CODE (ftype) == TYPE_CODE_INTERNAL_FUNCTION)
+       {
+         /* We don't know anything about what the internal
+            function might return, but we have to return
+            something.  */
+         return value_zero (builtin_type (exp->gdbarch)->builtin_int,
+                            not_lval);
+       }
+      else if (TYPE_CODE (ftype) == TYPE_CODE_XMETHOD)
+       {
+         type *return_type
+           = result_type_of_xmethod (argvec[0], nargs, argvec + 1);
+
+         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)
+       {
+         if (TYPE_GNU_IFUNC (ftype))
+           {
+             CORE_ADDR address = value_address (argvec[0]);
+             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)
+           return_type = default_return_type;
+
+         if (return_type == NULL)
+           error_call_unknown_return_type (function_name);
+
+         return allocate_value (return_type);
+       }
+      else
+       error (_("Expression of type other than "
+                "\"Function returning ...\" used as function"));
+    }
+  switch (TYPE_CODE (value_type (argvec[0])))
+    {
+    case TYPE_CODE_INTERNAL_FUNCTION:
+      return call_internal_function (exp->gdbarch, exp->language_defn,
+                                    argvec[0], nargs, argvec + 1);
+    case TYPE_CODE_XMETHOD:
+      return call_xmethod (argvec[0], nargs, argvec + 1);
+    default:
+      return call_function_by_hand (argvec[0], default_return_type,
+                                   nargs, argvec + 1);
+    }
+}
+
+/* Helper for evaluating an OP_FUNCALL.  */
+
+static value *
+evaluate_funcall (type *expect_type, expression *exp, int *pos,
+                 enum noside noside)
+{
+  int tem;
+  int pc2 = 0;
+  value *arg1 = NULL;
+  value *arg2 = NULL;
+  int save_pos1;
+  symbol *function = NULL;
+  char *function_name = NULL;
+  const char *var_func_name = NULL;
+
+  int pc = (*pos);
+  (*pos) += 2;
+
+  exp_opcode op = exp->elts[*pos].opcode;
+  int nargs = longest_to_int (exp->elts[pc].longconst);
+  /* Allocate arg vector, including space for the function to be
+     called in argvec[0], a potential `this', and a terminating
+     NULL.  */
+  value **argvec = (value **) alloca (sizeof (value *) * (nargs + 3));
+  if (op == STRUCTOP_MEMBER || op == STRUCTOP_MPTR)
+    {
+      /* First, evaluate the structure into arg2.  */
+      pc2 = (*pos)++;
+
+      if (op == STRUCTOP_MEMBER)
+       {
+         arg2 = evaluate_subexp_for_address (exp, pos, noside);
+       }
+      else
+       {
+         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+       }
+
+      /* If the function is a virtual function, then the aggregate
+        value (providing the structure) plays its part by providing
+        the vtable.  Otherwise, it is just along for the ride: call
+        the function directly.  */
+
+      arg1 = evaluate_subexp (NULL_TYPE, 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)
+       {
+         if (noside == EVAL_AVOID_SIDE_EFFECTS)
+           arg1 = value_zero (TYPE_TARGET_TYPE (a1_type), not_lval);
+         else
+           arg1 = cplus_method_ptr_to_value (&arg2, arg1);
+
+         /* Now, say which argument to start evaluating from.  */
+         nargs++;
+         tem = 2;
+         argvec[1] = arg2;
+       }
+      else if (TYPE_CODE (a1_type) == TYPE_CODE_MEMBERPTR)
+       {
+         struct type *type_ptr
+           = lookup_pointer_type (TYPE_SELF_TYPE (a1_type));
+         struct type *target_type_ptr
+           = lookup_pointer_type (TYPE_TARGET_TYPE (a1_type));
+
+         /* Now, convert these values to an address.  */
+         arg2 = value_cast (type_ptr, arg2);
+
+         long mem_offset = value_as_long (arg1);
+
+         arg1 = value_from_pointer (target_type_ptr,
+                                    value_as_long (arg2) + mem_offset);
+         arg1 = value_ind (arg1);
+         tem = 1;
+       }
+      else
+       error (_("Non-pointer-to-member value used in pointer-to-member "
+                "construct"));
+    }
+  else if (op == STRUCTOP_STRUCT || op == STRUCTOP_PTR)
+    {
+      /* Hair for method invocations.  */
+      int tem2;
+
+      nargs++;
+      /* First, evaluate the structure into arg2.  */
+      pc2 = (*pos)++;
+      tem2 = longest_to_int (exp->elts[pc2 + 1].longconst);
+      *pos += 3 + BYTES_TO_EXP_ELEM (tem2 + 1);
+
+      if (op == STRUCTOP_STRUCT)
+       {
+         /* If v is a variable in a register, and the user types
+            v.method (), this will produce an error, because v has no
+            address.
+
+            A possible way around this would be to allocate a copy of
+            the variable on the stack, copy in the contents, call the
+            function, and copy out the contents.  I.e. convert this
+            from call by reference to call by copy-return (or
+            whatever it's called).  However, this does not work
+            because it is not the same: the method being called could
+            stash a copy of the address, and then future uses through
+            that address (after the method returns) would be expected
+            to use the variable itself, not some copy of it.  */
+         arg2 = evaluate_subexp_for_address (exp, pos, noside);
+       }
+      else
+       {
+         arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+
+         /* Check to see if the operator '->' has been overloaded.
+            If the operator has been overloaded replace arg2 with the
+            value returned by the custom operator and continue
+            evaluation.  */
+         while (unop_user_defined_p (op, arg2))
+           {
+             struct value *value = NULL;
+             TRY
+               {
+                 value = value_x_unop (arg2, op, noside);
+               }
+
+             CATCH (except, RETURN_MASK_ERROR)
+               {
+                 if (except.error == NOT_FOUND_ERROR)
+                   break;
+                 else
+                   throw_exception (except);
+               }
+             END_CATCH
+
+               arg2 = value;
+           }
+       }
+      /* Now, say which argument to start evaluating from.  */
+      tem = 2;
+    }
+  else if (op == OP_SCOPE
+          && overload_resolution
+          && (exp->language_defn->la_language == language_cplus))
+    {
+      /* Unpack it locally so we can properly handle overload
+        resolution.  */
+      char *name;
+      int local_tem;
+
+      pc2 = (*pos)++;
+      local_tem = longest_to_int (exp->elts[pc2 + 2].longconst);
+      (*pos) += 4 + BYTES_TO_EXP_ELEM (local_tem + 1);
+      struct type *type = exp->elts[pc2 + 1].type;
+      name = &exp->elts[pc2 + 3].string;
+
+      function = NULL;
+      function_name = NULL;
+      if (TYPE_CODE (type) == TYPE_CODE_NAMESPACE)
+       {
+         function = cp_lookup_symbol_namespace (TYPE_NAME (type),
+                                                name,
+                                                get_selected_block (0),
+                                                VAR_DOMAIN).symbol;
+         if (function == NULL)
+           error (_("No symbol \"%s\" in namespace \"%s\"."),
+                  name, TYPE_NAME (type));
+
+         tem = 1;
+         /* arg2 is left as NULL on purpose.  */
+       }
+      else
+       {
+         gdb_assert (TYPE_CODE (type) == TYPE_CODE_STRUCT
+                     || TYPE_CODE (type) == TYPE_CODE_UNION);
+         function_name = name;
+
+         /* We need a properly typed value for method lookup.  For
+            static methods arg2 is otherwise unused.  */
+         arg2 = value_zero (type, lval_memory);
+         ++nargs;
+         tem = 2;
+       }
+    }
+  else if (op == OP_ADL_FUNC)
+    {
+      /* Save the function position and move pos so that the arguments
+        can be evaluated.  */
+      int func_name_len;
+
+      save_pos1 = *pos;
+      tem = 1;
+
+      func_name_len = longest_to_int (exp->elts[save_pos1 + 3].longconst);
+      (*pos) += 6 + BYTES_TO_EXP_ELEM (func_name_len + 1);
+    }
+  else
+    {
+      /* Non-method function call.  */
+      save_pos1 = *pos;
+      tem = 1;
+
+      /* If this is a C++ function wait until overload resolution.  */
+      if (op == OP_VAR_VALUE
+         && overload_resolution
+         && (exp->language_defn->la_language == language_cplus))
+       {
+         (*pos) += 4; /* Skip the evaluation of the symbol.  */
+         argvec[0] = NULL;
+       }
+      else
+       {
+         if (op == OP_VAR_MSYM_VALUE)
+           {
+             minimal_symbol *msym = exp->elts[*pos + 2].msymbol;
+             var_func_name = MSYMBOL_PRINT_NAME (msym);
+           }
+         else if (op == OP_VAR_VALUE)
+           {
+             symbol *sym = exp->elts[*pos + 2].symbol;
+             var_func_name = SYMBOL_PRINT_NAME (sym);
+           }
+
+         argvec[0] = evaluate_subexp_with_coercion (exp, pos, noside);
+         type *type = value_type (argvec[0]);
+         if (type && TYPE_CODE (type) == TYPE_CODE_PTR)
+           type = TYPE_TARGET_TYPE (type);
+         if (type && TYPE_CODE (type) == TYPE_CODE_FUNC)
+           {
+             for (; tem <= nargs && tem <= TYPE_NFIELDS (type); tem++)
+               {
+                 argvec[tem] = evaluate_subexp (TYPE_FIELD_TYPE (type,
+                                                                 tem - 1),
+                                                exp, pos, noside);
+               }
+           }
+       }
+    }
+
+  /* Evaluate arguments (if not already done, e.g., namespace::func()
+     and overload-resolution is off).  */
+  for (; tem <= nargs; tem++)
+    {
+      /* Ensure that array expressions are coerced into pointer
+        objects.  */
+      argvec[tem] = evaluate_subexp_with_coercion (exp, pos, noside);
+    }
+
+  /* Signal end of arglist.  */
+  argvec[tem] = 0;
+
+  if (noside == EVAL_SKIP)
+    return eval_skip_value (exp);
+
+  if (op == OP_ADL_FUNC)
+    {
+      struct symbol *symp;
+      char *func_name;
+      int  name_len;
+      int string_pc = save_pos1 + 3;
+
+      /* Extract the function name.  */
+      name_len = longest_to_int (exp->elts[string_pc].longconst);
+      func_name = (char *) alloca (name_len + 1);
+      strcpy (func_name, &exp->elts[string_pc + 1].string);
+
+      find_overload_match (&argvec[1], nargs, func_name,
+                          NON_METHOD, /* not method */
+                          NULL, NULL, /* pass NULL symbol since
+                                         symbol is unknown */
+                          NULL, &symp, NULL, 0, noside);
+
+      /* Now fix the expression being evaluated.  */
+      exp->elts[save_pos1 + 2].symbol = symp;
+      argvec[0] = evaluate_subexp_with_coercion (exp, &save_pos1, noside);
+    }
+
+  if (op == STRUCTOP_STRUCT || op == STRUCTOP_PTR
+      || (op == OP_SCOPE && function_name != NULL))
+    {
+      int static_memfuncp;
+      char *tstr;
+
+      /* Method invocation: stuff "this" as first parameter.  If the
+        method turns out to be static we undo this below.  */
+      argvec[1] = arg2;
+
+      if (op != OP_SCOPE)
+       {
+         /* Name of method from expression.  */
+         tstr = &exp->elts[pc2 + 2].string;
+       }
+      else
+       tstr = function_name;
+
+      if (overload_resolution && (exp->language_defn->la_language
+                                 == language_cplus))
+       {
+         /* Language is C++, do some overload resolution before
+            evaluation.  */
+         struct value *valp = NULL;
+
+         (void) find_overload_match (&argvec[1], nargs, tstr,
+                                     METHOD, /* method */
+                                     &arg2,  /* the object */
+                                     NULL, &valp, NULL,
+                                     &static_memfuncp, 0, noside);
+
+         if (op == OP_SCOPE && !static_memfuncp)
+           {
+             /* For the time being, we don't handle this.  */
+             error (_("Call to overloaded function %s requires "
+                      "`this' pointer"),
+                    function_name);
+           }
+         argvec[1] = arg2;     /* the ``this'' pointer */
+         argvec[0] = valp;     /* Use the method found after overload
+                                  resolution.  */
+       }
+      else
+       /* Non-C++ case -- or no overload resolution.  */
+       {
+         struct value *temp = arg2;
+
+         argvec[0] = value_struct_elt (&temp, argvec + 1, tstr,
+                                       &static_memfuncp,
+                                       op == STRUCTOP_STRUCT
+                                       ? "structure" : "structure pointer");
+         /* value_struct_elt updates temp with the correct value of
+            the ``this'' pointer if necessary, so modify argvec[1] to
+            reflect any ``this'' changes.  */
+         arg2
+           = value_from_longest (lookup_pointer_type(value_type (temp)),
+                                 value_address (temp)
+                                 + value_embedded_offset (temp));
+         argvec[1] = arg2;     /* the ``this'' pointer */
+       }
+
+      /* Take out `this' if needed.  */
+      if (static_memfuncp)
+       {
+         argvec[1] = argvec[0];
+         nargs--;
+         argvec++;
+       }
+    }
+  else if (op == STRUCTOP_MEMBER || op == STRUCTOP_MPTR)
+    {
+      /* Pointer to member.  argvec[1] is already set up.  */
+      argvec[0] = arg1;
+    }
+  else if (op == OP_VAR_VALUE || (op == OP_SCOPE && function != NULL))
+    {
+      /* Non-member function being called.  */
+      /* fn: This can only be done for C++ functions.  A C-style
+        function in a C++ program, for instance, does not have the
+        fields that are expected here.  */
+
+      if (overload_resolution && (exp->language_defn->la_language
+                                 == language_cplus))
+       {
+         /* Language is C++, do some overload resolution before
+            evaluation.  */
+         struct symbol *symp;
+         int no_adl = 0;
+
+         /* If a scope has been specified disable ADL.  */
+         if (op == OP_SCOPE)
+           no_adl = 1;
+
+         if (op == OP_VAR_VALUE)
+           function = exp->elts[save_pos1+2].symbol;
+
+         (void) find_overload_match (&argvec[1], nargs,
+                                     NULL,        /* no need for name */
+                                     NON_METHOD,  /* not method */
+                                     NULL, function, /* the function */
+                                     NULL, &symp, NULL, no_adl, noside);
+
+         if (op == OP_VAR_VALUE)
+           {
+             /* Now fix the expression being evaluated.  */
+             exp->elts[save_pos1+2].symbol = symp;
+             argvec[0] = evaluate_subexp_with_coercion (exp, &save_pos1,
+                                                        noside);
+           }
+         else
+           argvec[0] = value_of_variable (symp, get_selected_block (0));
+       }
+      else
+       {
+         /* Not C++, or no overload resolution allowed.  */
+         /* Nothing to be done; argvec already correctly set up.  */
+       }
+    }
+  else
+    {
+      /* It is probably a C-style function.  */
+      /* Nothing to be done; argvec already correctly set up.  */
+    }
 
-  return type;
+  return eval_call (exp, noside, nargs, argvec, var_func_name, expect_type);
 }
 
 struct value *
@@ -696,7 +1241,7 @@ evaluate_subexp_standard (struct type *expect_type,
 {
   enum exp_opcode op;
   int tem, tem2, tem3;
-  int pc, pc2 = 0, oldpos;
+  int pc, oldpos;
   struct value *arg1 = NULL;
   struct value *arg2 = NULL;
   struct value *arg3;
@@ -707,9 +1252,6 @@ evaluate_subexp_standard (struct type *expect_type,
   int ix;
   long mem_offset;
   struct type **arg_types;
-  int save_pos1;
-  struct symbol *function = NULL;
-  char *function_name = NULL;
 
   pc = (*pos)++;
   op = exp->elts[pc].opcode;
@@ -720,7 +1262,7 @@ evaluate_subexp_standard (struct type *expect_type,
       tem = longest_to_int (exp->elts[pc + 2].longconst);
       (*pos) += 4 + BYTES_TO_EXP_ELEM (tem + 1);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       arg1 = value_aggregate_elt (exp->elts[pc + 1].type,
                                  &exp->elts[pc + 3].string,
                                  expect_type, 0, noside);
@@ -733,57 +1275,45 @@ evaluate_subexp_standard (struct type *expect_type,
       return value_from_longest (exp->elts[pc + 1].type,
                                 exp->elts[pc + 2].longconst);
 
-    case OP_DOUBLE:
-      (*pos) += 3;
-      return value_from_double (exp->elts[pc + 1].type,
-                               exp->elts[pc + 2].doubleconst);
-
-    case OP_DECFLOAT:
+    case OP_FLOAT:
       (*pos) += 3;
-      return value_from_decfloat (exp->elts[pc + 1].type,
-                                 exp->elts[pc + 2].decfloatconst);
+      return value_from_contents (exp->elts[pc + 1].type,
+                                 exp->elts[pc + 2].floatconst);
 
     case OP_ADL_FUNC:
     case OP_VAR_VALUE:
       (*pos) += 3;
       if (noside == EVAL_SKIP)
-       goto nosideret;
-
-      /* JYG: We used to just return value_zero of the symbol type
-        if we're asked to avoid side effects.  Otherwise we return
-        value_of_variable (...).  However I'm not sure if
-        value_of_variable () has any side effect.
-        We need a full value object returned here for whatis_exp ()
-        to call evaluate_type () and then pass the full value to
-        value_rtti_target_type () if we are dealing with a pointer
-        or reference to a base class and print object is on.  */
+       return eval_skip_value (exp);
 
       {
-       volatile struct gdb_exception except;
-       struct value *ret = NULL;
+       symbol *var = exp->elts[pc + 2].symbol;
+       if (TYPE_CODE (SYMBOL_TYPE (var)) == TYPE_CODE_ERROR)
+         error_unknown_type (SYMBOL_PRINT_NAME (var));
 
-       TRY_CATCH (except, RETURN_MASK_ERROR)
-         {
-           ret = value_of_variable (exp->elts[pc + 2].symbol,
-                                    exp->elts[pc + 1].block);
-         }
+       return evaluate_var_value (noside, exp->elts[pc + 1].block, var);
+      }
 
-       if (except.reason < 0)
-         {
-           if (noside == EVAL_AVOID_SIDE_EFFECTS)
-             ret = value_zero (SYMBOL_TYPE (exp->elts[pc + 2].symbol),
-                               not_lval);
-           else
-             throw_exception (except);
-         }
+    case OP_VAR_MSYM_VALUE:
+      {
+       (*pos) += 3;
 
-       return ret;
+       minimal_symbol *msymbol = exp->elts[pc + 2].msymbol;
+       value *val = evaluate_var_msym_value (noside,
+                                             exp->elts[pc + 1].objfile,
+                                             msymbol);
+
+       type = value_type (val);
+       if (TYPE_CODE (type) == TYPE_CODE_ERROR
+           && (noside != EVAL_AVOID_SIDE_EFFECTS || pc != 0))
+         error_unknown_type (MSYMBOL_PRINT_NAME (msymbol));
+       return val;
       }
 
     case OP_VAR_ENTRY_VALUE:
       (*pos) += 2;
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
 
       {
        struct symbol *sym = exp->elts[pc + 1].symbol;
@@ -801,6 +1331,27 @@ evaluate_subexp_standard (struct type *expect_type,
        return SYMBOL_COMPUTED_OPS (sym)->read_variable_at_entry (sym, frame);
       }
 
+    case OP_FUNC_STATIC_VAR:
+      tem = longest_to_int (exp->elts[pc + 1].longconst);
+      (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
+      if (noside == EVAL_SKIP)
+       return eval_skip_value (exp);
+
+      {
+       value *func = evaluate_subexp_standard (NULL, exp, pos, noside);
+       CORE_ADDR addr = value_address (func);
+
+       const block *blk = block_for_pc (addr);
+       const char *var = &exp->elts[pc + 2].string;
+
+       struct block_symbol sym = lookup_symbol (var, blk, VAR_DOMAIN, NULL);
+
+       if (sym.symbol == NULL)
+         error (_("No symbol \"%s\" in specified context."), var);
+
+       return evaluate_var_value (noside, sym.block, sym.symbol);
+      }
+
     case OP_LAST:
       (*pos) += 2;
       return
@@ -848,7 +1399,7 @@ evaluate_subexp_standard (struct type *expect_type,
       tem = longest_to_int (exp->elts[pc + 1].longconst);
       (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       type = language_string_char_type (exp->language_defn, exp->gdbarch);
       return value_string (&exp->elts[pc + 2].string, tem, type);
 
@@ -857,9 +1408,7 @@ evaluate_subexp_standard (struct type *expect_type,
       tem = longest_to_int (exp->elts[pc + 1].longconst);
       (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
       if (noside == EVAL_SKIP)
-       {
-         goto nosideret;
-       }
+       return eval_skip_value (exp);
       return value_nsstring (exp->gdbarch, &exp->elts[pc + 2].string, tem + 1);
 
     case OP_ARRAY:
@@ -899,11 +1448,6 @@ evaluate_subexp_standard (struct type *expect_type,
              struct value *element;
              int index_pc = 0;
 
-             if (exp->elts[*pos].opcode == BINOP_RANGE)
-               {
-                 index_pc = ++(*pos);
-                 evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
-               }
              element = evaluate_subexp (element_type, exp, pos, noside);
              if (value_type (element) != element_type)
                element = value_cast (element_type, element);
@@ -954,22 +1498,10 @@ evaluate_subexp_standard (struct type *expect_type,
              struct type *range_low_type, *range_high_type;
              struct value *elem_val;
 
-             if (exp->elts[*pos].opcode == BINOP_RANGE)
-               {
-                 (*pos)++;
-                 elem_val = evaluate_subexp (element_type, exp, pos, noside);
-                 range_low_type = value_type (elem_val);
-                 range_low = value_as_long (elem_val);
-                 elem_val = evaluate_subexp (element_type, exp, pos, noside);
-                 range_high_type = value_type (elem_val);
-                 range_high = value_as_long (elem_val);
-               }
-             else
-               {
-                 elem_val = evaluate_subexp (element_type, exp, pos, noside);
-                 range_low_type = range_high_type = value_type (elem_val);
-                 range_low = range_high = value_as_long (elem_val);
-               }
+             elem_val = evaluate_subexp (element_type, exp, pos, noside);
+             range_low_type = range_high_type = value_type (elem_val);
+             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.  */
@@ -1008,7 +1540,7 @@ evaluate_subexp_standard (struct type *expect_type,
          return set;
        }
 
-      argvec = (struct value **) alloca (sizeof (struct value *) * nargs);
+      argvec = XALLOCAVEC (struct value *, nargs);
       for (tem = 0; tem < nargs; tem++)
        {
          /* Ensure that array expressions are coerced into pointer
@@ -1016,7 +1548,7 @@ evaluate_subexp_standard (struct type *expect_type,
          argvec[tem] = evaluate_subexp_with_coercion (exp, pos, noside);
        }
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       return value_array (tem2, tem3, argvec);
 
     case TERNOP_SLICE:
@@ -1028,7 +1560,7 @@ evaluate_subexp_standard (struct type *expect_type,
          = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
 
        if (noside == EVAL_SKIP)
-         goto nosideret;
+         return eval_skip_value (exp);
        return value_slice (array, lowbound, upper - lowbound + 1);
       }
 
@@ -1055,7 +1587,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
        (*pos) += 3 + BYTES_TO_EXP_ELEM (len + 1);
        if (noside == EVAL_SKIP)
-         goto nosideret;
+         return eval_skip_value (exp);
 
        if (sel[len] != 0)
          sel[len] = 0;         /* Make sure it's terminated.  */
@@ -1074,7 +1606,7 @@ evaluate_subexp_standard (struct type *expect_type,
        CORE_ADDR selector = 0;
 
        int struct_return = 0;
-       int sub_no_side = 0;
+       enum noside sub_no_side = EVAL_NORMAL;
 
        struct value *msg_send = NULL;
        struct value *msg_send_stret = NULL;
@@ -1092,8 +1624,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
        selector = exp->elts[pc + 1].longconst;
        nargs = exp->elts[pc + 2].longconst;
-       argvec = (struct value **) alloca (sizeof (struct value *) 
-                                          * (nargs + 5));
+       argvec = XALLOCAVEC (struct value *, nargs + 5);
 
        (*pos) += 3;
 
@@ -1110,7 +1641,7 @@ evaluate_subexp_standard (struct type *expect_type,
        if (value_as_long (target) == 0)
          return value_from_longest (long_type, 0);
        
-       if (lookup_minimal_symbol ("objc_msg_lookup", 0, 0))
+       if (lookup_minimal_symbol ("objc_msg_lookup", 0, 0).minsym)
          gnu_runtime = 1;
        
        /* Find the method dispatch (Apple runtime) or method lookup
@@ -1178,12 +1709,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], 3, argvec + 1);
+       ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
        if (gnu_runtime)
          {
            /* Function objc_msg_lookup returns a pointer.  */
            argvec[0] = ret;
-           ret = call_function_by_hand (argvec[0], 3, argvec + 1);
+           ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
          }
        if (value_as_long (ret) == 0)
          error (_("Target does not respond to this message selector."));
@@ -1200,11 +1731,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], 3, argvec + 1);
+       ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
        if (gnu_runtime)
          {
            argvec[0] = ret;
-           ret = call_function_by_hand (argvec[0], 3, argvec + 1);
+           ret = call_function_by_hand (argvec[0], NULL, 3, argvec + 1);
          }
 
        /* ret should now be the selector.  */
@@ -1217,7 +1748,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);
@@ -1237,7 +1768,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
            block_for_pc (funaddr);
 
-           CHECK_TYPEDEF (val_type);
+           val_type = check_typedef (val_type);
          
            if ((val_type == NULL) 
                || (TYPE_CODE(val_type) == TYPE_CODE_ERROR))
@@ -1298,7 +1829,7 @@ evaluate_subexp_standard (struct type *expect_type,
          }
 
        if (noside == EVAL_SKIP)
-         goto nosideret;
+         return eval_skip_value (exp);
 
        if (noside == EVAL_AVOID_SIDE_EFFECTS)
          {
@@ -1346,421 +1877,16 @@ evaluate_subexp_standard (struct type *expect_type,
            deprecated_set_value_type (argvec[0],
                                       lookup_pointer_type (lookup_function_type (value_type (argvec[0]))));
            argvec[0]
-             = call_function_by_hand (argvec[0], nargs + 2, argvec + 1);
+             = call_function_by_hand (argvec[0], NULL, nargs + 2, argvec + 1);
          }
 
-       ret = call_function_by_hand (argvec[0], nargs + 2, argvec + 1);
+       ret = call_function_by_hand (argvec[0], NULL, nargs + 2, argvec + 1);
        return ret;
       }
       break;
 
     case OP_FUNCALL:
-      (*pos) += 2;
-      op = exp->elts[*pos].opcode;
-      nargs = longest_to_int (exp->elts[pc + 1].longconst);
-      /* 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 + 3));
-      if (op == STRUCTOP_MEMBER || op == STRUCTOP_MPTR)
-       {
-         /* First, evaluate the structure into arg2.  */
-         pc2 = (*pos)++;
-
-         if (noside == EVAL_SKIP)
-           goto nosideret;
-
-         if (op == STRUCTOP_MEMBER)
-           {
-             arg2 = evaluate_subexp_for_address (exp, pos, noside);
-           }
-         else
-           {
-             arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-           }
-
-         /* If the function is a virtual function, then the
-            aggregate value (providing the structure) plays
-            its part by providing the vtable.  Otherwise,
-            it is just along for the ride: call the function
-            directly.  */
-
-         arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-
-         type = check_typedef (value_type (arg1));
-         if (TYPE_CODE (type) == TYPE_CODE_METHODPTR)
-           {
-             if (noside == EVAL_AVOID_SIDE_EFFECTS)
-               arg1 = value_zero (TYPE_TARGET_TYPE (type), not_lval);
-             else
-               arg1 = cplus_method_ptr_to_value (&arg2, arg1);
-
-             /* Now, say which argument to start evaluating from.  */
-             nargs++;
-             tem = 2;
-             argvec[1] = arg2;
-           }
-         else if (TYPE_CODE (type) == TYPE_CODE_MEMBERPTR)
-           {
-             struct type *type_ptr
-               = lookup_pointer_type (TYPE_DOMAIN_TYPE (type));
-             struct type *target_type_ptr
-               = lookup_pointer_type (TYPE_TARGET_TYPE (type));
-
-             /* Now, convert these values to an address.  */
-             arg2 = value_cast (type_ptr, arg2);
-
-             mem_offset = value_as_long (arg1);
-
-             arg1 = value_from_pointer (target_type_ptr,
-                                        value_as_long (arg2) + mem_offset);
-             arg1 = value_ind (arg1);
-             tem = 1;
-           }
-         else
-           error (_("Non-pointer-to-member value used in pointer-to-member "
-                    "construct"));
-       }
-      else if (op == STRUCTOP_STRUCT || op == STRUCTOP_PTR)
-       {
-         /* Hair for method invocations.  */
-         int tem2;
-
-         nargs++;
-         /* First, evaluate the structure into arg2.  */
-         pc2 = (*pos)++;
-         tem2 = longest_to_int (exp->elts[pc2 + 1].longconst);
-         *pos += 3 + BYTES_TO_EXP_ELEM (tem2 + 1);
-         if (noside == EVAL_SKIP)
-           goto nosideret;
-
-         if (op == STRUCTOP_STRUCT)
-           {
-             /* If v is a variable in a register, and the user types
-                v.method (), this will produce an error, because v has
-                no address.
-
-                A possible way around this would be to allocate a
-                copy of the variable on the stack, copy in the
-                contents, call the function, and copy out the
-                contents.  I.e. convert this from call by reference
-                to call by copy-return (or whatever it's called).
-                However, this does not work because it is not the
-                same: the method being called could stash a copy of
-                the address, and then future uses through that address
-                (after the method returns) would be expected to
-                use the variable itself, not some copy of it.  */
-             arg2 = evaluate_subexp_for_address (exp, pos, noside);
-           }
-         else
-           {
-             arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-
-             /* Check to see if the operator '->' has been
-                overloaded.  If the operator has been overloaded
-                replace arg2 with the value returned by the custom
-                operator and continue evaluation.  */
-             while (unop_user_defined_p (op, arg2))
-               {
-                 volatile struct gdb_exception except;
-                 struct value *value = NULL;
-                 TRY_CATCH (except, RETURN_MASK_ERROR)
-                   {
-                     value = value_x_unop (arg2, op, noside);
-                   }
-
-                 if (except.reason < 0)
-                   {
-                     if (except.error == NOT_FOUND_ERROR)
-                       break;
-                     else
-                       throw_exception (except);
-                   }
-                 arg2 = value;
-               }
-           }
-         /* Now, say which argument to start evaluating from.  */
-         tem = 2;
-       }
-      else if (op == OP_SCOPE
-              && overload_resolution
-              && (exp->language_defn->la_language == language_cplus))
-       {
-         /* Unpack it locally so we can properly handle overload
-            resolution.  */
-         char *name;
-         int local_tem;
-
-         pc2 = (*pos)++;
-         local_tem = longest_to_int (exp->elts[pc2 + 2].longconst);
-         (*pos) += 4 + BYTES_TO_EXP_ELEM (local_tem + 1);
-         type = exp->elts[pc2 + 1].type;
-         name = &exp->elts[pc2 + 3].string;
-
-         function = NULL;
-         function_name = NULL;
-         if (TYPE_CODE (type) == TYPE_CODE_NAMESPACE)
-           {
-             function = cp_lookup_symbol_namespace (TYPE_TAG_NAME (type),
-                                                    name,
-                                                    get_selected_block (0),
-                                                    VAR_DOMAIN);
-             if (function == NULL)
-               error (_("No symbol \"%s\" in namespace \"%s\"."), 
-                      name, TYPE_TAG_NAME (type));
-
-             tem = 1;
-           }
-         else
-           {
-             gdb_assert (TYPE_CODE (type) == TYPE_CODE_STRUCT
-                         || TYPE_CODE (type) == TYPE_CODE_UNION);
-             function_name = name;
-
-             arg2 = value_zero (type, lval_memory);
-             ++nargs;
-             tem = 2;
-           }
-       }
-      else if (op == OP_ADL_FUNC)
-        {
-          /* Save the function position and move pos so that the arguments
-             can be evaluated.  */
-          int func_name_len;
-
-          save_pos1 = *pos;
-          tem = 1;
-
-          func_name_len = longest_to_int (exp->elts[save_pos1 + 3].longconst);
-          (*pos) += 6 + BYTES_TO_EXP_ELEM (func_name_len + 1);
-        }
-      else
-       {
-         /* Non-method function call.  */
-         save_pos1 = *pos;
-         tem = 1;
-
-         /* If this is a C++ function wait until overload resolution.  */
-         if (op == OP_VAR_VALUE
-             && overload_resolution
-             && (exp->language_defn->la_language == language_cplus))
-           {
-             (*pos) += 4; /* Skip the evaluation of the symbol.  */
-             argvec[0] = NULL;
-           }
-         else
-           {
-             argvec[0] = evaluate_subexp_with_coercion (exp, pos, noside);
-             type = value_type (argvec[0]);
-             if (type && TYPE_CODE (type) == TYPE_CODE_PTR)
-               type = TYPE_TARGET_TYPE (type);
-             if (type && TYPE_CODE (type) == TYPE_CODE_FUNC)
-               {
-                 for (; tem <= nargs && tem <= TYPE_NFIELDS (type); tem++)
-                   {
-                     argvec[tem] = evaluate_subexp (TYPE_FIELD_TYPE (type,
-                                                                     tem - 1),
-                                                    exp, pos, noside);
-                   }
-               }
-           }
-       }
-
-      /* Evaluate arguments.  */
-      for (; tem <= nargs; tem++)
-       {
-         /* Ensure that array expressions are coerced into pointer
-            objects.  */
-         argvec[tem] = evaluate_subexp_with_coercion (exp, pos, noside);
-       }
-
-      /* Signal end of arglist.  */
-      argvec[tem] = 0;
-      if (op == OP_ADL_FUNC)
-        {
-          struct symbol *symp;
-          char *func_name;
-          int  name_len;
-          int string_pc = save_pos1 + 3;
-
-          /* Extract the function name.  */
-          name_len = longest_to_int (exp->elts[string_pc].longconst);
-          func_name = (char *) alloca (name_len + 1);
-          strcpy (func_name, &exp->elts[string_pc + 1].string);
-
-          find_overload_match (&argvec[1], nargs, func_name,
-                               NON_METHOD, /* not method */
-                               NULL, NULL, /* pass NULL symbol since
-                                             symbol is unknown */
-                               NULL, &symp, NULL, 0);
-
-          /* Now fix the expression being evaluated.  */
-          exp->elts[save_pos1 + 2].symbol = symp;
-          argvec[0] = evaluate_subexp_with_coercion (exp, &save_pos1, noside);
-        }
-
-      if (op == STRUCTOP_STRUCT || op == STRUCTOP_PTR
-         || (op == OP_SCOPE && function_name != NULL))
-       {
-         int static_memfuncp;
-         char *tstr;
-
-         /* Method invocation : stuff "this" as first parameter.  */
-         argvec[1] = arg2;
-
-         if (op != OP_SCOPE)
-           {
-             /* Name of method from expression.  */
-             tstr = &exp->elts[pc2 + 2].string;
-           }
-         else
-           tstr = function_name;
-
-         if (overload_resolution && (exp->language_defn->la_language
-                                     == language_cplus))
-           {
-             /* Language is C++, do some overload resolution before
-                evaluation.  */
-             struct value *valp = NULL;
-
-             (void) find_overload_match (&argvec[1], nargs, tstr,
-                                         METHOD, /* method */
-                                         &arg2,  /* the object */
-                                         NULL, &valp, NULL,
-                                         &static_memfuncp, 0);
-
-             if (op == OP_SCOPE && !static_memfuncp)
-               {
-                 /* For the time being, we don't handle this.  */
-                 error (_("Call to overloaded function %s requires "
-                          "`this' pointer"),
-                        function_name);
-               }
-             argvec[1] = arg2; /* the ``this'' pointer */
-             argvec[0] = valp; /* Use the method found after overload
-                                  resolution.  */
-           }
-         else
-           /* Non-C++ case -- or no overload resolution.  */
-           {
-             struct value *temp = arg2;
-
-             argvec[0] = value_struct_elt (&temp, argvec + 1, tstr,
-                                           &static_memfuncp,
-                                           op == STRUCTOP_STRUCT
-                                      ? "structure" : "structure pointer");
-             /* value_struct_elt updates temp with the correct value
-                of the ``this'' pointer if necessary, so modify argvec[1] to
-                reflect any ``this'' changes.  */
-             arg2
-               = value_from_longest (lookup_pointer_type(value_type (temp)),
-                                     value_address (temp)
-                                     + value_embedded_offset (temp));
-             argvec[1] = arg2; /* the ``this'' pointer */
-           }
-
-         if (static_memfuncp)
-           {
-             argvec[1] = argvec[0];
-             nargs--;
-             argvec++;
-           }
-       }
-      else if (op == STRUCTOP_MEMBER || op == STRUCTOP_MPTR)
-       {
-         /* Pointer to member.  argvec[1] is already set up.  */
-         argvec[0] = arg1;
-       }
-      else if (op == OP_VAR_VALUE || (op == OP_SCOPE && function != NULL))
-       {
-         /* Non-member function being called.  */
-          /* fn: This can only be done for C++ functions.  A C-style function
-             in a C++ program, for instance, does not have the fields that 
-             are expected here.  */
-
-         if (overload_resolution && (exp->language_defn->la_language
-                                     == language_cplus))
-           {
-             /* Language is C++, do some overload resolution before
-                evaluation.  */
-             struct symbol *symp;
-             int no_adl = 0;
-
-             /* If a scope has been specified disable ADL.  */
-             if (op == OP_SCOPE)
-               no_adl = 1;
-
-             if (op == OP_VAR_VALUE)
-               function = exp->elts[save_pos1+2].symbol;
-
-             (void) find_overload_match (&argvec[1], nargs,
-                                         NULL,        /* no need for name */
-                                         NON_METHOD,  /* not method */
-                                         NULL, function, /* the function */
-                                         NULL, &symp, NULL, no_adl);
-
-             if (op == OP_VAR_VALUE)
-               {
-                 /* Now fix the expression being evaluated.  */
-                 exp->elts[save_pos1+2].symbol = symp;
-                 argvec[0] = evaluate_subexp_with_coercion (exp, &save_pos1,
-                                                            noside);
-               }
-             else
-               argvec[0] = value_of_variable (symp, get_selected_block (0));
-           }
-         else
-           {
-             /* Not C++, or no overload resolution allowed.  */
-             /* Nothing to be done; argvec already correctly set up.  */
-           }
-       }
-      else
-       {
-         /* It is probably a C-style function.  */
-         /* Nothing to be done; argvec already correctly set up.  */
-       }
-
-    do_call_it:
-
-      if (noside == EVAL_SKIP)
-       goto nosideret;
-      if (argvec[0] == NULL)
-       error (_("Cannot evaluate function -- may be inlined"));
-      if (noside == EVAL_AVOID_SIDE_EFFECTS)
-       {
-         /* If the return type doesn't look like a function type, call an
-            error.  This can happen if somebody tries to turn a variable into
-            a function call.  This is here because people often want to
-            call, eg, strcmp, which gdb doesn't know is a function.  If
-            gdb isn't asked for it's opinion (ie. through "whatis"),
-            it won't offer it.  */
-
-         struct type *ftype = value_type (argvec[0]);
-
-         if (TYPE_CODE (ftype) == TYPE_CODE_INTERNAL_FUNCTION)
-           {
-             /* We don't know anything about what the internal
-                function might return, but we have to return
-                something.  */
-             return value_zero (builtin_type (exp->gdbarch)->builtin_int,
-                                not_lval);
-           }
-         else if (TYPE_GNU_IFUNC (ftype))
-           return allocate_value (TYPE_TARGET_TYPE (TYPE_TARGET_TYPE (ftype)));
-         else if (TYPE_TARGET_TYPE (ftype))
-           return allocate_value (TYPE_TARGET_TYPE (ftype));
-         else
-           error (_("Expression of type other than "
-                    "\"Function returning ...\" used as function"));
-       }
-      if (TYPE_CODE (value_type (argvec[0])) == TYPE_CODE_INTERNAL_FUNCTION)
-       return call_internal_function (exp->gdbarch, exp->language_defn,
-                                      argvec[0], nargs, argvec + 1);
-
-      return call_function_by_hand (argvec[0], nargs, argvec + 1);
-      /* pai: FIXME save value from call_function_by_hand, then adjust
-        pc by adjust_fn_pc if +ve.  */
+      return evaluate_funcall (expect_type, exp, pos, noside);
 
     case OP_F77_UNDETERMINED_ARGLIST:
 
@@ -1800,13 +1926,13 @@ evaluate_subexp_standard (struct type *expect_type,
       switch (code)
        {
        case TYPE_CODE_ARRAY:
-         if (exp->elts[*pos].opcode == OP_F90_RANGE)
+         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_F90_RANGE)
+         if (exp->elts[*pos].opcode == OP_RANGE)
            return value_f90_subarray (arg1, exp, pos, noside);
          else
            {
@@ -1826,7 +1952,9 @@ evaluate_subexp_standard (struct type *expect_type,
          for (; tem <= nargs; tem++)
            argvec[tem] = evaluate_subexp_with_coercion (exp, pos, noside);
          argvec[tem] = 0;      /* signal end of arglist */
-         goto do_call_it;
+         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"));
@@ -1846,36 +1974,39 @@ evaluate_subexp_standard (struct type *expect_type,
       (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
-      /* Also handle EVAL_AVOID_SIDE_EFFECTS.  */
-      return value_struct_elt (&arg1, NULL, &exp->elts[pc + 2].string,
+       return eval_skip_value (exp);
+      arg3 = value_struct_elt (&arg1, NULL, &exp->elts[pc + 2].string,
                               NULL, "structure");
+      if (noside == EVAL_AVOID_SIDE_EFFECTS)
+       arg3 = value_zero (value_type (arg3), VALUE_LVAL (arg3));
+      return arg3;
 
     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);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
 
       /* Check to see if operator '->' has been overloaded.  If so replace
          arg1 with the value returned by evaluating operator->().  */
       while (unop_user_defined_p (op, arg1))
        {
-         volatile struct gdb_exception except;
          struct value *value = NULL;
-         TRY_CATCH (except, RETURN_MASK_ERROR)
+         TRY
            {
              value = value_x_unop (arg1, op, noside);
            }
 
-         if (except.reason < 0)
+         CATCH (except, RETURN_MASK_ERROR)
            {
              if (except.error == NOT_FOUND_ERROR)
                break;
              else
                throw_exception (except);
            }
+         END_CATCH
+
          arg1 = value;
        }
 
@@ -1885,12 +2016,13 @@ evaluate_subexp_standard (struct type *expect_type,
       {
         struct type *type = value_type (arg1);
         struct type *real_type;
-        int full, top, using_enc;
+        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_CLASS))
+            && (TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_STRUCT))
           {
             real_type = value_rtti_indirect_type (arg1, &full, &top,
                                                  &using_enc);
@@ -1899,9 +2031,11 @@ evaluate_subexp_standard (struct type *expect_type,
           }
       }
 
-      /* Also handle EVAL_AVOID_SIDE_EFFECTS.  */
-      return value_struct_elt (&arg1, NULL, &exp->elts[pc + 2].string,
+      arg3 = value_struct_elt (&arg1, NULL, &exp->elts[pc + 2].string,
                               NULL, "structure pointer");
+      if (noside == EVAL_AVOID_SIDE_EFFECTS)
+       arg3 = value_zero (value_type (arg3), VALUE_LVAL (arg3));
+      return arg3;
 
     case STRUCTOP_MEMBER:
     case STRUCTOP_MPTR:
@@ -1913,7 +2047,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
 
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
 
       type = check_typedef (value_type (arg2));
       switch (TYPE_CODE (type))
@@ -1930,7 +2064,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
        case TYPE_CODE_MEMBERPTR:
          /* Now, convert these values to an address.  */
-         arg1 = value_cast_pointers (lookup_pointer_type (TYPE_DOMAIN_TYPE (type)),
+         arg1 = value_cast_pointers (lookup_pointer_type (TYPE_SELF_TYPE (type)),
                                      arg1, 1);
 
          mem_offset = value_as_long (arg2);
@@ -1945,24 +2079,24 @@ evaluate_subexp_standard (struct type *expect_type,
        }
 
     case TYPE_INSTANCE:
-      nargs = longest_to_int (exp->elts[pc + 1].longconst);
-      arg_types = (struct type **) alloca (nargs * sizeof (struct type *));
-      for (ix = 0; ix < nargs; ++ix)
-       arg_types[ix] = exp->elts[pc + 1 + ix + 1].type;
-
-      expect_type = make_params (nargs, arg_types);
-      *(pos) += 3 + nargs;
-      arg1 = evaluate_subexp_standard (expect_type, exp, pos, noside);
-      xfree (TYPE_FIELDS (expect_type));
-      xfree (TYPE_MAIN_TYPE (expect_type));
-      xfree (expect_type);
-      return arg1;
+      {
+       type_instance_flags flags
+         = (type_instance_flag_value) longest_to_int (exp->elts[pc + 1].longconst);
+       nargs = longest_to_int (exp->elts[pc + 2].longconst);
+       arg_types = (struct type **) alloca (nargs * sizeof (struct 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);
+       *(pos) += 4 + nargs;
+       return evaluate_subexp_standard (expect_type.type (), exp, pos, noside);
+      }
 
     case BINOP_CONCAT:
       arg1 = evaluate_subexp_with_coercion (exp, pos, noside);
       arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        return value_x_binop (arg1, arg2, op, OP_NULL, noside);
       else
@@ -2016,7 +2150,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp_with_coercion (exp, pos, noside);
       arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       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))
@@ -2035,7 +2169,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp_with_coercion (exp, pos, noside);
       arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       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))
@@ -2068,7 +2202,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        return value_x_binop (arg1, arg2, op, OP_NULL, noside);
       else
@@ -2107,18 +2241,11 @@ evaluate_subexp_standard (struct type *expect_type,
            }
        }
 
-    case BINOP_RANGE:
-      evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      if (noside == EVAL_SKIP)
-       goto nosideret;
-      error (_("':' operator used in invalid context"));
-
     case BINOP_SUBSCRIPT:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        return value_x_binop (arg1, arg2, op, OP_NULL, noside);
       else
@@ -2144,15 +2271,6 @@ evaluate_subexp_standard (struct type *expect_type,
          else
            return value_subscript (arg1, value_as_long (arg2));
        }
-
-    case BINOP_IN:
-      arg1 = evaluate_subexp_with_coercion (exp, pos, noside);
-      arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
-      if (noside == EVAL_SKIP)
-       goto nosideret;
-      type = language_bool_type (exp->language_defn, exp->gdbarch);
-      return value_from_longest (type, (LONGEST) value_in (arg1, arg2));
-
     case MULTI_SUBSCRIPT:
       (*pos) += 2;
       nargs = longest_to_int (exp->elts[pc + 1].longconst);
@@ -2164,13 +2282,8 @@ evaluate_subexp_standard (struct type *expect_type,
          if (noside == EVAL_SKIP)
            {
              if (nargs > 0)
-               {
-                 continue;
-               }
-             else
-               {
-                 goto nosideret;
-               }
+               continue;
+             return eval_skip_value (exp);
            }
          /* FIXME:  EVAL_AVOID_SIDE_EFFECTS handling may not be correct.  */
          if (noside == EVAL_AVOID_SIDE_EFFECTS)
@@ -2269,7 +2382,7 @@ evaluate_subexp_standard (struct type *expect_type,
       if (noside == EVAL_SKIP)
        {
          evaluate_subexp (NULL_TYPE, exp, pos, noside);
-         goto nosideret;
+         return eval_skip_value (exp);
        }
 
       oldpos = *pos;
@@ -2296,7 +2409,7 @@ evaluate_subexp_standard (struct type *expect_type,
       if (noside == EVAL_SKIP)
        {
          evaluate_subexp (NULL_TYPE, exp, pos, noside);
-         goto nosideret;
+         return eval_skip_value (exp);
        }
 
       oldpos = *pos;
@@ -2322,7 +2435,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        {
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
@@ -2339,7 +2452,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        {
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
@@ -2356,7 +2469,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        {
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
@@ -2373,7 +2486,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        {
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
@@ -2390,7 +2503,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        {
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
@@ -2407,7 +2520,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (binop_user_defined_p (op, arg1, arg2))
        {
          return value_x_binop (arg1, arg2, op, OP_NULL, noside);
@@ -2424,9 +2537,10 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       type = check_typedef (value_type (arg2));
-      if (TYPE_CODE (type) != TYPE_CODE_INT)
+      if (TYPE_CODE (type) != TYPE_CODE_INT
+          && TYPE_CODE (type) != TYPE_CODE_ENUM)
        error (_("Non-integral right operand for \"@\" operator."));
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
@@ -2443,7 +2557,7 @@ evaluate_subexp_standard (struct type *expect_type,
     case UNOP_PLUS:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
        return value_x_unop (arg1, op, noside);
       else
@@ -2455,7 +2569,7 @@ evaluate_subexp_standard (struct type *expect_type,
     case UNOP_NEG:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
        return value_x_unop (arg1, op, noside);
       else
@@ -2466,11 +2580,10 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case UNOP_COMPLEMENT:
       /* C++: check for and handle destructor names.  */
-      op = exp->elts[*pos].opcode;
 
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (unop_user_defined_p (UNOP_COMPLEMENT, arg1))
        return value_x_unop (arg1, UNOP_COMPLEMENT, noside);
       else
@@ -2482,7 +2595,7 @@ evaluate_subexp_standard (struct type *expect_type,
     case UNOP_LOGICAL_NOT:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
        return value_x_unop (arg1, op, noside);
       else
@@ -2501,14 +2614,14 @@ evaluate_subexp_standard (struct type *expect_type,
        error (_("Attempt to dereference pointer "
                 "to member without an object"));
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (unop_user_defined_p (op, arg1))
        return value_x_unop (arg1, op, noside);
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
          type = check_typedef (value_type (arg1));
          if (TYPE_CODE (type) == TYPE_CODE_PTR
-             || TYPE_CODE (type) == TYPE_CODE_REF
+             || TYPE_IS_REFERENCE (type)
          /* In C you can dereference an array to get the 1st elt.  */
              || TYPE_CODE (type) == TYPE_CODE_ARRAY
            )
@@ -2534,12 +2647,10 @@ evaluate_subexp_standard (struct type *expect_type,
     case UNOP_ADDR:
       /* C++: check for and handle pointer to members.  */
 
-      op = exp->elts[*pos].opcode;
-
       if (noside == EVAL_SKIP)
        {
          evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
-         goto nosideret;
+         return eval_skip_value (exp);
        }
       else
        {
@@ -2553,36 +2664,39 @@ evaluate_subexp_standard (struct type *expect_type,
       if (noside == EVAL_SKIP)
        {
          evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
-         goto nosideret;
+         return eval_skip_value (exp);
        }
-      return evaluate_subexp_for_sizeof (exp, pos);
+      return evaluate_subexp_for_sizeof (exp, pos, noside);
+
+    case UNOP_ALIGNOF:
+      {
+       struct type *type
+         = value_type (evaluate_subexp (NULL_TYPE, 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;
-      arg1 = evaluate_subexp (type, exp, pos, noside);
-      if (noside == EVAL_SKIP)
-       goto nosideret;
-      if (type != value_type (arg1))
-       arg1 = value_cast (type, arg1);
-      return arg1;
+      return evaluate_subexp_for_cast (exp, pos, noside, type);
 
     case UNOP_CAST_TYPE:
       arg1 = evaluate_subexp (NULL, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       type = value_type (arg1);
-      arg1 = evaluate_subexp (type, exp, pos, noside);
-      if (noside == EVAL_SKIP)
-       goto nosideret;
-      if (type != value_type (arg1))
-       arg1 = value_cast (type, arg1);
-      return arg1;
+      return evaluate_subexp_for_cast (exp, pos, noside, type);
 
     case UNOP_DYNAMIC_CAST:
       arg1 = evaluate_subexp (NULL, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       type = value_type (arg1);
       arg1 = evaluate_subexp (type, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       return value_dynamic_cast (type, arg1);
 
     case UNOP_REINTERPRET_CAST:
@@ -2590,14 +2704,14 @@ evaluate_subexp_standard (struct type *expect_type,
       type = value_type (arg1);
       arg1 = evaluate_subexp (type, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       return value_reinterpret_cast (type, arg1);
 
     case UNOP_MEMVAL:
       (*pos) += 2;
       arg1 = evaluate_subexp (expect_type, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
        return value_zero (exp->elts[pc + 1].type, lval_memory);
       else
@@ -2609,28 +2723,12 @@ evaluate_subexp_standard (struct type *expect_type,
       type = value_type (arg1);
       arg1 = evaluate_subexp (expect_type, exp, pos, noside);
       if (noside == EVAL_SKIP)
-       goto nosideret;
+       return eval_skip_value (exp);
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
        return value_zero (type, lval_memory);
       else
        return value_at_lazy (type, value_as_address (arg1));
 
-    case UNOP_MEMVAL_TLS:
-      (*pos) += 3;
-      arg1 = evaluate_subexp (expect_type, exp, pos, noside);
-      if (noside == EVAL_SKIP)
-       goto nosideret;
-      if (noside == EVAL_AVOID_SIDE_EFFECTS)
-       return value_zero (exp->elts[pc + 2].type, lval_memory);
-      else
-       {
-         CORE_ADDR tls_addr;
-
-         tls_addr = target_translate_tls_address (exp->elts[pc + 1].objfile,
-                                                  value_as_address (arg1));
-         return value_at_lazy (exp->elts[pc + 2].type, tls_addr);
-       }
-
     case UNOP_PREINCREMENT:
       arg1 = evaluate_subexp (expect_type, exp, pos, noside);
       if (noside == EVAL_SKIP || noside == EVAL_AVOID_SIDE_EFFECTS)
@@ -2742,20 +2840,9 @@ evaluate_subexp_standard (struct type *expect_type,
          easier to accommodate expressions that contain types.  */
       (*pos) += 2;
       if (noside == EVAL_SKIP)
-        goto nosideret;
+       return eval_skip_value (exp);
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
-       {
-         struct type *type = exp->elts[pc + 1].type;
-
-         /* If this is a typedef, then find its immediate target.  We
-            use check_typedef to resolve stubs, but we ignore its
-            result because we do not want to dig past all
-            typedefs.  */
-         check_typedef (type);
-         if (TYPE_CODE (type) == TYPE_CODE_TYPEDEF)
-           type = TYPE_TARGET_TYPE (type);
-         return allocate_value (type);
-       }
+       return allocate_value (exp->elts[pc + 1].type);
       else
         error (_("Attempt to use a type name as an expression"));
 
@@ -2764,7 +2851,7 @@ evaluate_subexp_standard (struct type *expect_type,
       if (noside == EVAL_SKIP)
        {
          evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
-         goto nosideret;
+         return eval_skip_value (exp);
        }
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
@@ -2786,9 +2873,9 @@ evaluate_subexp_standard (struct type *expect_type,
            {
              struct type *type = value_type (result);
 
-             if (TYPE_CODE (check_typedef (type)) != TYPE_CODE_REF)
+             if (!TYPE_IS_REFERENCE (type))
                {
-                 type = lookup_reference_type (type);
+                 type = lookup_lvalue_reference_type (type);
                  result = allocate_value (type);
                }
            }
@@ -2798,6 +2885,23 @@ evaluate_subexp_standard (struct type *expect_type,
       else
         error (_("Attempt to use a type as an expression"));
 
+    case OP_TYPEID:
+      {
+       struct value *result;
+       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);
+       else
+         result = evaluate_subexp (NULL_TYPE, exp, pos, noside);
+
+       if (noside != EVAL_NORMAL)
+         return allocate_value (cplus_typeid_type (exp->gdbarch));
+
+       return cplus_typeid (result);
+      }
+
     default:
       /* Removing this case and compiling with gcc -Wall reveals that
          a lot of cases are hitting this case.  Some of these should
@@ -2812,8 +2916,7 @@ evaluate_subexp_standard (struct type *expect_type,
               "evaluate that kind of expression"));
     }
 
-nosideret:
-  return value_from_longest (builtin_type (exp->gdbarch)->builtin_int, 1);
+  gdb_assert_not_reached ("missed return?");
 }
 \f
 /* Evaluate a subexpression of EXP, at index *POS,
@@ -2872,7 +2975,7 @@ evaluate_subexp_for_address (struct expression *exp, int *pos,
 
       /* C++: The "address" of a reference should yield the address
        * of the object pointed to.  Let value_addr() deal with it.  */
-      if (TYPE_CODE (SYMBOL_TYPE (var)) == TYPE_CODE_REF)
+      if (TYPE_IS_REFERENCE (SYMBOL_TYPE (var)))
        goto default_case;
 
       (*pos) += 4;
@@ -2893,6 +2996,22 @@ evaluate_subexp_for_address (struct expression *exp, int *pos,
       else
        return address_of_variable (var, exp->elts[pc + 1].block);
 
+    case OP_VAR_MSYM_VALUE:
+      {
+       (*pos) += 4;
+
+       value *val = evaluate_var_msym_value (noside,
+                                             exp->elts[pc + 1].objfile,
+                                             exp->elts[pc + 2].msymbol);
+       if (noside == EVAL_AVOID_SIDE_EFFECTS)
+         {
+           struct type *type = lookup_pointer_type (value_type (val));
+           return value_zero (type, not_lval);
+         }
+       else
+         return value_addr (val);
+      }
+
     case OP_SCOPE:
       tem = longest_to_int (exp->elts[pc + 2].longconst);
       (*pos) += 5 + BYTES_TO_EXP_ELEM (tem + 1);
@@ -2911,7 +3030,7 @@ evaluate_subexp_for_address (struct expression *exp, int *pos,
        {
          struct type *type = check_typedef (value_type (x));
 
-         if (TYPE_CODE (type) == TYPE_CODE_REF)
+         if (TYPE_IS_REFERENCE (type))
            return value_zero (lookup_pointer_type (TYPE_TARGET_TYPE (type)),
                               not_lval);
          else if (VALUE_LVAL (x) == lval_memory || value_must_coerce_to_target (x))
@@ -2972,10 +3091,13 @@ evaluate_subexp_with_coercion (struct expression *exp,
 
 /* Evaluate a subexpression of EXP, at index *POS,
    and return a value for the size of that subexpression.
-   Advance *POS over the subexpression.  */
+   Advance *POS over the subexpression.  If NOSIDE is EVAL_NORMAL
+   we allow side-effects on the operand if its type is a variable
+   length array.   */
 
 static struct value *
-evaluate_subexp_for_sizeof (struct expression *exp, int *pos)
+evaluate_subexp_for_sizeof (struct expression *exp, int *pos,
+                           enum noside noside)
 {
   /* FIXME: This should be size_t.  */
   struct type *size_type = builtin_type (exp->gdbarch)->builtin_int;
@@ -2998,34 +3120,157 @@ evaluate_subexp_for_sizeof (struct expression *exp, int *pos)
       val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       type = check_typedef (value_type (val));
       if (TYPE_CODE (type) != TYPE_CODE_PTR
-         && TYPE_CODE (type) != TYPE_CODE_REF
+         && !TYPE_IS_REFERENCE (type)
          && TYPE_CODE (type) != TYPE_CODE_ARRAY)
        error (_("Attempt to take contents of a non-pointer value."));
-      type = check_typedef (TYPE_TARGET_TYPE (type));
+      type = TYPE_TARGET_TYPE (type);
+      if (is_dynamic_type (type))
+       type = value_type (value_ind (val));
       return value_from_longest (size_type, (LONGEST) TYPE_LENGTH (type));
 
     case UNOP_MEMVAL:
       (*pos) += 3;
-      type = check_typedef (exp->elts[pc + 1].type);
-      return value_from_longest (size_type, (LONGEST) TYPE_LENGTH (type));
+      type = exp->elts[pc + 1].type;
+      break;
 
     case UNOP_MEMVAL_TYPE:
       (*pos) += 1;
       val = evaluate_subexp (NULL, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
-      type = check_typedef (value_type (val));
-      return value_from_longest (size_type, (LONGEST) TYPE_LENGTH (type));
+      type = value_type (val);
+      break;
 
     case OP_VAR_VALUE:
-      (*pos) += 4;
-      type = check_typedef (SYMBOL_TYPE (exp->elts[pc + 2].symbol));
-      return
-       value_from_longest (size_type, (LONGEST) TYPE_LENGTH (type));
+      type = SYMBOL_TYPE (exp->elts[pc + 2].symbol);
+      if (is_dynamic_type (type))
+       {
+         val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_NORMAL);
+         type = value_type (val);
+         if (TYPE_CODE (type) == TYPE_CODE_ARRAY
+              && is_dynamic_type (TYPE_INDEX_TYPE (type))
+              && TYPE_HIGH_BOUND_UNDEFINED (TYPE_INDEX_TYPE (type)))
+           return allocate_optimized_out_value (size_type);
+       }
+      else
+       (*pos) += 4;
+      break;
+
+    case OP_VAR_MSYM_VALUE:
+      {
+       (*pos) += 4;
+
+       minimal_symbol *msymbol = exp->elts[pc + 2].msymbol;
+       value *val = 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));
+
+       return value_from_longest (size_type, TYPE_LENGTH (type));
+      }
+      break;
+
+      /* 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
+        side-effects. */
+    case BINOP_SUBSCRIPT:
+      if (noside == EVAL_NORMAL)
+       {
+         int pc = (*pos) + 1;
+
+         val = evaluate_subexp (NULL_TYPE, exp, &pc, EVAL_AVOID_SIDE_EFFECTS);
+         type = check_typedef (value_type (val));
+         if (TYPE_CODE (type) == TYPE_CODE_ARRAY)
+           {
+             type = check_typedef (TYPE_TARGET_TYPE (type));
+             if (TYPE_CODE (type) == TYPE_CODE_ARRAY)
+               {
+                 type = TYPE_INDEX_TYPE (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)
+                   {
+                     val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_NORMAL);
+                     return value_from_longest
+                       (size_type, (LONGEST) TYPE_LENGTH (value_type (val)));
+                   }
+               }
+           }
+       }
+
+      /* Fall through.  */
 
     default:
       val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
-      return value_from_longest (size_type,
-                                (LONGEST) TYPE_LENGTH (value_type (val)));
+      type = value_type (val);
+      break;
+    }
+
+  /* $5.3.3/2 of the C++ Standard (n3290 draft) says of sizeof:
+     "When applied to a reference or a reference type, the result is
+     the size of the referenced type."  */
+  type = check_typedef (type);
+  if (exp->language_defn->la_language == language_cplus
+      && (TYPE_IS_REFERENCE (type)))
+    type = check_typedef (TYPE_TARGET_TYPE (type));
+  return value_from_longest (size_type, (LONGEST) TYPE_LENGTH (type));
+}
+
+/* Evaluate a subexpression of EXP, at index *POS, and return a value
+   for that subexpression cast to TO_TYPE.  Advance *POS over the
+   subexpression.  */
+
+static value *
+evaluate_subexp_for_cast (expression *exp, int *pos,
+                         enum noside noside,
+                         struct type *to_type)
+{
+  int pc = *pos;
+
+  /* Don't let symbols be evaluated with evaluate_subexp because that
+     throws an "unknown type" error for no-debug data symbols.
+     Instead, we want the cast to reinterpret the symbol.  */
+  if (exp->elts[pc].opcode == OP_VAR_MSYM_VALUE
+      || exp->elts[pc].opcode == OP_VAR_VALUE)
+    {
+      (*pos) += 4;
+
+      value *val;
+      if (exp->elts[pc].opcode == OP_VAR_MSYM_VALUE)
+       {
+         if (noside == EVAL_AVOID_SIDE_EFFECTS)
+           return value_zero (to_type, not_lval);
+
+         val = evaluate_var_msym_value (noside,
+                                        exp->elts[pc + 1].objfile,
+                                        exp->elts[pc + 2].msymbol);
+       }
+      else
+       val = evaluate_var_value (noside,
+                                 exp->elts[pc + 1].block,
+                                 exp->elts[pc + 2].symbol);
+
+      if (noside == EVAL_SKIP)
+       return eval_skip_value (exp);
+
+      val = value_cast (to_type, val);
+
+      /* Don't allow e.g. '&(int)var_with_no_debug_info'.  */
+      if (VALUE_LVAL (val) == lval_memory)
+       {
+         if (value_lazy (val))
+           value_fetch_lazy (val);
+         VALUE_LVAL (val) = not_lval;
+       }
+      return val;
     }
+
+  value *val = evaluate_subexp (to_type, exp, pos, noside);
+  if (noside == EVAL_SKIP)
+    return eval_skip_value (exp);
+  return value_cast (to_type, val);
 }
 
 /* Parse a type expression in the string [P..P+LENGTH).  */
@@ -3034,14 +3279,13 @@ struct type *
 parse_and_eval_type (char *p, int length)
 {
   char *tmp = (char *) alloca (length + 4);
-  struct expression *expr;
 
   tmp[0] = '(';
   memcpy (tmp + 1, p, length);
   tmp[length + 1] = ')';
   tmp[length + 2] = '0';
   tmp[length + 3] = '\0';
-  expr = parse_expression (tmp);
+  expression_up expr = parse_expression (tmp);
   if (expr->elts[0].opcode != UNOP_CAST)
     error (_("Internal error in eval_type."));
   return expr->elts[1].type;