-
Notifications
You must be signed in to change notification settings - Fork 795
[SYCL][FPGA][NFC] LSU builtin test and documentation update #3546
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
Conversation
romanovvlad
left a comment
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, but would like that @mlychkov takes a look as well.
mlychkov
left a comment
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.
|
merged to get this change: #3578 and enable the checks to pass. |
|
@bader Anything pending for the merge? |
|
@intel/dpcpp-specification-reviewers team should approve the spec. |
Updating documentation to reflect change to multi_ptr.
Updating test so it tests device, host ptrs as well as accessor method.