From: Richard Henderson Date: Fri, 1 Oct 1999 01:13:51 +0000 (-0700) Subject: jump.c (jump_optimize_1): Amend last change to test only the form of the operand... X-Git-Tag: prereleases/libstdc++-2.92~10285 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=39cc9917e21cb9f2145881c321a25ab8ed2bb54a;p=thirdparty%2Fgcc.git jump.c (jump_optimize_1): Amend last change to test only the form of the operand, not the insn. * jump.c (jump_optimize_1): Amend last change to test only the form of the operand, not the insn. From-SVN: r29741 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 605e26cb816d..34656de2a12b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,8 @@ Thu Sep 30 18:04:13 1999 Richard Henderson + * jump.c (jump_optimize_1): Amend last change to test only the + form of the operand, not the insn. + * jump.c (jump_optimize_1) [conditional_move]: Use general_operand not register_operand to determine when B may need preloading. diff --git a/gcc/jump.c b/gcc/jump.c index 377cfc966901..dcd1a6a5c337 100644 --- a/gcc/jump.c +++ b/gcc/jump.c @@ -1169,11 +1169,11 @@ jump_optimize_1 (f, cross_jump, noop_moves, after_regscan, mark_labels_only) start_sequence (); - /* If we're not dealing with a simple operand or the insn is - more complex than a simple SET, duplicate the computation - and replace the destination with a new temporary. */ - if (general_operand (temp2, GET_MODE (var)) - && GET_CODE (PATTERN (temp)) == SET) + /* We're dealing with a single_set insn with no side effects + on SET_SRC. We do need to be reasonably certain that if + we need to force BVAL into a register that we won't + clobber the flags -- general_operand should suffice. */ + if (general_operand (temp2, GET_MODE (var))) bval = temp2; else { @@ -1183,7 +1183,7 @@ jump_optimize_1 (f, cross_jump, noop_moves, after_regscan, mark_labels_only) SET_DEST (temp6) = bval; emit_insn (PATTERN (new_insn)); } - + target = emit_conditional_move (var, code, cond0, cond1, VOIDmode, aval, bval, GET_MODE (var),