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


> 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.

OK. However, I don't know how DEF_EXT_C99RES_BUILTIN should be
#defined. It should certainly be similar to

#define DEF_C99_C90RES_BUILTIN(ENUM, NAME, TYPE, ATTRS) \ 
  DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE,    \
               true, true, !flag_isoc99, ATTRS, TARGET_C99_FUNCTIONS, true)

but I can only guess what to do with most if the arguments to
DEF_BUILTIN. Can someone tell me what the correct definition should
be?

> 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 [...]

No, only because clog10* are the one needed for the fortran front-end.

FX


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