Re: [Patch, fortran] PR87151 - allocating array of character

2018-10-08 Thread Thomas Koenig
Hi Paul, Bootstraps and regtests on FC28/x86_64 - OK for trunk and later for 8-branch? OK. Thanks for the patch! Regards Thomas

Re: [Patch, fortran] PR87151 - allocating array of character

2018-10-08 Thread Paul Richard Thomas
Hi Dominique, Thanks for the testing. Although the tests for PR80931 and PR83196, comment #4, compiled OK, when I attempted to use the modules both segfaulted for the same reason ('span' not being set on the array descriptor) and these required a slightly different version of the same tweak. The

Re: [Patch, fortran] PR87151 - allocating array of character

2018-10-08 Thread Dominique d'Humières
Hi Paul, Your patch works as expected. It also fixes the ICEs for the tests in pr80931 (and the test accidentally attached to pr83196). Thanks for the patch. Dominique

[Patch, fortran] PR87151 - allocating array of character

2018-10-07 Thread Paul Richard Thomas
This turned out to be rather more than the allocation... The ChangeLog and the patch tell the story well enough. Bootstraps and regtests on FC28/x86_64 - OK for trunk and later for 8-branch? Cheers Paul 2018-10-07 Paul Thomas PR fortran/87151 * trans-array.c (gfc_get_array_span):