Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Burgess <andrew.burgess@embecosm.com>
To: gdb-patches@sourceware.org
Subject: [PATCHv2 00/10] Fortran Array Slicing and Striding Support
Date: Wed, 26 Aug 2020 15:49:07 +0100	[thread overview]
Message-ID: <cover.1598452395.git.andrew.burgess@embecosm.com> (raw)
In-Reply-To: <cover.1597319264.git.andrew.burgess@embecosm.com>

Since the V1 series I have rebased these patches on top of Pedro's
enum-flags rewrite[1].  As such patches 1 -> 4 should not be reviewed
in this sereis, but at[1].

The actual content of the remaining patches has not changed since the
V1 series.  I'm still keen to get any feedback.

Patches 5, 6, 7, and 9 are independent of the earlier patches, and could be
merged before the GDB 10 branching if I get positive feedback.

Patches 8 and 10 I'd like to leave until after the GDB 10 branch has
been created, it's these patches that depend on the earlier 1->4 patch
series.

All feedback welcome.

Thanks,
Andrew

[1] https://sourceware.org/pipermail/gdb-patches/2020-August/171392.html

----

Andrew Burgess (7):
  gdb: additional changes to make use of type_instance_flags more
  gdb/fortran: Clean up array/string expression evaluation
  gdb/fortran: Move Fortran expression handling into f-lang.c
  gdb/fortran: Change whitespace when printing arrays
  gdb: Convert enum range_type to a bit field enum
  gdb/testsuite: Add missing expected results
  gdb/fortran: Add support for Fortran array slices at the GDB prompt

Pedro Alves (3):
  Rewrite valid-expr.h's internals in terms of the detection idiom
    (C++17/N4502)
  Use type_instance_flags more throughout
  Rewrite enum_flags, add unit tests, fix problems

 gdb/ChangeLog                                 | 121 +++
 gdb/Makefile.in                               |   2 +
 gdb/NEWS                                      |  13 +
 gdb/avr-tdep.c                                |  13 +-
 gdb/btrace.c                                  |   4 +-
 gdb/compile/compile-c-types.c                 |   3 +-
 gdb/compile/compile-cplus-symbols.c           |   4 +-
 gdb/compile/compile-cplus-types.c             |  10 +-
 gdb/doc/ChangeLog                             |   7 +
 gdb/doc/gdb.texinfo                           |  30 +
 gdb/dwarf2/read.c                             |   7 +-
 gdb/eval.c                                    | 227 +----
 gdb/expprint.c                                | 114 ++-
 gdb/expression.h                              |  39 +-
 gdb/f-array-walker.h                          | 255 ++++++
 gdb/f-exp.y                                   |  52 +-
 gdb/f-lang.c                                  | 779 ++++++++++++++++++
 gdb/f-lang.h                                  |  19 +-
 gdb/f-valprint.c                              | 187 +++--
 gdb/fortran-operator.def                      |   8 +
 gdb/ft32-tdep.c                               |  13 +-
 gdb/gdbarch.c                                 |   8 +-
 gdb/gdbarch.h                                 |  16 +-
 gdb/gdbarch.sh                                |  10 +-
 gdb/gdbtypes.c                                |  70 +-
 gdb/gdbtypes.h                                |  15 +-
 gdb/go-exp.y                                  |   2 +-
 gdb/parse.c                                   |  25 +-
 gdb/parser-defs.h                             |  16 +
 gdb/record-btrace.c                           |  10 +-
 gdb/rust-exp.y                                |  21 +-
 gdb/rust-lang.c                               |  25 +-
 gdb/s390-tdep.c                               |  13 +-
 gdb/stabsread.c                               |   2 +-
 gdb/std-operator.def                          |   8 -
 gdb/testsuite/ChangeLog                       |  22 +
 .../gdb.fortran/array-slices-bad.exp          |  69 ++
 .../gdb.fortran/array-slices-bad.f90          |  42 +
 .../gdb.fortran/array-slices-sub-slices.exp   | 111 +++
 .../gdb.fortran/array-slices-sub-slices.f90   |  96 +++
 gdb/testsuite/gdb.fortran/array-slices.exp    | 264 +++++-
 gdb/testsuite/gdb.fortran/array-slices.f90    | 364 +++++++-
 .../gdb.fortran/class-allocatable-array.exp   |   2 +-
 gdb/testsuite/gdb.fortran/multi-dim.exp       |   2 +-
 gdb/testsuite/gdb.fortran/vla-sizeof.exp      |   4 +-
 gdb/testsuite/gdb.fortran/vla-type.exp        |   6 +-
 gdb/testsuite/gdb.mi/mi-vla-fortran.exp       |   2 +-
 gdb/type-stack.c                              |   4 +-
 gdb/unittests/enum-flags-selftests.c          | 586 +++++++++++++
 gdbsupport/enum-flags.h                       | 366 ++++++--
 gdbsupport/traits.h                           |  67 ++
 gdbsupport/valid-expr.h                       |  35 +-
 52 files changed, 3505 insertions(+), 685 deletions(-)
 create mode 100644 gdb/f-array-walker.h
 create mode 100644 gdb/testsuite/gdb.fortran/array-slices-bad.exp
 create mode 100644 gdb/testsuite/gdb.fortran/array-slices-bad.f90
 create mode 100644 gdb/testsuite/gdb.fortran/array-slices-sub-slices.exp
 create mode 100644 gdb/testsuite/gdb.fortran/array-slices-sub-slices.f90
 create mode 100644 gdb/unittests/enum-flags-selftests.c

