On Wed, 9 Jul 2025, Jan Hubicka wrote:

> > The following changes the percentage that determines how many
> > stmts are allowed for backwards jump threading from 50 to 54,
> > enabling the missed jump threading observed in PR109893.
> > 
> > Bootstrapped and tested on x86_64-unknown-linux-gnu.  It seems that
> > at least backward threading is prone to profile mismatches, I've
> > altered two testcases to deal with new ones to pop up (definitely
> > latent issues).
> > 
> > OK?
> I wonder if the duplication limit should not be controlled by statement
> size rather than statement count?

It is, it uses estimate_num_insns (stmt, &eni_size_weights) to
"count" stmts, the name (and variables used) are just "off".

Richard.

> Honza
> 

Reply via email to