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.