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
Implement parallel::merge. #2833
Conversation
…tainer algorithms.
1. Add some assertions. 2. Add some comments. 3. Fix inspection errors. 4. Remove non-meaning comments. 5. Remove non-meaning codes.
…tainer version of parallel::partition.
@hkaiser When will this PR merged?? |
@taeguk slowly catching up on things... |
@hkaiser Okay good :) I just worried that you forgot this PR. |
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.
LGTM, thanks!
@hkaiser Were there some improvements in runtime system of HPX since after PR was merged? |
@taeguk Everything is possible ;-) I wouldn't know from the top of my head however what was changed that may enable the performance improvements you are seeing. |
Check Box
Issues
-> For now, restrict the requirements of parallel::merge to only random access iterator.
Note for future