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: Add clog10 to builtins.def


On Tue, 28 Jun 2005, [ISO-8859-1] François-Xavier Coudert wrote:

> Hi all,
> 
> while trying to clean and add support for various complex math
> functions in the fortran front-end, I noticed that there is not
> mention of clog10 and friends (clog10f and clog10l) in builtins.def.
> That does look like a bug to me, and could be fixed by this patch.

These are reserved functions (7.26.1), not functions defined by C99.  So 
they shouldn't be DEF_C99_BUILTIN; DEF_EXT_C99RES_BUILTIN, similar to 
DEF_C99_C90RES_BUILTIN for names reserved in C90 and added in C99, would 
be more appropriate.

I suppose you're only adding clog10* not all 27 functions (cerf, cerfc, 
cexp2, cexpm1, clog10, clog1p, clog2, clgamma, ctgamma and float and long 
double versions) mentioned in 7.26.1 because glibc only has the clog10 
functions and you don't know of any systems which have implementations of 
the others which would benefit from the built-in functions?

-- 
Joseph S. Myers               http://www.srcf.ucam.org/~jsm28/gcc/
    jsm@polyomino.org.uk (personal mail)
    joseph@codesourcery.com (CodeSourcery mail)
    jsm28@gcc.gnu.org (Bugzilla assignments and CCs)

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