Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Remove test case using host accessor in interop_handle::get_na… #269

Merged

Conversation

v-klochkov
Copy link

…tive_mem()

This check verifies a situation that is always wrong and causing an exception
at runtime. The patch intel/llvm#3685 removes
the method throwing exception at runtime and lets see the error at compile
time.

The temporarily removed test6() will be returned and improved in consequent
changes: #263
Removal of test6() is needed to have a clean CI run for #3685

Signed-off-by: Vyacheslav N Klochkov [email protected]

…tive_mem()

This check verifies a situation that is always wrong and causing an exception
at runtime. The patch intel/llvm#3685 removes
the method throwing exception at runtime and lets see the error at compile
time.

The temporarily removed test6() will be returned and improved in consequent
changes: intel#263
Removal of test6() is needed to have a clean CI run for #3685

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
@v-klochkov v-klochkov requested a review from againull May 5, 2021 22:27
@v-klochkov v-klochkov marked this pull request as ready for review May 5, 2021 22:28
@v-klochkov v-klochkov requested a review from a team as a code owner May 5, 2021 22:28
@v-klochkov v-klochkov merged commit 8fafb9e into intel:intel May 5, 2021
@v-klochkov v-klochkov deleted the public_vklochkov_interop_handle_tmp branch May 5, 2021 23:34
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…tive_mem() (intel/llvm-test-suite#269)

This check verifies a situation that is always wrong and causing an exception
at runtime. The patch intel#3685 removes
the method throwing exception at runtime and lets see the error at compile
time.

The temporarily removed test6() will be returned and improved in consequent
changes: intel/llvm-test-suite#263
Removal of test6() is needed to have a clean CI run for intel#3685

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants