This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [PATCH] variable size arrays in nested functions

On Fri, Feb 09, 2001 at 10:36:19AM -0800, Richard Henderson wrote:
> On Fri, Feb 09, 2001 at 01:17:28PM +0100, Jakub Jelinek wrote:
> > 	* c-decl.c (push_parm_decl): Call grokdeclarator always with
> > 	current_function_decl == 0, even for nested functions.
> This seems strange.
> > +int b;
> > +int foo (void)
> > +{
> > +  int x[b];
> > +  int bar (int t[b])
> > +  {
> What about
>   int foo(int b)
> instead?  Doesn't dropping to no context cause this to fail?

It does not break (but ICEs without the patch).
Basically, what's going on is that variable_size takes care of putting the
SAVE_EXPR on pending list, but SAVE_EXPRs can be created deeply from fold
(the problematic SAVE_EXPR is actually created from within build_array_type
when building TREE_SIZE()). get_pending_sizes() takes care of fixing up
context of the SAVE_EXPRs recorded by variable_size but this one slips
Do you think a patch using a global variable which would force save_expr to
record things into pending_sizes (like variable_size does already) would be


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]