public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point
@ 2010-11-09 21:30 zsojka at seznam dot cz
2010-11-10 12:08 ` [Bug c++/46401] " rguenth at gcc dot gnu.org
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: zsojka at seznam dot cz @ 2010-11-09 21:30 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
Summary: [4.5/4.6 Regression] very slow compile time with
-Wsequence-point
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
AssignedTo: unassigned@gcc.gnu.org
ReportedBy: zsojka@seznam.cz
Created attachment 22356
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22356
reduced testcase
Compiler output - 4.4.5:
$ time gcc-4.4.5 -Wsequence-point -c testcase.C
real 0m0.514s
user 0m0.470s
sys 0m0.030s
Compiler output - 4.5.1: (disabled checking)
$ time gcc-4.5.1 -Wsequence-point -c testcase.C
real 4m22.328s
user 4m21.230s
sys 0m0.250s
Compiler output - r166492: (enabled checking)
$ time gcc -Wsequence-point -c testcase.C
real 6m2.734s
user 6m2.090s
sys 0m0.300s
Compiler output - r166492 without -Wsequence-point:
$ time gcc -c testcase.C
real 0m0.699s
user 0m0.520s
sys 0m0.050s
The time needed to analyze sequence points grew a lot in 4.5 for attached
testcase. The problem is more likely to appear since -Wsequence-point is
enabled by -Wall. The reduced testcase is from 'cantor', a KDE application.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5/4.6 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
@ 2010-11-10 12:08 ` rguenth at gcc dot gnu.org
2010-11-15 16:27 ` jakub at gcc dot gnu.org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-10 12:08 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.5.2
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5/4.6 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
2010-11-10 12:08 ` [Bug c++/46401] " rguenth at gcc dot gnu.org
@ 2010-11-15 16:27 ` jakub at gcc dot gnu.org
2010-11-16 21:11 ` jakub at gcc dot gnu.org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2010-11-15 16:27 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|unassigned at gcc dot |jakub at gcc dot gnu.org
|gnu.org |
--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-15 16:21:11 UTC ---
Created attachment 22401
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22401
gcc46-pr46401.patch
Untested fix.
No point tracking CALL_EXPRs that aren't ECF_CONST (because then
operand_equal_p fails anyway) nor STRING_CSTs (which can't be written into).
That said, -Wsequence-point algorithm is still quadratic (certainly in used
memory, and at least quadratic in compilation time).
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5/4.6 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
2010-11-10 12:08 ` [Bug c++/46401] " rguenth at gcc dot gnu.org
2010-11-15 16:27 ` jakub at gcc dot gnu.org
@ 2010-11-16 21:11 ` jakub at gcc dot gnu.org
2010-11-16 21:31 ` [Bug c++/46401] [4.5 " jakub at gcc dot gnu.org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2010-11-16 21:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-16 20:50:04 UTC ---
Author: jakub
Date: Tue Nov 16 20:49:57 2010
New Revision: 166823
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166823
Log:
PR c++/46401
* c-common.c (warning_candidate_p): Don't track non-const calls
or STRING_CSTs.
* g++.dg/warn/Wsequence-point-3.C: New test.
Added:
trunk/gcc/testsuite/g++.dg/warn/Wsequence-point-3.C
Modified:
trunk/gcc/c-family/ChangeLog
trunk/gcc/c-family/c-common.c
trunk/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
` (2 preceding siblings ...)
2010-11-16 21:11 ` jakub at gcc dot gnu.org
@ 2010-11-16 21:31 ` jakub at gcc dot gnu.org
2010-11-25 16:02 ` rguenth at gcc dot gnu.org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2010-11-16 21:31 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2010.11.16 21:28:46
Known to work| |4.6.0
Summary|[4.5/4.6 Regression] very |[4.5 Regression] very slow
|slow compile time with |compile time with
|-Wsequence-point |-Wsequence-point
Ever Confirmed|0 |1
Known to fail|4.6.0 |
--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-16 21:28:46 UTC ---
Fixed for 4.6 so far.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
` (3 preceding siblings ...)
2010-11-16 21:31 ` [Bug c++/46401] [4.5 " jakub at gcc dot gnu.org
@ 2010-11-25 16:02 ` rguenth at gcc dot gnu.org
2010-12-07 15:06 ` jakub at gcc dot gnu.org
2010-12-07 15:40 ` jakub at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-25 16:02 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P2
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
` (4 preceding siblings ...)
2010-11-25 16:02 ` rguenth at gcc dot gnu.org
@ 2010-12-07 15:06 ` jakub at gcc dot gnu.org
2010-12-07 15:40 ` jakub at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2010-12-07 15:06 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-07 15:06:31 UTC ---
Author: jakub
Date: Tue Dec 7 15:06:27 2010
New Revision: 167538
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167538
Log:
Backport from mainline
2010-11-16 Jakub Jelinek <jakub@redhat.com>
PR c++/46401
* c-common.c (warning_candidate_p): Don't track non-const calls
or STRING_CSTs.
* g++.dg/warn/Wsequence-point-3.C: New test.
Added:
branches/gcc-4_5-branch/gcc/testsuite/g++.dg/warn/Wsequence-point-3.C
Modified:
branches/gcc-4_5-branch/gcc/ChangeLog
branches/gcc-4_5-branch/gcc/c-common.c
branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/46401] [4.5 Regression] very slow compile time with -Wsequence-point
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
` (5 preceding siblings ...)
2010-12-07 15:06 ` jakub at gcc dot gnu.org
@ 2010-12-07 15:40 ` jakub at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2010-12-07 15:40 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46401
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-07 15:40:42 UTC ---
Fixed.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2010-12-07 15:40 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-09 21:30 [Bug c++/46401] New: [4.5/4.6 Regression] very slow compile time with -Wsequence-point zsojka at seznam dot cz
2010-11-10 12:08 ` [Bug c++/46401] " rguenth at gcc dot gnu.org
2010-11-15 16:27 ` jakub at gcc dot gnu.org
2010-11-16 21:11 ` jakub at gcc dot gnu.org
2010-11-16 21:31 ` [Bug c++/46401] [4.5 " jakub at gcc dot gnu.org
2010-11-25 16:02 ` rguenth at gcc dot gnu.org
2010-12-07 15:06 ` jakub at gcc dot gnu.org
2010-12-07 15:40 ` jakub at gcc dot gnu.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).