On Wed, 14 Oct 2020, Tom de Vries wrote:

> On 10/14/20 8:15 AM, Richard Biener wrote:
> >> I've tried to address this by merging can_duplicate_stmt_p and
> >> can_duplicate_last_stmt_p, and adding a default parameter.
> >>
> >> Better like this?
> > Sorry for iterating again but since we now would appropriately
> > handle things in the CFG hook there's no need for tracer.c to
> > do this on its own via the _stmt calls.  So I suggest to
> > remove the unifying of the stmt counting loop and use
> > the can_duplicate_block_p CFG hook directly instead (but of course
> > still cache its outcome).  That way we can simplify what is
> > exported.
> 
> Np :) . Fully retested, OK for trunk?

OK.

Thanks,
Richard.

Reply via email to