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

--- Comment #8 from Oleg Endo <olegendo at gcc dot gnu.org> 2012-06-05 20:12:51 
UTC ---
(In reply to comment #6)
> Looking into the __builtin_sh_media_FMAC_S implementation, it takes
> 2 floating arguments and one return float value.  It looks that
> "a = __builtin_sh_media_FMAC_S (b, c)" was expected to work.  I guess
> that this is broken in concept.  

I really wonder what the expected result should be in this case ;)

> We can make it take 3 arguments but
> I suspect that no one has used it at all.  It might be better to
> simply remove this intrinsic because now your patch makes fma usable
> naturally.

OK, I will just remove this built-in in the final patch.

Reply via email to