[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-23 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

Paul Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Paul Thomas  ---
Fixed on trunk.

Paul

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-23 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

--- Comment #6 from Paul Thomas  ---
Author: pault
Date: Thu Jan 24 07:21:36 2019
New Revision: 268232

URL: https://gcc.gnu.org/viewcvs?rev=268232&root=gcc&view=rev
Log:
2019-01-24  Paul Thomas  

PR fortran/88929
* trans-array.c (gfc_conv_descriptor_elem_len): New function.
* trans-array.h : Add prototype for above.
* trans-expr.c (gfc_conv_gfc_desc_to_cfi_desc): Take account of
assumed rank arrays being flagged by rank = -1 in expressions.
Intent in arrays need a pointer to a copy of the data to be
assigned to the descriptor passed for conversion. This should
then be freed, together with the CFI descriptor on return from
the C call.

2019-01-24  Paul Thomas  

PR fortran/88929
* gfortran.dg/ISO_Fortran_binding_3.f90 : New test
* gfortran.dg/ISO_Fortran_binding_3.c : Subsidiary source.

Modified:
trunk/gcc/testsuite/gfortran.dg/ISO_Fortran_binding_3.f90

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-23 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

--- Comment #5 from Paul Thomas  ---
Author: pault
Date: Thu Jan 24 07:19:49 2019
New Revision: 268231

URL: https://gcc.gnu.org/viewcvs?rev=268231&root=gcc&view=rev
Log:
2019-01-24  Paul Thomas  

PR fortran/88929
* trans-array.c (gfc_conv_descriptor_elem_len): New function.
* trans-array.h : Add prototype for above.
* trans-expr.c (gfc_conv_gfc_desc_to_cfi_desc): Take account of
assumed rank arrays being flagged by rank = -1 in expressions.
Intent in arrays need a pointer to a copy of the data to be
assigned to the descriptor passed for conversion. This should
then be freed, together with the CFI descriptor on return from
the C call.

2019-01-24  Paul Thomas  

PR fortran/88929
* gfortran.dg/ISO_Fortran_binding_3.f90 : New test
* gfortran.dg/ISO_Fortran_binding_3.c : Subsidiary source.

Added:
trunk/gcc/testsuite/gfortran.dg/ISO_Fortran_binding_3.c
trunk/gcc/testsuite/gfortran.dg/ISO_Fortran_binding_3.f90
Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/trans-array.c
trunk/gcc/fortran/trans-array.h
trunk/gcc/fortran/trans-expr.c
trunk/gcc/testsuite/ChangeLog

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-20 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

--- Comment #4 from Dominique d'Humieres  ---
There is a partially reduced test in pr88908 (TODO reduce the modules).

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-20 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

Dominique d'Humieres  changed:

   What|Removed |Added

 CC||marxin at gcc dot gnu.org

--- Comment #3 from Dominique d'Humieres  ---
*** Bug 88908 has been marked as a duplicate of this bug. ***

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-20 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

Dominique d'Humieres  changed:

   What|Removed |Added

 CC||pault at gcc dot gnu.org

--- Comment #2 from Dominique d'Humieres  ---
This looks like a duplicate of pr88908.

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-20 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

--- Comment #1 from Paul Thomas  ---
Created attachment 45469
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45469&action=edit
A proposed patch for the PR.

Handling of fortran dummy arguments was not implemented in the recent
ISO_Fortran_binding patch. The attached rectifies that omission.

Paul

[Bug fortran/88929] ICE on building MPICH 3.2 with GCC 9 with ISO_Fortran_binding

2019-01-20 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88929

Paul Thomas  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2019-01-20
 CC||damian at sourceryinstitute 
dot or
   ||g
 Ever confirmed|0   |1