[PATCH] sh target: bogus operand reference

Kaz Kojima kkojima@rr.iij4u.or.jp
Sun Dec 2 02:10:00 GMT 2007


Samuel Tardieu <sam@rfc1149.net> wrote:
> 	* config/sh/sh.md (cmpgeusi_t): Immediate operand is operand 1,
> 	not 0.
> 
> Compilation regression tested on a cross-compiler from i686-linux to
> sh4-linux.
> 
> Ok for trunk?

The patch itself is fine.  Thanks!

Our ChangeLog entry should say how the code is changed, not why.
"Fix condition." or "Fix condition so as to check operand 1."
would be ok.  Ok for trunk with such change.

Regards,
	kaz



More information about the Gcc-patches mailing list