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

CMD-Buf supported queue properties #126

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Oct 4, 2024

Updates to the command-buffer emulation layer to support changes from KhronosGroup/OpenCL-Docs#850

Requires using KhronosGroup/OpenCL-Headers#265 in external/OpenCL-Headers and can be used to test CTS changes from KhronosGroup/OpenCL-CTS#2101

@bashbaug
Copy link
Owner

bashbaug commented Oct 4, 2024

Thanks, this is great!

Do you want to hold off on this one until the headers changes are merged, or try to add in some #ifdefs so it'll work both with newer and older headers?

Updates to the command-buffer emulation layer to support
changes from KhronosGroup/OpenCL-Docs#850

Requires using KhronosGroup/OpenCL-Headers#265
in `external/OpenCL-Headers` and can be used to test CTS changes from
KhronosGroup/OpenCL-CTS#2101
@EwanC EwanC force-pushed the ewan/cmd_buf_supported_props branch from 35bc432 to e66f9d9 Compare January 8, 2025 11:05
@EwanC EwanC marked this pull request as ready for review January 8, 2025 11:06
@EwanC
Copy link
Contributor Author

EwanC commented Jan 8, 2025

Thanks, this is great!

Do you want to hold off on this one until the headers changes are merged, or try to add in some #ifdefs so it'll work both with newer and older headers?

Let's just wait for the headers to be merged, which hopefully won't be long now

bashbaug pushed a commit to KhronosGroup/OpenCL-CTS that referenced this pull request Jan 14, 2025
CTS test update to match OpenCL-Doc & OpenCL-Header PRs:
* KhronosGroup/OpenCL-Headers#265
* KhronosGroup/OpenCL-Docs#850

Tested with bashbaug/SimpleOpenCLSamples#126
locally using a checkout of the linked OpenCL-Headers branch
@EwanC
Copy link
Contributor Author

EwanC commented Jan 15, 2025

Headers are merged now, so this PR could be merged as-is (if approved)

Copy link
Owner

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this!

I'm going to merge this as-is, since I think it's needed to build with the latest headers now :).

We can always make follow-on changes in a separate PR.

@bashbaug bashbaug merged commit 23ab096 into bashbaug:main Jan 15, 2025
5 of 6 checks passed
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