Oleg Endo wrote:
> * 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.
Thanks! I've just applied your patch as revision 179073.
Regards,
kaz
> A few minor nits:
Thanks for the hints.
The attached patch and changelog below should do now.
> BTW, have you done your paper work with FSF?
>
Yep. Took a while but it's done now.
Cheers,
Oleg
2011-09-21 Oleg Endo
* config/sh/sh.c (andcosts): Renamed to and_xor_ior_c
Oleg Endo 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:
>
Hello,
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.
Tested with make -k check RUNTESTFLAGS="
