Skip to content

[SYCL] Fix SYCL accessor.get_count() and accessor.get_size() methods. #696

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

alexeyvoronov-intel
Copy link
Contributor

A non-local accessor returned incorrect values when it was created
with the provided range which was different from the buffer range.

Signed-off-by: Alexey Voronov [email protected]

romanovvlad
romanovvlad previously approved these changes Oct 7, 2019
@alexeyvoronov-intel alexeyvoronov-intel force-pushed the private/avoronov/fix_accessor_getters_size_and_count branch from 0891169 to 2176338 Compare October 22, 2019 11:45
@alexeyvoronov-intel
Copy link
Contributor Author

Just rebased.

…ods.

A non-local accessor returned incorrect values when it was created
with the provided range which was different from the buffer range.

Signed-off-by: Alexey Voronov <[email protected]>
@alexeyvoronov-intel alexeyvoronov-intel force-pushed the private/avoronov/fix_accessor_getters_size_and_count branch from 2176338 to 98950bb Compare November 1, 2019 14:56
@bader bader requested a review from romanovvlad November 6, 2019 10:47
@romanovvlad romanovvlad changed the title [SYCL] Fix SYCL accessor.get_count() and accessor.get_size() methods. [DO NOT MERGE] [SYCL] Fix SYCL accessor.get_count() and accessor.get_size() methods. Nov 6, 2019
@alexeyvoronov-intel alexeyvoronov-intel changed the title [DO NOT MERGE] [SYCL] Fix SYCL accessor.get_count() and accessor.get_size() methods. [SYCL] Fix SYCL accessor.get_count() and accessor.get_size() methods. Nov 11, 2019
@bader bader assigned romanovvlad and vladimirlaz and unassigned romanovvlad Nov 11, 2019
@romanovvlad romanovvlad merged commit 9dd68c5 into intel:sycl Nov 11, 2019
@alexeyvoronov-intel alexeyvoronov-intel deleted the private/avoronov/fix_accessor_getters_size_and_count branch November 11, 2019 16:27
vmaksimo pushed a commit to vmaksimo/llvm that referenced this pull request Oct 6, 2020
* Handle @llvm.memset.* with non-constant arguments

There is no SPIR-V counterpart for @llvm.memset.* intrinsic. Cases with constant
value and length arguments are emulated via "storing" a constant array to the
destination.

For other cases we wrap the intrinsic in @spirv.llvm_memset_* function and
expand the intrinsic to a loop via expandMemSetAsLoop() from
llvm/Transforms/Utils/LowerMemIntrinsics.h.

During reverse translation from SPIR-V to LLVM IR we can detect
@spirv.llvm_memset_* and replace it with @llvm.memset.

Signed-off-by: Alexey Sotkin <[email protected]>
Co-authored-by: Sven van Haastregt <[email protected]>
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.

3 participants