public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
@ 2004-02-13 9:23 John at xza5 dot com
2004-02-13 9:24 ` [Bug c++/14138] " John at xza5 dot com
` (10 more replies)
0 siblings, 11 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:23 UTC (permalink / raw)
To: gcc-bugs
We've got some generated C++ which compiles in about 10s (on a P4 2800) at -O0
with gcc-3.2 or 3.3.2, but takes about 1000s with 3.4.0 or 3.5.0.
-ftime-report says that the parser time has gone from 8.04s (gcc-3.3.2) to
958.19s (gcc-3.4.0)
I'll attach the .ii shortly.
--
Summary: 100x -O0 C++ Compile Time Regression {3.2,3.3} ->
{3.4,3.5}
Product: gcc
Version: 3.4.0
Status: UNCONFIRMED
Severity: normal
Priority: P2
Component: c++
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: John at xza5 dot com
CC: gcc-bugs at gcc dot gnu dot org
GCC build triplet: i686-pc-linux-gnu
GCC host triplet: i686-pc-linux-gnu
GCC target triplet: i686-pc-linux-gnu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
@ 2004-02-13 9:24 ` John at xza5 dot com
2004-02-13 9:27 ` John at xza5 dot com
` (9 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:24 UTC (permalink / raw)
To: gcc-bugs
--
What |Removed |Added
----------------------------------------------------------------------------
Known to fail| |3.4.0 3.5.0
Known to work| |3.2 3.3.2
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
2004-02-13 9:24 ` [Bug c++/14138] " John at xza5 dot com
@ 2004-02-13 9:27 ` John at xza5 dot com
2004-02-13 9:28 ` John at xza5 dot com
` (8 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:27 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From John at xza5 dot com 2004-02-13 09:27 -------
Created an attachment (id=5742)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5742&action=view)
A .ii illustrating the compile time regression
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
2004-02-13 9:24 ` [Bug c++/14138] " John at xza5 dot com
2004-02-13 9:27 ` John at xza5 dot com
@ 2004-02-13 9:28 ` John at xza5 dot com
2004-02-13 9:29 ` John at xza5 dot com
` (7 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:28 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From John at xza5 dot com 2004-02-13 09:28 -------
Created an attachment (id=5743)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5743&action=view)
time report + mem report for 3.4.0
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (2 preceding siblings ...)
2004-02-13 9:28 ` John at xza5 dot com
@ 2004-02-13 9:29 ` John at xza5 dot com
2004-02-13 9:30 ` John at xza5 dot com
` (6 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:29 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From John at xza5 dot com 2004-02-13 09:29 -------
Created an attachment (id=5744)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5744&action=view)
Time report + mem report for 3.3.2
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (3 preceding siblings ...)
2004-02-13 9:29 ` John at xza5 dot com
@ 2004-02-13 9:30 ` John at xza5 dot com
2004-02-13 9:32 ` John at xza5 dot com
` (5 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:30 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From John at xza5 dot com 2004-02-13 09:30 -------
Created an attachment (id=5745)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5745&action=view)
Time report + mem report for 3.5.0
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (4 preceding siblings ...)
2004-02-13 9:30 ` John at xza5 dot com
@ 2004-02-13 9:32 ` John at xza5 dot com
2004-02-13 16:05 ` [Bug c++/14138] [3.4/3.5 Regression] " pinskia at gcc dot gnu dot org
` (4 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: John at xza5 dot com @ 2004-02-13 9:32 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From John at xza5 dot com 2004-02-13 09:32 -------
Created an attachment (id=5746)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5746&action=view)
A .ii.bz2 generated with 3.3.2
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] [3.4/3.5 Regression] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (5 preceding siblings ...)
2004-02-13 9:32 ` John at xza5 dot com
@ 2004-02-13 16:05 ` pinskia at gcc dot gnu dot org
2004-03-01 4:56 ` mmitchel at gcc dot gnu dot org
` (3 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2004-02-13 16:05 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From pinskia at gcc dot gnu dot org 2004-02-13 16:05 -------
The problem is not really in the parser (-ftime-report is wrong but that is another bug).
The real problem is in name lookup and not using the sorted fields so it is doing a linear search almost
all the time (at least early on).
--
What |Removed |Added
----------------------------------------------------------------------------
CC| |pinskia at gcc dot gnu dot
| |org
Severity|normal |critical
Status|UNCONFIRMED |NEW
Ever Confirmed| |1
Keywords| |compile-time-hog
Last reconfirmed|0000-00-00 00:00:00 |2004-02-13 16:05:14
date| |
Summary|100x -O0 C++ Compile Time |[3.4/3.5 Regression] 100x -
|Regression {3.2,3.3} -> |O0 C++ Compile Time
|{3.4,3.5} |Regression {3.2,3.3} ->
| |{3.4,3.5}
Target Milestone|--- |3.4.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] [3.4/3.5 Regression] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (6 preceding siblings ...)
2004-02-13 16:05 ` [Bug c++/14138] [3.4/3.5 Regression] " pinskia at gcc dot gnu dot org
@ 2004-03-01 4:56 ` mmitchel at gcc dot gnu dot org
2004-03-01 6:21 ` cvs-commit at gcc dot gnu dot org
` (2 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2004-03-01 4:56 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From mmitchel at gcc dot gnu dot org 2004-03-01 04:55 -------
Working on a fix.
--
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|unassigned at gcc dot gnu |mark at codesourcery dot com
|dot org |
Status|NEW |ASSIGNED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] [3.4/3.5 Regression] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (7 preceding siblings ...)
2004-03-01 4:56 ` mmitchel at gcc dot gnu dot org
@ 2004-03-01 6:21 ` cvs-commit at gcc dot gnu dot org
2004-03-01 6:23 ` cvs-commit at gcc dot gnu dot org
2004-03-01 6:28 ` mmitchel at gcc dot gnu dot org
10 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-03-01 6:21 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-03-01 06:21 -------
Subject: Bug 14138
CVSROOT: /cvs/gcc
Module name: gcc
Branch: gcc-3_4-branch
Changes by: mmitchel@gcc.gnu.org 2004-03-01 06:21:40
Modified files:
gcc/cp : ChangeLog decl.c decl2.c name-lookup.c
name-lookup.h parser.c pt.c
Log message:
PR c++/14138
* name-lookup.h (push_scope): Change prototype.
* name-lookup.c (push_scope): Do not reenter the current class
scope.
* decl.c (grokfndecl): Check return code from push_scope before
calling pop_scope.
* decl2.c (check_classfn): Likewise.
* parser.c (cp_parser_conversion_function_id): Likewise.
(cp_parser_init_declarator): Likewise.
(cp_parser_direct_declarator): Likewise.
(cp_parser_class_specifier): Likewise.
(cp_parser_class_head): Likewise.
(cp_parser_lookup_name): Likewise.
(cp_parser_constructor_declarator_p): Likewise.
* pt.c (instantiate_class_template): Likewise.
(resolve_typename_type): Likewise.
Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3892.2.63&r2=1.3892.2.64
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/decl.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.1174.2.9&r2=1.1174.2.10
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/decl2.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.695.4.3&r2=1.695.4.4
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/name-lookup.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.34.2.4&r2=1.34.2.5
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/name-lookup.h.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.15.2.2&r2=1.15.2.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.157.2.15&r2=1.157.2.16
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.816.2.13&r2=1.816.2.14
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] [3.4/3.5 Regression] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (8 preceding siblings ...)
2004-03-01 6:21 ` cvs-commit at gcc dot gnu dot org
@ 2004-03-01 6:23 ` cvs-commit at gcc dot gnu dot org
2004-03-01 6:28 ` mmitchel at gcc dot gnu dot org
10 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-03-01 6:23 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-03-01 06:23 -------
Subject: Bug 14138
CVSROOT: /cvs/gcc
Module name: gcc
Changes by: mmitchel@gcc.gnu.org 2004-03-01 06:23:39
Modified files:
gcc/cp : ChangeLog decl.c decl2.c name-lookup.c
name-lookup.h parser.c pt.c
Log message:
PR c++/14138
* name-lookup.h (push_scope): Change prototype.
* name-lookup.c (push_scope): Do not reenter the current class
scope.
* decl.c (grokfndecl): Check return code from push_scope before
calling pop_scope.
* decl2.c (check_classfn): Likewise.
* parser.c (cp_parser_conversion_function_id): Likewise.
(cp_parser_init_declarator): Likewise.
(cp_parser_direct_declarator): Likewise.
(cp_parser_class_specifier): Likewise.
(cp_parser_class_head): Likewise.
(cp_parser_lookup_name): Likewise.
(cp_parser_constructor_declarator_p): Likewise.
* pt.c (instantiate_class_template): Likewise.
(resolve_typename_type): Likewise.
Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&r1=1.3977&r2=1.3978
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/decl.c.diff?cvsroot=gcc&r1=1.1189&r2=1.1190
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/decl2.c.diff?cvsroot=gcc&r1=1.699&r2=1.700
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/name-lookup.c.diff?cvsroot=gcc&r1=1.41&r2=1.42
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/name-lookup.h.diff?cvsroot=gcc&r1=1.17&r2=1.18
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&r1=1.176&r2=1.177
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&r1=1.832&r2=1.833
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug c++/14138] [3.4/3.5 Regression] 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5}
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
` (9 preceding siblings ...)
2004-03-01 6:23 ` cvs-commit at gcc dot gnu dot org
@ 2004-03-01 6:28 ` mmitchel at gcc dot gnu dot org
10 siblings, 0 replies; 12+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2004-03-01 6:28 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From mmitchel at gcc dot gnu dot org 2004-03-01 06:28 -------
Fixed in GCC 3.4 and GCC 3.5.
--
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14138
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2004-03-01 6:28 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-02-13 9:23 [Bug c++/14138] New: 100x -O0 C++ Compile Time Regression {3.2,3.3} -> {3.4,3.5} John at xza5 dot com
2004-02-13 9:24 ` [Bug c++/14138] " John at xza5 dot com
2004-02-13 9:27 ` John at xza5 dot com
2004-02-13 9:28 ` John at xza5 dot com
2004-02-13 9:29 ` John at xza5 dot com
2004-02-13 9:30 ` John at xza5 dot com
2004-02-13 9:32 ` John at xza5 dot com
2004-02-13 16:05 ` [Bug c++/14138] [3.4/3.5 Regression] " pinskia at gcc dot gnu dot org
2004-03-01 4:56 ` mmitchel at gcc dot gnu dot org
2004-03-01 6:21 ` cvs-commit at gcc dot gnu dot org
2004-03-01 6:23 ` cvs-commit at gcc dot gnu dot org
2004-03-01 6:28 ` mmitchel at gcc dot gnu dot org
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).