gdb: use gdb::optional instead of passing a pointer to gdb::array_view
[deliverable/binutils-gdb.git] / gdb / f-lang.c
index 554626e537b86a1d1b1fddb6a7b1a483941bb6fe..16ec9e04044b899b59560e9cdc72a15441f98372 100644 (file)
@@ -38,6 +38,7 @@
 #include "gdbarch.h"
 #include "gdbcmd.h"
 #include "f-array-walker.h"
+#include "f-exp.h"
 
 #include <math.h>
 
@@ -68,10 +69,10 @@ show_fortran_array_slicing_debug (struct ui_file *file, int from_tty,
 
 /* Local functions */
 
-static value *fortran_prepare_argument (struct expression *exp, int *pos,
+static value *fortran_prepare_argument (struct expression *exp,
+                                       expr::operation *subexp,
                                        int arg_num, bool is_internal_call_p,
-                                       struct type *func_type,
-                                       enum noside noside);
+                                       struct type *func_type, enum noside noside);
 
 /* Return the encoding that should be used for the character type
    TYPE.  */
@@ -102,33 +103,22 @@ f_language::get_encoding (struct type *type)
 
 \f
 
-/* Table of operators and their precedences for printing expressions.  */
+/* A helper function for the "bound" intrinsics that checks that TYPE
+   is an array.  LBOUND_P is true for lower bound; this is used for
+   the error message, if any.  */
 
-const struct op_print f_language::op_print_tab[] =
+static void
+fortran_require_array (struct type *type, bool lbound_p)
 {
-  {"+", BINOP_ADD, PREC_ADD, 0},
-  {"+", UNOP_PLUS, PREC_PREFIX, 0},
-  {"-", BINOP_SUB, PREC_ADD, 0},
-  {"-", UNOP_NEG, PREC_PREFIX, 0},
-  {"*", BINOP_MUL, PREC_MUL, 0},
-  {"/", BINOP_DIV, PREC_MUL, 0},
-  {"DIV", BINOP_INTDIV, PREC_MUL, 0},
-  {"MOD", BINOP_REM, PREC_MUL, 0},
-  {"=", BINOP_ASSIGN, PREC_ASSIGN, 1},
-  {".OR.", BINOP_LOGICAL_OR, PREC_LOGICAL_OR, 0},
-  {".AND.", BINOP_LOGICAL_AND, PREC_LOGICAL_AND, 0},
-  {".NOT.", UNOP_LOGICAL_NOT, PREC_PREFIX, 0},
-  {".EQ.", BINOP_EQUAL, PREC_EQUAL, 0},
-  {".NE.", BINOP_NOTEQUAL, PREC_EQUAL, 0},
-  {".LE.", BINOP_LEQ, PREC_ORDER, 0},
-  {".GE.", BINOP_GEQ, PREC_ORDER, 0},
-  {".GT.", BINOP_GTR, PREC_ORDER, 0},
-  {".LT.", BINOP_LESS, PREC_ORDER, 0},
-  {"**", UNOP_IND, PREC_PREFIX, 0},
-  {"@", BINOP_REPEAT, PREC_REPEAT, 0},
-  {NULL, OP_NULL, PREC_REPEAT, 0}
-};
-\f
+  type = check_typedef (type);
+  if (type->code () != TYPE_CODE_ARRAY)
+    {
+      if (lbound_p)
+       error (_("LBOUND can only be applied to arrays"));
+      else
+       error (_("UBOUND can only be applied to arrays"));
+    }
+}
 
 /* Create an array containing the lower bounds (when LBOUND_P is true) or
    the upper bounds (when LBOUND_P is false) of ARRAY (which must be of
@@ -394,595 +384,386 @@ private:
   struct value *m_val;
 };
 
-/* Called from evaluate_subexp_standard to perform array indexing, and
-   sub-range extraction, for Fortran.  As well as arrays this function
-   also handles strings as they can be treated like arrays of characters.
-   ARRAY is the array or string being accessed.  EXP, POS, and NOSIDE are
-   as for evaluate_subexp_standard, and NARGS is the number of arguments
-   in this access (e.g. 'array (1,2,3)' would be NARGS 3).  */
+
+/* Evaluate FORTRAN_ASSOCIATED expressions.  Both GDBARCH and LANG are
+   extracted from the expression being evaluated.  POINTER is the required
+   first argument to the 'associated' keyword, and TARGET is the optional
+   second argument, this will be nullptr if the user only passed one
+   argument to their use of 'associated'.  */
 
 static struct value *
