-
-
Notifications
You must be signed in to change notification settings - Fork 430
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
Adding parallel make_heap #4964
Conversation
Very cool to see this being revisited! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I take back my previous review. It's just some names that need to be updated though.
libs/parallelism/algorithms/include/hpx/parallel/algorithms/make_heap.hpp
Outdated
Show resolved
Hide resolved
Hmm, this is maybe not rebased on master? I think the CircleCI failure is just because of that. The jenkins builders build a merge with master. |
I rebased this again, let's see. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @hkaiser, looks good now!
This revives #1914, which has never made it in. This also converts the algorithm to CPOs and makes it conforming to C++20. Thanks again @Syntaf for working on this and sorry it took so long to get back to it.