Oleg Endo <oleg.e...@t-online.de> wrote:
> The attached patch does pretty much the same for the SH target.
> Tested also by setting LOG_COSTS to 1 and checking that multi-word modes
> are marked for splitting (except for DImode zero_extend lowering).
> Also verified that newlib compiles again.
> 
> OK?
> 
> Cheers,
> Oleg
> 
> ChangLog:
> 
>       PR target/53250
>       * config/sh/sh.c (sh_rtx_costs): Handle SET case to restore 
>       original behavior of lower-subreg.

Looks fine, though the terser ChangeLog entry would be better.
We usually don't include how and why into there.  MIPS's 
"(mips_rtx_costs): Handle SET." is enough, I think.
Ok with that change.  Thanks for fixing this!

Regards,
        kaz

Reply via email to