* [Bug middle-end/54114] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
@ 2012-07-28 20:50 ` jimis at gmx dot net
2012-07-30 9:11 ` [Bug middle-end/54114] [4.8 Regression] " rguenth at gcc dot gnu.org
` (15 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jimis at gmx dot net @ 2012-07-28 20:50 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
--- Comment #1 from jimis <jimis at gmx dot net> 2012-07-28 20:49:55 UTC ---
Sorry guys my machine died an ugly death, so the web interface for my profiling
runs is unfortunately offline. :-( From memory I remember that this regression
is more than 0.5s overhead out of ~5s. For more details I'll have to repair my
equipment.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug middle-end/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
2012-07-28 20:50 ` [Bug middle-end/54114] " jimis at gmx dot net
@ 2012-07-30 9:11 ` rguenth at gcc dot gnu.org
2012-07-30 12:20 ` jimis at gmx dot net
` (14 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-07-30 9:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |WAITING
Last reconfirmed| |2012-07-30
Version|unknown |4.8.0
Target Milestone|--- |4.8.0
Summary|variable-tracking |[4.8 Regression]
|performance regression from |variable-tracking
|4.8-20120610 to |performance regression from
|4.8-20120701 |4.8-20120610 to
| |4.8-20120701
Ever Confirmed|0 |1
--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-07-30 09:11:00 UTC ---
Please attach preprocessed source.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug middle-end/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
2012-07-28 20:50 ` [Bug middle-end/54114] " jimis at gmx dot net
2012-07-30 9:11 ` [Bug middle-end/54114] [4.8 Regression] " rguenth at gcc dot gnu.org
@ 2012-07-30 12:20 ` jimis at gmx dot net
2012-08-07 20:28 ` aoliva at gcc dot gnu.org
` (13 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jimis at gmx dot net @ 2012-07-30 12:20 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
--- Comment #3 from jimis <jimis at gmx dot net> 2012-07-30 12:18:49 UTC ---
Created attachment 27894
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27894
XPatch.cpp preprocessed source from xalanc
Hi thanks for your patience, I resurrected my PC so now I have access to my
files, and the webinterface is up for anyone caring to see the function
breakdown (just check footnote [6] in front page). Attached is the preprocessed
source, I compile with
$CC1PLUS -quiet -D_GNU_SOURCE -D LINUX -D _REENTRANT -D XALAN_INMEM_MSG_LOADER
XPath.ii -g -O2 -fno-elide-constructors -fPIC -o /dev/null
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug middle-end/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (2 preceding siblings ...)
2012-07-30 12:20 ` jimis at gmx dot net
@ 2012-08-07 20:28 ` aoliva at gcc dot gnu.org
2012-09-19 14:05 ` rguenth at gcc dot gnu.org
` (12 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: aoliva at gcc dot gnu.org @ 2012-08-07 20:28 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
--- Comment #4 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2012-08-07 20:28:05 UTC ---
It seems quite clear to me that the new cost arises due to
clobber_overlapping_mems, a relatively expensive operation that fixes a debug
info correctness bug and that we just didn't do before. I'm afraid I can't
think of a way to do it in a way that is significantly more efficient, so we'll
have to choose between this additional cost, or living with debug locations
that are no longer valid due to overlapping stores :-(
I'm tempted to mark this as WONTFIX, but I thought I'd leave this open for a
bit in case anyone else wants to give it a try.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug middle-end/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (3 preceding siblings ...)
2012-08-07 20:28 ` aoliva at gcc dot gnu.org
@ 2012-09-19 14:05 ` rguenth at gcc dot gnu.org
2012-12-18 11:01 ` jakub at gcc dot gnu.org
` (11 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-09-19 14:05 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P1
Status|WAITING |NEW
--- Comment #5 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-09-19 14:04:51 UTC ---
Confirmed.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug middle-end/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (4 preceding siblings ...)
2012-09-19 14:05 ` rguenth at gcc dot gnu.org
@ 2012-12-18 11:01 ` jakub at gcc dot gnu.org
2013-01-18 11:01 ` aoliva at gcc dot gnu.org
` (10 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-12-18 11:01 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
CC| |jakub at gcc dot gnu.org
Resolution| |WONTFIX
--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-12-18 11:01:24 UTC ---
variable tracking : 0.08 ( 1%) usr 0.01 ( 3%) sys 0.11 ( 1%) wall
3165 kB ( 3%) ggc
var-tracking dataflow : 0.36 ( 4%) usr 0.00 ( 0%) sys 0.38 ( 4%) wall
226 kB ( 0%) ggc
var-tracking emit : 0.31 ( 3%) usr 0.00 ( 0%) sys 0.36 ( 4%) wall
2186 kB ( 2%) ggc
for me, I'd call this insignificant, given that it was really a debug info
correctness fix.
We have PR54402 which tracks cases where the slowdown is actually significant.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug middle-end/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (5 preceding siblings ...)
2012-12-18 11:01 ` jakub at gcc dot gnu.org
@ 2013-01-18 11:01 ` aoliva at gcc dot gnu.org
2013-01-18 11:07 ` [Bug lto/54114] " aoliva at gcc dot gnu.org
` (9 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: aoliva at gcc dot gnu.org @ 2013-01-18 11:01 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
--- Comment #7 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2013-01-18 10:58:47 UTC ---
Author: aoliva
Date: Fri Jan 18 10:58:15 2013
New Revision: 195291
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195291
Log:
PR debug/54114
PR debug/54402
PR debug/49888
* var-tracking.c (negative_power_of_two_p): New.
(global_get_addr_cache, local_get_addr_cache): New.
(get_addr_from_global_cache, get_addr_from_local_cache): New.
(vt_canonicalize_addr): Rewrite using the above. Adjust the
heading comment.
(vt_stack_offset_p): Remove.
(vt_canon_true_dep): Always canonicalize loc's address.
(clobber_overlapping_mems): Make sure we have a MEM.
(local_get_addr_clear_given_value): New.
(val_reset): Clear local cached entries.
(compute_bb_dataflow): Create and release the local cache.
Disable duplicate MEMs clobbering.
(emit_notes_in_bb): Clobber MEMs likewise.
(vt_emit_notes): Create and release the local cache.
(vt_initialize, vt_finalize): Create and release the global
cache, respectively.
* alias.c (rtx_equal_for_memref_p): Compare operands of
ENTRY_VALUEs.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/alias.c
trunk/gcc/var-tracking.c
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug lto/54114] [4.8 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (6 preceding siblings ...)
2013-01-18 11:01 ` aoliva at gcc dot gnu.org
@ 2013-01-18 11:07 ` aoliva at gcc dot gnu.org
2013-01-21 18:57 ` [Bug debug/54114] [4.8/4.9 " jakub at gcc dot gnu.org
` (8 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: aoliva at gcc dot gnu.org @ 2013-01-18 11:07 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Alexandre Oliva <aoliva at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |NEW
Component|middle-end |lto
Resolution|WONTFIX |
--- Comment #8 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2013-01-18 11:06:38 UTC ---
This should be much better now. There still ought to be a better way to deal
with MEM clobbering than comparing to each one at every write, but we don't
have that yet.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] [4.8/4.9 Regression] variable-tracking performance regression from 4.8-20120610 to 4.8-20120701
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (7 preceding siblings ...)
2013-01-18 11:07 ` [Bug lto/54114] " aoliva at gcc dot gnu.org
@ 2013-01-21 18:57 ` jakub at gcc dot gnu.org
2013-12-20 16:38 ` [Bug debug/54114] VTA compile-time performance could be improved law at redhat dot com
` (7 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-01-21 18:57 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Component|lto |debug
Target Milestone|4.8.0 |4.9.0
Summary|[4.8 Regression] |[4.8/4.9 Regression]
|variable-tracking |variable-tracking
|performance regression from |performance regression from
|4.8-20120610 to |4.8-20120610 to
|4.8-20120701 |4.8-20120701
--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-01-21 18:56:44 UTC ---
Something to look for 4.9 then.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (8 preceding siblings ...)
2013-01-21 18:57 ` [Bug debug/54114] [4.8/4.9 " jakub at gcc dot gnu.org
@ 2013-12-20 16:38 ` law at redhat dot com
2014-04-22 11:36 ` jakub at gcc dot gnu.org
` (6 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: law at redhat dot com @ 2013-12-20 16:38 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jeffrey A. Law <law at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |law at redhat dot com
Summary|[4.8/4.9 Regression] |VTA compile-time
|variable-tracking |performance could be
|performance regression from |improved
|4.8-20120610 to |
|4.8-20120701 |
--- Comment #10 from Jeffrey A. Law <law at redhat dot com> ---
Removing regression marker based on c#7 and c#8. Sounds like there's still
work that can be done here, but that the regression aspect is significantly
improved.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (9 preceding siblings ...)
2013-12-20 16:38 ` [Bug debug/54114] VTA compile-time performance could be improved law at redhat dot com
@ 2014-04-22 11:36 ` jakub at gcc dot gnu.org
2014-07-16 13:29 ` jakub at gcc dot gnu.org
` (5 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-04-22 11:36 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.9.0 |4.9.1
--- Comment #11 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.0 has been released
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (10 preceding siblings ...)
2014-04-22 11:36 ` jakub at gcc dot gnu.org
@ 2014-07-16 13:29 ` jakub at gcc dot gnu.org
2014-10-30 10:39 ` jakub at gcc dot gnu.org
` (4 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-07-16 13:29 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.9.1 |4.9.2
--- Comment #12 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.1 has been released.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (11 preceding siblings ...)
2014-07-16 13:29 ` jakub at gcc dot gnu.org
@ 2014-10-30 10:39 ` jakub at gcc dot gnu.org
2015-05-12 20:59 ` dsl at dsl dot pp.ua
` (3 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-10-30 10:39 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.9.2 |4.9.3
--- Comment #13 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.2 has been released.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (12 preceding siblings ...)
2014-10-30 10:39 ` jakub at gcc dot gnu.org
@ 2015-05-12 20:59 ` dsl at dsl dot pp.ua
2015-06-26 19:54 ` jakub at gcc dot gnu.org
` (2 subsequent siblings)
16 siblings, 0 replies; 18+ messages in thread
From: dsl at dsl dot pp.ua @ 2015-05-12 20:59 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Eliot <dsl at dsl dot pp.ua> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |dsl at dsl dot pp.ua
--- Comment #14 from Eliot <dsl at dsl dot pp.ua> ---
Comment on attachment 27894
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=27894
XPatch.cpp preprocessed source from xalanc
http://legup.eecg.utoronto.ca
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (13 preceding siblings ...)
2015-05-12 20:59 ` dsl at dsl dot pp.ua
@ 2015-06-26 19:54 ` jakub at gcc dot gnu.org
2015-06-26 20:27 ` jakub at gcc dot gnu.org
2015-06-26 21:30 ` pinskia at gcc dot gnu.org
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2015-06-26 19:54 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
--- Comment #15 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.3 has been released.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (14 preceding siblings ...)
2015-06-26 19:54 ` jakub at gcc dot gnu.org
@ 2015-06-26 20:27 ` jakub at gcc dot gnu.org
2015-06-26 21:30 ` pinskia at gcc dot gnu.org
16 siblings, 0 replies; 18+ messages in thread
From: jakub at gcc dot gnu.org @ 2015-06-26 20:27 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.9.3 |4.9.4
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Bug debug/54114] VTA compile-time performance could be improved
2012-07-28 17:22 [Bug middle-end/54114] New: variable-tracking performance regression from 4.8-20120610 to 4.8-20120701 jimis at gmx dot net
` (15 preceding siblings ...)
2015-06-26 20:27 ` jakub at gcc dot gnu.org
@ 2015-06-26 21:30 ` pinskia at gcc dot gnu.org
16 siblings, 0 replies; 18+ messages in thread
From: pinskia at gcc dot gnu.org @ 2015-06-26 21:30 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54114
Andrew Pinski <pinskia at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |compile-time-hog
Target Milestone|4.9.4 |---
^ permalink raw reply [flat|nested] 18+ messages in thread