http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511

--- Comment #5 from Kazumoto Kojima <kkojima at gcc dot gnu.org> 2012-06-05 
04:47:22 UTC ---
(In reply to comment #4)
> This patch adds the FMA patterns and aligns the SH target with the 
> -mfused-madd
> / -mno-fused-madd option handling in other targets.

Make -mfused-madd no-op instead to remove for the backward
compatibility.  Other than that, the patch looks fine to me.

> I had to touch the __builtin_sh_media_FMAC_S mapping.  I think it should be 
> OK,
> since the number and semantics of the insn operands is the same as before.

It seems that __builtin_sh_media_FMAC_S is broken on trunk
in the first place, though I can test it only on sh64-elf
environment now.  Anyway the patch should OK in this regard.

Reply via email to