On 11/24/14 20:56, Segher Boessenkool wrote:
This caused ICEs on sh64.

`min_cost' and `def' here are supposed to refer to the same element;
removing it from the heap before asking the heap for the key doesn't
work (and at the end of the loop here we will ask for min_key on an
empty heap, which then does gcc_unreachable).

Bootstrapped and tested on powerpc64-linux, but I doubt it exercises
this code at all; only sh64 did ICE, and does not anymore.  Okay for
trunk?


Segher


2014-11-24  Segher Boessenkool  <seg...@kernel.crashing.org>

gcc/
        * bt-load.c (migrate_btr_defs): Get the key of a heap entry
        before removing it, not after.
OK.

Did sh64 ICE during a build, or was it during testing or something else? Trying to figure out if we need a distinct test in the suite or not.


jeff

Reply via email to