> 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. > What does the RTL for your conditional return look like > > (if_then_else (cond) (pc) (return)) > > Where the (pc) and (return) can be reversed as well? That's what the > later hunks in get_branch_condition seem to imply. It's the usual support for branches and inverted branches applied to returns, so I'm not sure what you're asking here. :-) > The cleanup stuff is OK, check that in whenver you'd like. Thanks. -- Eric Botcazou