On 4/28/20 6:06 PM, Peter Bergner wrote: > However, do you mean the change to be the following, since I don't think > simplify_gen_binary ever returns NULL? > > > validate_change (object, &XEXP (x, i), > cse_process_notes (XEXP (x, i), object, changed), 0); > > + switch (GET_RTX_CLASS (code)) > + { > + case RTX_BIN_ARITH: > + case RTX_COMM_ARITH: > + return simplify_gen_binary (code, GET_MODE (x), XEXP (x, 0), XEXP (x, > 1)); > + default: > + break; > + } > + > return x; > }
FYI, this passed bootstrap and regtesting with no regressions. Is the above ok (with comment from previous patch) for trunk? Peter