https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64694

--- Comment #5 from Jan Hubicka <hubicka at ucw dot cz> ---
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64694
> 
> --- Comment #4 from H.J. Lu <hjl.tools at gmail dot com> ---
> r219076 introduced:
> 
>       if (current_badness != badness)
>         {
>           if (edge_heap.min () && badness > edge_heap.min_key ())
>                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>                                   Does it look right?

This is a thinko, indeed.  I am testing a fix, but I doubdt this can affect
correctness
of a testcase.  I see it does inlining, but why it depends on inline decisions?

Honza
>             {
>               edge->aux = edge_heap.insert (current_badness, edge);
>               continue;
>             }    
>           else
>             badness = current_badness;
>         }
> 
> -- 
> You are receiving this mail because:
> You are on the CC list for the bug.

Reply via email to