Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

P0553R4 <bit> Rotating And Counting Functions #25

Closed
StephanTLavavej opened this issue Sep 6, 2019 · 5 comments
Closed

P0553R4 <bit> Rotating And Counting Functions #25

StephanTLavavej opened this issue Sep 6, 2019 · 5 comments
Assignees
Labels
cxx20 C++20 feature fixed Something works now, yay!
Milestone

Comments

@StephanTLavavej
Copy link
Member

P0553R4 <bit> Rotating And Counting Functions

This may need compiler support.

@crackedmind
Copy link
Contributor

How i can see, MSVC supports __builtin_clz/__builtin_popcount and some other helpers since 19.23. Can we start start to implement this? :)

@sylveon
Copy link
Contributor

sylveon commented Oct 25, 2019

There's _BitScanReverse for clz. Not sure if constexpr though.

@crackedmind
Copy link
Contributor

crackedmind commented Oct 25, 2019

There's _BitScanReverse for clz. Not sure if constexpr though.

if you will grep c1xx.dll from vc >= 19.23 you can find __builtin_clz and other functions.

And it's work. Tested on local compiler-explorer instance for constexpr functions

@StephanTLavavej
Copy link
Member Author

@barcharcraz is actively working on this. Charlie, would it be useful to submit a draft PR so people can see what you’ve been working on, as I did for Casey’s <coroutine> that I revised?

@crackedmind
Copy link
Contributor

@barcharcraz is actively working on this. Charlie, would it be useful to submit a draft PR so people can see what you’ve been working on, as I did for Casey’s <coroutine> that I revised?

How about lables 'i started' in issues or maybe WIP PRs? I think it can be useful. Specialy, if you start it iternal.

@StephanTLavavej StephanTLavavej added fixed Something works now, yay! and removed work in progress labels Nov 19, 2019
SuperWig pushed a commit to SuperWig/STL that referenced this issue Nov 21, 2019
Resolves microsoft#25 and resolves microsoft#26. Currently active for Clang and EDG, but not C1XX.
@cbezault cbezault added this to the Conformance milestone May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cxx20 C++20 feature fixed Something works now, yay!
Projects
None yet
Development

No branches or pull requests

5 participants