Timeline for answer to Near constant time rotate that does not violate the standards by Jarod42
Current License: CC BY-SA 3.0
Post Revisions
3 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Aug 17, 2015 at 20:40 | comment | added | Peter Cordes |
@MarkB: You're right, this unfortunately generates terrible asm even on gcc 5.2 and clang 3.7. godbolt.org/g/wJBthc (godbolt). Also, templating a rotate function seems like a terrible idea unless you just use it as a building block for named wrappers, like rotl32. If you want 32bit rotate, you don't want to accidentally do a 64bit rotate because of the size of your temporary.
|
|
| Jul 20, 2015 at 15:55 | comment | added | Mark B |
Are you sure that the !! doesn't force the compiler to generate a branch?
|
|
| Jul 13, 2015 at 19:16 | history | answered | Jarod42 | CC BY-SA 3.0 |
