public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Harald Anlauf <anlauf@gmx.de>
To: gcc-patches@gcc.gnu.org
Cc: fortran@gcc.gnu.org
Subject: Re: [Patch, fortran] PR37336 finalization
Date: Sat, 3 Jun 2023 21:10:22 +0200	[thread overview]
Message-ID: <c0cd3cd7-e4b1-4fe8-a272-e49cfe0a9510@gmx.de> (raw)
Message-ID: <20230603191022.aKP1-cJ5S6JPgH-lUPM5OWYMOSSkJECnjtVGJoNbQn4@z> (raw)
In-Reply-To: <CAGkQGi+uKg5kaonRhTZau5G159W7UFHn4frXsj0D06gxNWXFPQ@mail.gmail.com>

Hi Paul, all,

On 6/3/23 15:16, Paul Richard Thomas via Gcc-patches wrote:
> Hi Thomas,
> 
> I want to get something approaching correct finalization to the
> distros, which implies 12-branch at present. Hopefully I can do the
> same with associate in a month or two's time.

IMHO it is not only distros, but also installations at (scientific)
computing centers with a larger user base and a large software stack.
Migrating to a different major version of gcc/gfortran is not a trivial
task for them.

I'd fully support the idea of backporting the finalization fixes, as
IIUC this on the one hand touches a rather isolated part, and on the
other hand already got quite some testing.  It is also already in the
13-branch (or only mostly?).  Given that 12.3 was released recently
and 12.4 is far away, there'd be sufficient time to fix any fallout.

Regarding the associate fixes, we could get as much of those into 13.2,
which we'd normally expect in just a few months.  As long as spare time
to work on gfortran is limited, I'd rather prefer to get as much fixed
for that release.

(This is not a no: I simply expect that real regression testing for the
associate changes may take more time.)

> I am dithering about changing the F2003/08 part of finalization since
> the default is 2018 compliance. That said, it does need a change since
> the suppression of constructor finalization is also suppressing
> finalization of function results within the compilers. I'll do that
> first, perhaps?

That sounds like a good idea.

Cheers,
Harald

> Cheers
> 
> Paul
> 
> 
> 
> On Sat, 3 Jun 2023 at 06:50, Thomas Koenig <tkoenig@netcologne.de> wrote:
>>
>> Hi Paul,
>>
>>> I propose to backport
>>> r13-6747-gd7caf313525a46f200d7f5db1ba893f853774aee to 12-branch very
>>> soon.
>>
>> Is this something that we usually do?
>>
>> While finalization was basically broken before, some people still used
>> working subsets (or subsets that were broken, and they adapted or
>> wrote their code accordingly).
>>
>> What is the general opinion on that?  I'm undecided.
>>
>>> Before that, I propose to remove the F2003/2008 finalization of
>>> structure and array constructors in 13- and 14-branches. I can see why
>>> it was removed from the standard in a correction to F2008 and think
>>> that it is likely to cause endless confusion and maintenance
>>> complications. However, finalization of function results within
>>> constructors will be retained.
>>
>> That, I agree with.  Should it be noted somewhere as an intentional
>> deviation from the standard?
>>
>> Best regards
>>
>>          Thomas
>>
> 
> 
> --
> "If you can't explain it simply, you don't understand it well enough"
> - Albert Einstein
> 



  reply	other threads:[~2023-06-03 19:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-07 13:45 Paul Richard Thomas
2023-03-07 14:58 ` Thomas Koenig
2023-03-07 17:15   ` Steve Kargl
2023-06-02 13:42 ` Paul Richard Thomas
2023-06-03  5:50   ` Thomas Koenig
2023-06-03  7:32     ` Steve Kargl
2023-06-03 13:16     ` Paul Richard Thomas
2023-06-03 19:10       ` Harald Anlauf [this message]
2023-06-03 19:10         ` Harald Anlauf
2023-06-03 19:22       ` Thomas Koenig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c0cd3cd7-e4b1-4fe8-a272-e49cfe0a9510@gmx.de \
    --to=anlauf@gmx.de \
    --cc=fortran@gcc.gnu.org \
    --cc=gcc-patches@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).