Skip to content
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

[SYCL] Throw exceptions while creating sub buffer #822

Merged

Conversation

KarachunIvan
Copy link
Contributor

Throw invalid_object_error exception if memory region of
constructed sub buffer specified by offset and sub buffer size
doesn't represent contiguous region or if it exceeds parent bounds.

Signed-off-by: Ivan Karachun ivan.karachun@intel.com

@bader
Copy link
Contributor

bader commented Nov 12, 2019

"Throw exceptions while creating sub buffer" - it says that I can't create sub-buffer!

@KarachunIvan KarachunIvan force-pushed the private/ivankara/subbuffer_reinterpret_elements branch 2 times, most recently from 8e027c6 to 040307f Compare November 12, 2019 10:54
@KarachunIvan
Copy link
Contributor Author

"Throw exceptions while creating sub buffer" - it says that I can't create sub-buffer!

Sure, the commit title is a bit confusing. Changed.

Throw `invalid_object_error` exception if memory region of
constructed sub buffer specified by offset and sub buffer size
doesn't represent contiguous region or if it exceeds parent bounds.

Signed-off-by: Ivan Karachun <ivan.karachun@intel.com>
@KarachunIvan KarachunIvan force-pushed the private/ivankara/subbuffer_reinterpret_elements branch from 040307f to 102ffdf Compare November 13, 2019 11:14
@bader bader merged commit 0bd58df into intel:sycl Nov 14, 2019
@KarachunIvan KarachunIvan deleted the private/ivankara/subbuffer_reinterpret_elements branch December 6, 2019 14:06
vmaksimo pushed a commit to vmaksimo/llvm that referenced this pull request Nov 30, 2020
Spec: KhronosGroup/SPIRV-Registry#85

Signed-off-by: amochalo <anastasiya.mochalova@intel.com>
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.

6 participants