On Wed, Mar 22, 2023 at 11:18:33AM +0100, Richard Biener via Gcc-patches wrote:
> Is/was the code ever exercised for non-x86?  HJ, what was the reason to
> abstract this?

Initially the hook looked like
rtx
ix86_gen_scratch_sse_rtx (machine_mode mode)
{
  if (TARGET_SSE)
    return gen_rtx_REG (mode, (TARGET_64BIT
                              ? LAST_REX_SSE_REG
                              : LAST_SSE_REG));
  else
    return gen_reg_rtx (mode);
}
on x86, but then PR104704 changed it to the same definition as the default
hook.  So I think it is fine to remove it.

> OK if HJ thinks it was really unnecessary abstraction unlikely to be
> required by another target.

        Jakub

Reply via email to