[Bug target/53511] SH Target: Add support for fma patterns

2012-07-23 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 Oleg Endo changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|

[Bug target/53511] SH Target: Add support for fma patterns

2012-07-23 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #14 from Oleg Endo 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=gcc&view=rev&rev=189796 Log: PR target/53511 * config/sh/sh.md (mulsf3_i

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-12 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #13 from Oleg Endo 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=gcc&view=rev&rev=188471 Log: PR target/53511 * gcc.target/sh/pr51340-1.c

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-11 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #12 from Oleg Endo 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=gcc&view=rev&rev=188396 Log: PR target/53511 * config/sh/sh.md (fmasf4):

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-05 Thread kkojima at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #11 from Kazumoto Kojima 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 other targets and the overa

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-05 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #10 from Oleg Endo 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 like (*). > OTOH deleting th

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-05 Thread kkojima at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #9 from Kazumoto Kojima 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. -mfused-madd is for a mi

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-05 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #8 from Oleg Endo 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_F

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-05 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #7 from Oleg Endo 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 understanding we have something l

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-05 Thread kkojima at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #6 from Kazumoto Kojima 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 should OK in this regard

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-04 Thread kkojima at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #5 from Kazumoto Kojima 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

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-04 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 Oleg Endo changed: What|Removed |Added Attachment #27547|0 |1 is obsolete|

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-03 Thread kkojima at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #3 from Kazumoto Kojima 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, the '*macsf3' / 'mac_medi

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-03 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 Oleg Endo changed: What|Removed |Added CC||kkojima at gcc dot gnu.org --- Comment #2 fro

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-03 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 --- Comment #1 from Oleg Endo 2012-06-03 18:01:31 UTC --- Created attachment 27547 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27547 Propo

[Bug target/53511] SH Target: Add support for fma patterns

2012-06-03 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53511 Oleg Endo changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|