Timeline for C++ PEXT/PDEP implementations
Current License: CC BY-SA 4.0
7 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Nov 29, 2022 at 15:53 | comment | added | Glenn Teitelbaum | The original used a macro, I directly used constexpr | |
| Aug 20, 2022 at 8:31 | comment | added | Toby Speight | I don't see any macro. Did you intend to write "the constexpr keyword" there? | |
| Aug 19, 2022 at 12:46 | comment | added | Glenn Teitelbaum | ownership can be discussed here: codereview.meta.stackexchange.com/questions/10829/… | |
| Aug 19, 2022 at 5:49 | comment | added | G. Sliepen | Yes, I did see the difference in assembly output, and with your changes it is indeed a bit better. However, if you are not the author of the code you post, it's not considered on-topic for CodeReview.. | |
| Aug 18, 2022 at 23:25 | comment | added | Glenn Teitelbaum | Its the result of quite a bit of optimizing. Take a look at godbolt.org/z/3TqeerMzc | |
| Aug 18, 2022 at 20:01 | comment | added | G. Sliepen | So it looks like you took Matthew Fioravante's version and turned the for-loop in a do-while loop, and the if into a ternary? I'm not really sure this is suitable for Code Review. | |
| Aug 18, 2022 at 17:24 | history | asked | Glenn Teitelbaum | CC BY-SA 4.0 |