Re: [patch] simplify emit_delay_sequence

2013-04-17 Thread Steven Bosscher
On Wed, Apr 17, 2013 at 10:39 AM, Eric Botcazou wrote: > @@ -538,6 +502,8 @@ emit_delay_sequence (rtx insn, rtx list, int lengt > INSN_LOCATION (seq_insn) = INSN_LOCATION (tem); >INSN_LOCATION (tem) = 0; > > + /* Remove any REG_DEAD notes because we can't rely on them now > +

Re: [patch] simplify emit_delay_sequence

2013-04-17 Thread Eric Botcazou
> This patch is also necessary for my new delay-slot scheduler to keep > basic block boundaries correctly up-to-date. The emit-rtl API does > that already. > > Cross-tested powerpc64 x mips. Currently running bootstrap&test on > sparc64-unknown-linux-gnu. OK if it passes? Yes, modulo @@ -538,6 +

[patch] simplify emit_delay_sequence

2013-04-16 Thread Steven Bosscher
Hello, With the recent add_insn_* cleanups, we can now simplify reorg.c:emit_delay_sequence. Without this patch, emit_delay_sequence hacks the insns chain "manually", setting PREV_INSN and NEXT_INSN to chain everything together properly. But emit-rtl provides an abstraction of sorts for that, and