[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-02-23 Thread jason at gcc dot gnu dot org 


--- Comment #9 from jason at gcc dot gnu dot org  2010-02-23 18:32 ---
Subject: Bug 42800

Author: jason
Date: Tue Feb 23 18:32:09 2010
New Revision: 157014

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157014
Log:
PR debug/42800
* cfgexpand.c (expand_used_vars): Keep artificial non-ignored vars
in cfun->local_decls even if they have register types.

Added:
trunk/gcc/testsuite/c-c++-common/dwarf2/
trunk/gcc/testsuite/c-c++-common/dwarf2/vla1.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/cfgexpand.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/debug/dwarf2/dwarf2.exp
trunk/gcc/testsuite/gcc.dg/debug/dwarf2/dwarf2.exp


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-02-23 Thread jason at gcc dot gnu dot org 


--- Comment #8 from jason at gcc dot gnu dot org  2010-02-23 18:32 ---
Fixed.


-- 

jason at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-02-19 Thread jason at gcc dot gnu dot org 


--- Comment #7 from jason at gcc dot gnu dot org  2010-02-20 05:16 ---
Created an attachment (id=19929)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19929&action=view)
patch


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-02-19 Thread jason at gcc dot gnu dot org 


--- Comment #6 from jason at gcc dot gnu dot org  2010-02-19 20:59 ---
Yeah, the Expand from SSA patch broke Jakub's fix for PR 34037.  I'm testing a
patch that just tweaks the logic a bit to make it work again.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-02-19 Thread jason at gcc dot gnu dot org 


--- Comment #5 from jason at gcc dot gnu dot org  2010-02-19 20:13 ---
The problem is that instantiate_decl_rtl isn't being run on the rtl for the
upper bound temporary, because it isn't visible to instantiate_decls.  This
happens because even though the temp got added to cfun->local_decls,
expand_used_vars clears it out before instantiate_decls gets a chance to look
at it.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-02-19 Thread jason at gcc dot gnu dot org 


-- 

jason at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |jason at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-01-19 15:43:55 |2010-02-19 18:01:22
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-01-27 Thread jakub at gcc dot gnu dot org 


-- 

jakub at gcc dot gnu dot org changed:

   What|Removed |Added

   Priority|P3  |P1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-01-19 Thread jakub at gcc dot gnu dot org 


--- Comment #4 from jakub at gcc dot gnu dot org  2010-01-19 17:19 ---
Yes.  This is -O0, so we definitely should ensure that the debug info has the
correct upper bound.
Look at gimplify_type_sizes which for -O0 clears DECL_IGNORED_P on the
temporaries to make sure they are not optimized out.  That worked well in 4.4,
but probably doesn't work any longer.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-01-19 Thread hjl dot tools at gmail dot com 


--- Comment #3 from hjl dot tools at gmail dot com  2010-01-19 15:52 ---
(In reply to comment #2)
> What's the brokenness?  The missing upper bound in the subrange type?
> 

It blocks PR 42801.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-01-19 Thread matz at gcc dot gnu dot org 


--- Comment #2 from matz at gcc dot gnu dot org  2010-01-19 15:49 ---
What's the brokenness?  The missing upper bound in the subrange type?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-01-19 Thread hjl dot tools at gmail dot com 


--- Comment #1 from hjl dot tools at gmail dot com  2010-01-19 15:43 ---
It is caused by Expand from SSA:

http://gcc.gnu.org/ml/gcc-cvs/2009-04/msg01459.html


-- 

hjl dot tools at gmail dot com changed:

   What|Removed |Added

 CC||matz at gcc dot gnu dot org
 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-01-19 15:43:55
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800

[Bug debug/42800] [4.5 Regression] VLA DW_AT_upper_bound is no longer emitted

2010-01-19 Thread rguenth at gcc dot gnu dot org 


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

Summary|VLA DW_AT_upper_bound is no |[4.5 Regression] VLA
   |longer emitted  |DW_AT_upper_bound is no
   ||longer emitted
   Target Milestone|--- |4.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42800