Hi Christophe, 2017-04-24 10:25 GMT+02:00 Christophe Lyon <christophe.l...@linaro.org>: >>>> the patch in the attachment fixes a memory leak by auto-deallocating >>>> the allocatable components of an allocatable intent(out) argument. >>>> >>>> Regtests cleanly on x86_64-linux-gnu. Ok for trunk? >>> >>> OK for trunk. >> >> thanks for the review! Committed as r247083. > > This patch causes an error message from DejaGnu: > (DejaGnu) proc "cleanup-tree-dump original" does not exist.
thanks for letting me know. I didn't notice that ... > I'm not familiar with fortran, so I'm not sure it is as obvious as > removing cleanup-tree-dump as it is done in the other neighboring tests? Yes, probably it should just be removed. I assume this kind of cleanup is being done automatically now? I actually took it from this wiki page: https://gcc.gnu.org/wiki/TestCaseWriting So I guess this needs to be updated as well. Will take care of both points tonight ... Cheers, Janus
