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

Showcase OpenCL #250

Closed
wants to merge 1 commit into from
Closed

Conversation

jerryct
Copy link

@jerryct jerryct commented Sep 19, 2021

Hi. The PR in its current state is only to start a discussion about a potential change to run OpenCL kernels. See #251

@jerryct jerryct marked this pull request as draft September 19, 2021 16:13
@axsaucedo
Copy link
Member

This is really interesting @jerryct, thank you for opening the PR to initiate the discussion. Before diving into the discussion could you also create an issue that points to this PR so we can have a way to track on the issues as well?

In regards to your current suggetsions, it seems quite interesting that it does seem to work with OpenCL kernels that compile into SpirV, very cool - is the current feature request to just allow changing the name of the "main" entrypoint? If that is the case, this should be a relatively simple extension that could be exposed through helper / property functions.

@jerryct
Copy link
Author

jerryct commented Sep 19, 2021

I created this issue #251 and moved most of the description from this PR.

is the current feature request to just allow changing the name of the "main" entrypoint?

Yes, this is the only change to make the above code in the issue work.

@axsaucedo
Copy link
Member

Closing to continue discussion through issue when picked up

@axsaucedo axsaucedo closed this Dec 3, 2022
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