On 11/08/2015 08:09 PM, Segher Boessenkool wrote:
The code mistakenly thinks any cond_jump has two successors.  This is
not true if both destinations are the same, as can happen with weird
patterns as in the PR.

Bootstrapped and tested on powerpc64-linux; also tested the simplified
test in the PR on an x86_64-linux cross.

Sorry for the breakage.  Is this okay for trunk?


Segher


2015-11-09  Segher Boessenkool  <seg...@kernel.crashing.org>

        * gcc/bb-reorder.c (reorder_basic_blocks_simple): Treat a conditional
        branch with only one successor just like unconditional branches.
OK. Though this begs the question, should something have cleaned that up prior to bb-reorder?

Don't forget the PR marker in the committed ChangeLog.

jeff

Reply via email to