Hi Steve,
Sorry about the delay. Daytime stuff caught up with me.
While I was about it, I committed the fix for PR89841 with the fix for
PR89842. The latter is even safer than the former.
Committed as revision 270037.
Thanks
Paul
2019-03-30 Paul Thomas <pa...@gcc.gnu.org>
PR fortran/89841
* trans-expr.c (gfc_conv_gfc_desc_to_cfi_desc): Use the formal
argument attributes rather than those of the actual argument.
PR fortran/89842
* trans-expr.c (gfc_conv_gfc_desc_to_cfi_desc): Call
'set_dtype_for_unallocated' for any type of arrayspec.
2019-03-30 Paul Thomas <pa...@gcc.gnu.org>
PR fortran/89841
* gfortran.dg/ISO_Fortran_binding_1.f90: Change the interfaces
for c_deallocate, c_allocate and c_assumed_size so that the
attributes of the array arguments are correct and are typed.
* gfortran.dg/ISO_Fortran_binding_7.f90: New test.
* gfortran.dg/ISO_Fortran_binding_7.c: Additional source.
PR fortran/89842
* gfortran.dg/ISO_Fortran_binding_8.f90: New test.
* gfortran.dg/ISO_Fortran_binding_8.c: Additional source.
On Wed, 27 Mar 2019 at 18:55, Steve Kargl
<s...@troutmask.apl.washington.edu> wrote:
>
> On Wed, Mar 27, 2019 at 06:50:41PM +0000, Paul Richard Thomas wrote:
> > This corrects a screw-up on my part. The attribute field of the CFI
> > descriptor must be set by the formal argument in the interface and not
> > the actual argument.
> >
> > Most of the work was in correcting
> >
> > Bootstrapped and regtested on FC29/x86_64 - OK for trunk?
> >
>
> OK.
>
> --
> Steve
--
"If you can't explain it simply, you don't understand it well enough"
- Albert Einstein
