http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #14 from Oleg Endo olegendo at gcc dot gnu.org 2012-07-23
22:54:13 UTC ---
Author: olegendo
Date: Mon Jul 23 22:54:06 2012
New Revision: 189796
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189796
Log:
PR target/53511
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
Oleg Endo olegendo at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #13 from Oleg Endo olegendo at gcc dot gnu.org 2012-06-12
18:25:46 UTC ---
Author: olegendo
Date: Tue Jun 12 18:25:40 2012
New Revision: 188471
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=188471
Log:
PR target/53511
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #12 from Oleg Endo olegendo at gcc dot gnu.org 2012-06-11
19:24:24 UTC ---
Author: olegendo
Date: Mon Jun 11 19:24:20 2012
New Revision: 188396
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=188396
Log:
PR target/53511
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #6 from Kazumoto Kojima kkojima at gcc dot gnu.org 2012-06-05
09:21:49 UTC ---
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
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #7 from Oleg Endo olegendo at gcc dot gnu.org 2012-06-05 20:08:08
UTC ---
(In reply to comment #4)
Make -mfused-madd no-op instead to remove for the backward
compatibility.
Sorry, I don't quite follow. According to my
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
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #9 from Kazumoto Kojima kkojima at gcc dot gnu.org 2012-06-05
22:35:28 UTC ---
(In reply to comment #7)
Making -mfused-madd a no-op with special handling will only differ in the (*)
marked case. Is this what you had in mind?
Yes.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #10 from Oleg Endo olegendo at gcc dot gnu.org 2012-06-05
23:06:33 UTC ---
(In reply to comment #9)
Yes. -mfused-madd is for a micro optimization after all. It would
be tolerable even if it can't generate fmac insns in the case
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #11 from Kazumoto Kojima kkojima at gcc dot gnu.org 2012-06-05
23:49:48 UTC ---
(In reply to comment #10)
Yeah, probably. I also don't care that much which option is on/off by
default.
I just wanted it to be aligned with the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
Oleg Endo olegendo at gcc dot gnu.org changed:
What|Removed |Added
Attachment #27547|0 |1
is
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
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
Oleg Endo olegendo at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #1 from Oleg Endo olegendo at gcc dot gnu.org 2012-06-03 18:01:31
UTC ---
Created attachment 27547
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=27547
Propo
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
Oleg Endo olegendo at gcc dot gnu.org changed:
What|Removed |Added
CC||kkojima at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511
--- Comment #3 from Kazumoto Kojima kkojima at gcc dot gnu.org 2012-06-03
22:48:17 UTC ---
(In reply to comment #2)
It would be better to ask these generic issues to the experts
on the gcc list, I think. About the SH specific one,
Effectively,
16 matches
Mail list logo
