On Tue, Oct 23, 2012 at 03:08:07PM +0200, Dodji Seketeli wrote: > +static gimple_stmt_iterator > +create_cond_insert_point_before_iter (gimple_stmt_iterator *iter, > + bool then_more_likely_p, > + basic_block *then_block, > + basic_block *fallthrough_block) > +{ > + gcc_assert (then_block != NULL && fallthrough_block != NULL);
I think this assert is useless, if they are NULL > + *then_block = then_bb; > + *fallthrough_block = fallthru_bb; the above two stmts will just crash and be as useful for debugging as the assert. Jakub