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

gpu: amd: Enable SYCL kernels #2024

Merged
merged 4 commits into from
Aug 9, 2024

Conversation

sgeor255
Copy link
Contributor

@sgeor255 sgeor255 commented Aug 5, 2024

Description

This PR enables the generic SYCL kernels on intel & AMD backends.

For AMD backend, a new internal cmake option was added (DNNL_HIP_TARGET_ARCH) used to specify the AMD architecture to compile the kernels for. If the option is not passed, the kernels are not compiled and not added to the implementation lists.

@sgeor255 sgeor255 requested review from a team as code owners August 5, 2024 14:17
@sgeor255 sgeor255 changed the title gpu: sycl: bugfix kernel includes gpu: amd: intel: Enable SYCL kernels Aug 5, 2024
@vpirogov vpirogov added this to the v3.6 milestone Aug 6, 2024
@vpirogov vpirogov added platform:gpu-amd Codeowner: @oneapi-src/onednn-gpu-amd platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel labels Aug 6, 2024
@sgeor255
Copy link
Contributor Author

sgeor255 commented Aug 6, 2024

I think the failing gtests on CI are not related to the changes in this PR. I am seeing similar failures on master when I run the tests locally.

cmake/options.cmake Outdated Show resolved Hide resolved
cmake/options.cmake Outdated Show resolved Hide resolved
cmake/options.cmake Outdated Show resolved Hide resolved
src/gpu/generic/CMakeLists.txt Outdated Show resolved Hide resolved
src/gpu/generic/CMakeLists.txt Outdated Show resolved Hide resolved
examples/CMakeLists.txt Outdated Show resolved Hide resolved
cmake/options.cmake Outdated Show resolved Hide resolved
cmake/options.cmake Outdated Show resolved Hide resolved
src/gpu/gpu_impl_list.hpp Outdated Show resolved Hide resolved
src/gpu/gpu_shuffle_list.cpp Outdated Show resolved Hide resolved
@sgeor255 sgeor255 requested review from a team as code owners August 7, 2024 15:42
@sgeor255 sgeor255 force-pushed the svet/enable-sycl-kernels branch from fe16e2e to f12a48e Compare August 7, 2024 15:44
@sgeor255 sgeor255 changed the title gpu: amd: intel: Enable SYCL kernels gpu: amd: Enable SYCL kernels Aug 8, 2024
@sgeor255 sgeor255 force-pushed the svet/enable-sycl-kernels branch from f12a48e to 9a4f95e Compare August 8, 2024 16:35
@sgeor255 sgeor255 requested review from a team as code owners August 8, 2024 16:35
@vpirogov vpirogov removed request for a team August 8, 2024 20:08
@densamoilov densamoilov merged commit d4d58cf into oneapi-src:main Aug 9, 2024
0 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform:gpu-amd Codeowner: @oneapi-src/onednn-gpu-amd platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants