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

Add distribution_policy_executor #1701

Merged
merged 2 commits into from Aug 9, 2015
Merged

Conversation

hkaiser
Copy link
Member

@hkaiser hkaiser commented Aug 3, 2015

  • integrates executors with distribution policies
  • adding test

- integrates executors with distribution policies
- adding test
@hkaiser hkaiser force-pushed the distribution_policy_executors branch from 706ed5d to 183e81c Compare August 5, 2015 01:30
auto exec = make_distribution_policy_executor(hpx::colocated(loc));
typedef executor_traits<decltype(exec)> executor_traits;

HPX_TEST_EQ(executor_traits::async_execute(exec, call_pfo()).get(), loc);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this work without a serialize function in call_pfo?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See: https://github.com/STEllAR-GROUP/hpx/blob/master/hpx/runtime/serialization/access.hpp#L66. We are able to serialize empty types without explicit serialization support.

hkaiser added a commit that referenced this pull request Aug 9, 2015
@hkaiser hkaiser merged commit 9806a56 into master Aug 9, 2015
@hkaiser hkaiser deleted the distribution_policy_executors branch August 9, 2015 19:41
hkaiser added a commit that referenced this pull request Aug 10, 2015
…cy_executors"

This reverts commit 9806a56, reversing
changes made to aaa8eb5.

This revert is necessary as the changes break the builds for gcc 4.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants