This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag

From: Janus Weil <janus at gcc dot gnu dot org>
To: Rainer Orth <ro at cebitec dot uni-bielefeld dot de>
Cc: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>, David Edelsohn <dje dot gcc at gmail dot com>, Janne Blomqvist <blomqvist dot janne at gmail dot com>, Fortran List <fortran at gcc dot gnu dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Tue, 26 Sep 2017 11:47:00 +0200
Subject: Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
Authentication-results: sourceware.org; auth=none
References: <CAGWvnykd97Ly0UpvX+NUF7QGh9EHt6_kg2d-N==-0_ZAkicQeQ@mail.gmail.com> <CAKwh3qjwQDNTpsDUZE_SFKkdQuC9EBk_UbkE52172+7Ea1UQ0g@mail.gmail.com> <20170925212335.GA44284@troutmask.apl.washington.edu> <CAKwh3qi2N4LCiMzv1obi5t53YiFn6eABecNFHWYfsWkRp=YE+Q@mail.gmail.com> <CAKwh3qhkWVV-DzyN1JQPhse=Q9ZZSQaDReGoHi5=drgLMLBY+Q@mail.gmail.com> <yddlgl1x1r4.fsf@CeBiTec.Uni-Bielefeld.DE>

Hi Rainer,

>> Attached is a more complete patch, which should fix all problems that
>> were reported concerning these two test cases. Would be great if
>> someone could confirm that it works on a failing target (I currently
>> only have access to x86_64-linux-gnu machines).
>
> I've just checked sparc-sun-solaris2.11: works fine.  promotion_3.f90
> PASSes as before, but promotion_4.f90 is now UNSUPPORTED instead of
> failing.

thanks for checking!


>> Ok for trunk?
>
> The new fortran_real_10 effective-target keyword needs documenting in
> sourcebuild.texi.

Good point. fortran_real_16 was missing there as well. Added both (new
patch attached).

I'll commit this tonight, unless there are further comments ...

Cheers,
Janus

Attachment: pr82324_v2.diff
Description: Text document

References:
- Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
  - From: David Edelsohn
- Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
  - From: Janus Weil
- Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
  - From: Steve Kargl
- Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
  - From: Janus Weil
- Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
  - From: Janus Weil
- Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag
  - From: Rainer Orth

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]