Oleg Endo <oleg.e...@t-online.de> wrote:
> the attached patch corrects the costs calculation for AND / OR / XOR
> operations that could be done with an 8 bit immediate value and R0.
> The change doesn't seem to have an impact on the generated code but will
> be (probably) required to fix PR 49263.
A few minor nits:
> * config/sh/sh.c (andcosts): Renamed to and_xor_ior_costs.
> Added AND special case. Adapted comments.
> (sh_rtx_costs): Added XOR and IOR case.
Please use a tab instead of 8 spaces for ChangeLog entry.
Use two spaces between the period and the next sentence.
>- /* Anding with a register is a single cycle and instruction. */
>+ /* register x register is a single cycle instruction. */
s/register x register/A logical operation with a register/
would be better.
>+ /* Constants that can be used in an instruction with an immediate are a
>single
> cycle, but this requires r0, so make it a little more expensive. */
The line should be wrapped so it is less than ~75 characters on a line.
OK with those changes.
BTW, have you done your paper work with FSF?
Regards,
kaz