-fortran_value_subarray (struct value *array, struct expression *exp,
-                       int *pos, int nargs, enum noside noside)
+fortran_associated (struct gdbarch *gdbarch, const language_defn *lang,
+                   struct value *pointer, struct value *target = nullptr)
 {
-  type *original_array_type = check_typedef (value_type (array));
-  bool is_string_p = original_array_type->code () == TYPE_CODE_STRING;
+  struct type *result_type = language_bool_type (lang, gdbarch);
 
-  /* Perform checks for ARRAY not being available.  The somewhat overly
-     complex logic here is just to keep backward compatibility with the
-     errors that we used to get before FORTRAN_VALUE_SUBARRAY was
-     rewritten.  Maybe a future task would streamline the error messages we
-     get here, and update all the expected test results.  */
-  if (exp->elts[*pos].opcode != OP_RANGE)
-    {
-      if (type_not_associated (original_array_type))
-       error (_("no such vector element (vector not associated)"));
-      else if (type_not_allocated (original_array_type))
-       error (_("no such vector element (vector not allocated)"));
-    }
+  /* All Fortran pointers should have the associated property, this is
+     how we know the pointer is pointing at something or not.  */
+  struct type *pointer_type = check_typedef (value_type (pointer));
+  if (TYPE_ASSOCIATED_PROP (pointer_type) == nullptr
+      && pointer_type->code () != TYPE_CODE_PTR)
+    error (_("ASSOCIATED can only be applied to pointers"));
+
+  /* Get an address from POINTER.  Fortran (or at least gfortran) models
+     array pointers as arrays with a dynamic data address, so we need to
+     use two approaches here, for real pointers we take the contents of the
+     pointer as an address.  For non-pointers we take the address of the
+     content.  */
+  CORE_ADDR pointer_addr;
+  if (pointer_type->code () == TYPE_CODE_PTR)
+    pointer_addr = value_as_address (pointer);
   else
+    pointer_addr = value_address (pointer);
+
+  /* The single argument case, is POINTER associated with anything?  */
+  if (target == nullptr)
     {
-      if (type_not_associated (original_array_type))
-       error (_("array not associated"));
-      else if (type_not_allocated (original_array_type))
-       error (_("array not allocated"));
+      bool is_associated = false;
+
+      /* If POINTER is an actual pointer and doesn't have an associated
+        property then we need to figure out whether this pointer is
+        associated by looking at the value of the pointer itself.  We make
+        the assumption that a non-associated pointer will be set to 0.
+        This is probably true for most targets, but might not be true for
+        everyone.  */
+      if (pointer_type->code () == TYPE_CODE_PTR
+         && TYPE_ASSOCIATED_PROP (pointer_type) == nullptr)
+       is_associated = (pointer_addr != 0);
+      else
+       is_associated = !type_not_associated (pointer_type);
+      return value_from_longest (result_type, is_associated ? 1 : 0);
     }
 
-  /* First check that the number of dimensions in the type we are slicing
-     matches the number of arguments we were passed.  */
-  int ndimensions = calc_f77_array_dims (original_array_type);
-  if (nargs != ndimensions)
-    error (_("Wrong number of subscripts"));
+  /* The two argument case, is POINTER associated with TARGET?  */
 
-  /* This will be initialised below with the type of the elements held in
-     ARRAY.  */
-  struct type *inner_element_type;
+  struct type *target_type = check_typedef (value_type (target));
 
-  /* Extract the types of each array dimension from the original array
-     type.  We need these available so we can fill in the default upper and
-     lower bounds if the user requested slice doesn't provide that
-     information.  Additionally unpacking the dimensions like this gives us
-     the inner element type.  */
-  std::vector<struct type *> dim_types;
-  {
-    dim_types.reserve (ndimensions);
-    struct type *type = original_array_type;
-    for (int i = 0; i < ndimensions; ++i)
-      {
-       dim_types.push_back (type);
-       type = TYPE_TARGET_TYPE (type);
-      }
-    /* TYPE is now the inner element type of the array, we start the new
-       array slice off as this type, then as we process the requested slice
-       (from the user) we wrap new types around this to build up the final
-       slice type.  */
-    inner_element_type = type;
-  }
+  struct type *pointer_target_type;
+  if (pointer_type->code () == TYPE_CODE_PTR)
+    pointer_target_type = TYPE_TARGET_TYPE (pointer_type);
+  else
+    pointer_target_type = pointer_type;
 
-  /* As we analyse the new slice type we need to understand if the data
-     being referenced is contiguous.  Do decide this we must track the size
-     of an element at each dimension of the new slice array.  Initially the
-     elements of the inner most dimension of the array are the same inner
-     most elements as the original ARRAY.  */
-  LONGEST slice_element_size = TYPE_LENGTH (inner_element_type);
+  struct type *target_target_type;
+  if (target_type->code () == TYPE_CODE_PTR)
+    target_target_type = TYPE_TARGET_TYPE (target_type);
+  else
+    target_target_type = target_type;
 
-  /* Start off assuming all data is contiguous, this will be set to false
-     if access to any dimension results in non-contiguous data.  */
-  bool is_all_contiguous = true;
+  if (pointer_target_type->code () != target_target_type->code ()
+      || (pointer_target_type->code () != TYPE_CODE_ARRAY
+         && (TYPE_LENGTH (pointer_target_type)
+             != TYPE_LENGTH (target_target_type))))
+    error (_("arguments to associated must be of same type and kind"));
 
-  /* The TOTAL_OFFSET is the distance in bytes from the start of the
-     original ARRAY to the start of the new slice.  This is calculated as
-     we process the information from the user.  */
-  LONGEST total_offset = 0;
+  /* If TARGET is not in memory, or the original pointer is specifically
+     known to be not associated with anything, then the answer is obviously
+     false.  Alternatively, if POINTER is an actual pointer and has no
+     associated property, then we have to check if its associated by
+     looking the value of the pointer itself.  We make the assumption that
+     a non-associated pointer will be set to 0.  This is probably true for
+     most targets, but might not be true for everyone.  */
+  if (value_lval_const (target) != lval_memory
+      || type_not_associated (pointer_type)
+      || (TYPE_ASSOCIATED_PROP (pointer_type) == nullptr
+         && pointer_type->code () == TYPE_CODE_PTR
+         && pointer_addr == 0))
+    return value_from_longest (result_type, 0);
 
-  /* A structure representing information about each dimension of the
-     resulting slice.  */
-  struct slice_dim
-  {
-    /* Constructor.  */
-    slice_dim (LONGEST l, LONGEST h, LONGEST s, struct type *idx)
-      : low (l),
-       high (h),
-       stride (s),
-       index (idx)
-    { /* Nothing.  */ }
+  /* See the comment for POINTER_ADDR above.  */
+  CORE_ADDR target_addr;
+  if (target_type->code () == TYPE_CODE_PTR)
+    target_addr = value_as_address (target);
+  else
+    target_addr = value_address (target);
 
-    /* The low bound for this dimension of the slice.  */
-    LONGEST low;
+  /* Wrap the following checks inside a do { ... } while (false) loop so
+     that we can use `break' to jump out of the loop.  */
+  bool is_associated = false;
+  do
+    {
+      /* If the addresses are different then POINTER is definitely not
+        pointing at TARGET.  */
+      if (pointer_addr != target_addr)
+       break;
 
-    /* The high bound for this dimension of the slice.  */
-    LONGEST high;
+      /* If POINTER is a real pointer (i.e. not an array pointer, which are
+        implemented as arrays with a dynamic content address), then this
+        is all the checking that is needed.  */
+      if (pointer_type->code () == TYPE_CODE_PTR)
+       {
+         is_associated = true;
+         break;
+       }
 
-    /* The byte stride for this dimension of the slice.  */
-    LONGEST stride;
+      /* We have an array pointer.  Check the number of dimensions.  */
+      int pointer_dims = calc_f77_array_dims (pointer_type);
+      int target_dims = calc_f77_array_dims (target_type);
+      if (pointer_dims != target_dims)
+       break;
 
-    struct type *index;
-  };
+      /* Now check that every dimension has the same upper bound, lower
+        bound, and stride value.  */
+      int dim = 0;
+      while (dim < pointer_dims)
+       {
+         LONGEST pointer_lowerbound, pointer_upperbound, pointer_stride;
+         LONGEST target_lowerbound, target_upperbound, target_stride;
 
-  /* The dimensions of the resulting slice.  */
-  std::vector<slice_dim> slice_dims;
+         pointer_type = check_typedef (pointer_type);
+         target_type = check_typedef (target_type);
 
-  /* Process the incoming arguments.   These arguments are in the reverse
-     order to the array dimensions, that is the first argument refers to
-     the last array dimension.  */
-  if (fortran_array_slicing_debug)
-    debug_printf ("Processing array access:\n");
-  for (int i = 0; i < nargs; ++i)
-    {
-      /* For each dimension of the array the user will have either provided
-        a ranged access with optional lower bound, upper bound, and
-        stride, or the user will have supplied a single index.  */
-      struct type *dim_type = dim_types[ndimensions - (i + 1)];
-      if (exp->elts[*pos].opcode == OP_RANGE)
-       {
-         int pc = (*pos) + 1;
-         enum range_flag range_flag = (enum range_flag) exp->elts[pc].longconst;
-         *pos += 3;
+         struct type *pointer_range = pointer_type->index_type ();
+         struct type *target_range = target_type->index_type ();
 
-         LONGEST low, high, stride;
-         low = high = stride = 0;
+         if (!get_discrete_bounds (pointer_range, &pointer_lowerbound,
+                                   &pointer_upperbound))
+           break;
 
-         if ((range_flag & RANGE_LOW_BOUND_DEFAULT) == 0)
-           low = value_as_long (evaluate_subexp (nullptr, exp, pos, noside));
-         else
-           low = f77_get_lowerbound (dim_type);
-         if ((range_flag & RANGE_HIGH_BOUND_DEFAULT) == 0)
-           high = value_as_long (evaluate_subexp (nullptr, exp, pos, noside));
-         else
-           high = f77_get_upperbound (dim_type);
-         if ((range_flag & RANGE_HAS_STRIDE) == RANGE_HAS_STRIDE)
-           stride = value_as_long (evaluate_subexp (nullptr, exp, pos, noside));
-         else
-           stride = 1;
+         if (!get_discrete_bounds (target_range, &target_lowerbound,
+                                   &target_upperbound))
+           break;
 
-         if (stride == 0)
-           error (_("stride must not be 0"));
+         if (pointer_lowerbound != target_lowerbound
+             || pointer_upperbound != target_upperbound)
+           break;
 
-         /* Get information about this dimension in the original ARRAY.  */
-         struct type *target_type = TYPE_TARGET_TYPE (dim_type);
-         struct type *index_type = dim_type->index_type ();
-         LONGEST lb = f77_get_lowerbound (dim_type);
-         LONGEST ub = f77_get_upperbound (dim_type);
-         LONGEST sd = index_type->bit_stride ();
-         if (sd == 0)
-           sd = TYPE_LENGTH (target_type) * 8;
+         /* Figure out the stride (in bits) for both pointer and target.
+            If either doesn't have a stride then we take the element size,
+            but we need to convert to bits (hence the * 8).  */
+         pointer_stride = pointer_range->bounds ()->bit_stride ();
+         if (pointer_stride == 0)
+           pointer_stride
+             = type_length_units (check_typedef
+                                    (TYPE_TARGET_TYPE (pointer_type))) * 8;
+         target_stride = target_range->bounds ()->bit_stride ();
+         if (target_stride == 0)
+           target_stride
+             = type_length_units (check_typedef
+                                    (TYPE_TARGET_TYPE (target_type))) * 8;
+         if (pointer_stride != target_stride)
+           break;
 
-         if (fortran_array_slicing_debug)
-           {
-             debug_printf ("|-> Range access\n");
-             std::string str = type_to_string (dim_type);
-             debug_printf ("|   |-> Type: %s\n", str.c_str ());
-             debug_printf ("|   |-> Array:\n");
-             debug_printf ("|   |   |-> Low bound: %s\n", plongest (lb));
-             debug_printf ("|   |   |-> High bound: %s\n", plongest (ub));
-             debug_printf ("|   |   |-> Bit stride: %s\n", plongest (sd));
-             debug_printf ("|   |   |-> Byte stride: %s\n", plongest (sd / 8));
-             debug_printf ("|   |   |-> Type size: %s\n",
-                           pulongest (TYPE_LENGTH (dim_type)));
-             debug_printf ("|   |   '-> Target type size: %s\n",
-                           pulongest (TYPE_LENGTH (target_type)));
-             debug_printf ("|   |-> Accessing:\n");
-             debug_printf ("|   |   |-> Low bound: %s\n",
-                           plongest (low));
-             debug_printf ("|   |   |-> High bound: %s\n",
-                           plongest (high));
-             debug_printf ("|   |   '-> Element stride: %s\n",
-                           plongest (stride));
-           }
-
-         /* Check the user hasn't asked for something invalid.  */
-         if (high > ub || low < lb)
-           error (_("array subscript out of bounds"));
-
-         /* Calculate what this dimension of the new slice array will look
-            like.  OFFSET is the byte offset from the start of the
-            previous (more outer) dimension to the start of this
-            dimension.  E_COUNT is the number of elements in this
-            dimension.  REMAINDER is the number of elements remaining
-            between the last included element and the upper bound.  For
-            example an access '1:6:2' will include elements 1, 3, 5 and
-            have a remainder of 1 (element #6).  */
-         LONGEST lowest = std::min (low, high);
-         LONGEST offset = (sd / 8) * (lowest - lb);
-         LONGEST e_count = std::abs (high - low) + 1;
-         e_count = (e_count + (std::abs (stride) - 1)) / std::abs (stride);
-         LONGEST new_low = 1;
-         LONGEST new_high = new_low + e_count - 1;
-         LONGEST new_stride = (sd * stride) / 8;
-         LONGEST last_elem = low + ((e_count - 1) * stride);
-         LONGEST remainder = high - last_elem;
-         if (low > high)
-           {
-             offset += std::abs (remainder) * TYPE_LENGTH (target_type);
-             if (stride > 0)
-               error (_("incorrect stride and boundary combination"));
-           }
-         else if (stride < 0)
-           error (_("incorrect stride and boundary combination"));
-
-         /* Is the data within this dimension contiguous?  It is if the
-            newly computed stride is the same size as a single element of
-            this dimension.  */
-         bool is_dim_contiguous = (new_stride == slice_element_size);
-         is_all_contiguous &= is_dim_contiguous;
+         ++dim;
+       }
 
-         if (fortran_array_slicing_debug)
-           {
-             debug_printf ("|   '-> Results:\n");
-             debug_printf ("|       |-> Offset = %s\n", plongest (offset));
-             debug_printf ("|       |-> Elements = %s\n", plongest (e_count));
-             debug_printf ("|       |-> Low bound = %s\n", plongest (new_low));
-             debug_printf ("|       |-> High bound = %s\n",
-                           plongest (new_high));
-             debug_printf ("|       |-> Byte stride = %s\n",
-                           plongest (new_stride));
-             debug_printf ("|       |-> Last element = %s\n",
-                           plongest (last_elem));
-             debug_printf ("|       |-> Remainder = %s\n",
-                           plongest (remainder));
-             debug_printf ("|       '-> Contiguous = %s\n",
-                           (is_dim_contiguous ? "Yes" : "No"));
-           }
+      if (dim < pointer_dims)
+       break;
 
-         /* Figure out how big (in bytes) an element of this dimension of
-            the new array slice will be.  */
-         slice_element_size = std::abs (new_stride * e_count);
+      is_associated = true;
+    }
+  while (false);
 
-         slice_dims.emplace_back (new_low, new_high, new_stride,
-                                  index_type);
+  return value_from_longest (result_type, is_associated ? 1 : 0);
+}
 
-         /* Update the total offset.  */
-         total_offset += offset;
-       }
-      else
-       {
-         /* There is a single index for this dimension.  */
-         LONGEST index
-           = value_as_long (evaluate_subexp_with_coercion (exp, pos, noside));
+struct value *
+eval_op_f_associated (struct type *expect_type,
+                     struct expression *exp,
+                     enum noside noside,
+                     enum exp_opcode opcode,
+                     struct value *arg1)
+{
+  return fortran_associated (exp->gdbarch, exp->language_defn, arg1);
+}
 
-         /* Get information about this dimension in the original ARRAY.  */
-         struct type *target_type = TYPE_TARGET_TYPE (dim_type);
-         struct type *index_type = dim_type->index_type ();
-         LONGEST lb = f77_get_lowerbound (dim_type);
-         LONGEST ub = f77_get_upperbound (dim_type);
-         LONGEST sd = index_type->bit_stride () / 8;
-         if (sd == 0)
-           sd = TYPE_LENGTH (target_type);
+struct value *
+eval_op_f_associated (struct type *expect_type,
+                     struct expression *exp,
+                     enum noside noside,
+                     enum exp_opcode opcode,
+                     struct value *arg1,
+                     struct value *arg2)
+{
+  return fortran_associated (exp->gdbarch, exp->language_defn, arg1, arg2);
+}
 
-         if (fortran_array_slicing_debug)
-           {
-             debug_printf ("|-> Index access\n");
-             std::string str = type_to_string (dim_type);
-             debug_printf ("|   |-> Type: %s\n", str.c_str ());
-             debug_printf ("|   |-> Array:\n");
-             debug_printf ("|   |   |-> Low bound: %s\n", plongest (lb));
-             debug_printf ("|   |   |-> High bound: %s\n", plongest (ub));
-             debug_printf ("|   |   |-> Byte stride: %s\n", plongest (sd));
-             debug_printf ("|   |   |-> Type size: %s\n",
-                           pulongest (TYPE_LENGTH (dim_type)));
-             debug_printf ("|   |   '-> Target type size: %s\n",
-                           pulongest (TYPE_LENGTH (target_type)));
-             debug_printf ("|   '-> Accessing:\n");
-             debug_printf ("|       '-> Index: %s\n",
-                           plongest (index));
-           }
+/* Implement FORTRAN_ARRAY_SIZE expression, this corresponds to the 'SIZE'
+   keyword.  Both GDBARCH and LANG are extracted from the expression being
+   evaluated.  ARRAY is the value that should be an array, though this will
+   not have been checked before calling this function.  DIM is optional, if
+   present then it should be an integer identifying a dimension of the
+   array to ask about.  As with ARRAY the validity of DIM is not checked
+   before calling this function.
 
-         /* If the array has actual content then check the index is in
-            bounds.  An array without content (an unbound array) doesn't
-            have a known upper bound, so don't error check in that
-            situation.  */
-         if (index < lb
-             || (dim_type->index_type ()->bounds ()->high.kind () != PROP_UNDEFINED
-                 && index > ub)
-             || (VALUE_LVAL (array) != lval_memory
-                 && dim_type->index_type ()->bounds ()->high.kind () == PROP_UNDEFINED))
-           {
-             if (type_not_associated (dim_type))
-               error (_("no such vector element (vector not associated)"));
-             else if (type_not_allocated (dim_type))
-               error (_("no such vector element (vector not allocated)"));
-             else
-               error (_("no such vector element"));
-           }
+   Return either the total number of elements in ARRAY (when DIM is
+   nullptr), or the number of elements in dimension DIM.  */
 
-         /* Calculate using the type stride, not the target type size.  */
-         LONGEST offset = sd * (index - lb);
-         total_offset += offset;
-       }
-    }
+static struct value *
+fortran_array_size (struct gdbarch *gdbarch, const language_defn *lang,
+                   struct value *array, struct value *dim_val = nullptr)
+{
+  /* Check that ARRAY is the correct type.  */
+  struct type *array_type = check_typedef (value_type (array));
+  if (array_type->code () != TYPE_CODE_ARRAY)
+    error (_("SIZE can only be applied to arrays"));
+  if (type_not_allocated (array_type) || type_not_associated (array_type))
+    error (_("SIZE can only be used on allocated/associated arrays"));
 
-  if (noside == EVAL_SKIP)
-    return array;
+  int ndimensions = calc_f77_array_dims (array_type);
+  int dim = -1;
+  LONGEST result = 0;
 
-  /* Build a type that represents the new array slice in the target memory
-     of the original ARRAY, this type makes use of strides to correctly
-     find only those elements that are part of the new slice.  */
-  struct type *array_slice_type = inner_element_type;
-  for (const auto &d : slice_dims)
+  if (dim_val != nullptr)
     {
-      /* Create the range.  */
-      dynamic_prop p_low, p_high, p_stride;
+      if (check_typedef (value_type (dim_val))->code () != TYPE_CODE_INT)
+       error (_("DIM argument to SIZE must be an integer"));
+      dim = (int) value_as_long (dim_val);
 
-      p_low.set_const_val (d.low);
-      p_high.set_const_val (d.high);
-      p_stride.set_const_val (d.stride);
-
-      struct type *new_range
-       = create_range_type_with_stride ((struct type *) NULL,
-                                        TYPE_TARGET_TYPE (d.index),
-                                        &p_low, &p_high, 0, &p_stride,
-                                        true);
-      array_slice_type
-       = create_array_type (nullptr, array_slice_type, new_range);
-    }
-
-  if (fortran_array_slicing_debug)
-    {
-      debug_printf ("'-> Final result:\n");
-      debug_printf ("    |-> Type: %s\n",
-                   type_to_string (array_slice_type).c_str ());
-      debug_printf ("    |-> Total offset: %s\n",
-                   plongest (total_offset));
-      debug_printf ("    |-> Base address: %s\n",
-                   core_addr_to_string (value_address (array)));
-      debug_printf ("    '-> Contiguous = %s\n",
-                   (is_all_contiguous ? "Yes" : "No"));
+      if (dim < 1 || dim > ndimensions)
+       error (_("DIM argument to SIZE must be between 1 and %d"),
+              ndimensions);
     }
 
-  /* Should we repack this array slice?  */
-  if (!is_all_contiguous && (repack_array_slices || is_string_p))
+  /* Now walk over all the dimensions of the array totalling up the
+     elements in each dimension.  */
+  for (int i = ndimensions - 1; i >= 0; --i)
     {
-      /* Build a type for the repacked slice.  */
-      struct type *repacked_array_type = inner_element_type;
-      for (const auto &d : slice_dims)
+      /* If this is the requested dimension then we're done.  Grab the
+        bounds and return.  */
+      if (i == dim - 1 || dim == -1)
        {
-         /* Create the range.  */
-         dynamic_prop p_low, p_high, p_stride;
+         LONGEST lbound, ubound;
+         struct type *range = array_type->index_type ();
 
-         p_low.set_const_val (d.low);
-         p_high.set_const_val (d.high);
-         p_stride.set_const_val (TYPE_LENGTH (repacked_array_type));
+         if (!get_discrete_bounds (range, &lbound, &ubound))
+           error (_("failed to find array bounds"));
 
-         struct type *new_range
-           = create_range_type_with_stride ((struct type *) NULL,
-                                            TYPE_TARGET_TYPE (d.index),
-                                            &p_low, &p_high, 0, &p_stride,
-                                            true);
-         repacked_array_type
-           = create_array_type (nullptr, repacked_array_type, new_range);
-       }
-
-      /* Now copy the elements from the original ARRAY into the packed
-        array value DEST.  */
-      struct value *dest = allocate_value (repacked_array_type);
-      if (value_lazy (array)
-         || (total_offset + TYPE_LENGTH (array_slice_type)
-             > TYPE_LENGTH (check_typedef (value_type (array)))))
-       {
-         fortran_array_walker<fortran_lazy_array_repacker_impl> p
-           (array_slice_type, value_address (array) + total_offset, dest);
-         p.walk ();
-       }
-      else
-       {
-         fortran_array_walker<fortran_array_repacker_impl> p
-           (array_slice_type, value_address (array) + total_offset,
-            total_offset, array, dest);
-         p.walk ();
-       }
-      array = dest;
-    }
-  else
-    {
-      if (VALUE_LVAL (array) == lval_memory)
-       {
-         /* If the value we're taking a slice from is not yet loaded, or
-            the requested slice is outside the values content range then
-            just create a new lazy value pointing at the memory where the
-            contents we're looking for exist.  */
-         if (value_lazy (array)
-             || (total_offset + TYPE_LENGTH (array_slice_type)
-                 > TYPE_LENGTH (check_typedef (value_type (array)))))
-           array = value_at_lazy (array_slice_type,
-                                  value_address (array) + total_offset);
+         LONGEST dim_size = (ubound - lbound + 1);
+         if (result == 0)
+           result = dim_size;
          else
-           array = value_from_contents_and_address (array_slice_type,
-                                                    (value_contents (array)
-                                                     + total_offset),
-                                                    (value_address (array)
-                                                     + total_offset));
+           result *= dim_size;
+
+         if (dim != -1)
+           break;
        }
-      else if (!value_lazy (array))
-       array = value_from_component (array, array_slice_type, total_offset);
-      else
-       error (_("cannot subscript arrays that are not in memory"));
+
+      /* Peel off another dimension of the array.  */
+      array_type = TYPE_TARGET_TYPE (array_type);
     }
 
-  return array;
+  struct type *result_type
+    = builtin_f_type (gdbarch)->builtin_integer;
+  return value_from_longest (result_type, result);
 }
 
-/* Evaluate FORTRAN_ASSOCIATED expressions.  Both GDBARCH and LANG are
-   extracted from the expression being evaluated.  POINTER is the required
-   first argument to the 'associated' keyword, and TARGET is the optional
-   second argument, this will be nullptr if the user only passed one
-   argument to their use of 'associated'.  */
+/* See f-exp.h.  */
 
-static struct value *
-fortran_associated (struct gdbarch *gdbarch, const language_defn *lang,
-                   struct value *pointer, struct value *target = nullptr)
+struct value *
+eval_op_f_array_size (struct type *expect_type,
+                     struct expression *exp,
+                     enum noside noside,
+                     enum exp_opcode opcode,
+                     struct value *arg1)
 {
-  struct type *result_type = language_bool_type (lang, gdbarch);
+  gdb_assert (opcode == FORTRAN_ARRAY_SIZE);
+  return fortran_array_size (exp->gdbarch, exp->language_defn, arg1);
+}
 
-  /* All Fortran pointers should have the associated property, this is
-     how we know the pointer is pointing at something or not.  */
-  struct type *pointer_type = check_typedef (value_type (pointer));
-  if (TYPE_ASSOCIATED_PROP (pointer_type) == nullptr
-      && pointer_type->code () != TYPE_CODE_PTR)
-    error (_("ASSOCIATED can only be applied to pointers"));
+/* See f-exp.h.  */
 
-  /* Get an address from POINTER.  Fortran (or at least gfortran) models
-     array pointers as arrays with a dynamic data address, so we need to
-     use two approaches here, for real pointers we take the contents of the
-     pointer as an address.  For non-pointers we take the address of the
-     content.  */
-  CORE_ADDR pointer_addr;
-  if (pointer_type->code () == TYPE_CODE_PTR)
-    pointer_addr = value_as_address (pointer);
-  else
-    pointer_addr = value_address (pointer);
+struct value *
+eval_op_f_array_size (struct type *expect_type,
+                     struct expression *exp,
+                     enum noside noside,
+                     enum exp_opcode opcode,
+                     struct value *arg1,
+                     struct value *arg2)
+{
+  gdb_assert (opcode == FORTRAN_ARRAY_SIZE);
+  return fortran_array_size (exp->gdbarch, exp->language_defn, arg1, arg2);
+}
 
-  /* The single argument case, is POINTER associated with anything?  */
-  if (target == nullptr)
-    {
-      bool is_associated = false;
+/* Implement UNOP_FORTRAN_SHAPE expression.  Both GDBARCH and LANG are
+   extracted from the expression being evaluated.  VAL is the value on
+   which 'shape' was used, this can be any type.
 
-      /* If POINTER is an actual pointer and doesn't have an associated
-        property then we need to figure out whether this pointer is
-        associated by looking at the value of the pointer itself.  We make
-        the assumption that a non-associated pointer will be set to 0.
-        This is probably true for most targets, but might not be true for
-        everyone.  */
-      if (pointer_type->code () == TYPE_CODE_PTR
-         && TYPE_ASSOCIATED_PROP (pointer_type) == nullptr)
-       is_associated = (pointer_addr != 0);
-      else
-       is_associated = !type_not_associated (pointer_type);
-      return value_from_longest (result_type, is_associated ? 1 : 0);
-    }
+   Return an array of integers.  If VAL is not an array then the returned
+   array should have zero elements.  If VAL is an array then the returned
+   array should have one element per dimension, with the element
+   containing the extent of that dimension from VAL.  */
 
