public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jerry DeLisle <jvdelisle@frontier.com>
To: Tobias Burnus <tobias.burnus@physik.fu-berlin.de>
Cc: gcc-patches@gcc.gnu.org, fortran@gcc.gnu.org
Subject: Re: [testsuite] Compile gfortran.dg/fmt_g0_6.f08 with -ffloat-store [fwd: tobias.burnus@physik.fu-berlin.de]
Date: Tue, 03 May 2011 11:50:00 -0000	[thread overview]
Message-ID: <4DBFEBE6.1060201@frontier.com> (raw)
In-Reply-To: <20110503101407.GA29709@physik.fu-berlin.de>

On 05/03/2011 03:14 AM, Tobias Burnus wrote:
> Rainer,
>
> Rainer Orth wrote:
>> Jerry DeLisle wrote:
>>> This is OK.  We are working on a solution, so maybe it would be better to
>>> XFAIL it so that when it does get fixed it will be flagged. It also fails
>>> on i686-pc-gnu.
>>
>> is this a solution for this particular case or for the general need to
>> sometimes use -ffloat-store?  In the former case, I'd agree, in the
>> latter it's probably better to consistently use -ffloat-store in the
>> testcases and review those uses once a solution is in place.
>
> I think the issue is PR 48602 - wrong I/O output due to wrong rounding
> and algorithm bugs. The PR has been mostly fixed, but the excess
> precision of the x87 processor still causes failures. One solution
> might be to compile the whole libgfortran/io/ with
> -fexcess-precision=standard. Adding a single
> "asm volatile ("" : "+m" (temp))" proved to be insufficient.
>
> Tobias
>
> PS: Especially as you are listed as testsuite maintainer, I would be
> happy if you could comment on the testsuite patch at
> http://gcc.gnu.org/ml/fortran/2011-04/msg00331.html
>

My goal is to completely not use any floating point operations for that section 
of code.  We have a concept on how to do it.  I am still working out tweaks to 
48787, then I will turn my attention to this. Feel free to quiet that test case 
any way you wish in the meantime.

Jerry

  reply	other threads:[~2011-05-03 11:50 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-03 10:14 Tobias Burnus
2011-05-03 11:50 ` Jerry DeLisle [this message]
2011-05-05 14:02   ` Rainer Orth
2011-05-03 15:28 ` Mike Stump

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=4DBFEBE6.1060201@frontier.com \
    --to=jvdelisle@frontier.com \
    --cc=fortran@gcc.gnu.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=tobias.burnus@physik.fu-berlin.de \
    /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).