subject:"\[Bug target\/52624\] missing __builtin

[Bug target/52624] missing __builtin_bswap16

2012-04-11 Thread ebotcazou at gcc dot gnu.org

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 Eric Botcazou changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|

[Bug target/52624] missing __builtin_bswap16

2012-04-11 Thread ebotcazou at gcc dot gnu.org

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #11 from Eric Botcazou 2012-04-11 11:13:43 UTC --- Author: ebotcazou Date: Wed Apr 11 11:13:39 2012 New Revision: 186308 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186308 Log: PR target/52624 * doc/extend.texi (Oth

[Bug target/52624] missing __builtin_bswap16

2012-03-22 Thread ubizjak at gmail dot com

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #10 from Uros Bizjak 2012-03-22 16:59:43 UTC --- (In reply to comment #9) > Do we need to optimize for partial register stall? xchg is enabled only for Pentium4, and this is not partial reg stall target. BTW: According to the docs,

[Bug target/52624] missing __builtin_bswap16

2012-03-22 Thread hjl.tools at gmail dot com

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #9 from H.J. Lu 2012-03-22 16:24:41 UTC --- Do we need to optimize for partial register stall?

[Bug target/52624] missing __builtin_bswap16

2012-03-22 Thread ebotcazou at gcc dot gnu.org

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #8 from Eric Botcazou 2012-03-22 10:04:44 UTC --- > Please note that the rotate is split back into bswap, so we can avoid rotate > by > using "xchg %rh, %rl" on P4. Sure, 16-bit rotates are already emitted as xchg when appropriate.

[Bug target/52624] missing __builtin_bswap16

2012-03-22 Thread ubizjak at gmail dot com

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #7 from Uros Bizjak 2012-03-22 09:56:18 UTC --- (In reply to comment #6) > I think that the builtin should be expanded into a rotate (left or right) if > they are available. On x86 this works out of the box since the rotates are > t

[Bug target/52624] missing __builtin_bswap16

2012-03-22 Thread ebotcazou at gcc dot gnu.org

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 Eric Botcazou changed: What|Removed |Added Target Milestone|--- |4.8.0 Summary|Missing __builti

[Bug target/52624] Missing __builtin_bswap16

2012-03-22 Thread ebotcazou at gcc dot gnu.org

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 Eric Botcazou changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|unassigned at

[Bug target/52624] Missing __builtin_bswap16

2012-03-22 Thread ubizjak at gmail dot com

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #5 from Uros Bizjak 2012-03-22 07:51:20 UTC --- (In reply to comment #4) > I think that it should be available on all architectures, like the 32-bit and > 64-bit flavors. And, for x86, you don't really need to add new patterns. I ag

[Bug target/52624] Missing __builtin_bswap16

2012-03-22 Thread ebotcazou at gcc dot gnu.org

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #4 from Eric Botcazou 2012-03-22 07:41:25 UTC --- I think that it should be available on all architectures, like the 32-bit and 64-bit flavors. And, for x86, you don't really need to add new patterns.

[Bug target/52624] Missing __builtin_bswap16

2012-03-21 Thread ubizjak at gmail dot com

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 --- Comment #3 from Uros Bizjak 2012-03-21 23:25:37 UTC --- Created attachment 26946 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26946 Patch that implements __builtin_bswap16 Untested patch.