Hi,
the Fortran FE sets flag_errno_math and flag_associative_math
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=94447
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=159620
Seemingly it (now?) needs to communicate that it is doing so, since otherwise
this is overwritten later on
Dominque,
after Janne's OK, and FX judgement, I wonder if you have reached a conclusion.
If so, the fsf assignment is now complete, and the patch could be applied.
Joost
> Ok for trunk. Thanks!
Please don't rush! The behavior of -fno-signed-zeros -fno-trapping-math
implying associative math h
Hi Joost,
Why do you want -fno-math-errno to be the default for gfortran?
AFAICT such default is not documented and the flag works
(checked on your test gfortran.dg/errnocheck_1.f90).
For -fassociative-math, the manual says
For Fortran the option is automatically enabled when both -fno-signed-ze
> Why do you want -fno-math-errno to be the default for gfortran?
because it was with rth's commit? This makes sense also because errno is a
variable that is defined for C, and not accessible from Fortran. Why would you
want -fmath-errno to be the default ?
> if (flag_associative_math == -1)
>
> Why do you want -fno-math-errno to be the default for gfortran?
> AFAICT such default is not documented and the flag works
> (checked on your test gfortran.dg/errnocheck_1.f90).
It should be the default, and it used to be. Fortran doesn’t care about math
routines setting errno (its existence is
I have now verified that both new testcases indeed pass with
gcc version 4.6.0 20100520 (experimental) [trunk revision 159620] (GCC)
that is the revision where Tobias enabled associative math by default. So that
this patch fixes a regression.
On Thu, Jun 5, 2014 at 3:54 PM, VandeVondele Joost
wrote:
> I have now verified that both new testcases indeed pass with
>
> gcc version 4.6.0 20100520 (experimental) [trunk revision 159620] (GCC)
>
> that is the revision where Tobias enabled associative math by default. So
> that this patch fix
> Ok for trunk. Thanks!
Please don't rush! The behavior of -fno-signed-zeros -fno-trapping-math
implying associative math has been changed (as in reverted) between r165758
(implied associative math) and r165930 (lost associative math). AFAICT
it could be due to 165823. Investigating! I am also loo
ping ?
https://gcc.gnu.org/ml/fortran/2014-05/msg00162.html
On Mon, Aug 18, 2014 at 6:48 PM, VandeVondele Joost
wrote:
> ping ?
>
> https://gcc.gnu.org/ml/fortran/2014-05/msg00162.html
Ok, again. If Dominique comes up with some reason why the patch should
be modified, it can be done then. AFAICT your patch is better than the
status quo.
--
Janne Blomq
Thanks, can somebody with svn write access commit ?
On Tue, Aug 19, 2014 at 1:07 PM, VandeVondele Joost
wrote:
> Thanks, can somebody with svn write access commit ?
Please get yourself write-after-approval access per instructions at
https://gcc.gnu.org/svnwrite.html . You can use me as your sponsor.
Thanks.
--
Janne Blomqvist
Committed revision 214211.
13 matches
Mail list logo