[Bug debug/63623] Lots of functions get -fvar-tracking silently turned off unnecessarily

2015-06-13 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63623

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Jakub Jelinek jakub at gcc dot gnu.org ---
Fixed.

[Bug debug/63623] Lots of functions get -fvar-tracking silently turned off unnecessarily

2015-06-12 Thread matz at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63623

--- Comment #4 from Michael Matz matz at gcc dot gnu.org ---
Author: matz
Date: Fri Jun 12 14:06:41 2015
New Revision: 224434

URL: https://gcc.gnu.org/viewcvs?rev=224434root=gccview=rev
Log:
Backported from mainline
2014-10-23  Jakub Jelinek  ja...@redhat.com

PR debug/63623
* var-tracking.c (stack_adjust_offset_pre_post_cb): New function.
(stack_adjust_offset_pre_post): Use it through for_each_inc_dec,
instead of only handling autoinc in dest if it is a MEM.
(vt_stack_adjustments): Fix up formatting.

Modified:
branches/gcc-4_8-branch/gcc/ChangeLog
branches/gcc-4_8-branch/gcc/var-tracking.c

[Bug debug/63623] Lots of functions get -fvar-tracking silently turned off unnecessarily

2015-06-12 Thread matz at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63623

--- Comment #5 from Michael Matz matz at gcc dot gnu.org ---
Author: matz
Date: Fri Jun 12 14:13:33 2015
New Revision: 224435

URL: https://gcc.gnu.org/viewcvs?rev=224435root=gccview=rev
Log:
Backported from mainline
2014-10-23  Jakub Jelinek  ja...@redhat.com

PR debug/63623
* var-tracking.c (stack_adjust_offset_pre_post_cb): New function.
(stack_adjust_offset_pre_post): Use it through for_each_inc_dec,
instead of only handling autoinc in dest if it is a MEM.

Modified:
branches/gcc-4_9-branch/gcc/ChangeLog
branches/gcc-4_9-branch/gcc/var-tracking.c

[Bug debug/63623] Lots of functions get -fvar-tracking silently turned off unnecessarily

2014-10-23 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63623

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2014-10-23
   Assignee|unassigned at gcc dot gnu.org  |jakub at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Jakub Jelinek jakub at gcc dot gnu.org ---
Created attachment 33789
  -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=33789action=edit
gcc5-pr63623.patch

Patch (successfully tested).

[Bug debug/63623] Lots of functions get -fvar-tracking silently turned off unnecessarily

2014-10-23 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63623

--- Comment #2 from Jakub Jelinek jakub at gcc dot gnu.org ---
dwlocstat --tabulate=0.0:10,99
results on cc1plus bootstrapped --enable-checking=yes,rtl binaries before/after
the patch (in the tree without the patch I've applied the patch afterwards and
did make cc1plus in the gcc directory, so var-tracking.o in stage3 got rebuilt
again and thus the compared binaries contain exactly the same code, just
different debug info):

x86_64-linux cc1plus built without the patch:
cov%samplescumul
0.0506230/38%506230/38%
0..1010327/0%516557/39%
11..2012390/0%528947/39%
21..3031265/2%560212/42%
31..4018775/1%578987/43%
41..5020631/1%599618/45%
51..6024921/1%624539/47%
61..7040959/3%665498/50%
71..8023771/1%689269/52%
81..9041771/3%731040/55%
91..9981667/6%812707/61%
100510564/38%1323271/100%

x86_64-linux cc1plus built with the patch:
cov%samplescumul
0.0382214/28%382214/28%
0..1013100/0%395314/29%
11..2014568/1%409882/30%
21..3033708/2%443590/33%
31..4021927/1%465517/35%
41..5023924/1%489441/36%
51..6028736/2%518177/39%
61..7045847/3%564024/42%
71..8029284/2%593308/44%
81..9052085/3%645393/48%
91..9999971/7%745364/56%
100577907/43%1323271/100%

i686-linux cc1plus built without the patch:
cov%samplescumul
0.0631348/48%631348/48%
0..107764/0%639112/48%
11..209690/0%648802/49%
21..3025036/1%673838/51%
31..4016113/1%689951/52%
41..5019753/1%709704/54%
51..6014563/1%724267/55%
61..7034093/2%758360/58%
71..8017450/1%775810/59%
81..9031339/2%807149/61%
91..9960368/4%867517/66%
100437548/33%1305065/100%

i686-linux cc1plus built with the patch:
cov%samplescumul
0.0377352/28%377352/28%
0..1016077/1%393429/30%
11..2015390/1%408819/31%
21..3031790/2%440609/33%
31..4023889/1%464498/35%
41..5029267/2%493765/37%
51..6022902/1%516667/39%
61..7045629/3%562296/43%
71..8029511/2%591807/45%
81..9050536/3%642343/49%
91..9993584/7%735927/56%
100569138/43%1305065/100%

.debug_info/.debug_loc sizes in bytes:
x86_64-linux cc1plus without patch .debug_info 75411710, .debug_loc 75421077
x86_64-linux cc1plus withpatch .debug_info 78498790, .debug_loc 90530117
  i686-linux cc1plus without patch .debug_info 59921183, .debug_loc 37823166
  i686-linux cc1plus withpatch .debug_info 63009554, .debug_loc 59535100

[Bug debug/63623] Lots of functions get -fvar-tracking silently turned off unnecessarily

2014-10-23 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63623

--- Comment #3 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: jakub
Date: Thu Oct 23 21:58:41 2014
New Revision: 216600

URL: https://gcc.gnu.org/viewcvs?rev=216600root=gccview=rev
Log:
PR debug/63623
* var-tracking.c (stack_adjust_offset_pre_post_cb): New function.
(stack_adjust_offset_pre_post): Use it through for_each_inc_dec,
instead of only handling autoinc in dest if it is a MEM.
(vt_stack_adjustments): Fix up formatting.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/var-tracking.c