On 01/06/2013 01:11 PM, Alan Lehotsky wrote: > Do I need to do something like change my define_expand for addsi3 to be > something like > > > [ (parallel [ (clobber:CCmode (reg:CCmode CCREG)) > (set:SI (match_operand:SI 0 "" "") > (plus:SI (match_operand::SI 1 "" "" ) > (match_operand:SI 2 "" > "")))])] > > So that there's an explicitly stated dependency that would prevent motioning > before the use in the jump_insn?
Yes. (With the note that SETs appear first in parallels, with CLOBBERs last.) r~