Skip to content

Conversation

zibaiwan
Copy link
Contributor

@zibaiwan zibaiwan commented Mar 27, 2023

A continuation of #5838. Accompanying runtime change is #7468.

Add a memory order parameter to device-side read/write members and
default to sycl::memory_order::seq_cst. This parameter is in place but not being used at this moment, it's intended for the future work.

Add host pipe read/write members with additional sycl::queue parameter.

@zibaiwan zibaiwan marked this pull request as ready for review March 27, 2023 20:37
@zibaiwan zibaiwan requested a review from a team as a code owner March 27, 2023 20:37
@zibaiwan zibaiwan marked this pull request as draft March 27, 2023 20:37
@zibaiwan zibaiwan marked this pull request as ready for review March 28, 2023 14:09
@zibaiwan
Copy link
Contributor Author

@rho180 , can you please take a look at this Spec?

Copy link
Contributor

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

Thanks @zibaiwan! LGTM!

@zibaiwan
Copy link
Contributor Author

@steffenlarsen , thanks for your review. The test passed for the doc change.

@bader bader requested review from gmlueck and rho180 March 29, 2023 22:07
@steffenlarsen steffenlarsen merged commit 6b2d66b into intel:sycl Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants