public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
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

  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).