[PATCH] rtl cse: Fix PR94740, ICE on testsuite/gcc.dg/sso/t5.c with -mcpu=future -mpcrel -O1
Richard Sandiford
richard.sandiford@arm.com
Tue Apr 28 16:57:48 GMT 2020
Peter Bergner <bergner@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
More information about the Gcc-patches
mailing list