[Bug target/55295] [SH] Add support for fipr instruction

2024-09-21 Thread olegendo at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #18 from Oleg Endo --- A rather recent case and request to add a __builtin_fipr function, when trying to optimize quaternion multiplication. This one includes hand-written inline asm and register pre-allocation. https://godbolt.org/

[Bug target/55295] [SH] Add support for fipr instruction

2023-03-21 Thread olegendo at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #17 from Oleg Endo --- (In reply to Luke Benstead from comment #16) > OK so perhaps adding __builtin_sh_fipr is a good first step? Yeah, you can try and see if it produces any useful results for you.

[Bug target/55295] [SH] Add support for fipr instruction

2023-03-21 Thread kazade at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #16 from Luke Benstead --- OK so perhaps adding __builtin_sh_fipr is a good first step?

[Bug target/55295] [SH] Add support for fipr instruction

2023-03-21 Thread olegendo at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #15 from Oleg Endo --- It's been too long since I've looked into it. Maybe some middle-end parts got more suitable over the time, but it was difficult to make it generate the fipr instruction automatically due to the reasons stated a

[Bug target/55295] [SH] Add support for fipr instruction

2023-03-21 Thread kazade at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 Luke Benstead changed: What|Removed |Added CC||kazade at gmail dot com --- Comment #14

[Bug target/55295] [SH] Add support for fipr instruction

2015-03-02 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #13 from Oleg Endo --- (In reply to Manu Evans from comment #12) > Hey, I'm still following this with great interest. > > Is it possible to make an intrinsic for this instruction so it can be issued > at will? Yes, that's what I wan

[Bug target/55295] [SH] Add support for fipr instruction

2015-03-01 Thread turkeyman at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #12 from Manu Evans --- Hey, I'm still following this with great interest. Is it possible to make an intrinsic for this instruction so it can be issued at will? What I'm still more interested in at this point, would be some support

[Bug target/55295] [SH] Add support for fipr instruction

2015-03-01 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #11 from Oleg Endo --- A note on the side... As mentioned above, fipr can also be used to do a 3D dot product. However, GCC's vector extensions do not allow specifying vectors of length 3. To support that I guess the easiest way is

[Bug target/55295] [SH] Add support for fipr instruction

2014-12-09 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #10 from Oleg Endo --- (In reply to Oleg Endo from comment #9) > Created attachment 34213 [details] > Combine patterns for matching fipr > > An updated patch for trunk. As for the redundant fp moves and/or ferries > through fpul, th

[Bug target/55295] [SH] Add support for fipr instruction

2014-12-07 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 Oleg Endo changed: What|Removed |Added Attachment #28671|0 |1 is obsolete|

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-13 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #8 from Oleg Endo 2013-03-13 18:21:37 UTC --- (In reply to comment #5) > > This is another reason for adding a new ABI, BTW. Just for the record, I've opened a new PR 56592 for this.

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-05 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #7 from Oleg Endo 2013-03-06 01:05:14 UTC --- (In reply to comment #5) > > > > I'm also noticing that -ffast-math is inhibiting fmac emission in some > > cases: > > > > Compiled with: -O3 -mfused-madd -ffast-math > > >

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-05 Thread turkeyman at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #6 from Manu Evans 2013-03-05 12:53:26 UTC --- Awesome, thanks for the info and help! Strange -m4-single won't work with my toolchain, it says 'not compatible with this configuration' >_< Looking forward to all these fixes

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-05 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #5 from Oleg Endo 2013-03-05 12:28:22 UTC --- (In reply to comment #4) > > Why is a new ABI important? > Because currently, there is no way to pass something like struct { float x, y, z, w }; as function arguments i

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-04 Thread turkeyman at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #4 from Manu Evans 2013-03-05 01:55:08 UTC --- (In reply to comment #3) > (In reply to comment #2) > > +1 > > > > I'm seeing the same pattern. > > Infact, I'm noticing a lot of my maths code seems to be performing a lot of

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-04 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #3 from Oleg Endo 2013-03-04 21:50:58 UTC --- (In reply to comment #2) > +1 > > I'm seeing the same pattern. > Infact, I'm noticing a lot of my maths code seems to be performing a lot of > redundant moves. Some examples

[Bug target/55295] [SH] Add support for fipr instruction

2013-03-04 Thread turkeyman at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 Manu Evans changed: What|Removed |Added CC||turkeyman at gmail dot com --- Com

[Bug target/55295] [SH] Add support for fipr instruction

2012-11-12 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55295 --- Comment #1 from Oleg Endo 2012-11-12 22:39:27 UTC --- I forgot to mention that at least there should be a target specific built-in function to generate the fipr insn. There is already a SHmedia built-in for that, so adding one for SH4