> 
> I don't think so.  But I don't know much about that bug, it is something
> with AVX I think?  If you are talking about PR79224.

I see, we have separate PR for that, good ;)
> 
> > Also with profile feedback perhaps you have enough info to tell that the
> > speculative path is almost as likely as the original placement.
> 
> Maybe.  The gain will be tiny though, afaics.

Usually profile is useful while driving speculation.  I would add an
param that prevents moving insn when bb->count or bb->frequency increases
by large amount (like 100 times).  Speculatively executing insn that will
be ignored 99% of times is probably not paying back...
Without profile that will probably only trigger for paths leading to abort()
or similar cases, but with profiles that may hit quite common scenarios.

Honza
> 
> > I wonder if we can provie resonable default by RTX cost...
> 
> Not for rs6000 at least.

Reply via email to