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
Fix tuple serialization #2505
Fix tuple serialization #2505
Conversation
@@ -174,7 +174,7 @@ namespace hpx { namespace parcelset { namespace policies { namespace mpi | |||
int ret = 0; | |||
ret = MPI_Test(&r, &completed, status); | |||
HPX_ASSERT(ret == MPI_SUCCESS); | |||
if(completed)// && status->MPI_ERROR != MPI_ERR_PENDING) | |||
if(completed) |
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.
What's the rationale of this change?
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.
Just deleting the comment. The code was never used and is not required.
struct is_bitwise_serializable< | ||
::hpx::util::tuple<Ts...> | ||
::hpx::util::detail::tuple_impl<Is, Ts...> |
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.
Excellent catch!
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!
085a509
to
a4c4982
Compare
Only tuple_impl<Is, Ts...> should be bitwise serialized
- Removed commented code - Removed unnecessary call to request_done - Removing impossible check in tag_provider
a4c4982
to
62a16f8
Compare
Only tuple_impl<Is, Ts...> should be bitwise serialized
Flyby: Cleaning up MPI parcelport.