public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/21743] New: Enable __builtin_clog
@ 2005-05-24 22:04 pcarlini at suse dot de
2005-05-24 22:14 ` [Bug middle-end/21743] " pinskia at gcc dot gnu dot org
` (10 more replies)
0 siblings, 11 replies; 12+ messages in thread
From: pcarlini at suse dot de @ 2005-05-24 22:04 UTC (permalink / raw)
To: gcc-bugs
This builtin, that would be *so* useful in the implementation of std::complex
cannot be enabled at the moment due to this issue (see also builtins.def):
http://gcc.gnu.org/ml/gcc-patches/2003-09/msg00510.html
We should *really* find a way to work around the problem!
(not sure whether the proper component is middle-end or libstdc++-v3 ;)
--
Summary: Enable __builtin_clog
Product: gcc
Version: 4.0.0
Status: UNCONFIRMED
Severity: enhancement
Priority: P2
Component: middle-end
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: pcarlini at suse dot de
CC: gcc-bugs at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
@ 2005-05-24 22:14 ` pinskia at gcc dot gnu dot org
2005-05-24 22:20 ` pcarlini at suse dot de
` (9 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-05-24 22:14 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pinskia at gcc dot gnu dot org 2005-05-24 22:07 -------
We could just define the __builtin_ version of the function.
I keep on wondering if (and when) C++ gets the C99 complex functions and types, what would they do
about complex log.
Confirmed.
--
What |Removed |Added
----------------------------------------------------------------------------
CC| |pinskia at gcc dot gnu dot
| |org
Status|UNCONFIRMED |NEW
Ever Confirmed| |1
Last reconfirmed|0000-00-00 00:00:00 |2005-05-24 22:07:21
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
2005-05-24 22:14 ` [Bug middle-end/21743] " pinskia at gcc dot gnu dot org
@ 2005-05-24 22:20 ` pcarlini at suse dot de
2005-05-24 22:22 ` pinskia at gcc dot gnu dot org
` (8 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pcarlini at suse dot de @ 2005-05-24 22:20 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pcarlini at suse dot de 2005-05-24 22:14 -------
> We could just define the __builtin_ version of the function.
I don't understand, can you explain? In my understanding everything is in place,
indeed all the other builtins are there, *only* we cannot do that for clog
because ends up conflicting with iostream's clog. That's the whole point: it
conflicts. If you don't believe that, just enable it and run the testsuite:
it conflicts as soon as you do 'using std::clog;' in the global namespace.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
2005-05-24 22:14 ` [Bug middle-end/21743] " pinskia at gcc dot gnu dot org
2005-05-24 22:20 ` pcarlini at suse dot de
@ 2005-05-24 22:22 ` pinskia at gcc dot gnu dot org
2005-05-24 23:02 ` pcarlini at suse dot de
` (7 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-05-24 22:22 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pinskia at gcc dot gnu dot org 2005-05-24 22:20 -------
If you uncomment what is in builtins.def, you get both clog and __builtin_clog.
If we define it as DEF_LIB_BUILTIN instead of DEF_C99_BUILTIN, we only get __builtin_clog.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (2 preceding siblings ...)
2005-05-24 22:22 ` pinskia at gcc dot gnu dot org
@ 2005-05-24 23:02 ` pcarlini at suse dot de
2005-05-26 18:15 ` pcarlini at suse dot de
` (6 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pcarlini at suse dot de @ 2005-05-24 23:02 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pcarlini at suse dot de 2005-05-24 22:46 -------
Ah, ah, cool, thanks! I'm going to propose that!
--
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|unassigned at gcc dot gnu |pcarlini at suse dot de
|dot org |
Status|NEW |ASSIGNED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (3 preceding siblings ...)
2005-05-24 23:02 ` pcarlini at suse dot de
@ 2005-05-26 18:15 ` pcarlini at suse dot de
2005-05-27 14:48 ` pcarlini at suse dot de
` (5 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pcarlini at suse dot de @ 2005-05-26 18:15 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pcarlini at suse dot de 2005-05-26 17:59 -------
A fix along the way suggested by Andrew doesn't seem implementable in a very clean
way, because no DEF_*_BUILTIN category does exactly what we want: 1- only
__builtin_* variant (BOTH_P == false); 2- fall back to library function call
(FALLBACK_P == true). I'm still looking a bit into it...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (4 preceding siblings ...)
2005-05-26 18:15 ` pcarlini at suse dot de
@ 2005-05-27 14:48 ` pcarlini at suse dot de
2005-05-30 16:34 ` mmitchel at gcc dot gnu dot org
` (4 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pcarlini at suse dot de @ 2005-05-27 14:48 UTC (permalink / raw)
To: gcc-bugs
--
What |Removed |Added
----------------------------------------------------------------------------
BugsThisDependsOn| |21784
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (5 preceding siblings ...)
2005-05-27 14:48 ` pcarlini at suse dot de
@ 2005-05-30 16:34 ` mmitchel at gcc dot gnu dot org
2005-05-30 20:17 ` cvs-commit at gcc dot gnu dot org
` (3 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2005-05-30 16:34 UTC (permalink / raw)
To: gcc-bugs
--
Bug 21743 depends on bug 21784, which changed state.
Bug 21784 Summary: [3.4/4.0/4.1 Regression] Using vs builtin names
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21784
What |Old Value |New Value
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Status|NEW |ASSIGNED
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (6 preceding siblings ...)
2005-05-30 16:34 ` mmitchel at gcc dot gnu dot org
@ 2005-05-30 20:17 ` cvs-commit at gcc dot gnu dot org
2005-05-30 20:19 ` cvs-commit at gcc dot gnu dot org
` (2 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2005-05-30 20:17 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-05-30 20:15 -------
Subject: Bug 21743
CVSROOT: /cvs/gcc
Module name: gcc
Changes by: paolo@gcc.gnu.org 2005-05-30 20:15:48
Modified files:
gcc : ChangeLog builtins.def
gcc/doc : extend.texi
Log message:
2005-05-30 Paolo Carlini <pcarlini@suse.de>
PR middle-end/21743
* builtins.def (BUILT_IN_CLOG, BUILT_IN_CLOGF, BUILT_IN_CLOGL):
Enable.
* doc/extend.texi: Add clog, clogf, clogl.
Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.8948&r2=2.8949
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/builtins.def.diff?cvsroot=gcc&r1=1.102&r2=1.103
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/doc/extend.texi.diff?cvsroot=gcc&r1=1.253&r2=1.254
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (7 preceding siblings ...)
2005-05-30 20:17 ` cvs-commit at gcc dot gnu dot org
@ 2005-05-30 20:19 ` cvs-commit at gcc dot gnu dot org
2005-05-30 20:55 ` pcarlini at suse dot de
2005-05-30 23:19 ` giovannibajo at libero dot it
10 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2005-05-30 20:19 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-05-30 20:17 -------
Subject: Bug 21743
CVSROOT: /cvs/gcc
Module name: gcc
Changes by: paolo@gcc.gnu.org 2005-05-30 20:16:59
Modified files:
gcc/testsuite : ChangeLog
gcc/testsuite/gcc.dg: builtins-1.c
gcc/testsuite/gcc.dg/torture: builtin-attr-1.c
Log message:
2005-05-30 Paolo Carlini <pcarlini@suse.de>
PR middle-end/21743
* gcc.dg/builtins-1.c: Activate disabled clog test.
* gcc.dg/torture/builtin-attr-1.c: Likewise.
Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.5561&r2=1.5562
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/builtins-1.c.diff?cvsroot=gcc&r1=1.10&r2=1.11
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/torture/builtin-attr-1.c.diff?cvsroot=gcc&r1=1.7&r2=1.8
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (8 preceding siblings ...)
2005-05-30 20:19 ` cvs-commit at gcc dot gnu dot org
@ 2005-05-30 20:55 ` pcarlini at suse dot de
2005-05-30 23:19 ` giovannibajo at libero dot it
10 siblings, 0 replies; 12+ messages in thread
From: pcarlini at suse dot de @ 2005-05-30 20:55 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pcarlini at suse dot de 2005-05-30 20:19 -------
Fixed for 4.1.0
--
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
Target Milestone|--- |4.1.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug middle-end/21743] Enable __builtin_clog
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
` (9 preceding siblings ...)
2005-05-30 20:55 ` pcarlini at suse dot de
@ 2005-05-30 23:19 ` giovannibajo at libero dot it
10 siblings, 0 replies; 12+ messages in thread
From: giovannibajo at libero dot it @ 2005-05-30 23:19 UTC (permalink / raw)
To: gcc-bugs
--
Bug 21743 depends on bug 21784, which changed state.
Bug 21784 Summary: [3.4/4.0/4.1 Regression] Using vs builtin names
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21784
What |Old Value |New Value
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21743
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2005-05-30 22:46 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-05-24 22:04 [Bug middle-end/21743] New: Enable __builtin_clog pcarlini at suse dot de
2005-05-24 22:14 ` [Bug middle-end/21743] " pinskia at gcc dot gnu dot org
2005-05-24 22:20 ` pcarlini at suse dot de
2005-05-24 22:22 ` pinskia at gcc dot gnu dot org
2005-05-24 23:02 ` pcarlini at suse dot de
2005-05-26 18:15 ` pcarlini at suse dot de
2005-05-27 14:48 ` pcarlini at suse dot de
2005-05-30 16:34 ` mmitchel at gcc dot gnu dot org
2005-05-30 20:17 ` cvs-commit at gcc dot gnu dot org
2005-05-30 20:19 ` cvs-commit at gcc dot gnu dot org
2005-05-30 20:55 ` pcarlini at suse dot de
2005-05-30 23:19 ` giovannibajo at libero dot it
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).