On Fri, 15 Mar 2019, Jakub Jelinek wrote:

> On Fri, Mar 15, 2019 at 01:25:57PM +0300, Andrey Belevantsev wrote:
> > As explained in the PR trail, we incorrectly update the availability sets
> > in the rare case of several successors and one of them having another
> > fence.  Fixed as follows.  Ok for trunk?
> > 
> > Best,
> > Andrey
> > 
> > 2019-03-15  Andrey Belevantsev  <a...@ispras.ru>
> > 
> >     PR middle-end/89676
> >     * sel-sched.c (compute_av_set_at_bb_end): When we have an ineligible
> > successor,
> >     use NULL as its av set.
> 
> Just formatting nits, will defer actual review to some scheduler maintainer.

I can do this, it falls under sel-sched maintenance and Andrey showed to me
in person the algorithmic corner-case here, so: OK for trunk with formatting
fixed.

Thanks!
Alexander

Reply via email to