Skip to content

Conversation

AlexGuteniev
Copy link
Contributor

@AlexGuteniev AlexGuteniev commented Oct 4, 2021

The paper misses __cpp_lib_byteswap value, picked the current date as a speculation

Fixes #2247.

@AlexGuteniev AlexGuteniev requested a review from a team as a code owner October 4, 2021 18:14
@CaseyCarter CaseyCarter added the cxx23 C++23 feature label Oct 4, 2021
@StephanTLavavej StephanTLavavej self-assigned this Oct 13, 2021
@StephanTLavavej StephanTLavavej removed their assignment Oct 19, 2021
@StephanTLavavej
Copy link
Member

@AlexGuteniev @CaseyCarter Looks good, thanks. I pushed a couple of tiny changes - please meow if you disagree!

@StephanTLavavej StephanTLavavej self-assigned this Oct 19, 2021
@StephanTLavavej
Copy link
Member

I'm mirroring this to an MSVC-internal PR; please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit a6718c2 into microsoft:main Oct 20, 2021
@StephanTLavavej
Copy link
Member

🚀 😻 🎉 !sknahT

@AlexGuteniev AlexGuteniev deleted the bswap branch October 20, 2021 10:29
AZero13 pushed a commit to AZero13/STL that referenced this pull request Nov 4, 2021
Co-authored-by: Stephan T. Lavavej <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cxx23 C++23 feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

P1272R4 byteswap()
4 participants