Re: [Patch, Fortran, F03] PR 80983: memory leak when calling procedure-pointer component with allocatable result

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [Patch, Fortran, F03] PR 80983: memory leak when calling procedure-pointer component with allocatable result

Janus Weil-3
2017-06-15 15:33 GMT+02:00 Thomas Koenig <[hidden email]>:
> Hi Janus,
>
>> It regtests cleanly on x86_64-linux-gnu. Ok for trunk?
>
>
> OK.
>
> Thanks for the patch!

Thanks, Thomas! Committed as r249227.

Cheers,
Janus
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [Patch, Fortran, F03] PR 80983: memory leak when calling procedure-pointer component with allocatable result

Janus Weil-3
2017-06-16 11:11 GMT+02:00 Christophe Lyon <[hidden email]>:

> Hi,
>
> On 15 June 2017 at 23:18, Janus Weil <[hidden email]> wrote:
>> 2017-06-15 15:33 GMT+02:00 Thomas Koenig <[hidden email]>:
>>> Hi Janus,
>>>
>>>> It regtests cleanly on x86_64-linux-gnu. Ok for trunk?
>>>
>>>
>>> OK.
>>>
>>> Thanks for the patch!
>>
>> Thanks, Thomas! Committed as r249227.
>>
>
> There's a problem with the new test. It says:
> gfortran.dg/proc_ptr_comp_51.f90   -O  : dump file does not exist
> UNRESOLVED: gfortran.dg/proc_ptr_comp_51.f90   -O
> scan-tree-dump-times original "__builtin_free" 3
>
> You probably either want to add:
> ! { dg-options "-fdump-tree-original" }
> or to remove:
> ! { dg-final { scan-tree-dump-times "__builtin_free" 3 "original" } }

Thanks for your attention, Christophe. I wanted the first of these two
options, but apparently forgot that line ... :(

Fixed with r249243.

Cheers,
Janus
Loading...