From: Jerry DeLisle <jvdelisle@charter.net>
To: Lukasz Kolodziejczyk <lukasz.kolodziejczyk@codethink.com>
Cc: Jakub Jelinek <jakub@redhat.com>, Jeff Law <law@redhat.com>,
Janne Blomqvist <blomqvist.janne@gmail.com>,
Fortran List <fortran@gcc.gnu.org>,
fritzoreese@gmail.com
Subject: Re: Desire to allocate bit in DT_PARM bitmask for DEC FORMAT compatibility purposes
Date: Fri, 23 Mar 2018 14:47:00 -0000 [thread overview]
Message-ID: <8b1c3989-d1a1-bbe9-5ba8-28ff846153f5@charter.net> (raw)
In-Reply-To: <c257a5ea-6a26-72ad-4465-5f4b21e2d901@codethink.com>
On 03/23/2018 06:26 AM, Lukasz Kolodziejczyk wrote:
> On 23/03/18 08:12, Jakub Jelinek wrote:
>> On Thu, Mar 22, 2018 at 07:07:11PM -0700, Jerry DeLisle wrote:
>>>> But in the tables I referenced and in the new code we want 15, 7, 2
>>>> respectively.
>>> The widths we use now were picked specifically to assure that values
>>> written
>>> and then read back in result in the same values.
>>>
>>> I have some philosophical questions.
>>>
>>> 1) Why do you want 15, 7, 2? Why is this so critical?
>> Because that is what all those compilers document and agree on.
>> See e.g.
>> https://software.intel.com/en-us/node/678750#32937290-265D-4805-B2B7-4E78F6AAD0D8
>>
>> for Intel documentation, I'm sure Jeff has links to other documentations.
>
> In addition to the Intel and Oracle compiler documentation linked by
> Jakub, the Compaq Visual Fortran documentation is in support of 15, 7, 2:
> http://jp.xlsoft.com/documents/intel/cvf/cvf_lref.pdf ("Default Widths
> for Data Edit Descriptors", page 305)
>
>>> 2) If it is critical, why are you trying to make gfortran do non
>>> standard
>>> stuff? Why is it so difficult to write a python script to translate
>>> any and
>>> all legacy DEC code to standard conforming format specifiers? The only
>>> thing that would not be straight forward is if the format strings are
>>> being
>>> computed on the fly. Not the best practice, but nothing surprises me
>>> with
>>> these legacy codes.
>> Changing this stuff isn't that easy with python scripts, because for that
>> you need to understand the kinds of the arguments. If the format
>> string is
>> visible to compiler and so are the kinds of the arguments, sure, the
>> compiler could change it at compile time, passing a different format
>> string
>> to the runtime and the runtime wouldn't have to worry. If the format
>> string
>> is built at runtime, or even compile time, but the compiler can't see it
>> (say the format string in one TU and used in another TU), then that is
>> something that can only be handled in the runtime.
>
> While we agree that modernising Fortran is the appropriate long-term
> goal, in the short term it might not be feasible for users of gfortran.
> Therefore we believe that implementing the change in this way is the
> most accessible approach for those that require this feature in the
> immediate future.
>
> We have tested Jeff's patch on trunk and can confirm it works as
> expected with no regressions.
>
>
> Lukasz
>
>
OK, this makes sense to me. From where I sit, I have a limited view of
these use cases.
Jerry
next prev parent reply other threads:[~2018-03-23 14:47 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-20 18:41 Jeff Law
2018-03-21 18:35 ` Jakub Jelinek
2018-03-21 17:29 ` Jeff Law
2018-03-21 18:38 ` Janne Blomqvist
2018-03-21 19:26 ` Jeff Law
2018-03-22 5:49 ` Jeff Law
2018-03-23 2:07 ` Jerry DeLisle
2018-03-23 9:02 ` Janne Blomqvist
2018-03-23 9:08 ` Jakub Jelinek
2018-03-23 14:22 ` Jerry DeLisle
2018-03-23 9:35 ` Jakub Jelinek
2018-03-23 8:39 ` Jakub Jelinek
2018-03-23 14:46 ` Jerry DeLisle
2018-03-23 19:12 ` Jeff Law
2018-03-23 9:35 ` Jakub Jelinek
2018-03-23 13:26 ` Lukasz Kolodziejczyk
2018-03-23 14:47 ` Jerry DeLisle [this message]
2018-03-23 19:39 ` Jeff Law
2018-03-23 21:36 ` Steve Kargl
2018-03-23 19:58 ` Jeff Law
2018-03-23 21:32 ` Fritz Reese
2018-03-24 9:35 ` Jakub Jelinek
2018-03-24 16:04 ` Steve Kargl
2018-03-24 17:35 ` Jakub Jelinek
2018-03-24 17:11 ` Steve Kargl
2018-03-24 18:20 ` Jerry DeLisle
2018-03-25 18:48 ` Jeff Law
2018-03-25 19:09 ` Jerry DeLisle
2018-03-26 3:41 ` Jeff Law
2018-03-26 3:44 ` Steve Kargl
2018-03-26 7:42 ` Jakub Jelinek
2018-03-26 12:14 ` Fritz Reese
2018-03-26 19:15 ` Steve Kargl
2018-03-26 21:04 ` Jeff Law
2018-03-26 21:03 ` Jeff Law
2018-03-27 9:14 ` Jakub Jelinek
2018-03-28 1:02 ` Jerry DeLisle
2018-03-23 19:56 ` Jeff Law
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=8b1c3989-d1a1-bbe9-5ba8-28ff846153f5@charter.net \
--to=jvdelisle@charter.net \
--cc=blomqvist.janne@gmail.com \
--cc=fortran@gcc.gnu.org \
--cc=fritzoreese@gmail.com \
--cc=jakub@redhat.com \
--cc=law@redhat.com \
--cc=lukasz.kolodziejczyk@codethink.com \
/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).