This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: Ping: [Patch, fortran] Bug 68241 - [meta-bug] Deferred-length character - PRs50221, 68216, 63932, 66408, 67674 and 49954
- From: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Damian Rouson <damian at sourceryinstitute dot org>, Louis Krupp <louis dot krupp at zoho dot com>, Dominique Dhumieres <dominiq at lps dot ens dot fr>
- Date: Sat, 14 Nov 2015 09:49:43 -0800
- Subject: Re: Ping: [Patch, fortran] Bug 68241 - [meta-bug] Deferred-length character - PRs50221, 68216, 63932, 66408, 67674 and 49954
- Authentication-results: sourceware.org; auth=none
- References: <CAGkQGiJ0ZptE31WH9bPaJJoO1n5Fg7d7Wj2oozHjcCYcJtqunw at mail dot gmail dot com>
On Sat, Nov 14, 2015 at 06:39:28PM +0100, Paul Richard Thomas wrote:
>
> I am completely unable to reproduce the problems that Dominique is
> reporting for deferred_character_4.f90. This might be because the
> patch has moved on to fix PR49554 :-)
>
> Concatenation expressions assigned to deferred length character arrays
> need careful handling to ensure that the temporary creation for the
> concatenation operator occurs at the right place, that the descriptor
> dtype is updated and an array temporary is created if there is any
> dependency between lhs and rhs. This latter has been implemented in
> resolve.c.
>
> Testcases 4-6 have been added to reflect the additional fixes afforded
> by the original patch, as reported by Dominique (thanks!).
>
> As soon as this patch has been committed, I will prepare a version for
> 4.9 and 5 branches
>
> Bootstrapped and regtested on FC21/x86_64 - OK for trunk?
>
Hi Paul,
I was going to cast an eye over your diff today. I'll
build and run some tests on FreeBSD. Dominiq uses
MacOS. So, perhaps, some latent memory corruption
issue.
--
steve