PATCH Re: [ast-optimizer-branch]: Simplify STMT_EXPR's

Diego Novillo dnovillo@redhat.com
Wed Jun 19 22:17:00 GMT 2002


On Wed, 19 Jun 2002, Daniel Berlin wrote:

> On Wed, 19 Jun 2002, Diego Novillo wrote:
> 
> > On Thu, 20 Jun 2002, Jason Merrill wrote:
> > 
> > > --- 193,207 ----
> > >         tree next, pre, post;
> > >         int keep_stmt_p;
> > >         tree stmt = *stmt_p;
> > > +       int saved_stmts_are_full_exprs_p;
> > > + 
> > > +       /* Set up context appropriately for handling this statement.  */
> > > +       saved_stmts_are_full_exprs_p = stmts_are_full_exprs_p ();
> > > +       prep_stmt (stmt);
> > >   
> > What's the purpose of saved_stmts_are_full_exprs_p?  I don't see
> > it used anywhere else.
> 
> Look at c-semantics.c, expand_expr, and the gen* functions.
> 
> Or, to cut to the chase, expand_start_target_temps in stmt.c (which is 
> called if (stmts_are_full_exprs) from the gen* functions in c-semantics.c:
> 
What I meant is that variable 'saved_stmts_are_full_exprs_p' is a
local variable inside simplify_stmt() that is set but never used
in that function.


Diego.



More information about the Gcc-patches mailing list