* Add clog10 to builtins.def, round 2
@ 2005-06-29 21:51 François-Xavier Coudert
2005-07-03 6:53 ` FX Coudert
0 siblings, 1 reply; 5+ messages in thread
From: François-Xavier Coudert @ 2005-06-29 21:51 UTC (permalink / raw)
To: gcc, patch
[-- Attachment #1: Type: text/plain, Size: 272 bytes --]
The fortran front-end needs to recognize clog10, clog10f and clog10l a
proper built-ins. Attached patch tries to add them to clog10, under a
new category: DEF_EXT_C99RES_BUILTIN (as suggested by jsm28).
Can someone review this? Is it OK?
Thanks,
François-Xavier
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: clog10.diff --]
[-- Type: text/x-patch; name="clog10.diff", Size: 2087 bytes --]
Index: gcc/builtins.def
===================================================================
RCS file: /cvsroot/gcc/gcc/gcc/builtins.def,v
retrieving revision 1.104
diff -u -3 -p -r1.104 builtins.def
--- gcc/builtins.def 25 Jun 2005 01:59:13 -0000 1.104
+++ gcc/builtins.def 29 Jun 2005 21:25:38 -0000
@@ -119,6 +119,13 @@ Software Foundation, 51 Franklin Street,
DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE, \
true, true, !flag_isoc99, ATTRS, TARGET_C99_FUNCTIONS, true)
+/* Builtin that C99 reserve the name for future use. We can still recognize
+ the builtin in C99 mode but we can't produce it implicitly. */
+#undef DEF_EXT_C99RES_BUILTIN
+#define DEF_EXT_C99RES_BUILTIN(ENUM, NAME, TYPE, ATTRS) \
+ DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE, \
+ true, true, true, ATTRS, false, true)
+
/* Allocate the enum and the name for a builtin, but do not actually
define it here at all. */
#undef DEF_BUILTIN_STUB
@@ -436,6 +443,9 @@ DEF_C99_BUILTIN (BUILT_IN_CIMAGL,
DEF_C99_BUILTIN (BUILT_IN_CLOG, "clog", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CLOGF, "clogf", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CLOGL, "clogl", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10, "clog10", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10F, "clog10f", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10L, "clog10l", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CONJ, "conj", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_CONST_NOTHROW_LIST)
DEF_C99_BUILTIN (BUILT_IN_CONJF, "conjf", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_CONST_NOTHROW_LIST)
DEF_C99_BUILTIN (BUILT_IN_CONJL, "conjl", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_CONST_NOTHROW_LIST)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Add clog10 to builtins.def, round 2
2005-06-29 21:51 Add clog10 to builtins.def, round 2 François-Xavier Coudert
@ 2005-07-03 6:53 ` FX Coudert
2005-07-19 8:46 ` FX Coudert
0 siblings, 1 reply; 5+ messages in thread
From: FX Coudert @ 2005-07-03 6:53 UTC (permalink / raw)
To: François-Xavier Coudert; +Cc: gcc, patch
[-- Attachment #1: Type: text/plain, Size: 323 bytes --]
> The fortran front-end needs to recognize clog10, clog10f and clog10l a
> proper built-ins. Attached patch tries to add them to clog10, under a
> new category: DEF_EXT_C99RES_BUILTIN (as suggested by jsm28).
>
> Can someone review this? Is it OK?
Just realized I forgot the ChangeLog entry to go with it.
OK to commit?
[-- Attachment #2: allkinds2_middle-end.ChangeLog --]
[-- Type: text/plain, Size: 208 bytes --]
2005-06-28 Francois-Xavier Coudert <coudert@clipper.ens.fr>
* builtins.def: Add DEF_EXT_C99RES_BUILTIN to define builtins
that C99 reserve for future use. Use it to define clog10, clog10f
and clog10l.
[-- Attachment #3: allkinds2_middle-end.diff --]
[-- Type: text/plain, Size: 2093 bytes --]
Index: gcc/builtins.def
===================================================================
RCS file: /cvsroot/gcc/gcc/gcc/builtins.def,v
retrieving revision 1.105
diff -u -3 -p -r1.105 builtins.def
--- gcc/builtins.def 27 Jun 2005 12:17:18 -0000 1.105
+++ gcc/builtins.def 3 Jul 2005 02:44:06 -0000
@@ -119,6 +119,13 @@ Software Foundation, 51 Franklin Street,
DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE, \
true, true, !flag_isoc99, ATTRS, TARGET_C99_FUNCTIONS, true)
+/* Builtin that C99 reserve the name for future use. We can still recognize
+ the builtin in C99 mode but we can't produce it implicitly. */
+#undef DEF_EXT_C99RES_BUILTIN
+#define DEF_EXT_C99RES_BUILTIN(ENUM, NAME, TYPE, ATTRS) \
+ DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE, \
+ true, true, true, ATTRS, false, true)
+
/* Allocate the enum and the name for a builtin, but do not actually
define it here at all. */
#undef DEF_BUILTIN_STUB
@@ -436,6 +443,9 @@ DEF_C99_BUILTIN (BUILT_IN_CIMAGL,
DEF_C99_BUILTIN (BUILT_IN_CLOG, "clog", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CLOGF, "clogf", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CLOGL, "clogl", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10, "clog10", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10F, "clog10f", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10L, "clog10l", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CONJ, "conj", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_CONST_NOTHROW_LIST)
DEF_C99_BUILTIN (BUILT_IN_CONJF, "conjf", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_CONST_NOTHROW_LIST)
DEF_C99_BUILTIN (BUILT_IN_CONJL, "conjl", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_CONST_NOTHROW_LIST)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Add clog10 to builtins.def, round 2
2005-07-03 6:53 ` FX Coudert
@ 2005-07-19 8:46 ` FX Coudert
2005-07-21 20:06 ` Richard Henderson
0 siblings, 1 reply; 5+ messages in thread
From: FX Coudert @ 2005-07-19 8:46 UTC (permalink / raw)
To: FX Coudert; +Cc: gcc, patch, gfortran
[-- Attachment #1: Type: text/plain, Size: 171 bytes --]
> OK to commit?
Ping.
This problem is really blocking a very nice gfortran feature (support
for large real kinds). Since it is very short, could someone review it?
FX
[-- Attachment #2: allkinds2_middle-end.ChangeLog --]
[-- Type: text/plain, Size: 208 bytes --]
2005-07-19 Francois-Xavier Coudert <coudert@clipper.ens.fr>
* builtins.def: Add DEF_EXT_C99RES_BUILTIN to define builtins
that C99 reserve for future use. Use it to define clog10, clog10f
and clog10l.
[-- Attachment #3: allkinds2_middle-end.diff --]
[-- Type: text/plain, Size: 2093 bytes --]
Index: gcc/builtins.def
===================================================================
RCS file: /cvsroot/gcc/gcc/gcc/builtins.def,v
retrieving revision 1.105
diff -u -3 -p -r1.105 builtins.def
--- gcc/builtins.def 27 Jun 2005 12:17:18 -0000 1.105
+++ gcc/builtins.def 3 Jul 2005 02:44:06 -0000
@@ -119,6 +119,13 @@ Software Foundation, 51 Franklin Street,
DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE, \
true, true, !flag_isoc99, ATTRS, TARGET_C99_FUNCTIONS, true)
+/* Builtin that C99 reserve the name for future use. We can still recognize
+ the builtin in C99 mode but we can't produce it implicitly. */
+#undef DEF_EXT_C99RES_BUILTIN
+#define DEF_EXT_C99RES_BUILTIN(ENUM, NAME, TYPE, ATTRS) \
+ DEF_BUILTIN (ENUM, "__builtin_" NAME, BUILT_IN_NORMAL, TYPE, TYPE, \
+ true, true, true, ATTRS, false, true)
+
/* Allocate the enum and the name for a builtin, but do not actually
define it here at all. */
#undef DEF_BUILTIN_STUB
@@ -436,6 +443,9 @@ DEF_C99_BUILTIN (BUILT_IN_CIMAGL,
DEF_C99_BUILTIN (BUILT_IN_CLOG, "clog", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CLOGF, "clogf", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CLOGL, "clogl", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10, "clog10", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10F, "clog10f", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_MATHFN_FPROUNDING)
+DEF_EXT_C99RES_BUILTIN (BUILT_IN_CLOG10L, "clog10l", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_MATHFN_FPROUNDING)
DEF_C99_BUILTIN (BUILT_IN_CONJ, "conj", BT_FN_COMPLEX_DOUBLE_COMPLEX_DOUBLE, ATTR_CONST_NOTHROW_LIST)
DEF_C99_BUILTIN (BUILT_IN_CONJF, "conjf", BT_FN_COMPLEX_FLOAT_COMPLEX_FLOAT, ATTR_CONST_NOTHROW_LIST)
DEF_C99_BUILTIN (BUILT_IN_CONJL, "conjl", BT_FN_COMPLEX_LONGDOUBLE_COMPLEX_LONGDOUBLE, ATTR_CONST_NOTHROW_LIST)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Add clog10 to builtins.def, round 2
2005-07-19 8:46 ` FX Coudert
@ 2005-07-21 20:06 ` Richard Henderson
2005-07-24 8:36 ` François-Xavier Coudert
0 siblings, 1 reply; 5+ messages in thread
From: Richard Henderson @ 2005-07-21 20:06 UTC (permalink / raw)
To: FX Coudert; +Cc: gcc, patch, gfortran
On Tue, Jul 19, 2005 at 10:46:34AM +0200, FX Coudert wrote:
> * builtins.def: Add DEF_EXT_C99RES_BUILTIN to define builtins
> that C99 reserve for future use. Use it to define clog10, clog10f
> and clog10l.
Ok.
r~
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Add clog10 to builtins.def, round 2
2005-07-21 20:06 ` Richard Henderson
@ 2005-07-24 8:36 ` François-Xavier Coudert
0 siblings, 0 replies; 5+ messages in thread
From: François-Xavier Coudert @ 2005-07-24 8:36 UTC (permalink / raw)
To: Richard Henderson, FX Coudert, gcc, patch, gfortran
>> * builtins.def: Add DEF_EXT_C99RES_BUILTIN to define builtins
>> that C99 reserve for future use. Use it to define clog10, clog10f
>> and clog10l.
>
> Ok.
Commited.
Thanks,
FX
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2005-07-24 8:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-06-29 21:51 Add clog10 to builtins.def, round 2 François-Xavier Coudert
2005-07-03 6:53 ` FX Coudert
2005-07-19 8:46 ` FX Coudert
2005-07-21 20:06 ` Richard Henderson
2005-07-24 8:36 ` François-Xavier Coudert
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).