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