-
Notifications
You must be signed in to change notification settings - Fork 67
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 "std::byte" support to stream #183
base: main
Are you sure you want to change the base?
Conversation
Since we encourage people to migrate from the old `sycl::byte` to `std::byte`, I presume we want to support `std::byte` in the `stream` class.
Interesting, are bytes streamable? The example in https://en.cppreference.com/w/cpp/types/byte suggests you need to convert it to an integer first. |
Good point. |
Yes, I agree that this is a good point. I think there should still be some way for applications to emit the value of an
For that to be legal, we need to say someplace that Should we add a new section that lists functions and types from the standard C++ library that are safe in device code? It seems like the typical implementation of the functions related to |
stream op for byte does not align with C++ |
2021/09/02 SYCL Khronos committee meetingWaiting for new wording from @gmlueck. |
awaiting draft for NEXT or KHR |
Since we encourage people to migrate from the old
sycl::byte
tostd::byte
, I presume we want to supportstd::byte
in thestream
class.