-  /* The two argument case, is POINTER associated with TARGET?  */
-
-  struct type *target_type = check_typedef (value_type (target));
+static struct value *
+fortran_array_shape (struct gdbarch *gdbarch, const language_defn *lang,
+                    struct value *val)
+{
+  struct type *val_type = check_typedef (value_type (val));
 
-  struct type *pointer_target_type;
-  if (pointer_type->code () == TYPE_CODE_PTR)
-    pointer_target_type = TYPE_TARGET_TYPE (pointer_type);
-  else
-    pointer_target_type = pointer_type;
+  /* If we are passed an array that is either not allocated, or not
+     associated, then this is explicitly not allowed according to the
+     Fortran specification.  */
+  if (val_type->code () == TYPE_CODE_ARRAY
+      && (type_not_associated (val_type) || type_not_allocated (val_type)))
+    error (_("The array passed to SHAPE must be allocated or associated"));
 
-  struct type *target_target_type;
-  if (target_type->code () == TYPE_CODE_PTR)
-    target_target_type = TYPE_TARGET_TYPE (target_type);
-  else
-    target_target_type = target_type;
+  /* The Fortran specification allows non-array types to be passed to this
+     function, in which case we get back an empty array.
 
-  if (pointer_target_type->code () != target_target_type->code ()
-      || (pointer_target_type->code () != TYPE_CODE_ARRAY
-         && (TYPE_LENGTH (pointer_target_type)
-             != TYPE_LENGTH (target_target_type))))
-    error (_("arguments to associated must be of same type and kind"));
+     Calculate the number of dimensions for the resulting array.  */
+  int ndimensions = 0;
+  if (val_type->code () == TYPE_CODE_ARRAY)
+    ndimensions = calc_f77_array_dims (val_type);
 
-  /* If TARGET is not in memory, or the original pointer is specifically
-     known to be not associated with anything, then the answer is obviously
-     false.  Alternatively, if POINTER is an actual pointer and has no
-     associated property, then we have to check if its associated by
-     looking the value of the pointer itself.  We make the assumption that
-     a non-associated pointer will be set to 0.  This is probably true for
-     most targets, but might not be true for everyone.  */
-  if (value_lval_const (target) != lval_memory
-      || type_not_associated (pointer_type)
-      || (TYPE_ASSOCIATED_PROP (pointer_type) == nullptr
-         && pointer_type->code () == TYPE_CODE_PTR
-         && pointer_addr == 0))
-    return value_from_longest (result_type, 0);
+  /* Allocate a result value of the correct type.  */
+  struct type *range
+    = create_static_range_type (nullptr,
+                               builtin_type (gdbarch)->builtin_int,
+                               1, ndimensions);
+  struct type *elm_type = builtin_f_type (gdbarch)->builtin_integer;
+  struct type *result_type = create_array_type (nullptr, elm_type, range);
+  struct value *result = allocate_value (result_type);
+  LONGEST elm_len = TYPE_LENGTH (elm_type);
 
-  /* See the comment for POINTER_ADDR above.  */
-  CORE_ADDR target_addr;
-  if (target_type->code () == TYPE_CODE_PTR)
-    target_addr = value_as_address (target);
-  else
-    target_addr = value_address (target);
+  /* Walk the array dimensions backwards due to the way the array will be
+     laid out in memory, the first dimension will be the most inner.
 
-  /* Wrap the following checks inside a do { ... } while (false) loop so
-     that we can use `break' to jump out of the loop.  */
-  bool is_associated = false;
-  do
+     If VAL was not an array then ndimensions will be 0, in which case we
+     will never go around this loop.  */
+  for (LONGEST dst_offset = elm_len * (ndimensions - 1);
+       dst_offset >= 0;
+       dst_offset -= elm_len)
     {
-      /* If the addresses are different then POINTER is definitely not
-        pointing at TARGET.  */
-      if (pointer_addr != target_addr)
-       break;
+      LONGEST lbound, ubound;
 
-      /* If POINTER is a real pointer (i.e. not an array pointer, which are
-        implemented as arrays with a dynamic content address), then this
-        is all the checking that is needed.  */
-      if (pointer_type->code () == TYPE_CODE_PTR)
-       {
-         is_associated = true;
-         break;
-       }
-
-      /* We have an array pointer.  Check the number of dimensions.  */
-      int pointer_dims = calc_f77_array_dims (pointer_type);
-      int target_dims = calc_f77_array_dims (target_type);
-      if (pointer_dims != target_dims)
-       break;
-
-      /* Now check that every dimension has the same upper bound, lower
-        bound, and stride value.  */
-      int dim = 0;
-      while (dim < pointer_dims)
-       {
-         LONGEST pointer_lowerbound, pointer_upperbound, pointer_stride;
-         LONGEST target_lowerbound, target_upperbound, target_stride;
-
-         pointer_type = check_typedef (pointer_type);
-         target_type = check_typedef (target_type);
-
-         struct type *pointer_range = pointer_type->index_type ();
-         struct type *target_range = target_type->index_type ();
-
-         if (!get_discrete_bounds (pointer_range, &pointer_lowerbound,
-                                   &pointer_upperbound))
-           break;
-
-         if (!get_discrete_bounds (target_range, &target_lowerbound,
-                                   &target_upperbound))
-           break;
-
-         if (pointer_lowerbound != target_lowerbound
-             || pointer_upperbound != target_upperbound)
-           break;
-
-         /* Figure out the stride (in bits) for both pointer and target.
-            If either doesn't have a stride then we take the element size,
-            but we need to convert to bits (hence the * 8).  */
-         pointer_stride = pointer_range->bounds ()->bit_stride ();
-         if (pointer_stride == 0)
-           pointer_stride
-             = type_length_units (check_typedef
-                                    (TYPE_TARGET_TYPE (pointer_type))) * 8;
-         target_stride = target_range->bounds ()->bit_stride ();
-         if (target_stride == 0)
-           target_stride
-             = type_length_units (check_typedef
-                                    (TYPE_TARGET_TYPE (target_type))) * 8;
-         if (pointer_stride != target_stride)
-           break;
-
-         ++dim;
-       }
+      if (!get_discrete_bounds (val_type->index_type (), &lbound, &ubound))
+       error (_("failed to find array bounds"));
 
-      if (dim < pointer_dims)
-       break;
+      LONGEST dim_size = (ubound - lbound + 1);
 
-      is_associated = true;
+      /* And copy the value into the result value.  */
+      struct value *v = value_from_longest (elm_type, dim_size);
+      gdb_assert (dst_offset + TYPE_LENGTH (value_type (v))
+                 <= TYPE_LENGTH (value_type (result)));
+      gdb_assert (TYPE_LENGTH (value_type (v)) == elm_len);
+      value_contents_copy (result, dst_offset, v, 0, elm_len);
+
+      /* Peel another dimension of the array.  */
+      val_type = TYPE_TARGET_TYPE (val_type);
     }
-  while (false);
 
-  return value_from_longest (result_type, is_associated ? 1 : 0);
+  return result;
 }
 
+/* See f-exp.h.  */
+
+struct value *
+eval_op_f_array_shape (struct type *expect_type, struct expression *exp,
+                      enum noside noside, enum exp_opcode opcode,
+                      struct value *arg1)
+{
+  gdb_assert (opcode == UNOP_FORTRAN_SHAPE);
+  return fortran_array_shape (exp->gdbarch, exp->language_defn, arg1);
+}
 
 /* A helper function for UNOP_ABS.  */
 
