This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCHv2 0/8] Series of Fortran type printing patches
- From: Andrew Burgess <andrew dot burgess at embecosm dot com>
- To: gdb-patches at sourceware dot org
- Cc: Richard Bunt <Richard dot Bunt at arm dot com>
- Date: Tue, 23 Apr 2019 23:16:24 +0100
- Subject: Re: [PATCHv2 0/8] Series of Fortran type printing patches
- References: <cover.1552913183.git.andrew.burgess@embecosm.com> <cover.1554249182.git.andrew.burgess@embecosm.com>
* Andrew Burgess <andrew.burgess@embecosm.com> [2019-04-03 00:58:34 +0100]:
> Changes since the original series:
>
> + Original patches 1 and 2 have been committed, and are no longer
> included in this series.
>
> + Two new patches are introduced in order to support an updated
> version of patch 3. We now have the fortran specific operators in
> a separate file (like ada does).
>
> + Patch 4 has had a clean up, removing some dead code that I should
> have removed before posting the first time. The remaining code is
> unchanged, but the patch commit message has been expanded to
> better explain the patch.
>
> + Patches 5 to 8 are unchanged other than having been rebased onto
> latest HEAD.
Ping! Anyone have any feedback? Especially around adding new
fortran-operator.def file...
Thanks,
Andrew
>
> Thanks,
> Andrew
>
> --
>
> Andrew Burgess (8):
> gdb: Remove an unbalanced stray double quote from a comment
> gdb/fortran: Introduce fortran-operator.def file
> gdb/fortran: Additional builtin procedures
> gdb/fortran: better types for components of complex numbers
> gdb/fortran: Print 'void' type in lower case
> gdb/fortran: print function arguments when printing function type
> gdb/fortran: Update rules for printing whitespace in types
> gdb/fortran: Add allocatable type qualifier
>
> gdb/ChangeLog | 76 ++++++
> gdb/dwarf2read.c | 35 ++-
> gdb/expprint.c | 1 -
> gdb/expression.h | 1 +
> gdb/f-exp.y | 15 +-
> gdb/f-lang.c | 298 +++++++++++++++++++++-
> gdb/f-typeprint.c | 55 +++-
> gdb/f-valprint.c | 2 +-
> gdb/fortran-operator.def | 27 ++
> gdb/gdbtypes.h | 4 +
> gdb/parse.c | 1 -
> gdb/std-operator.def | 3 +-
> gdb/testsuite/ChangeLog | 36 +++
> gdb/testsuite/gdb.dwarf2/dw2-case-insensitive.exp | 2 +-
> gdb/testsuite/gdb.fortran/complex.exp | 41 ++-
> gdb/testsuite/gdb.fortran/complex.f | 24 --
> gdb/testsuite/gdb.fortran/complex.f90 | 44 ++++
> gdb/testsuite/gdb.fortran/exprs.exp | 2 +-
> gdb/testsuite/gdb.fortran/intrinsics.exp | 35 +++
> gdb/testsuite/gdb.fortran/ptr-indentation.exp | 2 +-
> gdb/testsuite/gdb.fortran/ptype-on-functions.exp | 45 ++++
> gdb/testsuite/gdb.fortran/ptype-on-functions.f90 | 74 ++++++
> gdb/testsuite/gdb.fortran/vla-datatypes.exp | 10 +-
> gdb/testsuite/gdb.fortran/vla-ptr-info.exp | 2 +-
> gdb/testsuite/gdb.fortran/vla-ptype.exp | 8 +-
> gdb/testsuite/gdb.fortran/vla-type.exp | 22 +-
> gdb/testsuite/gdb.fortran/vla-value.exp | 8 +-
> gdb/testsuite/gdb.mi/mi-vla-fortran.exp | 4 +-
> 28 files changed, 778 insertions(+), 99 deletions(-)
> create mode 100644 gdb/fortran-operator.def
> delete mode 100644 gdb/testsuite/gdb.fortran/complex.f
> create mode 100644 gdb/testsuite/gdb.fortran/complex.f90
> create mode 100644 gdb/testsuite/gdb.fortran/ptype-on-functions.exp
> create mode 100644 gdb/testsuite/gdb.fortran/ptype-on-functions.f90
>
> --
> 2.14.5
>