2005-02-07 Andrew Cagney <cagney@gnu.org>
[deliverable/binutils-gdb.git] / gdb / eval.c
index 3b1a06c94cbf03185abf83ba5306c5eaa8280ff3..79a8ffd329ec4533ae63d1f56cb7c82f023aa7af 100644 (file)
@@ -1,7 +1,8 @@
 /* Evaluate expressions for GDB.
-   Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995,
-   1996, 1997, 1998, 1999, 2000, 2001
-   Free Software Foundation, Inc.
+
+   Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994,
+   1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2005 Free
+   Software Foundation, Inc.
 
    This file is part of GDB.
 
 #include "language.h"          /* For CAST_IS_CONVERSION */
 #include "f-lang.h"            /* for array bound stuff */
 #include "cp-abi.h"
-
-/* Defined in symtab.c */
-extern int hp_som_som_object_present;
+#include "infcall.h"
+#include "objc-lang.h"
+#include "block.h"
+#include "parser-defs.h"
 
 /* This is defined in valops.c */
 extern int overload_resolution;
@@ -63,10 +65,11 @@ static LONGEST init_array_element (struct value *, struct value *,
                                   LONGEST, LONGEST);
 
 static struct value *
-evaluate_subexp (struct type *expect_type, register struct expression *exp,
-                register int *pos, enum noside noside)
+evaluate_subexp (struct type *expect_type, struct expression *exp,
+                int *pos, enum noside noside)
 {
-  return (*exp->language_defn->evaluate_exp) (expect_type, exp, pos, noside);
+  return (*exp->language_defn->la_exp_desc->evaluate_exp) 
+    (expect_type, exp, pos, noside);
 }
 \f
 /* Parse the string EXP as a C expression, evaluate it,
@@ -76,9 +79,9 @@ CORE_ADDR
 parse_and_eval_address (char *exp)
 {
   struct expression *expr = parse_expression (exp);
-  register CORE_ADDR addr;
-  register struct cleanup *old_chain =
-  make_cleanup (free_current_contents, &expr);
+  CORE_ADDR addr;
+  struct cleanup *old_chain =
+    make_cleanup (free_current_contents, &expr);
 
   addr = value_as_address (evaluate_expression (expr));
   do_cleanups (old_chain);
@@ -92,9 +95,9 @@ CORE_ADDR
 parse_and_eval_address_1 (char **expptr)
 {
   struct expression *expr = parse_exp_1 (expptr, (struct block *) 0, 0);
-  register CORE_ADDR addr;
-  register struct cleanup *old_chain =
-  make_cleanup (free_current_contents, &expr);
+  CORE_ADDR addr;
+  struct cleanup *old_chain =
+    make_cleanup (free_current_contents, &expr);
 
   addr = value_as_address (evaluate_expression (expr));
   do_cleanups (old_chain);
@@ -107,8 +110,8 @@ LONGEST
 parse_and_eval_long (char *exp)
 {
   struct expression *expr = parse_expression (exp);
-  register LONGEST retval;
-  register struct cleanup *old_chain =
+  LONGEST retval;
+  struct cleanup *old_chain =
     make_cleanup (free_current_contents, &expr);
 
   retval = value_as_long (evaluate_expression (expr));
@@ -121,8 +124,8 @@ parse_and_eval (char *exp)
 {
   struct expression *expr = parse_expression (exp);
   struct value *val;
-  register struct cleanup *old_chain
-  = make_cleanup (free_current_contents, &expr);
+  struct cleanup *old_chain =
+    make_cleanup (free_current_contents, &expr);
 
   val = evaluate_expression (expr);
   do_cleanups (old_chain);
@@ -138,8 +141,8 @@ parse_to_comma_and_eval (char **expp)
 {
   struct expression *expr = parse_exp_1 (expp, (struct block *) 0, 1);
   struct value *val;
-  register struct cleanup *old_chain
-  = make_cleanup (free_current_contents, &expr);
+  struct cleanup *old_chain =
+    make_cleanup (free_current_contents, &expr);
 
   val = evaluate_expression (expr);
   do_cleanups (old_chain);
@@ -172,7 +175,7 @@ evaluate_type (struct expression *exp)
    returning the label.  Otherwise, does nothing and returns NULL. */
 
 static char *