-static struct value *
+struct value *
 eval_op_f_abs (struct type *expect_type, struct expression *exp,
               enum noside noside,
+              enum exp_opcode opcode,
               struct value *arg1)
 {
-  if (noside == EVAL_SKIP)
-    return eval_skip_value (exp);
   struct type *type = value_type (arg1);
   switch (type->code ())
     {
@@ -1005,13 +786,12 @@ eval_op_f_abs (struct type *expect_type, struct expression *exp,
 
 /* A helper function for BINOP_MOD.  */
 
-static struct value *
+struct value *
 eval_op_f_mod (struct type *expect_type, struct expression *exp,
               enum noside noside,
+              enum exp_opcode opcode,
               struct value *arg1, struct value *arg2)
 {
-  if (noside == EVAL_SKIP)
-    return eval_skip_value (exp);
   struct type *type = value_type (arg1);
   if (type->code () != value_type (arg2)->code ())
     error (_("non-matching types for parameters to MOD ()"));
@@ -1043,13 +823,12 @@ eval_op_f_mod (struct type *expect_type, struct expression *exp,
 
 /* A helper function for UNOP_FORTRAN_CEILING.  */
 
-static struct value *
+struct value *
 eval_op_f_ceil (struct type *expect_type, struct expression *exp,
                enum noside noside,
+               enum exp_opcode opcode,
                struct value *arg1)
 {
-  if (noside == EVAL_SKIP)
-    return eval_skip_value (exp);
   struct type *type = value_type (arg1);
   if (type->code () != TYPE_CODE_FLT)
     error (_("argument to CEILING must be of type float"));
@@ -1062,13 +841,12 @@ eval_op_f_ceil (struct type *expect_type, struct expression *exp,
 
 /* A helper function for UNOP_FORTRAN_FLOOR.  */
 
-static struct value *
+struct value *
 eval_op_f_floor (struct type *expect_type, struct expression *exp,
                 enum noside noside,
+                enum exp_opcode opcode,
                 struct value *arg1)
 {
-  if (noside == EVAL_SKIP)
-    return eval_skip_value (exp);
   struct type *type = value_type (arg1);
   if (type->code () != TYPE_CODE_FLT)
     error (_("argument to FLOOR must be of type float"));
@@ -1081,13 +859,12 @@ eval_op_f_floor (struct type *expect_type, struct expression *exp,
 
 /* A helper function for BINOP_FORTRAN_MODULO.  */
 
-static struct value *
+struct value *
 eval_op_f_modulo (struct type *expect_type, struct expression *exp,
                  enum noside noside,
+                 enum exp_opcode opcode,
                  struct value *arg1, struct value *arg2)
 {
-  if (noside == EVAL_SKIP)
-    return eval_skip_value (exp);
   struct type *type = value_type (arg1);
   if (type->code () != value_type (arg2)->code ())
     error (_("non-matching types for parameters to MODULO ()"));
@@ -1122,482 +899,638 @@ eval_op_f_modulo (struct type *expect_type, struct expression *exp,
 
 /* A helper function for BINOP_FORTRAN_CMPLX.  */
 
-static struct value *
-eval_op_f_cmplx (struct type *expect_type, struct expression *exp,
-                enum noside noside,
-                struct value *arg1, struct value *arg2)
-{
-  if (noside == EVAL_SKIP)
-    return eval_skip_value (exp);
-  struct type *type = builtin_f_type(exp->gdbarch)->builtin_complex_s16;
-  return value_literal_complex (arg1, arg2, type);
-}
+struct value *
+eval_op_f_cmplx (struct type *expect_type, struct expression *exp,
+                enum noside noside,
+                enum exp_opcode opcode,
+                struct value *arg1, struct value *arg2)
+{
+  struct type *type = builtin_f_type(exp->gdbarch)->builtin_complex_s16;
+  return value_literal_complex (arg1, arg2, type);
+}
+
+/* A helper function for UNOP_FORTRAN_KIND.  */
+
+struct value *
+eval_op_f_kind (struct type *expect_type, struct expression *exp,
+               enum noside noside,
+               enum exp_opcode opcode,
+               struct value *arg1)
+{
+  struct type *type = value_type (arg1);
+
+  switch (type->code ())
+    {
+    case TYPE_CODE_STRUCT:
+    case TYPE_CODE_UNION:
+    case TYPE_CODE_MODULE:
+    case TYPE_CODE_FUNC:
+      error (_("argument to kind must be an intrinsic type"));
+    }
+
+  if (!TYPE_TARGET_TYPE (type))
+    return value_from_longest (builtin_type (exp->gdbarch)->builtin_int,
+                              TYPE_LENGTH (type));
+  return value_from_longest (builtin_type (exp->gdbarch)->builtin_int,
+                            TYPE_LENGTH (TYPE_TARGET_TYPE (type)));
+}
+
+/* A helper function for UNOP_FORTRAN_ALLOCATED.  */
+
+struct value *
+eval_op_f_allocated (struct type *expect_type, struct expression *exp,
+                    enum noside noside, enum exp_opcode op,
+                    struct value *arg1)
+{
+  struct type *type = check_typedef (value_type (arg1));
+  if (type->code () != TYPE_CODE_ARRAY)
+    error (_("ALLOCATED can only be applied to arrays"));
+  struct type *result_type
+    = builtin_f_type (exp->gdbarch)->builtin_logical;
+  LONGEST result_value = type_not_allocated (type) ? 0 : 1;
+  return value_from_longest (result_type, result_value);
+}
+
+/* See f-exp.h.  */
+
+struct value *
+eval_op_f_rank (struct type *expect_type,
+               struct expression *exp,
+               enum noside noside,
+               enum exp_opcode op,
+               struct value *arg1)
+{
+  gdb_assert (op == UNOP_FORTRAN_RANK);
+
+  struct type *result_type
+    = builtin_f_type (exp->gdbarch)->builtin_integer;
+  struct type *type = check_typedef (value_type (arg1));
+  if (type->code () != TYPE_CODE_ARRAY)
+    return value_from_longest (result_type, 0);
+  LONGEST ndim = calc_f77_array_dims (type);
+  return value_from_longest (result_type, ndim);
+}
+
+/* A helper function for UNOP_FORTRAN_LOC.  */
+
+struct value *
+eval_op_f_loc (struct type *expect_type, struct expression *exp,
+                    enum noside noside, enum exp_opcode op,
+                    struct value *arg1)
+{
+  struct type *result_type;
+  if (gdbarch_ptr_bit (exp->gdbarch) == 16)
+    result_type = builtin_f_type (exp->gdbarch)->builtin_integer_s2;
+  else if (gdbarch_ptr_bit (exp->gdbarch) == 32)
+    result_type = builtin_f_type (exp->gdbarch)->builtin_integer;
+  else
+    result_type = builtin_f_type (exp->gdbarch)->builtin_integer_s8;
+
+  LONGEST result_value = value_address (arg1);
+  return value_from_longest (result_type, result_value);
+}
+
+namespace expr
+{
+
+/* Called from evaluate to perform array indexing, and sub-range
+   extraction, for Fortran.  As well as arrays this function also
+   handles strings as they can be treated like arrays of characters.
+   ARRAY is the array or string being accessed.  EXP and NOSIDE are as
+   for evaluate.  */
+
+value *
+fortran_undetermined::value_subarray (value *array,
+                                     struct expression *exp,
+                                     enum noside noside)
+{
+  type *original_array_type = check_typedef (value_type (array));
+  bool is_string_p = original_array_type->code () == TYPE_CODE_STRING;
+  const std::vector<operation_up> &ops = std::get<1> (m_storage);
+  int nargs = ops.size ();
+
+  /* Perform checks for ARRAY not being available.  The somewhat overly
+     complex logic here is just to keep backward compatibility with the
+     errors that we used to get before FORTRAN_VALUE_SUBARRAY was
+     rewritten.  Maybe a future task would streamline the error messages we
+     get here, and update all the expected test results.  */
+  if (ops[0]->opcode () != OP_RANGE)
+    {
+      if (type_not_associated (original_array_type))
+       error (_("no such vector element (vector not associated)"));
+      else if (type_not_allocated (original_array_type))
+       error (_("no such vector element (vector not allocated)"));
+    }
+  else
+    {
+      if (type_not_associated (original_array_type))
+       error (_("array not associated"));
+      else if (type_not_allocated (original_array_type))
+       error (_("array not allocated"));
+    }
+
+  /* First check that the number of dimensions in the type we are slicing
+     matches the number of arguments we were passed.  */
+  int ndimensions = calc_f77_array_dims (original_array_type);
+  if (nargs != ndimensions)
+    error (_("Wrong number of subscripts"));
+
+  /* This will be initialised below with the type of the elements held in
+     ARRAY.  */
+  struct type *inner_element_type;
+
+  /* Extract the types of each array dimension from the original array
+     type.  We need these available so we can fill in the default upper and
+     lower bounds if the user requested slice doesn't provide that
+     information.  Additionally unpacking the dimensions like this gives us
+     the inner element type.  */
+  std::vector<struct type *> dim_types;
+  {
+    dim_types.reserve (ndimensions);
+    struct type *type = original_array_type;
+    for (int i = 0; i < ndimensions; ++i)
+      {
+       dim_types.push_back (type);
+       type = TYPE_TARGET_TYPE (type);
+      }
+    /* TYPE is now the inner element type of the array, we start the new
+       array slice off as this type, then as we process the requested slice
+       (from the user) we wrap new types around this to build up the final
+       slice type.  */
+    inner_element_type = type;
+  }
+
+  /* As we analyse the new slice type we need to understand if the data
+     being referenced is contiguous.  Do decide this we must track the size
+     of an element at each dimension of the new slice array.  Initially the
+     elements of the inner most dimension of the array are the same inner
+     most elements as the original ARRAY.  */
+  LONGEST slice_element_size = TYPE_LENGTH (inner_element_type);
+
+  /* Start off assuming all data is contiguous, this will be set to false
+     if access to any dimension results in non-contiguous data.  */
+  bool is_all_contiguous = true;
+
+  /* The TOTAL_OFFSET is the distance in bytes from the start of the
+     original ARRAY to the start of the new slice.  This is calculated as
+     we process the information from the user.  */
+  LONGEST total_offset = 0;
+
+  /* A structure representing information about each dimension of the
+     resulting slice.  */
+  struct slice_dim
+  {
+    /* Constructor.  */
+    slice_dim (LONGEST l, LONGEST h, LONGEST s, struct type *idx)
+      : low (l),
+       high (h),
+       stride (s),
+       index (idx)
+    { /* Nothing.  */ }
+
+    /* The low bound for this dimension of the slice.  */
+    LONGEST low;
+
+    /* The high bound for this dimension of the slice.  */
+    LONGEST high;
+
+    /* The byte stride for this dimension of the slice.  */
+    LONGEST stride;
+
+    struct type *index;
+  };
+
+  /* The dimensions of the resulting slice.  */
+  std::vector<slice_dim> slice_dims;
+
+  /* Process the incoming arguments.   These arguments are in the reverse
+     order to the array dimensions, that is the first argument refers to
+     the last array dimension.  */
+  if (fortran_array_slicing_debug)
+    debug_printf ("Processing array access:\n");
+  for (int i = 0; i < nargs; ++i)
+    {
+      /* For each dimension of the array the user will have either provided
+        a ranged access with optional lower bound, upper bound, and
+        stride, or the user will have supplied a single index.  */
+      struct type *dim_type = dim_types[ndimensions - (i + 1)];
+      fortran_range_operation *range_op
+       = dynamic_cast<fortran_range_operation *> (ops[i].get ());
+      if (range_op != nullptr)
+       {
+         enum range_flag range_flag = range_op->get_flags ();
+
+         LONGEST low, high, stride;
+         low = high = stride = 0;
+
+         if ((range_flag & RANGE_LOW_BOUND_DEFAULT) == 0)
+           low = value_as_long (range_op->evaluate0 (exp, noside));
+         else
+           low = f77_get_lowerbound (dim_type);
+         if ((range_flag & RANGE_HIGH_BOUND_DEFAULT) == 0)
+           high = value_as_long (range_op->evaluate1 (exp, noside));
+         else
+           high = f77_get_upperbound (dim_type);
+         if ((range_flag & RANGE_HAS_STRIDE) == RANGE_HAS_STRIDE)
+           stride = value_as_long (range_op->evaluate2 (exp, noside));
+         else
+           stride = 1;
+
+         if (stride == 0)
+           error (_("stride must not be 0"));
+
+         /* Get information about this dimension in the original ARRAY.  */
+         struct type *target_type = TYPE_TARGET_TYPE (dim_type);
+         struct type *index_type = dim_type->index_type ();
+         LONGEST lb = f77_get_lowerbound (dim_type);
+         LONGEST ub = f77_get_upperbound (dim_type);
+         LONGEST sd = index_type->bit_stride ();
+         if (sd == 0)
+           sd = TYPE_LENGTH (target_type) * 8;
+
+         if (fortran_array_slicing_debug)
+           {
+             debug_printf ("|-> Range access\n");
+             std::string str = type_to_string (dim_type);
+             debug_printf ("|   |-> Type: %s\n", str.c_str ());
+             debug_printf ("|   |-> Array:\n");
+             debug_printf ("|   |   |-> Low bound: %s\n", plongest (lb));
+             debug_printf ("|   |   |-> High bound: %s\n", plongest (ub));
+             debug_printf ("|   |   |-> Bit stride: %s\n", plongest (sd));
+             debug_printf ("|   |   |-> Byte stride: %s\n", plongest (sd / 8));
+             debug_printf ("|   |   |-> Type size: %s\n",
+                           pulongest (TYPE_LENGTH (dim_type)));
+             debug_printf ("|   |   '-> Target type size: %s\n",
+                           pulongest (TYPE_LENGTH (target_type)));
+             debug_printf ("|   |-> Accessing:\n");
+             debug_printf ("|   |   |-> Low bound: %s\n",
+                           plongest (low));
+             debug_printf ("|   |   |-> High bound: %s\n",
+                           plongest (high));
+             debug_printf ("|   |   '-> Element stride: %s\n",
+                           plongest (stride));
+           }
+
+         /* Check the user hasn't asked for something invalid.  */
+         if (high > ub || low < lb)
+           error (_("array subscript out of bounds"));
+
+         /* Calculate what this dimension of the new slice array will look
+            like.  OFFSET is the byte offset from the start of the
+            previous (more outer) dimension to the start of this
+            dimension.  E_COUNT is the number of elements in this
+            dimension.  REMAINDER is the number of elements remaining
+            between the last included element and the upper bound.  For
+            example an access '1:6:2' will include elements 1, 3, 5 and
+            have a remainder of 1 (element #6).  */
+         LONGEST lowest = std::min (low, high);
+         LONGEST offset = (sd / 8) * (lowest - lb);
+         LONGEST e_count = std::abs (high - low) + 1;
+         e_count = (e_count + (std::abs (stride) - 1)) / std::abs (stride);
+         LONGEST new_low = 1;
+         LONGEST new_high = new_low + e_count - 1;
+         LONGEST new_stride = (sd * stride) / 8;
+         LONGEST last_elem = low + ((e_count - 1) * stride);
+         LONGEST remainder = high - last_elem;
+         if (low > high)
+           {
+             offset += std::abs (remainder) * TYPE_LENGTH (target_type);
+             if (stride > 0)
+               error (_("incorrect stride and boundary combination"));
+           }
+         else if (stride < 0)
+           error (_("incorrect stride and boundary combination"));
+
+         /* Is the data within this dimension contiguous?  It is if the
+            newly computed stride is the same size as a single element of
+            this dimension.  */
+         bool is_dim_contiguous = (new_stride == slice_element_size);
+         is_all_contiguous &= is_dim_contiguous;
+
+         if (fortran_array_slicing_debug)
+           {
+             debug_printf ("|   '-> Results:\n");
+             debug_printf ("|       |-> Offset = %s\n", plongest (offset));
+             debug_printf ("|       |-> Elements = %s\n", plongest (e_count));
+             debug_printf ("|       |-> Low bound = %s\n", plongest (new_low));
+             debug_printf ("|       |-> High bound = %s\n",
+                           plongest (new_high));
+             debug_printf ("|       |-> Byte stride = %s\n",
+                           plongest (new_stride));
+             debug_printf ("|       |-> Last element = %s\n",
+                           plongest (last_elem));
+             debug_printf ("|       |-> Remainder = %s\n",
+                           plongest (remainder));
+             debug_printf ("|       '-> Contiguous = %s\n",
+                           (is_dim_contiguous ? "Yes" : "No"));
+           }
 
-/* Special expression evaluation cases for Fortran.  */
+         /* Figure out how big (in bytes) an element of this dimension of
+            the new array slice will be.  */
+         slice_element_size = std::abs (new_stride * e_count);
 
-static struct value *
-evaluate_subexp_f (struct type *expect_type, struct expression *exp,
-                  int *pos, enum noside noside)
-{
-  struct value *arg1 = NULL, *arg2 = NULL;
-  enum exp_opcode op;
-  int pc;
-  struct type *type;
+         slice_dims.emplace_back (new_low, new_high, new_stride,
+                                  index_type);
 
-  pc = *pos;
-  *pos += 1;
-  op = exp->elts[pc].opcode;
+         /* Update the total offset.  */
+         total_offset += offset;
+       }
+      else
+       {
+         /* There is a single index for this dimension.  */
+         LONGEST index
+           = value_as_long (ops[i]->evaluate_with_coercion (exp, noside));
 
-  switch (op)
-    {
-    default:
-      *pos -= 1;
-      return evaluate_subexp_standard (expect_type, exp, pos, noside);
+         /* Get information about this dimension in the original ARRAY.  */
+         struct type *target_type = TYPE_TARGET_TYPE (dim_type);
+         struct type *index_type = dim_type->index_type ();
+         LONGEST lb = f77_get_lowerbound (dim_type);
+         LONGEST ub = f77_get_upperbound (dim_type);
+         LONGEST sd = index_type->bit_stride () / 8;
+         if (sd == 0)
+           sd = TYPE_LENGTH (target_type);
 
-    case UNOP_ABS:
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      return eval_op_f_abs (expect_type, exp, noside, arg1);
+         if (fortran_array_slicing_debug)
+           {
+             debug_printf ("|-> Index access\n");
+             std::string str = type_to_string (dim_type);
+             debug_printf ("|   |-> Type: %s\n", str.c_str ());
+             debug_printf ("|   |-> Array:\n");
+             debug_printf ("|   |   |-> Low bound: %s\n", plongest (lb));
+             debug_printf ("|   |   |-> High bound: %s\n", plongest (ub));
+             debug_printf ("|   |   |-> Byte stride: %s\n", plongest (sd));
+             debug_printf ("|   |   |-> Type size: %s\n",
+                           pulongest (TYPE_LENGTH (dim_type)));
+             debug_printf ("|   |   '-> Target type size: %s\n",
+                           pulongest (TYPE_LENGTH (target_type)));
+             debug_printf ("|   '-> Accessing:\n");
+             debug_printf ("|       '-> Index: %s\n",
+                           plongest (index));
+           }
 
-    case BINOP_MOD:
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
-      return eval_op_f_mod (expect_type, exp, noside, arg1, arg2);
+         /* If the array has actual content then check the index is in
+            bounds.  An array without content (an unbound array) doesn't
+            have a known upper bound, so don't error check in that
+            situation.  */
+         if (index < lb
+             || (dim_type->index_type ()->bounds ()->high.kind () != PROP_UNDEFINED
+                 && index > ub)
+             || (VALUE_LVAL (array) != lval_memory
+                 && dim_type->index_type ()->bounds ()->high.kind () == PROP_UNDEFINED))
+           {
+             if (type_not_associated (dim_type))
+               error (_("no such vector element (vector not associated)"));
+             else if (type_not_allocated (dim_type))
+               error (_("no such vector element (vector not allocated)"));
+             else
+               error (_("no such vector element"));
+           }
 
-    case UNOP_FORTRAN_CEILING:
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      return eval_op_f_ceil (expect_type, exp, noside, arg1);
+         /* Calculate using the type stride, not the target type size.  */
+         LONGEST offset = sd * (index - lb);
+         total_offset += offset;
+       }
+    }
 
-    case UNOP_FORTRAN_FLOOR:
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      return eval_op_f_floor (expect_type, exp, noside, arg1);
+  /* Build a type that represents the new array slice in the target memory
+     of the original ARRAY, this type makes use of strides to correctly
+     find only those elements that are part of the new slice.  */
+  struct type *array_slice_type = inner_element_type;
+  for (const auto &d : slice_dims)
+    {
+      /* Create the range.  */
+      dynamic_prop p_low, p_high, p_stride;
 
-    case UNOP_FORTRAN_ALLOCATED:
-      {
-       arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-       if (noside == EVAL_SKIP)
-         return eval_skip_value (exp);
-       type = check_typedef (value_type (arg1));
-       if (type->code () != TYPE_CODE_ARRAY)
-         error (_("ALLOCATED can only be applied to arrays"));
-       struct type *result_type
-         = builtin_f_type (exp->gdbarch)->builtin_logical;
-       LONGEST result_value = type_not_allocated (type) ? 0 : 1;
-       return value_from_longest (result_type, result_value);
-      }
+      p_low.set_const_val (d.low);
+      p_high.set_const_val (d.high);
+      p_stride.set_const_val (d.stride);
 
-    case BINOP_FORTRAN_MODULO:
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
-      return eval_op_f_modulo (expect_type, exp, noside, arg1, arg2);
+      struct type *new_range
+       = create_range_type_with_stride ((struct type *) NULL,
+                                        TYPE_TARGET_TYPE (d.index),
+                                        &p_low, &p_high, 0, &p_stride,
+                                        true);
+      array_slice_type
+       = create_array_type (nullptr, array_slice_type, new_range);
+    }
 
-    case FORTRAN_LBOUND:
-    case FORTRAN_UBOUND:
-      {
-       int nargs = longest_to_int (exp->elts[pc + 1].longconst);
-       (*pos) += 2;
-
-       /* This assertion should be enforced by the expression parser.  */
-       gdb_assert (nargs == 1 || nargs == 2);
-
-       bool lbound_p = op == FORTRAN_LBOUND;
-
-       /* Check that the first argument is array like.  */
-       arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-       type = check_typedef (value_type (arg1));
-       if (type->code () != TYPE_CODE_ARRAY)
-         {
-           if (lbound_p)
-             error (_("LBOUND can only be applied to arrays"));
-           else
-             error (_("UBOUND can only be applied to arrays"));
-         }
-
-       if (nargs == 1)
-         return fortran_bounds_all_dims (lbound_p, exp->gdbarch, arg1);
-
-       /* User asked for the bounds of a specific dimension of the array.  */
-       arg2 = evaluate_subexp (nullptr, exp, pos, noside);
-       type = check_typedef (value_type (arg2));
-       if (type->code () != TYPE_CODE_INT)
-         {
-           if (lbound_p)
-             error (_("LBOUND second argument should be an integer"));
-           else
-             error (_("UBOUND second argument should be an integer"));
-         }
-
-       return fortran_bounds_for_dimension (lbound_p, exp->gdbarch, arg1,
-                                            arg2);
-      }
-      break;
+  if (fortran_array_slicing_debug)
+    {
+      debug_printf ("'-> Final result:\n");
+      debug_printf ("    |-> Type: %s\n",
+                   type_to_string (array_slice_type).c_str ());
+      debug_printf ("    |-> Total offset: %s\n",
+                   plongest (total_offset));
+      debug_printf ("    |-> Base address: %s\n",
+                   core_addr_to_string (value_address (array)));
+      debug_printf ("    '-> Contiguous = %s\n",
+                   (is_all_contiguous ? "Yes" : "No"));
+    }
 
-    case FORTRAN_ASSOCIATED:
-      {
-       int nargs = longest_to_int (exp->elts[pc + 1].longconst);
-       (*pos) += 2;
-
-       /* This assertion should be enforced by the expression parser.  */
-       gdb_assert (nargs == 1 || nargs == 2);
-
-       arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-
-       if (nargs == 1)
-         {
-           if (noside == EVAL_SKIP)
-             return eval_skip_value (exp);
-           return fortran_associated (exp->gdbarch, exp->language_defn,
-                                      arg1);
-         }
-
-       arg2 = evaluate_subexp (nullptr, exp, pos, noside);
-       if (noside == EVAL_SKIP)
-         return eval_skip_value (exp);
-       return fortran_associated (exp->gdbarch, exp->language_defn,
-                                  arg1, arg2);
-      }
-      break;
+  /* Should we repack this array slice?  */
+  if (!is_all_contiguous && (repack_array_slices || is_string_p))
+    {
+      /* Build a type for the repacked slice.  */
+      struct type *repacked_array_type = inner_element_type;
+      for (const auto &d : slice_dims)
+       {
+         /* Create the range.  */
+         dynamic_prop p_low, p_high, p_stride;
 
-    case BINOP_FORTRAN_CMPLX:
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      arg2 = evaluate_subexp (value_type (arg1), exp, pos, noside);
-      return eval_op_f_cmplx (expect_type, exp, noside, arg1, arg2);
+         p_low.set_const_val (d.low);
+         p_high.set_const_val (d.high);
+         p_stride.set_const_val (TYPE_LENGTH (repacked_array_type));
 
-    case UNOP_FORTRAN_KIND:
-      arg1 = evaluate_subexp (NULL, exp, pos, EVAL_AVOID_SIDE_EFFECTS);
-      type = value_type (arg1);
+         struct type *new_range
+           = create_range_type_with_stride ((struct type *) NULL,
+                                            TYPE_TARGET_TYPE (d.index),
+                                            &p_low, &p_high, 0, &p_stride,
+                                            true);
+         repacked_array_type
+           = create_array_type (nullptr, repacked_array_type, new_range);
+       }
 
-      switch (type->code ())
+      /* Now copy the elements from the original ARRAY into the packed
+        array value DEST.  */
+      struct value *dest = allocate_value (repacked_array_type);
+      if (value_lazy (array)
+         || (total_offset + TYPE_LENGTH (array_slice_type)
+             > TYPE_LENGTH (check_typedef (value_type (array)))))
        {
-         case TYPE_CODE_STRUCT:
-         case TYPE_CODE_UNION:
-         case TYPE_CODE_MODULE:
-         case TYPE_CODE_FUNC:
-           error (_("argument to kind must be an intrinsic type"));
+         fortran_array_walker<fortran_lazy_array_repacker_impl> p
+           (array_slice_type, value_address (array) + total_offset, dest);
+         p.walk ();
        }
-
-      if (!TYPE_TARGET_TYPE (type))
-       return value_from_longest (builtin_type (exp->gdbarch)->builtin_int,
-                                  TYPE_LENGTH (type));
-      return value_from_longest (builtin_type (exp->gdbarch)->builtin_int,
-                                TYPE_LENGTH (TYPE_TARGET_TYPE (type)));
-
-
-    case OP_F77_UNDETERMINED_ARGLIST:
-      /* Remember that in F77, functions, substring ops and array subscript
-        operations cannot be disambiguated at parse time.  We have made
-        all array subscript operations, substring operations as well as
-        function calls come here and we now have to discover what the heck
-        this thing actually was.  If it is a function, we process just as
-        if we got an OP_FUNCALL.  */
-      int nargs = longest_to_int (exp->elts[pc + 1].longconst);
-      (*pos) += 2;
-
-      /* First determine the type code we are dealing with.  */
-      arg1 = evaluate_subexp (nullptr, exp, pos, noside);
-      type = check_typedef (value_type (arg1));
-      enum type_code code = type->code ();
-
-      if (code == TYPE_CODE_PTR)
+      else
        {
-         /* Fortran always passes variable to subroutines as pointer.
-            So we need to look into its target type to see if it is
-            array, string or function.  If it is, we need to switch
-            to the target value the original one points to.  */
-         struct type *target_type = check_typedef (TYPE_TARGET_TYPE (type));
-
-         if (target_type->code () == TYPE_CODE_ARRAY
-             || target_type->code () == TYPE_CODE_STRING
-             || target_type->code () == TYPE_CODE_FUNC)
-           {
-             arg1 = value_ind (arg1);
-             type = check_typedef (value_type (arg1));
-             code = type->code ();
-           }
+         fortran_array_walker<fortran_array_repacker_impl> p
+           (array_slice_type, value_address (array) + total_offset,
+            total_offset, array, dest);
+         p.walk ();
        }
-
-      switch (code)
+      array = dest;
+    }
+  else
+    {
+      if (VALUE_LVAL (array) == lval_memory)
        {
-       case TYPE_CODE_ARRAY:
-       case TYPE_CODE_STRING:
-         return fortran_value_subarray (arg1, exp, pos, nargs, noside);
-
-       case TYPE_CODE_PTR:
-       case TYPE_CODE_FUNC:
-       case TYPE_CODE_INTERNAL_FUNCTION:
-         {
-           /* It's a function call.  Allocate arg vector, including
-           space for the function to be called in argvec[0] and a
-           termination NULL.  */
-           struct value **argvec = (struct value **)
-             alloca (sizeof (struct value *) * (nargs + 2));
-           argvec[0] = arg1;
-           int tem = 1;
-           for (; tem <= nargs; tem++)
-             {
-               bool is_internal_func = (code == TYPE_CODE_INTERNAL_FUNCTION);
-               argvec[tem]
-                 = fortran_prepare_argument (exp, pos, (tem - 1),
-                                             is_internal_func,
-                                             value_type (arg1), noside);
-             }
-           argvec[tem] = 0;    /* signal end of arglist */
-           if (noside == EVAL_SKIP)
-             return eval_skip_value (exp);
-           return evaluate_subexp_do_call (exp, noside, argvec[0],
-                                           gdb::make_array_view (argvec + 1,
-                                                                 nargs),
-                                           NULL, expect_type);
-         }
-
-       default:
-         error (_("Cannot perform substring on this type"));
+         /* If the value we're taking a slice from is not yet loaded, or
+            the requested slice is outside the values content range then
+            just create a new lazy value pointing at the memory where the
+            contents we're looking for exist.  */
+         if (value_lazy (array)
+             || (total_offset + TYPE_LENGTH (array_slice_type)
+                 > TYPE_LENGTH (check_typedef (value_type (array)))))
+           array = value_at_lazy (array_slice_type,
+                                  value_address (array) + total_offset);
+         else
+           array = value_from_contents_and_address (array_slice_type,
+                                                    (value_contents (array)
+                                                     + total_offset),
+                                                    (value_address (array)
+                                                     + total_offset));
        }
+      else if (!value_lazy (array))
+       array = value_from_component (array, array_slice_type, total_offset);
+      else
+       error (_("cannot subscript arrays that are not in memory"));
     }
 
-  /* Should be unreachable.  */
-  return nullptr;
+  return array;
 }
 
-/* Special expression lengths for Fortran.  */
-
-static void
-operator_length_f (const struct expression *exp, int pc, int *oplenp,
-                  int *argsp)
+value *
+fortran_undetermined::evaluate (struct type *expect_type,
+                               struct expression *exp,
+                               enum noside noside)
 {
-  int oplen = 1;
-  int args = 0;
-
-  switch (exp->elts[pc - 1].opcode)
+  value *callee = std::get<0> (m_storage)->evaluate (nullptr, exp, noside);
+  if (noside == EVAL_AVOID_SIDE_EFFECTS
+      && is_dynamic_type (value_type (callee)))
+    callee = std::get<0> (m_storage)->evaluate (nullptr, exp, EVAL_NORMAL);
+  struct type *type = check_typedef (value_type (callee));
+  enum type_code code = type->code ();
+
+  if (code == TYPE_CODE_PTR)
     {
-    default:
-      operator_length_standard (exp, pc, oplenp, argsp);
-      return;
-
-    case UNOP_FORTRAN_KIND:
-    case UNOP_FORTRAN_FLOOR:
-    case UNOP_FORTRAN_CEILING:
-    case UNOP_FORTRAN_ALLOCATED:
-      oplen = 1;
-      args = 1;
-      break;
-
-    case BINOP_FORTRAN_CMPLX:
-    case BINOP_FORTRAN_MODULO:
-      oplen = 1;
-      args = 2;
-      break;
-
-    case FORTRAN_ASSOCIATED:
-    case FORTRAN_LBOUND:
-    case FORTRAN_UBOUND:
-      oplen = 3;
-      args = longest_to_int (exp->elts[pc - 2].longconst);
-      break;
-
-    case OP_F77_UNDETERMINED_ARGLIST:
-      oplen = 3;
-      args = 1 + longest_to_int (exp->elts[pc - 2].longconst);
-      break;
+      /* Fortran always passes variable to subroutines as pointer.
+        So we need to look into its target type to see if it is
+        array, string or function.  If it is, we need to switch
+        to the target value the original one points to.  */
+      struct type *target_type = check_typedef (TYPE_TARGET_TYPE (type));
+
+      if (target_type->code () == TYPE_CODE_ARRAY
+         || target_type->code () == TYPE_CODE_STRING
+         || target_type->code () == TYPE_CODE_FUNC)
+       {
+         callee = value_ind (callee);
+         type = check_typedef (value_type (callee));
+         code = type->code ();
+       }
     }
 
-  *oplenp = oplen;
-  *argsp = args;
-}
+  switch (code)
+    {
+    case TYPE_CODE_ARRAY:
+    case TYPE_CODE_STRING:
+      return value_subarray (callee, exp, noside);
 
-/* Helper for PRINT_SUBEXP_F.  Arguments are as for PRINT_SUBEXP_F, except
-   the extra argument NAME which is the text that should be printed as the
-   name of this operation.  */
+    case TYPE_CODE_PTR:
+    case TYPE_CODE_FUNC:
+    case TYPE_CODE_INTERNAL_FUNCTION:
+      {
+       /* It's a function call.  Allocate arg vector, including
+          space for the function to be called in argvec[0] and a
+          termination NULL.  */
+       const std::vector<operation_up> &actual (std::get<1> (m_storage));
+       std::vector<value *> argvec (actual.size ());
+       bool is_internal_func = (code == TYPE_CODE_INTERNAL_FUNCTION);
+       for (int tem = 0; tem < argvec.size (); tem++)
+         argvec[tem] = fortran_prepare_argument (exp, actual[tem].get (),
+                                                 tem, is_internal_func,
+                                                 value_type (callee),
+                                                 noside);
+       return evaluate_subexp_do_call (exp, noside, callee, argvec,
+                                       nullptr, expect_type);
+      }
 
-static void
-print_unop_subexp_f (struct expression *exp, int *pos,
-                    struct ui_file *stream, enum precedence prec,
-                    const char *name)
-{
-  (*pos)++;
-  fprintf_filtered (stream, "%s(", name);
-  print_subexp (exp, pos, stream, PREC_SUFFIX);
-  fputs_filtered (")", stream);
+    default:
+      error (_("Cannot perform substring on this type"));
+    }
 }
 
-/* Helper for PRINT_SUBEXP_F.  Arguments are as for PRINT_SUBEXP_F, except
-   the extra argument NAME which is the text that should be printed as the
-   name of this operation.  */
-
-static void
-print_binop_subexp_f (struct expression *exp, int *pos,
-                     struct ui_file *stream, enum precedence prec,
-                     const char *name)
+value *
+fortran_bound_1arg::evaluate (struct type *expect_type,
+                             struct expression *exp,
+                             enum noside noside)
 {
-  (*pos)++;
-  fprintf_filtered (stream, "%s(", name);
-  print_subexp (exp, pos, stream, PREC_SUFFIX);
-  fputs_filtered (",", stream);
-  print_subexp (exp, pos, stream, PREC_SUFFIX);
-  fputs_filtered (")", stream);
+  bool lbound_p = std::get<0> (m_storage) == FORTRAN_LBOUND;
+  value *arg1 = std::get<1> (m_storage)->evaluate (nullptr, exp, noside);
+  fortran_require_array (value_type (arg1), lbound_p);
+  return fortran_bounds_all_dims (lbound_p, exp->gdbarch, arg1);
 }
 
-/* Helper for PRINT_SUBEXP_F.  Arguments are as for PRINT_SUBEXP_F, except
-   the extra argument NAME which is the text that should be printed as the
-   name of this operation.  */
-
-static void
-print_unop_or_binop_subexp_f (struct expression *exp, int *pos,
-                             struct ui_file *stream, enum precedence prec,
-                             const char *name)
+value *
+fortran_bound_2arg::evaluate (struct type *expect_type,
+                             struct expression *exp,
+                             enum noside noside)
 {
-  unsigned nargs = longest_to_int (exp->elts[*pos + 1].longconst);
-  (*pos) += 3;
-  fprintf_filtered (stream, "%s (", name);
-  for (unsigned tem = 0; tem < nargs; tem++)
+  bool lbound_p = std::get<0> (m_storage) == FORTRAN_LBOUND;
+  value *arg1 = std::get<1> (m_storage)->evaluate (nullptr, exp, noside);
+  fortran_require_array (value_type (arg1), lbound_p);
+
+  /* User asked for the bounds of a specific dimension of the array.  */
+  value *arg2 = std::get<2> (m_storage)->evaluate (nullptr, exp, noside);
+  struct type *type = check_typedef (value_type (arg2));
+  if (type->code () != TYPE_CODE_INT)
     {
-      if (tem != 0)
-       fputs_filtered (", ", stream);
-      print_subexp (exp, pos, stream, PREC_ABOVE_COMMA);
+      if (lbound_p)
+       error (_("LBOUND second argument should be an integer"));
+      else
+       error (_("UBOUND second argument should be an integer"));
     }
-  fputs_filtered (")", stream);
-}
-
-/* Special expression printing for Fortran.  */
-
-static void
-print_subexp_f (struct expression *exp, int *pos,
-               struct ui_file *stream, enum precedence prec)
-{
-  int pc = *pos;
-  enum exp_opcode op = exp->elts[pc].opcode;
-
-  switch (op)
-    {
-    default:
-      print_subexp_standard (exp, pos, stream, prec);
-      return;
-
-    case UNOP_FORTRAN_KIND:
-      print_unop_subexp_f (exp, pos, stream, prec, "KIND");
-      return;
-
-    case UNOP_FORTRAN_FLOOR:
-      print_unop_subexp_f (exp, pos, stream, prec, "FLOOR");
-      return;
-
-    case UNOP_FORTRAN_CEILING:
-      print_unop_subexp_f (exp, pos, stream, prec, "CEILING");
-      return;
-
-    case UNOP_FORTRAN_ALLOCATED:
-      print_unop_subexp_f (exp, pos, stream, prec, "ALLOCATED");
-      return;
-
-    case BINOP_FORTRAN_CMPLX:
-      print_binop_subexp_f (exp, pos, stream, prec, "CMPLX");
-      return;
-
-    case BINOP_FORTRAN_MODULO:
-      print_binop_subexp_f (exp, pos, stream, prec, "MODULO");
-      return;
-
-    case FORTRAN_ASSOCIATED:
-      print_unop_or_binop_subexp_f (exp, pos, stream, prec, "ASSOCIATED");
-      return;
 
-    case FORTRAN_LBOUND:
-      print_unop_or_binop_subexp_f (exp, pos, stream, prec, "LBOUND");
-      return;
-
-    case FORTRAN_UBOUND:
-      print_unop_or_binop_subexp_f (exp, pos, stream, prec, "UBOUND");
-      return;
-
-    case OP_F77_UNDETERMINED_ARGLIST:
-      (*pos)++;
-      print_subexp_funcall (exp, pos, stream);
-      return;
-    }
+  return fortran_bounds_for_dimension (lbound_p, exp->gdbarch, arg1, arg2);
 }
 
-/* Special expression dumping for Fortran.  */
+/* Implement STRUCTOP_STRUCT for Fortran.  See operation::evaluate in
+   expression.h for argument descriptions.  */
 
-static int
-dump_subexp_body_f (struct expression *exp,
-                   struct ui_file *stream, int elt)
+value *
+fortran_structop_operation::evaluate (struct type *expect_type,
+                                     struct expression *exp,
+                                     enum noside noside)
 {
-  int opcode = exp->elts[elt].opcode;
-  int oplen, nargs, i;
-
-  switch (opcode)
+  value *arg1 = std::get<0> (m_storage)->evaluate (nullptr, exp, noside);
+  const char *str = std::get<1> (m_storage).c_str ();
+  if (noside == EVAL_AVOID_SIDE_EFFECTS)
     {
-    default:
-      return dump_subexp_body_standard (exp, stream, elt);
-
-    case UNOP_FORTRAN_KIND:
-    case UNOP_FORTRAN_FLOOR:
-    case UNOP_FORTRAN_CEILING:
-    case UNOP_FORTRAN_ALLOCATED:
-    case BINOP_FORTRAN_CMPLX:
-    case BINOP_FORTRAN_MODULO:
-      operator_length_f (exp, (elt + 1), &oplen, &nargs);
-      break;
-
-    case FORTRAN_ASSOCIATED:
-    case FORTRAN_LBOUND:
-    case FORTRAN_UBOUND:
-      operator_length_f (exp, (elt + 3), &oplen, &nargs);
-      break;
+      struct type *type = lookup_struct_elt_type (value_type (arg1), str, 1);
 
-    case OP_F77_UNDETERMINED_ARGLIST:
-      return dump_subexp_body_funcall (exp, stream, elt + 1);
+      if (type != nullptr && is_dynamic_type (type))
+       arg1 = std::get<0> (m_storage)->evaluate (nullptr, exp, EVAL_NORMAL);
     }
 
-  elt += oplen;
-  for (i = 0; i < nargs; i += 1)
-    elt = dump_subexp (exp, stream, elt);
+  value *elt = value_struct_elt (&arg1, {}, str, NULL, "structure");
 
-  return elt;
-}
-
-/* Special expression checking for Fortran.  */
-
-static int
-operator_check_f (struct expression *exp, int pos,
-                 int (*objfile_func) (struct objfile *objfile,
-                                      void *data),
-                 void *data)
-{
-  const union exp_element *const elts = exp->elts;
-
-  switch (elts[pos].opcode)
+  if (noside == EVAL_AVOID_SIDE_EFFECTS)
     {
-    case UNOP_FORTRAN_KIND:
-    case UNOP_FORTRAN_FLOOR:
-    case UNOP_FORTRAN_CEILING:
-    case UNOP_FORTRAN_ALLOCATED:
-    case BINOP_FORTRAN_CMPLX:
-    case BINOP_FORTRAN_MODULO:
-    case FORTRAN_ASSOCIATED:
-    case FORTRAN_LBOUND:
-    case FORTRAN_UBOUND:
-      /* Any references to objfiles are held in the arguments to this
-        expression, not within the expression itself, so no additional
-        checking is required here, the outer expression iteration code
-        will take care of checking each argument.  */
-      break;
-
-    default:
-      return operator_check_standard (exp, pos, objfile_func, data);
+      struct type *elt_type = value_type (elt);
+      if (is_dynamic_type (elt_type))
+       {
+         const gdb_byte *valaddr = value_contents_for_printing (elt);
+         CORE_ADDR address = value_address (elt);
+         gdb::array_view<const gdb_byte> view
+           = gdb::make_array_view (valaddr, TYPE_LENGTH (elt_type));
+         elt_type = resolve_dynamic_type (elt_type, view, address);
+       }
+      elt = value_zero (elt_type, VALUE_LVAL (elt));
     }
 
-  return 0;
+  return elt;
 }
 
-/* Expression processing for Fortran.  */
-const struct exp_descriptor f_language::exp_descriptor_tab =
-{
-  print_subexp_f,
-  operator_length_f,
-  operator_check_f,
-  dump_subexp_body_f,
-  evaluate_subexp_f
-};
+} /* namespace expr */
 
 /* See language.h.  */
 
@@ -1752,11 +1685,11 @@ _initialize_f_language ()
 
   add_basic_prefix_cmd ("fortran", no_class,
                        _("Prefix command for changing Fortran-specific settings."),
-                       &set_fortran_list, "set fortran ", 0, &setlist);
+                       &set_fortran_list, 0, &setlist);
 
   add_show_prefix_cmd ("fortran", no_class,
                       _("Generic command for showing Fortran-specific settings."),
-                      &show_fortran_list, "show fortran ", 0, &showlist);
+                      &show_fortran_list, 0, &showlist);
 
   add_setshow_boolean_cmd ("repack-array-slices", class_vars,
                           &repack_array_slices, _("\
@@ -1844,12 +1777,13 @@ fortran_argument_convert (struct value *value, bool is_artificial)
    malloc in target memory.  Infinite recursion ensues.  */
 
 static value *
-fortran_prepare_argument (struct expression *exp, int *pos,
-                         int arg_num, bool is_internal_call_p,
-                         struct type *func_type, enum noside noside)
+fortran_prepare_argument (struct expression *exp,
+                         expr::operation *subexp,
+                         int arg_num, bool is_internal_call_p,
+                         struct type *func_type, enum noside noside)
 {
   if (is_internal_call_p)
-    return evaluate_subexp_with_coercion (exp, pos, noside);
+    return subexp->evaluate_with_coercion (exp, noside);
 
   bool is_artificial = ((arg_num >= func_type->num_fields ())
                        ? true
@@ -1868,13 +1802,18 @@ fortran_prepare_argument (struct expression *exp, int *pos,
      As we already pass the address of non-artificial arguments, all we
      need to do if skip the UNOP_ADDR operator in the expression and mark
      the argument as non-artificial.  */
-  if (is_artificial && exp->elts[*pos].opcode == UNOP_ADDR)
+  if (is_artificial)
     {
-      (*pos)++;
-      is_artificial = false;
+      expr::unop_addr_operation *addrop
+       = dynamic_cast<expr::unop_addr_operation *> (subexp);
+      if (addrop != nullptr)
+       {
+         subexp = addrop->get_expression ().get ();
+         is_artificial = false;
+       }
     }
 
-  struct value *arg_val = evaluate_subexp_with_coercion (exp, pos, noside);
+  struct value *arg_val = subexp->evaluate_with_coercion (exp, noside);
   return fortran_argument_convert (arg_val, is_artificial);
 }
 
This page took 0.048548 seconds and 4 git commands to generate.