On 04/01/2013 01:47 PM, Eric Botcazou wrote:
I'm not sure what this is supposed to do. How is pat == target ever
going to be true when ANY_RETURN_P (pat) is true? Isn't target supposed
to be a CODE_LABEL or NULL? What am I missing?
The simple_return change from Bernd. The JUMP_LABEL of a JUMP_INSN is now
either a CODE_LABEL or a RETURN or a SIMPLE_RETURN.
Ah, OK, yea, it makes perfect sense now. Approved.
If you wanted to get ambitious, rewriting reorg.c to compute and use
dependency links to drive its candidate selection instead of the insane
tracking code it uses would be a huge step forward, both in terms of
cleanliness. It'd also help compile-time performance; we do a lot of
work to track resources that would be totally unnecessary.
I don't work on targets with delay slots anymore, so it's not something
I'm likely to ever tackle myself.
jeff
