-
Notifications
You must be signed in to change notification settings - Fork 30
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
[SYCLomatic] Adding dpct::null_type test #219
[SYCLomatic] Adding dpct::null_type test #219
Conversation
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
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, can you please rebase this patch?
Signed-off-by: Dan Hoeflinger <[email protected]>
Signed-off-by: Dan Hoeflinger <[email protected]>
Signed-off-by: Dan Hoeflinger <[email protected]>
67cb0bc
to
92118a4
Compare
@yihanwg done |
The failing test for onedpl_test_copy_if is unrelated to this PR. At first glance, it looks related to what should be resolved by #243. |
Adding example usage of
dpct::null_type
.It is intended to be used to indicate that the argument is unused and does not need to be acted upon. Here we are redirecting to a key swap instead of a pair swap using metaprogramming when
dpct::null_type
is used for the value type.This test requires oneapi-src/SYCLomatic#528 to be merged.