-get_label (register struct expression *exp, int *pos)
+get_label (struct expression *exp, int *pos)
 {
   if (exp->elts[*pos].opcode == OP_LABELED)
     {
@@ -186,15 +189,15 @@ get_label (register struct expression *exp, int *pos)
     return NULL;
 }
 
-/* This function evaluates tuples (in Chill) or brace-initializers
-   (in C/C++) for structure types.  */
+/* This function evaluates tuples (in (the deleted) Chill) or
+   brace-initializers (in C/C++) for structure types.  */
 
 static struct value *
 evaluate_struct_tuple (struct value *struct_val,
-                      register struct expression *exp,
-                      register int *pos, enum noside noside, int nargs)
+                      struct expression *exp,
+                      int *pos, enum noside noside, int nargs)
 {
-  struct type *struct_type = check_typedef (VALUE_TYPE (struct_val));
+  struct type *struct_type = check_typedef (value_type (struct_val));
   struct type *substruct_type = struct_type;
   struct type *field_type;
   int fieldno = -1;
@@ -206,7 +209,7 @@ evaluate_struct_tuple (struct value *struct_val,
       struct value *val = NULL;
       int nlabels = 0;
       int bitpos, bitsize;
-      char *addr;
+      bfd_byte *addr;
 
       /* Skip past the labels, and count them. */
       while (get_label (exp, pos) != NULL)
@@ -221,7 +224,7 @@ evaluate_struct_tuple (struct value *struct_val,
                   fieldno++)
                {
                  char *field_name = TYPE_FIELD_NAME (struct_type, fieldno);
-                 if (field_name != NULL && STREQ (field_name, label))
+                 if (field_name != NULL && DEPRECATED_STREQ (field_name, label))
                    {
                      variantno = -1;
                      subfieldno = fieldno;
@@ -249,7 +252,7 @@ evaluate_struct_tuple (struct value *struct_val,
                                 subfieldno < TYPE_NFIELDS (substruct_type);
                                   subfieldno++)
                                {
-                                 if (STREQ (TYPE_FIELD_NAME (substruct_type,
+                                 if (DEPRECATED_STREQ (TYPE_FIELD_NAME (substruct_type,
                                                              subfieldno),
                                             label))
                                    {
@@ -305,41 +308,40 @@ evaluate_struct_tuple (struct value *struct_val,
          /* Now actually set the field in struct_val. */
 
          /* Assign val to field fieldno. */
-         if (VALUE_TYPE (val) != field_type)
+         if (value_type (val) != field_type)
            val = value_cast (field_type, val);
 
          bitsize = TYPE_FIELD_BITSIZE (substruct_type, subfieldno);
          bitpos = TYPE_FIELD_BITPOS (struct_type, fieldno);
          if (variantno >= 0)
            bitpos += TYPE_FIELD_BITPOS (substruct_type, subfieldno);
-         addr = VALUE_CONTENTS (struct_val) + bitpos / 8;
+         addr = value_contents_writeable (struct_val) + bitpos / 8;
          if (bitsize)
            modify_field (addr, value_as_long (val),
                          bitpos % 8, bitsize);
          else
-           memcpy (addr, VALUE_CONTENTS (val),
-                   TYPE_LENGTH (VALUE_TYPE (val)));
+           memcpy (addr, value_contents (val),
+                   TYPE_LENGTH (value_type (val)));
        }
       while (--nlabels > 0);
     }
   return struct_val;
 }
 
-/* Recursive helper function for setting elements of array tuples for 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 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.  */
 
 static LONGEST
 init_array_element (struct value *array, struct value *element,
-                   register struct expression *exp, register int *pos,
+                   struct expression *exp, int *pos,
                    enum noside noside, LONGEST low_bound, LONGEST high_bound)
 {
   LONGEST index;
-  int element_size = TYPE_LENGTH (VALUE_TYPE (element));
+  int element_size = TYPE_LENGTH (value_type (element));
   if (exp->elts[*pos].opcode == BINOP_COMMA)
     {
       (*pos)++;
@@ -358,9 +360,9 @@ init_array_element (struct value *array, struct value *element,
        error ("tuple range index out of range");
       for (index = low; index <= high; index++)
        {
-         memcpy (VALUE_CONTENTS_RAW (array)
+         memcpy (value_contents_raw (array)
                  + (index - low_bound) * element_size,
-                 VALUE_CONTENTS (element), element_size);
+                 value_contents (element), element_size);
        }
     }
   else
@@ -368,20 +370,20 @@ init_array_element (struct value *array, struct value *element,
       index = value_as_long (evaluate_subexp (NULL_TYPE, exp, pos, noside));
       if (index < low_bound || index > high_bound)
        error ("tuple index out of range");
-      memcpy (VALUE_CONTENTS_RAW (array) + (index - low_bound) * element_size,
-             VALUE_CONTENTS (element), element_size);
+      memcpy (value_contents_raw (array) + (index - low_bound) * element_size,
+             value_contents (element), element_size);
     }
   return index;
 }
 
 struct value *
 evaluate_subexp_standard (struct type *expect_type,
-                         register struct expression *exp, register int *pos,
+                         struct expression *exp, int *pos,
                          enum noside noside)
 {
   enum exp_opcode op;
   int tem, tem2, tem3;
-  register int pc, pc2 = 0, oldpos;
+  int pc, pc2 = 0, oldpos;
   struct value *arg1 = NULL;
   struct value *arg2 = NULL;
   struct value *arg3;
@@ -403,11 +405,9 @@ evaluate_subexp_standard (struct type *expect_type,
     case OP_SCOPE:
       tem = longest_to_int (exp->elts[pc + 2].longconst);
       (*pos) += 4 + BYTES_TO_EXP_ELEM (tem + 1);
-      arg1 = value_struct_elt_for_reference (exp->elts[pc + 1].type,
-                                            0,
-                                            exp->elts[pc + 1].type,
-                                            &exp->elts[pc + 3].string,
-                                            NULL_TYPE);
+      arg1 = value_aggregate_elt (exp->elts[pc + 1].type,
+                                 &exp->elts[pc + 3].string,
+                                 noside);
       if (arg1 == NULL)
        error ("There is no field named %s", &exp->elts[pc + 3].string);
       return arg1;
@@ -447,11 +447,11 @@ evaluate_subexp_standard (struct type *expect_type,
     case OP_REGISTER:
       {
        int regno = longest_to_int (exp->elts[pc + 1].longconst);
-       struct value *val = value_of_register (regno);
-
+       struct value *val = value_of_register (regno, get_selected_frame (NULL));
        (*pos) += 2;
        if (val == NULL)
-         error ("Value of register %s not available.", REGISTER_NAME (regno));
+         error ("Value of register %s not available.",
+                frame_map_regnum_to_name (get_selected_frame (NULL), regno));
        else
          return val;
       }
@@ -471,6 +471,15 @@ evaluate_subexp_standard (struct type *expect_type,
        goto nosideret;
       return value_string (&exp->elts[pc + 2].string, tem);
 
+    case OP_OBJC_NSSTRING:             /* Objective C Foundation Class NSString constant.  */
+      tem = longest_to_int (exp->elts[pc + 1].longconst);
+      (*pos) += 3 + BYTES_TO_EXP_ELEM (tem + 1);
+      if (noside == EVAL_SKIP)
+       {
+         goto nosideret;
+       }
+      return (struct value *) value_nsstring (&exp->elts[pc + 2].string, tem + 1);
+
     case OP_BITSTRING:
       tem = longest_to_int (exp->elts[pc + 1].longconst);
       (*pos)
@@ -491,7 +500,7 @@ evaluate_subexp_standard (struct type *expect_type,
          && TYPE_CODE (type) == TYPE_CODE_STRUCT)
        {
          struct value *rec = allocate_value (expect_type);
-         memset (VALUE_CONTENTS_RAW (rec), '\0', TYPE_LENGTH (type));
+         memset (value_contents_raw (rec), '\0', TYPE_LENGTH (type));
          return evaluate_struct_tuple (rec, exp, pos, noside, nargs);
        }
 
@@ -509,7 +518,7 @@ evaluate_subexp_standard (struct type *expect_type,
              high_bound = (TYPE_LENGTH (type) / element_size) - 1;
            }
          index = low_bound;
-         memset (VALUE_CONTENTS_RAW (array), 0, TYPE_LENGTH (expect_type));
+         memset (value_contents_raw (array), 0, TYPE_LENGTH (expect_type));
          for (tem = nargs; --nargs >= 0;)
            {
              struct value *element;
@@ -520,7 +529,7 @@ evaluate_subexp_standard (struct type *expect_type,
                  evaluate_subexp (NULL_TYPE, exp, pos, EVAL_SKIP);
                }
              element = evaluate_subexp (element_type, exp, pos, noside);
-             if (VALUE_TYPE (element) != element_type)
+             if (value_type (element) != element_type)
                element = value_cast (element_type, element);
              if (index_pc)
                {
@@ -535,9 +544,9 @@ evaluate_subexp_standard (struct type *expect_type,
                  if (index > high_bound)
                    /* to avoid memory corruption */
                    error ("Too many array elements");
-                 memcpy (VALUE_CONTENTS_RAW (array)
+                 memcpy (value_contents_raw (array)
                          + (index - low_bound) * element_size,
-                         VALUE_CONTENTS (element),
+                         value_contents (element),
                          element_size);
                }
              index++;
@@ -549,7 +558,7 @@ evaluate_subexp_standard (struct type *expect_type,
          && TYPE_CODE (type) == TYPE_CODE_SET)
        {
          struct value *set = allocate_value (expect_type);
-         char *valaddr = VALUE_CONTENTS_RAW (set);
+         char *valaddr = value_contents_raw (set);
          struct type *element_type = TYPE_INDEX_TYPE (type);
          struct type *check_type = element_type;
          LONGEST low_bound, high_bound;
@@ -571,16 +580,16 @@ evaluate_subexp_standard (struct type *expect_type,
                {
                  (*pos)++;
                  elem_val = evaluate_subexp (element_type, exp, pos, noside);
-                 range_low_type = VALUE_TYPE (elem_val);
+                 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_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_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
@@ -620,7 +629,7 @@ evaluate_subexp_standard (struct type *expect_type,
          return set;
        }
 
-      argvec = (value_ptr *) alloca (sizeof (value_ptr) * nargs);
+      argvec = (struct value **) alloca (sizeof (struct value *) * nargs);
       for (tem = 0; tem < nargs; tem++)
        {
          /* Ensure that array expressions are coerced into pointer objects. */
@@ -667,13 +676,292 @@ evaluate_subexp_standard (struct type *expect_type,
          return arg2;
        }
 
+    case OP_OBJC_SELECTOR:
+      {                                /* Objective C @selector operator.  */
+       char *sel = &exp->elts[pc + 2].string;
+       int len = longest_to_int (exp->elts[pc + 1].longconst);
+
+       (*pos) += 3 + BYTES_TO_EXP_ELEM (len + 1);
+       if (noside == EVAL_SKIP)
+         goto nosideret;
+
+       if (sel[len] != 0)
+         sel[len] = 0;         /* Make sure it's terminated.  */
+       return value_from_longest (lookup_pointer_type (builtin_type_void),
+                                  lookup_child_selector (sel));
+      }
+
+    case OP_OBJC_MSGCALL:
+      {                                /* Objective C message (method) call.  */
+
+       static CORE_ADDR responds_selector = 0;
+       static CORE_ADDR method_selector = 0;
+
+       CORE_ADDR selector = 0;
+
+       int using_gcc = 0;
+       int struct_return = 0;
+       int sub_no_side = 0;
+
+       static struct value *msg_send = NULL;
+       static struct value *msg_send_stret = NULL;
+       static int gnu_runtime = 0;
+
+       struct value *target = NULL;
+       struct value *method = NULL;
+       struct value *called_method = NULL; 
+
+       struct type *selector_type = NULL;
+
+       struct value *ret = NULL;
+       CORE_ADDR addr = 0;
+
+       selector = exp->elts[pc + 1].longconst;
+       nargs = exp->elts[pc + 2].longconst;
+       argvec = (struct value **) alloca (sizeof (struct value *) 
+                                          * (nargs + 5));
+
+       (*pos) += 3;
+
+       selector_type = lookup_pointer_type (builtin_type_void);
+       if (noside == EVAL_AVOID_SIDE_EFFECTS)
+         sub_no_side = EVAL_NORMAL;
+       else
+         sub_no_side = noside;
+
+       target = evaluate_subexp (selector_type, exp, pos, sub_no_side);
+
+       if (value_as_long (target) == 0)
+         return value_from_longest (builtin_type_long, 0);
+       
+       if (lookup_minimal_symbol ("objc_msg_lookup", 0, 0))
+         gnu_runtime = 1;
+       
+       /* Find the method dispatch (Apple runtime) or method lookup
+          (GNU runtime) function for Objective-C.  These will be used
+          to lookup the symbol information for the method.  If we
+          can't find any symbol information, then we'll use these to
+          call the method, otherwise we can call the method
+          directly. The msg_send_stret function is used in the special
+          case of a method that returns a structure (Apple runtime 
+          only).  */
+       if (gnu_runtime)
+         {
+           struct type *type;
+           type = lookup_pointer_type (builtin_type_void);
+           type = lookup_function_type (type);
+           type = lookup_pointer_type (type);
+           type = lookup_function_type (type);
+           type = lookup_pointer_type (type);
+
+           msg_send = find_function_in_inferior ("objc_msg_lookup");
+           msg_send_stret = find_function_in_inferior ("objc_msg_lookup");
+
+           msg_send = value_from_pointer (type, value_as_address (msg_send));
+           msg_send_stret = value_from_pointer (type, 
+                                       value_as_address (msg_send_stret));
+         }
+       else
+         {
+           msg_send = find_function_in_inferior ("objc_msgSend");
+           /* Special dispatcher for methods returning structs */
+           msg_send_stret = find_function_in_inferior ("objc_msgSend_stret");
+         }
+
+       /* Verify the target object responds to this method. The
+          standard top-level 'Object' class uses a different name for
+          the verification method than the non-standard, but more
+          often used, 'NSObject' class. Make sure we check for both. */
+
+       responds_selector = lookup_child_selector ("respondsToSelector:");
+       if (responds_selector == 0)
+         responds_selector = lookup_child_selector ("respondsTo:");
+       
+       if (responds_selector == 0)
+         error ("no 'respondsTo:' or 'respondsToSelector:' method");
+       
+       method_selector = lookup_child_selector ("methodForSelector:");
+       if (method_selector == 0)
+         method_selector = lookup_child_selector ("methodFor:");
+       
+       if (method_selector == 0)
+         error ("no 'methodFor:' or 'methodForSelector:' method");
+
+       /* Call the verification method, to make sure that the target
+        class implements the desired method. */
+
+       argvec[0] = msg_send;
+       argvec[1] = target;
+       argvec[2] = value_from_longest (builtin_type_long, responds_selector);
+       argvec[3] = value_from_longest (builtin_type_long, selector);
+       argvec[4] = 0;
+
+       ret = call_function_by_hand (argvec[0], 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);
+         }
+       if (value_as_long (ret) == 0)
+         error ("Target does not respond to this message selector.");
+
+       /* Call "methodForSelector:" method, to get the address of a
+          function method that implements this selector for this
+          class.  If we can find a symbol at that address, then we
+          know the return type, parameter types etc.  (that's a good
+          thing). */
+
+       argvec[0] = msg_send;
+       argvec[1] = target;
+       argvec[2] = value_from_longest (builtin_type_long, method_selector);
+       argvec[3] = value_from_longest (builtin_type_long, selector);
+       argvec[4] = 0;
+
+       ret = call_function_by_hand (argvec[0], 3, argvec + 1);
+       if (gnu_runtime)
+         {
+           argvec[0] = ret;
+           ret = call_function_by_hand (argvec[0], 3, argvec + 1);
+         }
+
+       /* ret should now be the selector.  */
+
+       addr = value_as_long (ret);
+       if (addr)
+         {
+           struct symbol *sym = NULL;
+           /* Is it a high_level symbol?  */
+
+           sym = find_pc_function (addr);
+           if (sym != NULL) 
+             method = value_of_variable (sym, 0);
+         }
+
+       /* If we found a method with symbol information, check to see
+           if it returns a struct.  Otherwise assume it doesn't.  */
+
+       if (method)
+         {
+           struct block *b;
+           CORE_ADDR funaddr;
+           struct type *value_type;
+
+           funaddr = find_function_addr (method, &value_type);
+
+           b = block_for_pc (funaddr);
+
+           /* If compiled without -g, assume GCC 2.  */
+           using_gcc = (b == NULL ? 2 : BLOCK_GCC_COMPILED (b));
+
+           CHECK_TYPEDEF (value_type);
+         
+           if ((value_type == NULL) 
+               || (TYPE_CODE(value_type) == TYPE_CODE_ERROR))
+             {
+               if (expect_type != NULL)
+                 value_type = expect_type;
+             }
+
+           struct_return = using_struct_return (value_type, using_gcc);
+         }
+       else if (expect_type != NULL)
+         {
+           struct_return = using_struct_return (check_typedef (expect_type), using_gcc);
+         }
+       
+       /* Found a function symbol.  Now we will substitute its
+          value in place of the message dispatcher (obj_msgSend),
+          so that we call the method directly instead of thru
+          the dispatcher.  The main reason for doing this is that
+          we can now evaluate the return value and parameter values
+          according to their known data types, in case we need to
+          do things like promotion, dereferencing, special handling
+          of structs and doubles, etc.
+         
+          We want to use the type signature of 'method', but still
+          jump to objc_msgSend() or objc_msgSend_stret() to better
+          mimic the behavior of the runtime.  */
+       
+       if (method)
+         {
+           if (TYPE_CODE (value_type (method)) != TYPE_CODE_FUNC)
+             error ("method address has symbol information with non-function type; skipping");
+           if (struct_return)
+             VALUE_ADDRESS (method) = value_as_address (msg_send_stret);
+           else
+             VALUE_ADDRESS (method) = value_as_address (msg_send);
+           called_method = method;
+         }
+       else
+         {
+           if (struct_return)
+             called_method = msg_send_stret;
+           else
+             called_method = msg_send;
+         }
+
+       if (noside == EVAL_SKIP)
+         goto nosideret;
+
+       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 *type = value_type (called_method);
+           if (type && TYPE_CODE (type) == TYPE_CODE_PTR)
+             type = TYPE_TARGET_TYPE (type);
+           type = TYPE_TARGET_TYPE (type);
+
+           if (type)
+           {
+             if ((TYPE_CODE (type) == TYPE_CODE_ERROR) && expect_type)
+               return allocate_value (expect_type);
+             else
+               return allocate_value (type);
+           }
+           else
+             error ("Expression of type other than \"method returning ...\" used as a method");
+         }
+
+       /* Now depending on whether we found a symbol for the method,
+          we will either call the runtime dispatcher or the method
+          directly.  */
+
+       argvec[0] = called_method;
+       argvec[1] = target;
+       argvec[2] = value_from_longest (builtin_type_long, selector);
+       /* User-supplied arguments.  */
+       for (tem = 0; tem < nargs; tem++)
+         argvec[tem + 3] = evaluate_subexp_with_coercion (exp, pos, noside);
+       argvec[tem + 3] = 0;
+
+       if (gnu_runtime && (method != NULL))
+         {
+           /* Function objc_msg_lookup returns a pointer.  */
+           argvec[0]->type
+             = lookup_function_type (lookup_pointer_type (value_type (argvec[0])));
+           argvec[0] = call_function_by_hand (argvec[0], nargs + 2, argvec + 1);
+         }
+
+       ret = call_function_by_hand (argvec[0], 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 = (value_ptr *) alloca (sizeof (value_ptr) * (nargs + 3));
+      argvec = (struct value **) alloca (sizeof (struct value *) * (nargs + 3));
       if (op == STRUCTOP_MEMBER || op == STRUCTOP_MPTR)
        {
          LONGEST fnptr;
@@ -681,7 +969,7 @@ evaluate_subexp_standard (struct type *expect_type,
          /* 1997-08-01 Currently we do not support function invocation
             via pointers-to-methods with HP aCC. Pointer does not point
             to the function, but possibly to some thunk. */
-         if (hp_som_som_object_present)
+         if (deprecated_hp_som_som_object_present)
            {
              error ("Not implemented: function invocation through pointer to method with HP aCC");
            }
@@ -717,9 +1005,9 @@ evaluate_subexp_standard (struct type *expect_type,
              int fnoffset = METHOD_PTR_TO_VOFFSET (fnptr);
              struct type *basetype;
              struct type *domain_type =
-             TYPE_DOMAIN_TYPE (TYPE_TARGET_TYPE (VALUE_TYPE (arg1)));
+             TYPE_DOMAIN_TYPE (TYPE_TARGET_TYPE (value_type (arg1)));
              int i, j;
-             basetype = TYPE_TARGET_TYPE (VALUE_TYPE (arg2));
+             basetype = TYPE_TARGET_TYPE (value_type (arg2));
              if (domain_type != basetype)
                arg2 = value_cast (lookup_pointer_type (domain_type), arg2);
              basetype = TYPE_VPTR_BASETYPE (domain_type);
@@ -742,7 +1030,7 @@ evaluate_subexp_standard (struct type *expect_type,
            }
          else
            {
-             VALUE_TYPE (arg1) = lookup_pointer_type (TYPE_TARGET_TYPE (VALUE_TYPE (arg1)));
+             arg1->type = lookup_pointer_type (TYPE_TARGET_TYPE (value_type (arg1)));
            }
        got_it:
 
@@ -793,7 +1081,7 @@ evaluate_subexp_standard (struct type *expect_type,
          save_pos1 = *pos;
          argvec[0] = evaluate_subexp_with_coercion (exp, pos, noside);
          tem = 1;
-         type = VALUE_TYPE (argvec[0]);
+         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)
@@ -821,15 +1109,10 @@ evaluate_subexp_standard (struct type *expect_type,
       if (op == STRUCTOP_STRUCT || op == STRUCTOP_PTR)
        {
          int static_memfuncp;
-         struct value *temp = arg2;
          char tstr[256];
 
          /* Method invocation : stuff "this" as first parameter */
-         /* pai: this used to have lookup_pointer_type for some reason,
-          * but temp is already a pointer to the object */
-         argvec[1]
-           = value_from_pointer (VALUE_TYPE (temp),
-                                 VALUE_ADDRESS (temp) + VALUE_OFFSET (temp));
+         argvec[1] = arg2;
          /* Name of method from expression */
          strcpy (tstr, &exp->elts[pc2 + 2].string);
 
@@ -839,13 +1122,13 @@ evaluate_subexp_standard (struct type *expect_type,
              struct value *valp = NULL;
 
              /* Prepare list of argument types for overload resolution */
-             arg_types = (struct type **) xmalloc (nargs * (sizeof (struct type *)));
+             arg_types = (struct type **) alloca (nargs * (sizeof (struct type *)));
              for (ix = 1; ix <= nargs; ix++)
-               arg_types[ix - 1] = VALUE_TYPE (argvec[ix]);
+               arg_types[ix - 1] = value_type (argvec[ix]);
 
              (void) find_overload_match (arg_types, nargs, tstr,
                                     1 /* method */ , 0 /* strict match */ ,
-                                         arg2 /* the object */ , NULL,
+                                         &arg2 /* the object */ , NULL,
                                          &valp, NULL, &static_memfuncp);
 
 
@@ -855,11 +1138,17 @@ evaluate_subexp_standard (struct type *expect_type,
          else
            /* Non-C++ case -- or no overload resolution */
            {
-             temp = arg2;
+             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_offset (temp)
+                                        + value_embedded_offset (temp));
              argvec[1] = arg2; /* the ``this'' pointer */
            }
 
@@ -888,9 +1177,9 @@ evaluate_subexp_standard (struct type *expect_type,
              struct symbol *symp;
 
              /* Prepare list of argument types for overload resolution */
-             arg_types = (struct type **) xmalloc (nargs * (sizeof (struct type *)));
+             arg_types = (struct type **) alloca (nargs * (sizeof (struct type *)));
              for (ix = 1; ix <= nargs; ix++)
-               arg_types[ix - 1] = VALUE_TYPE (argvec[ix]);
+               arg_types[ix - 1] = value_type (argvec[ix]);
 
              (void) find_overload_match (arg_types, nargs, NULL /* no need for name */ ,
                                 0 /* not method */ , 0 /* strict match */ ,
@@ -917,6 +1206,8 @@ evaluate_subexp_standard (struct type *expect_type,
 
       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
@@ -927,15 +1218,13 @@ evaluate_subexp_standard (struct type *expect_type,
             it won't offer it. */
 
          struct type *ftype =
-         TYPE_TARGET_TYPE (VALUE_TYPE (argvec[0]));
+         TYPE_TARGET_TYPE (value_type (argvec[0]));
 
          if (ftype)
-           return allocate_value (TYPE_TARGET_TYPE (VALUE_TYPE (argvec[0])));
+           return allocate_value (TYPE_TARGET_TYPE (value_type (argvec[0])));
          else
            error ("Expression of type other than \"Function returning ...\" used as function");
        }
-      if (argvec[0] == NULL)
-       error ("Cannot evaluate function -- may be inlined");
       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  */
 
@@ -953,7 +1242,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
       /* First determine the type code we are dealing with.  */
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      type = check_typedef (VALUE_TYPE (arg1));
+      type = check_typedef (value_type (arg1));
       code = TYPE_CODE (type);
 
       switch (code)
@@ -969,7 +1258,7 @@ evaluate_subexp_standard (struct type *expect_type,
          /* It's a function call. */
          /* Allocate arg vector, including space for the function to be
             called in argvec[0] and a terminating NULL */
-         argvec = (value_ptr *) alloca (sizeof (value_ptr) * (nargs + 2));
+         argvec = (struct value **) alloca (sizeof (struct value *) * (nargs + 2));
          argvec[0] = arg1;
          tem = 1;
          for (; tem <= nargs; tem++)
@@ -1017,7 +1306,7 @@ evaluate_subexp_standard (struct type *expect_type,
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
-       return value_zero (lookup_struct_elt_type (VALUE_TYPE (arg1),
+       return value_zero (lookup_struct_elt_type (value_type (arg1),
                                                   &exp->elts[pc + 2].string,
                                                   0),
                           lval_memory);
@@ -1039,7 +1328,7 @@ evaluate_subexp_standard (struct type *expect_type,
         with rtti type in order to continue on with successful
         lookup of member / method only available in the rtti type. */
       {
-        struct type *type = VALUE_TYPE (arg1);
+        struct type *type = value_type (arg1);
         struct type *real_type;
         int full, top, using_enc;
         
@@ -1060,7 +1349,7 @@ evaluate_subexp_standard (struct type *expect_type,
       }
 
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
-       return value_zero (lookup_struct_elt_type (VALUE_TYPE (arg1),
+       return value_zero (lookup_struct_elt_type (value_type (arg1),
                                                   &exp->elts[pc + 2].string,
                                                   0),
                           lval_memory);
@@ -1076,9 +1365,9 @@ evaluate_subexp_standard (struct type *expect_type,
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
 
       /* With HP aCC, pointers to methods do not point to the function code */
-      if (hp_som_som_object_present &&
-         (TYPE_CODE (VALUE_TYPE (arg2)) == TYPE_CODE_PTR) &&
-      (TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (arg2))) == TYPE_CODE_METHOD))
+      if (deprecated_hp_som_som_object_present &&
+         (TYPE_CODE (value_type (arg2)) == TYPE_CODE_PTR) &&
+      (TYPE_CODE (TYPE_TARGET_TYPE (value_type (arg2))) == TYPE_CODE_METHOD))
        error ("Pointers to methods not supported with HP aCC");        /* 1997-08-19 */
 
       mem_offset = value_as_long (arg2);
@@ -1089,9 +1378,9 @@ evaluate_subexp_standard (struct type *expect_type,
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
 
       /* With HP aCC, pointers to methods do not point to the function code */
-      if (hp_som_som_object_present &&
-         (TYPE_CODE (VALUE_TYPE (arg2)) == TYPE_CODE_PTR) &&
-      (TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (arg2))) == TYPE_CODE_METHOD))
+      if (deprecated_hp_som_som_object_present &&
+         (TYPE_CODE (value_type (arg2)) == TYPE_CODE_PTR) &&
+      (TYPE_CODE (TYPE_TARGET_TYPE (value_type (arg2))) == TYPE_CODE_METHOD))
        error ("Pointers to methods not supported with HP aCC");        /* 1997-08-19 */
 
       mem_offset = value_as_long (arg2);
@@ -1099,7 +1388,7 @@ evaluate_subexp_standard (struct type *expect_type,
     handle_pointer_to_member:
       /* HP aCC generates offsets that have bit #29 set; turn it off to get
          a real offset to the member. */
-      if (hp_som_som_object_present)
+      if (deprecated_hp_som_som_object_present)
        {
          if (!mem_offset)      /* no bias -> really null */
            error ("Attempted dereference of null pointer-to-member");
@@ -1107,7 +1396,7 @@ evaluate_subexp_standard (struct type *expect_type,
        }
       if (noside == EVAL_SKIP)
        goto nosideret;
-      type = check_typedef (VALUE_TYPE (arg2));
+      type = check_typedef (value_type (arg2));
       if (TYPE_CODE (type) != TYPE_CODE_PTR)
        goto bad_pointer_to_member;
       type = check_typedef (TYPE_TARGET_TYPE (type));
@@ -1136,24 +1425,24 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_ASSIGN:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
 
       /* Do special stuff for HP aCC pointers to members */
-      if (hp_som_som_object_present)
+      if (deprecated_hp_som_som_object_present)
        {
          /* 1997-08-19 Can't assign HP aCC pointers to methods. No details of
             the implementation yet; but the pointer appears to point to a code
             sequence (thunk) in memory -- in any case it is *not* the address
             of the function as it would be in a naive implementation. */
-         if ((TYPE_CODE (VALUE_TYPE (arg1)) == TYPE_CODE_PTR) &&
-             (TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (arg1))) == TYPE_CODE_METHOD))
+         if ((TYPE_CODE (value_type (arg1)) == TYPE_CODE_PTR) &&
+             (TYPE_CODE (TYPE_TARGET_TYPE (value_type (arg1))) == TYPE_CODE_METHOD))
            error ("Assignment to pointers to methods not implemented with HP aCC");
 
          /* HP aCC pointers to data members require a constant bias */
-         if ((TYPE_CODE (VALUE_TYPE (arg1)) == TYPE_CODE_PTR) &&
-             (TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (arg1))) == TYPE_CODE_MEMBER))
+         if ((TYPE_CODE (value_type (arg1)) == TYPE_CODE_PTR) &&
+             (TYPE_CODE (TYPE_TARGET_TYPE (value_type (arg1))) == TYPE_CODE_MEMBER))
            {
-             unsigned int *ptr = (unsigned int *) VALUE_CONTENTS (arg2);       /* forces evaluation */
+             unsigned int *ptr = (unsigned int *) value_contents (arg2);       /* forces evaluation */
              *ptr |= 0x20000000;       /* set 29th bit */
            }
        }
@@ -1168,7 +1457,7 @@ evaluate_subexp_standard (struct type *expect_type,
     case BINOP_ASSIGN_MODIFY:
       (*pos) += 2;
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP || noside == EVAL_AVOID_SIDE_EFFECTS)
        return arg1;
       op = exp->elts[pc + 1].opcode;
@@ -1219,7 +1508,7 @@ evaluate_subexp_standard (struct type *expect_type,
        return value_x_binop (arg1, arg2, op, OP_NULL, noside);
       else if (noside == EVAL_AVOID_SIDE_EFFECTS
               && (op == BINOP_DIV || op == BINOP_REM || op == BINOP_MOD))
-       return value_zero (VALUE_TYPE (arg1), not_lval);
+       return value_zero (value_type (arg1), not_lval);
       else
        return value_binop (arg1, arg2, op);
 
@@ -1243,8 +1532,8 @@ evaluate_subexp_standard (struct type *expect_type,
             array or pointer type (like a plain int variable for example),
             then report this as an error. */
 
-         COERCE_REF (arg1);
-         type = check_typedef (VALUE_TYPE (arg1));
+         arg1 = coerce_ref (arg1);
+         type = check_typedef (value_type (arg1));
          if (TYPE_CODE (type) != TYPE_CODE_ARRAY
              && TYPE_CODE (type) != TYPE_CODE_PTR)
            {
@@ -1294,7 +1583,7 @@ evaluate_subexp_standard (struct type *expect_type,
                 type (like a plain int variable for example), then report this
                 as an error. */
 
-             type = TYPE_TARGET_TYPE (check_typedef (VALUE_TYPE (arg1)));
+             type = TYPE_TARGET_TYPE (check_typedef (value_type (arg1)));
              if (type != NULL)
                {
                  arg1 = value_zero (type, VALUE_LVAL (arg1));
@@ -1304,7 +1593,7 @@ evaluate_subexp_standard (struct type *expect_type,
              else
                {
                  error ("cannot subscript something of type `%s'",
-                        TYPE_NAME (VALUE_TYPE (arg1)));
+                        TYPE_NAME (value_type (arg1)));
                }
            }
 
@@ -1321,9 +1610,8 @@ evaluate_subexp_standard (struct type *expect_type,
 
     multi_f77_subscript:
       {
-       int subscript_array[MAX_FORTRAN_DIMS + 1];      /* 1-based array of 
-                                                          subscripts, max == 7 */
-       int array_size_array[MAX_FORTRAN_DIMS + 1];
+       int subscript_array[MAX_FORTRAN_DIMS];
+       int array_size_array[MAX_FORTRAN_DIMS];
        int ndimensions = 1, i;
        struct type *tmp_type;
        int offset_item;        /* The array offset where the item lives */
@@ -1331,7 +1619,7 @@ evaluate_subexp_standard (struct type *expect_type,
        if (nargs > MAX_FORTRAN_DIMS)
          error ("Too many subscripts for F77 (%d Max)", MAX_FORTRAN_DIMS);
 
-       tmp_type = check_typedef (VALUE_TYPE (arg1));
+       tmp_type = check_typedef (value_type (arg1));
        ndimensions = calc_f77_array_dims (type);
 
        if (nargs != ndimensions)
@@ -1341,7 +1629,8 @@ evaluate_subexp_standard (struct type *expect_type,
           let us actually find out where this element exists in the array. */
 
        offset_item = 0;
-       for (i = 1; i <= nargs; i++)
+       /* Take array indices left to right */
+       for (i = 0; i < nargs; i++)
          {
            /* Evaluate each subscript, It must be a legal integer in F77 */
            arg2 = evaluate_subexp_with_coercion (exp, pos, noside);
@@ -1349,7 +1638,11 @@ evaluate_subexp_standard (struct type *expect_type,
            /* Fill in the subscript and array size arrays */
 
            subscript_array[i] = value_as_long (arg2);
+         }
 
+       /* Internal type of array is arranged right to left */
+       for (i = 0; i < nargs; i++)
+         {
            retcode = f77_get_dynamic_upperbound (tmp_type, &upper);
            if (retcode == BOUND_FETCH_ERROR)
              error ("Cannot obtain dynamic upper bound");
@@ -1358,11 +1651,11 @@ evaluate_subexp_standard (struct type *expect_type,
            if (retcode == BOUND_FETCH_ERROR)
              error ("Cannot obtain dynamic lower bound");
 
-           array_size_array[i] = upper - lower + 1;
+           array_size_array[nargs - i - 1] = upper - lower + 1;
 
            /* Zero-normalize subscripts so that offsetting will work. */
 
-           subscript_array[i] -= lower;
+           subscript_array[nargs - i - 1] -= lower;
 
            /* If we are at the bottom of a multidimensional 
               array type then keep a ptr to the last ARRAY
@@ -1372,17 +1665,17 @@ evaluate_subexp_standard (struct type *expect_type,
               of base element type that we apply a simple 
               offset to. */
 
-           if (i < nargs)
+           if (i < nargs - 1)
              tmp_type = check_typedef (TYPE_TARGET_TYPE (tmp_type));
          }
 
        /* Now let us calculate the offset for this item */
 
-       offset_item = subscript_array[ndimensions];
+       offset_item = subscript_array[ndimensions - 1];
 
-       for (i = ndimensions - 1; i >= 1; i--)
+       for (i = ndimensions - 1; i > 0; --i)
          offset_item =
-           array_size_array[i] * offset_item + subscript_array[i];
+           array_size_array[i - 1] * offset_item + subscript_array[i - 1];
 
        /* Construct a value node with the value of the offset */
 
@@ -1395,7 +1688,7 @@ evaluate_subexp_standard (struct type *expect_type,
           type, this will ensure that value_subscript()
           returns the correct type value */
 
-       VALUE_TYPE (arg1) = tmp_type;
+       arg1->type = tmp_type;
        return value_ind (value_add (value_coerce_array (arg1), arg2));
       }
 
@@ -1453,7 +1746,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_EQUAL:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (binop_user_defined_p (op, arg1, arg2))
@@ -1468,7 +1761,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_NOTEQUAL:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (binop_user_defined_p (op, arg1, arg2))
@@ -1483,7 +1776,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_LESS:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (binop_user_defined_p (op, arg1, arg2))
@@ -1498,7 +1791,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_GTR:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (binop_user_defined_p (op, arg1, arg2))
@@ -1513,7 +1806,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_GEQ:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (binop_user_defined_p (op, arg1, arg2))
@@ -1528,7 +1821,7 @@ evaluate_subexp_standard (struct type *expect_type,
 
     case BINOP_LEQ:
       arg1 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
-      arg2 = evaluate_subexp (VALUE_TYPE (arg1), exp, pos, noside);
+      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
       if (binop_user_defined_p (op, arg1, arg2))
@@ -1546,12 +1839,12 @@ evaluate_subexp_standard (struct type *expect_type,
       arg2 = evaluate_subexp (NULL_TYPE, exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
-      type = check_typedef (VALUE_TYPE (arg2));
+      type = check_typedef (value_type (arg2));
       if (TYPE_CODE (type) != TYPE_CODE_INT)
        error ("Non-integral right operand for \"@\" operator.");
       if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
-         return allocate_repeat_value (VALUE_TYPE (arg1),
+         return allocate_repeat_value (value_type (arg1),
                                     longest_to_int (value_as_long (arg2)));
        }
       else
@@ -1596,9 +1889,9 @@ evaluate_subexp_standard (struct type *expect_type,
       if (expect_type && TYPE_CODE (expect_type) == TYPE_CODE_PTR)
        expect_type = TYPE_TARGET_TYPE (check_typedef (expect_type));
       arg1 = evaluate_subexp (expect_type, exp, pos, noside);
-      if ((TYPE_TARGET_TYPE (VALUE_TYPE (arg1))) &&
-         ((TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (arg1))) == TYPE_CODE_METHOD) ||
-          (TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (arg1))) == TYPE_CODE_MEMBER)))
+      if ((TYPE_TARGET_TYPE (value_type (arg1))) &&
+         ((TYPE_CODE (TYPE_TARGET_TYPE (value_type (arg1))) == TYPE_CODE_METHOD) ||
+          (TYPE_CODE (TYPE_TARGET_TYPE (value_type (arg1))) == TYPE_CODE_MEMBER)))
        error ("Attempt to dereference pointer to member without an object");
       if (noside == EVAL_SKIP)
        goto nosideret;
@@ -1606,7 +1899,7 @@ evaluate_subexp_standard (struct type *expect_type,
        return value_x_unop (arg1, op, noside);
       else if (noside == EVAL_AVOID_SIDE_EFFECTS)
        {
-         type = check_typedef (VALUE_TYPE (arg1));
+         type = check_typedef (value_type (arg1));
          if (TYPE_CODE (type) == TYPE_CODE_PTR
              || TYPE_CODE (type) == TYPE_CODE_REF
          /* In C you can dereference an array to get the 1st elt.  */
@@ -1642,11 +1935,11 @@ evaluate_subexp_standard (struct type *expect_type,
        {
          struct value *retvalp = evaluate_subexp_for_address (exp, pos, noside);
          /* If HP aCC object, use bias for pointers to members */
-         if (hp_som_som_object_present &&
-             (TYPE_CODE (VALUE_TYPE (retvalp)) == TYPE_CODE_PTR) &&
-             (TYPE_CODE (TYPE_TARGET_TYPE (VALUE_TYPE (retvalp))) == TYPE_CODE_MEMBER))
+         if (deprecated_hp_som_som_object_present &&
+             (TYPE_CODE (value_type (retvalp)) == TYPE_CODE_PTR) &&
+             (TYPE_CODE (TYPE_TARGET_TYPE (value_type (retvalp))) == TYPE_CODE_MEMBER))
            {
-             unsigned int *ptr = (unsigned int *) VALUE_CONTENTS (retvalp);    /* forces evaluation */
+             unsigned int *ptr = (unsigned int *) value_contents (retvalp);    /* forces evaluation */
              *ptr |= 0x20000000;       /* set 29th bit */
            }
          return retvalp;
@@ -1666,7 +1959,7 @@ evaluate_subexp_standard (struct type *expect_type,
       arg1 = evaluate_subexp (type, exp, pos, noside);
       if (noside == EVAL_SKIP)
        goto nosideret;
-      if (type != VALUE_TYPE (arg1))
+      if (type != value_type (arg1))
        arg1 = value_cast (type, arg1);
       return arg1;
 
@@ -1679,8 +1972,7 @@ evaluate_subexp_standard (struct type *expect_type,
        return value_zero (exp->elts[pc + 1].type, lval_memory);
       else
        return value_at_lazy (exp->elts[pc + 1].type,
-                             value_as_address (arg1),
-                             NULL);
+                             value_as_address (arg1));
 
     case UNOP_PREINCREMENT:
       arg1 = evaluate_subexp (expect_type, exp, pos, noside);
@@ -1748,6 +2040,10 @@ evaluate_subexp_standard (struct type *expect_type,
       (*pos) += 1;
       return value_of_this (1);
 
+    case OP_OBJC_SELF:
+      (*pos) += 1;
+      return value_of_local ("self", 1);
+
     case OP_TYPE:
       error ("Attempt to use a type name as an expression");
 
@@ -1777,11 +2073,11 @@ nosideret:
    then only the type of the result need be correct.  */
 
 static struct value *
-evaluate_subexp_for_address (register struct expression *exp, register int *pos,
+evaluate_subexp_for_address (struct expression *exp, int *pos,
                             enum noside noside)
 {
   enum exp_opcode op;
-  register int pc;
+  int pc;
   struct symbol *var;
 
   pc = (*pos);
@@ -1834,7 +2130,7 @@ evaluate_subexp_for_address (register struct expression *exp, register int *pos,
        {
          struct value *x = evaluate_subexp (NULL_TYPE, exp, pos, noside);
          if (VALUE_LVAL (x) == lval_memory)
-           return value_zero (lookup_pointer_type (VALUE_TYPE (x)),
+           return value_zero (lookup_pointer_type (value_type (x)),
                               not_lval);
          else
            error ("Attempt to take address of non-lval");
@@ -1847,7 +2143,7 @@ evaluate_subexp_for_address (register struct expression *exp, register int *pos,
    When used in contexts where arrays will be coerced anyway, this is
    equivalent to `evaluate_subexp' but much faster because it avoids
    actually fetching array contents (perhaps obsolete now that we have
-   VALUE_LAZY).
+   value_lazy()).
 
    Note that we currently only do the coercion for C expressions, where
    arrays are zero based and the coercion is correct.  For other languages,
@@ -1857,11 +2153,11 @@ evaluate_subexp_for_address (register struct expression *exp, register int *pos,
  */
 
 struct value *
-evaluate_subexp_with_coercion (register struct expression *exp,
-                              register int *pos, enum noside noside)
+evaluate_subexp_with_coercion (struct expression *exp,
+                              int *pos, enum noside noside)
 {
-  register enum exp_opcode op;
-  register int pc;
+  enum exp_opcode op;
+  int pc;
   struct value *val;
   struct symbol *var;
 
@@ -1894,10 +2190,10 @@ evaluate_subexp_with_coercion (register struct expression *exp,
    Advance *POS over the subexpression.  */
 
 static struct value *
-evaluate_subexp_for_sizeof (register struct expression *exp, register int *pos)
+evaluate_subexp_for_sizeof (struct expression *exp, int *pos)
 {
   enum exp_opcode op;
-  register int pc;
+  int pc;
   struct type *type;
   struct value *val;
 
@@ -1913,7 +2209,7 @@ evaluate_subexp_for_sizeof (register struct expression *exp, register int *pos)
     case UNOP_IND:
       (*pos)++;
       val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
-      type = check_typedef (VALUE_TYPE (val));
+      type = check_typedef (value_type (val));
       if (TYPE_CODE (type) != TYPE_CODE_PTR
          && TYPE_CODE (type) != TYPE_CODE_REF
          && TYPE_CODE (type) != TYPE_CODE_ARRAY)
@@ -1937,7 +2233,7 @@ evaluate_subexp_for_sizeof (register struct expression *exp, register int *pos)
     default:
       val = evaluate_subexp (NULL_TYPE, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
       return value_from_longest (builtin_type_int,
-                                (LONGEST) TYPE_LENGTH (VALUE_TYPE (val)));
+                                (LONGEST) TYPE_LENGTH (value_type (val)));
     }
 }
 
This page took 0.042133 seconds and 4 git commands to generate.