Skip to content

[SYCL][E2E] Fix online compiler test for accelerator #15270

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

Merged

Conversation

steffenlarsen
Copy link
Contributor

This commit fixes an issue where the online_compiler_OpenCL test would assume that the clCreateProgramWithIL function is supported by all OpenCL targets, while it was only supported with >=2.1 and had an extension in older versions. This also enables the test for accelerator.

This commit fixes an issue where the online_compiler_OpenCL test would
assume that the clCreateProgramWithIL function is supported by all
OpenCL targets, while it was only supported with >=2.1 and had an
extension in older versions. This also enables the test for accelerator.

Signed-off-by: Larsen, Steffen <[email protected]>
Signed-off-by: Larsen, Steffen <[email protected]>
Copy link
Contributor

@cperkinsintel cperkinsintel left a comment

Choose a reason for hiding this comment

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

we are mostly deprecating the oneline-compiler in favor of the kernel_compiler.

I wonder if when we can remove it? It was always experimental.

@steffenlarsen
Copy link
Contributor Author

we are mostly deprecating the oneline-compiler in favor of the kernel_compiler.

I wonder if when we can remove it? It was always experimental.

Do we have it deprecated currently? If not, I think first step should be to point users to the replacement.

@steffenlarsen steffenlarsen merged commit 729d6f6 into intel:sycl Sep 11, 2024
13 checks passed
@steffenlarsen steffenlarsen deleted the steffen/fix_opencl_compiler_test_acc branch September 11, 2024 09:22
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.

2 participants