[Bug objc++/49221] [4.7 Regression] Several ICEs in the obj-c++ test suite after revision 174307

2011-06-05 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49221

Jason Merrill jason at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|unassigned at gcc dot   |jason at gcc dot gnu.org
   |gnu.org |

--- Comment #8 from Jason Merrill jason at gcc dot gnu.org 2011-06-06 
03:51:55 UTC ---
During EH lowering the back end wants to replace a TYPE with something else
that represents the type at runtime.  objc_eh_runtime_type generates such a
thing, and tries to generate a DECL, but we're in function scope at the time so
we try to add a DECL_EXPR for the DECL, which breaks.

We definitely shouldn't be trying to add a DECL_EXPR for a non-local variable. 
I suppose it's easy enough to work around this in cp_finish_decl, though there
might be other code that assumes that any new variable while we're in function
scope belongs to that scope.


[Bug objc++/49221] [4.7 Regression] Several ICEs in the obj-c++ test suite after revision 174307

2011-06-05 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49221

--- Comment #9 from Jason Merrill jason at gcc dot gnu.org 2011-06-06 
04:18:08 UTC ---
Author: jason
Date: Mon Jun  6 04:18:03 2011
New Revision: 174683

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=174683
Log:
PR objc++/49221
* decl.c (cp_finish_decl): Check DECL_FUNCTION_SCOPE_P rather than
at_function_scope_p.

Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/decl.c


[Bug objc++/49221] [4.7 Regression] Several ICEs in the obj-c++ test suite after revision 174307

2011-06-05 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49221

Jason Merrill jason at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #10 from Jason Merrill jason at gcc dot gnu.org 2011-06-06 
04:32:06 UTC ---
Fixed.


[Bug objc++/49221] [4.7 Regression] Several ICEs in the obj-c++ test suite after revision 174307

2011-06-02 Thread danglin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49221

John David Anglin danglin at gcc dot gnu.org changed:

   What|Removed |Added

 CC||danglin at gcc dot gnu.org

--- Comment #7 from John David Anglin danglin at gcc dot gnu.org 2011-06-03 
00:55:33 UTC ---
Same on hppa64-hp-hpux11.11.


[Bug objc++/49221] [4.7 Regression] Several ICEs in the obj-c++ test suite after revision 174307

2011-06-01 Thread froydnj at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49221

Nathan Froyd froydnj at gcc dot gnu.org changed:

   What|Removed |Added

 CC||jason at gcc dot gnu.org

--- Comment #6 from Nathan Froyd froydnj at gcc dot gnu.org 2011-06-01 
12:01:06 UTC ---
I'm not sure what to do in this case; maybe Jason has an opinion.  Jason,
should we be push_stmt'ing somewhere in the objc code (objc_eh_runtime_type,
perhaps)?