> I wonder how many other problems of this nature are lurking in reorg.c.
> For example steal_delay_list_from_{target,fallthrough} or the code
> which searches for arithmetic at the branch target, and puts the
> opposite insn in a delay slot.

Right, and the latter has already been dealt with by Richard:

2012-02-11  Richard Sandiford  <rdsandif...@googlemail.com>

        PR rtl-optimization/52175
        * reorg.c (fill_slots_from_thread): Don't apply add/sub optimization
        to frame-related instructions.

> In fact, I really wonder if we should be allowing anything frame related
> outside fill_simple_delay_slots.

That could well be the end result after a few more years of tweaking. :-)

-- 
Eric Botcazou

Reply via email to