This is the mail archive of the
mailing list for the GCC project.
Re: [FORTRAN PATCH] Use more mathematical builtins
- From: "Toon Moene" <moene at knmi dot nl>
- To: <roger at eyesopen dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 12 May 2003 08:06:40 +0000
- Subject: Re: [FORTRAN PATCH] Use more mathematical builtins
- Organization: Royal Netherlands Meteorological Institute
> The following patch is a tweak to the g77 front-end to make more
> use of GCC's built-in mathematical functions. Previously g77 used
> GCC builtins for cos, sin and sqrt. With this patch, it does the
> same for atan2, exp, floor, fmod, log and pow. This enables
> FORTRAN to take advantage of the middle-end's constant folding
> transformations and (when available) the inline intrinsic
> implementations provided by the target's backend.
> The g77 testsuite showed up a bug in duplicate_decls when calling
> __builtin_fmod, which is always converted straight to a standard C
> library call. The code to call COPY_DECL_ASSEMBLER_NAME is copied
> directly from the C front-end's duplicate_decls in c-decl.c. This
> is another example of a change to duplicate_decls since the code
> was originally duplicated into the FORTRAN front-end.
Yes, sigh :-(
As far as I can see you also fixed another bug (r_atn2) - BRRR, probably
nobody ever uses that ...
> The following patch has been tested on i686-pc-linux-gnu, by a
> full bootstrap configured with --enable-languages=c,f77 and a
> top-level "make -k check" with no new regressions. I've also
> confirmed by hand that "FOO = EXP(0.0)" no longer calls the C
> library but instead assigns 1.0 to FOO at run-time.
> Ok for mainline?
Sure, thanks very much !
Toon Moene, KNMI, PO Box 201, 3730 AE De Bilt, The Netherlands.
Tel. +31302206443, Fax +31302210407, e-mail email@example.com
Zie ook/see also: http://www.knmi.nl/maildisclaimer.html