[tree-ssa] [PATCH]: Fix DCE not updating bb->end_tree_p
Daniel Berlin
dberlin@dberlin.org
Tue Jul 15 22:22:00 GMT 2003
On Tue, 15 Jul 2003, Diego Novillo wrote:
> On Tue, 2003-07-15 at 18:15, Daniel Berlin wrote:
>
> > > > + bb->end_tree_p = bsi_container (bsi);
> > > > + }
> > > > +
> > > This should be bsi_stmt_ptr (bsi).
> > Nope, that won't work, that was my first try.
> >
> Oh, probably so that we can start reverse iterators at block's end.
Right.
> OK
> then. Out of curiosity, remember what was the failure?
As you mentioned above. if you point it at the statement, rather than the
compound_expr containing it, it can't go backwards.
I think bsi_prev is actually broken, in that it doesn't do the opposite of
bsi_next/bsi_stmt in this regard (IE it should look in the first operand
of a compound_expr, like bsi_stmt would, when trying to find the prev)
>
>
> Thanks. Diego.
>
>
More information about the Gcc-patches
mailing list