This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fill more delay slots in conditional returns
- From: Jeff Law <law at redhat dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 15 Apr 2013 08:02:45 -0600
- Subject: Re: Fill more delay slots in conditional returns
- References: <1769120 dot Os7kk4Zj9I at polaris> <2373459 dot H4ikUyDJrc at polaris> <51645569 dot 1090501 at redhat dot com> <5082198 dot kTrRiqZjhC at polaris>
On 04/14/2013 03:43 AM, Eric Botcazou wrote:
I don't recall ever working on this aspect of reorg. The obvious worry
is that with reorg moving stuff around those notes may not be valid
anymore in the general case.
Yes, in the general case I agree that's too dangerous. In this particular
case, i.e. backward scan only, this might be plausible, although one has
probably to worry about what happens if the insn is removed from the delay
slot and put back into the RTL stream.
And if I remember correctly, relax_delay_slots can do this sort of thing.
* reorg.c (fill_simple_delay_slots): Reindent block of code.
* resource.c (mark_target_live_regs): Reformat conditional block.
Thanks for taking care of these.
Jeff