Re: [patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-26 Thread Jeff Law
On 11/25/2012 11:30 AM, Richard Sandiford wrote: FWIW, the vast majority of condjumps have a REG_BR_PROB note so the part of mostly_true_jump that handles the jumps without a REG_BR_PROB note is itself a rare_destination :-) GCC is very careful about preserving the notes. Perhaps all that code

[patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-25 Thread Steven Bosscher
Hello, reorg.c:rare_destination() tries to determine whether a given insn is a likely destination of a branch. To make this determination, it walks the insns chain from insn and stops at barriers, labels, return insns, or if more than 10 jumps have been followed. The function is supposed to

Re: [patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-25 Thread Richard Sandiford
Steven Bosscher stevenb@gmail.com writes: Hello, reorg.c:rare_destination() tries to determine whether a given insn is a likely destination of a branch. To make this determination, it walks the insns chain from insn and stops at barriers, labels, return insns, or if more than 10 jumps

Re: [patch] reorg.c janitor patch 3: remove rare_destination()

2012-11-25 Thread Steven Bosscher
On Sun, Nov 25, 2012 at 6:13 PM, Richard Sandiford wrote: Steven Bosscher writes: Hello, reorg.c:rare_destination() tries to determine whether a given insn is a likely destination of a branch. To make this determination, it walks the insns chain from insn and stops at barriers, labels,