Peter Bergner <berg...@linux.ibm.com> writes:
> On 4/28/20 2:38 AM, Richard Sandiford wrote:
>>     case RTX_BIN_ARITH:
>>     case RTX_COMM_ARITH:
>>       op0 = simplify_replace_fn_rtx (XEXP (x, 0), old_rtx, fn, data);
>>       op1 = simplify_replace_fn_rtx (XEXP (x, 1), old_rtx, fn, data);
>>       if (op0 == XEXP (x, 0) && op1 == XEXP (x, 1))
>>      return x;
>>       return simplify_gen_binary (code, mode, op0, op1);
>
> Is there a reason you use simplify_replace_fn_rtx here, rather than
> just using op0 = simplify_rtx (XEXP (x, 0))?  Ditto for op1.
> Does simplify_replace_fn_rtx do something that simplify_rtx doesn't?

I was just quoting code from simplify_replace_fn_rtx as an example of
something that handles a similar situation.  The recursive calls would
be different for cse_process_notes_1.

Sorry for the confusion :-)

Richard

Reply via email to