From d684a9bb4bcc3952649a347133b86c87a45c8930 Mon Sep 17 00:00:00 2001 From: Jerry DeLisle Date: Wed, 23 Aug 2017 00:41:10 +0000 Subject: [PATCH] backport: re PR fortran/81296 (derived type I/o problem) 2017-08-22 Jerry DeLisle Backport from trunk PR fortran/81296 * trans-io.c (get_dtio_proc): Add check for format label and set formatted flag accordingly. Reorganize the code a little. * gfortran.dg/dtio_12.f90: Update test. From-SVN: r251301 --- gcc/fortran/ChangeLog | 7 +++++++ gcc/fortran/trans-io.c | 26 ++++++++++++++++---------- gcc/testsuite/ChangeLog | 6 ++++++ gcc/testsuite/gfortran.dg/dtio_12.f90 | 6 ++++++ 4 files changed, 35 insertions(+), 10 deletions(-) diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index c70444d1f9e..63064268970 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,10 @@ +2017-08-22 Jerry DeLisle + + Backport from trunk + PR fortran/81296 + * trans-io.c (get_dtio_proc): Add check for format label and set + formatted flag accordingly. Reorganize the code a little. + 2017-08-18 Jerry DeLisle Backport from trunk diff --git a/gcc/fortran/trans-io.c b/gcc/fortran/trans-io.c index c3c56f29623..aa974eb3805 100644 --- a/gcc/fortran/trans-io.c +++ b/gcc/fortran/trans-io.c @@ -2214,18 +2214,24 @@ get_dtio_proc (gfc_typespec * ts, gfc_code * code, gfc_symbol **dtio_sub) bool formatted = false; gfc_dt *dt = code->ext.dt; - if (dt && dt->format_expr) + if (dt) { - char *fmt; - fmt = gfc_widechar_to_char (dt->format_expr->value.character.string, - -1); - if (strtok (fmt, "DT") != NULL) + char *fmt = NULL; + + if (dt->format_label == &format_asterisk) + { + /* List directed io must call the formatted DTIO procedure. */ + formatted = true; + } + else if (dt->format_expr) + fmt = gfc_widechar_to_char (dt->format_expr->value.character.string, + -1); + else if (dt->format_label) + fmt = gfc_widechar_to_char (dt->format_label->format->value.character.string, + -1); + if (fmt && strtok (fmt, "DT") != NULL) formatted = true; - } - else if (dt && dt->format_label == &format_asterisk) - { - /* List directed io must call the formatted DTIO procedure. */ - formatted = true; + } if (ts->type == BT_CLASS) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 63deb9177e2..e483873b3ca 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2017-08-22 Jerry DeLisle + + Backport from trunk + PR fortran/81296 + * gfortran.dg/dtio_12.f90: Update test. + 2017-08-22 Peter Bergner Backport from mainline diff --git a/gcc/testsuite/gfortran.dg/dtio_12.f90 b/gcc/testsuite/gfortran.dg/dtio_12.f90 index 213f7ebbb1e..cf1bfe38e2f 100644 --- a/gcc/testsuite/gfortran.dg/dtio_12.f90 +++ b/gcc/testsuite/gfortran.dg/dtio_12.f90 @@ -67,6 +67,12 @@ end module if (trim (msg) .ne. "42") call abort rewind (10) write (10,"(DT)") child (77) ! The original testcase + rewind (10) + read (10, *) msg + if (trim (msg) .ne. "77") call abort + rewind (10) + write (10,40) child (77) ! Modified using format label +40 format(DT) rewind (10) read (10, *) msg if (trim (msg) .ne. "77") call abort