-- 
2.25.4



  parent reply	other threads:[~2020-08-26 14:49 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-13 12:58 [PATCH 0/8] " Andrew Burgess
2020-08-13 12:58 ` [PATCH 1/8] gdbsupport: Provide global operators |=, &=, and ^= for enum bit flags Andrew Burgess
2020-08-15 17:16   ` Tom Tromey
2020-08-16  9:13     ` Andrew Burgess
2020-08-17 10:40     ` Andrew Burgess
2020-08-20 16:00       ` Pedro Alves
2020-08-21 14:49       ` Pedro Alves
2020-08-21 15:57         ` Andrew Burgess
2020-08-21 18:10           ` Pedro Alves
2020-08-13 12:58 ` [PATCH 2/8] gdbsupport: Make function arguments constant in enum-flags.h Andrew Burgess
2020-08-15 19:45   ` Tom Tromey
2020-08-16  9:08     ` Andrew Burgess
2020-08-13 12:58 ` [PATCH 3/8] gdb/fortran: Clean up array/string expression evaluation Andrew Burgess
2020-08-13 12:58 ` [PATCH 4/8] gdb/fortran: Move Fortran expression handling into f-lang.c Andrew Burgess
2020-08-13 12:58 ` [PATCH 5/8] gdb/fortran: Change whitespace when printing arrays Andrew Burgess
2020-08-13 12:58 ` [PATCH 6/8] gdb: Convert enum range_type to a bit field enum Andrew Burgess
2020-08-13 12:58 ` [PATCH 7/8] gdb/testsuite: Add missing expected results Andrew Burgess
2020-08-13 12:58 ` [PATCH 8/8] gdb/fortran: Add support for Fortran array slices at the GDB prompt Andrew Burgess
2020-08-13 13:31   ` Eli Zaretskii
2020-08-26 14:49 ` Andrew Burgess [this message]
2020-08-26 14:49   ` [PATCHv2 01/10] Rewrite valid-expr.h's internals in terms of the detection idiom (C++17/N4502) Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 02/10] Use type_instance_flags more throughout Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 03/10] Rewrite enum_flags, add unit tests, fix problems Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 04/10] gdb: additional changes to make use of type_instance_flags more Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 05/10] gdb/fortran: Clean up array/string expression evaluation Andrew Burgess
2020-09-19  8:53     ` Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 06/10] gdb/fortran: Move Fortran expression handling into f-lang.c Andrew Burgess
2020-09-19  8:53     ` Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 07/10] gdb/fortran: Change whitespace when printing arrays Andrew Burgess
2020-09-19  8:54     ` Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 08/10] gdb: Convert enum range_type to a bit field enum Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 09/10] gdb/testsuite: Add missing expected results Andrew Burgess
2020-09-18  9:53     ` Andrew Burgess
2020-08-26 14:49   ` [PATCHv2 10/10] gdb/fortran: Add support for Fortran array slices at the GDB prompt Andrew Burgess
2020-08-26 17:02     ` Eli Zaretskii
2020-09-19  9:47   ` [PATCHv3 0/2] Fortran Array Slicing and Striding Support Andrew Burgess
2020-09-19  9:48     ` [PATCHv3 1/2] gdb: Convert enum range_type to a bit field enum Andrew Burgess
2020-09-19 13:50       ` Simon Marchi
2020-09-19  9:48     ` [PATCHv3 2/2] gdb/fortran: Add support for Fortran array slices at the GDB prompt Andrew Burgess
2020-09-19 10:03       ` Eli Zaretskii
2020-09-28  9:40     ` [PATCHv4 0/3] Fortran Array Slicing and Striding Support Andrew Burgess
2020-09-28  9:40       ` [PATCHv4 1/3] gdb: Convert enum range_type to a bit field enum Andrew Burgess
2020-09-28  9:40       ` [PATCHv4 2/3] gdb: rename 'enum range_type' to 'enum range_flag' Andrew Burgess
2020-09-28  9:40       ` [PATCHv4 3/3] gdb/fortran: Add support for Fortran array slices at the GDB prompt Andrew Burgess
2020-09-28  9:52         ` Eli Zaretskii via Gdb-patches
2020-10-11 18:12       ` [PATCHv5 0/4] Fortran Array Slicing and Striding Support Andrew Burgess
2020-10-11 18:12         ` [PATCHv5 1/4] gdb: Convert enum range_type to a bit field enum Andrew Burgess
2020-10-20 20:16           ` Tom Tromey
2020-10-11 18:12         ` [PATCHv5 2/4] gdb: rename 'enum range_type' to 'enum range_flag' Andrew Burgess
2020-10-20 20:16           ` Tom Tromey
2020-10-11 18:12         ` [PATCHv5 3/4] gdb/fortran: add support for parsing array strides in expressions Andrew Burgess
2020-10-12 13:21           ` Simon Marchi
2020-10-20 20:17           ` Tom Tromey
2020-10-22 10:42           ` Andrew Burgess
2020-10-11 18:12         ` [PATCHv5 4/4] gdb/fortran: Add support for Fortran array slices at the GDB prompt Andrew Burgess
2020-10-12 14:10           ` Simon Marchi
2020-10-20 20:45           ` Tom Tromey
2020-10-29 11:08             ` Andrew Burgess
2020-10-31 22:16           ` [PATCHv6] " Andrew Burgess
2020-11-12 12:09             ` Andrew Burgess
2020-11-12 18:58             ` Tom Tromey
2020-11-19 11:56             ` Andrew Burgess

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1598452395.git.andrew.burgess@embecosm.com \
    --to=andrew.burgess@embecosm.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox