Steven Bosscher wrote:
> This patch merely papers over another issue, probably a missing
> CROSSING_JUMP_P test.
Perhaps. Surely it has looked the current DBR is not so well for
crossing jumps and my fix might be a bit ad-hoc.
The first part of the patch could be rewritten with checking if one
o
On Tue, Jan 27, 2015 at 12:52 AM, Kaz Kojima wrote:
> This patch is to fix 2 issues found in dbr_schedule when trying to
> fix PR target/64761. The first is relax_delay_slots removes
> the jump insn in the insns like below:
>
> (jump_insn/j 74 58 59 (set (pc) (label_ref:SI 29)) ...)
> (barrier 59
Jeff Law wrote:
> OK. I'm a bit surprised we're still finding this kind of stuff 10
> years after NOTE_INSN_SWITCH_TEXT_SECTIONS went in. Sigh.
Thanks for reviewing. I've committed it as revision 220235
after testing the patch independently.
Regards,
kaz
On 01/26/15 16:52, Kaz Kojima wrote:
This patch is to fix 2 issues found in dbr_schedule when trying to
fix PR target/64761. The first is relax_delay_slots removes
the jump insn in the insns like below:
(jump_insn/j 74 58 59 (set (pc) (label_ref:SI 29)) ...)
(barrier 59 74 105)
(note 105 59 29
