Re: [patch, fortran] PR77828 Linking gfortran-7 compiled program with libgfortran of 5.x allowed but crashes

[Janne Blomqvist <blomqvist dot janne at gmail dot com>]

On Mon, Oct 24, 2016 at 2:53 PM, Jerry DeLisle <jvdelisle@charter.net> wrote:
> On 10/18/2016 04:48 PM, Jerry DeLisle wrote:
>>
>> Hi Folks,
>>
>> The attached patch does some minor cleanup and bumps the libgfortran
>> version
>> number.  I have wanted to reorder the dtp structure for many years now.
>> Not
>> strictly needed but it has bugged me forever.
>>
>> The bump is needed because of the significant changes from implementation
>> of DTIO.
>>
>
> --- snip ---
>
> Revised patch. Changed a single byte char to int for better memory alignment
> on 32 bit boundaries. Regression tested on x86-64-linux, FreeBSD, and
> Power8.
>
> I plan to commit this to trunk sometime today.

I understood from https://gcc.gnu.org/ml/fortran/2016-10/msg00168.html
that fortran-dev won't be in shape for merging until GCC 8 stage 1. If
so, shouldn't we avoid breaking the ABI until then?

>
>
>>
>> Regression tested on x86-64-linux.
>>
>> OK for trunk?
>>
>> Regards,
>>
>> Jerry
>>
>> 2016-10-18  Jerry DeLisle  <jvdelisle@gcc.gnu.org>
>>
>>     PR fortran/77828
>>     * io/io.h (st_parameter_dt): Reorder for readability and sanity.
>>     * io/transfer.c (data_transfer_init): Remove TODO and enable the
>>     runtime error message, rec= specifier not allowed in STREAM
>>     access.
>>     * libtool-version: Bump major version of libgfortran to 4.
>>
>> 2016-10-18  Jerry DeLisle  <jvdelisle@gcc.gnu.org>
>>
>>     PR fortran/77828
>>     * ioparm.def: Reorder dt parameters to match libgfortran.
>>
>>
>>
>



-- 
Janne Blomqvist