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

[SYCL] Remove deprecated buffer interop case after removal #1175

Conversation

steffenlarsen
Copy link

Removed the deprecated SYCL 1.2.1 interop constructor for buffer. This causes buffer_interop.cpp to fail in a selection of cases. This commit removes the failing cases.

Removed the deprecated SYCL 1.2.1 interop constructor for buffer. This
causes buffer_interop.cpp to fail in a selection of cases. This commit
removes the failing cases.

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen steffenlarsen requested a review from a team as a code owner August 23, 2022 20:21
Copy link

@aelovikov-intel aelovikov-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume SYCL2020 interop feature had its own tests and hence these can simply be removed.

@steffenlarsen
Copy link
Author

I assume SYCL2020 interop feature had its own tests and hence these can simply be removed.

SYCL/Basic/interop/construction_ocl.cpp, SYCL/Plugin/interop-opencl.cpp, and SYCL/Plugin/interop-level-zero-buffer.cpp should cover sycl::make_buffer for OpenCL and L0. I would expect CTS to also cover the OpenCL side.

@romanovvlad
Copy link

I think we will remove rest of the test together with the program class.

@romanovvlad
Copy link

romanovvlad commented Aug 24, 2022

The run lines could be updated to:

// REQUIRES: opencl
// RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -D__SYCL_INTERNAL_API -o %t.out

@steffenlarsen steffenlarsen merged commit dcec231 into intel:intel Aug 24, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
Removed the deprecated SYCL 1.2.1 interop constructor for buffer. This
causes buffer_interop.cpp to fail in a selection of cases. This commit
removes the failing cases.

Signed-off-by: Larsen, Steffen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…m-test-suite#1175)

Removed the deprecated SYCL 1.2.1 interop constructor for buffer. This
causes buffer_interop.cpp to fail in a selection of cases. This commit
removes the failing cases.

Signed-off-by: Larsen, Steffen <[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.

3 participants