Skip to content

Enable OpenCL Support on Windows #3230

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

Closed
wants to merge 2 commits into from

Conversation

djeong20
Copy link
Contributor

@djeong20 djeong20 commented Jun 2, 2025

This PR enables the use of OpenCL on Windows.
Please note that OpenCL must be installed in order to use OpenCL.

djeong20 added 2 commits June 2, 2025 19:29
This commit fixes OpenCL library name to properly support on Windows.

**Self-evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test:   [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Donghyeon Jeong <dhyeon.jeong@samsung.com>
This commit temporary disable CLBlast build on Windows. Currently, Meson subproject CMake build for CLBlast is throwing an error. This can be resolved after nnstreamer#3196.

**Self-evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test:   [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Donghyeon Jeong <dhyeon.jeong@samsung.com>
@dkjung
Copy link
Collaborator

dkjung commented Jun 3, 2025

I think we need to check if CI Windows build test passes with -Denable-opencl = true is given for meson setup. How about postpone merging this PR until then?

Copy link
Collaborator

@dkjung dkjung left a comment

Choose a reason for hiding this comment

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

Let's discuss about this PR more face to face.

Comment on lines +47 to +50

#if defined(_WIN32)
static const char *kClLibName = "OpenCL.dll";
#else
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thank you for the bug fix

Copy link
Collaborator

@dkjung dkjung left a comment

Choose a reason for hiding this comment

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

This disables clblast for Windows.

@dkjung
Copy link
Collaborator

dkjung commented Jun 4, 2025

I think I got to know that how to set dependency clblast on a Windows machine. Have a chat later.

Copy link
Collaborator

@dkjung dkjung left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 3 days.

@github-actions github-actions bot added the Stale label Jun 18, 2025
Copy link

This PR was closed because it has been stalled for 3 days with no activity.

@github-actions github-actions bot closed this Jun 22, 2025
@djeong20 djeong20 deleted the windows/opencl branch July 9, 2025 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants