Skip to content

Move urMemImageGetInfo success test from a switch to individual test #2619

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

martygrant
Copy link
Contributor

For #2290

Also adds a few missing enums to the ur_image_info_t struct

intel/llvm#16655

@github-actions github-actions bot added loader Loader related feature/bug conformance Conformance test suite issues. specification Changes or additions to the specification opencl OpenCL adapter specific issues labels Jan 27, 2025
@martygrant martygrant force-pushed the martin/memimage-info-unswitch-redo branch from 2ef738b to fc2c774 Compare January 27, 2025 10:28
@martygrant martygrant marked this pull request as ready for review January 27, 2025 12:10
@martygrant martygrant requested review from a team as code owners January 27, 2025 12:10
@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label Jan 27, 2025
Added implementation details to OpenCL adapter for processing image format structs.
Added a couple missing enums to spec for ur_image_info_t and added CTS tests for these.
@martygrant martygrant force-pushed the martin/memimage-info-unswitch-redo branch from fc2c774 to c57c181 Compare January 29, 2025 11:38
@kbenzie kbenzie merged commit 8e6ea72 into oneapi-src:main Jan 29, 2025
26 checks passed
sommerlukas pushed a commit to intel/llvm that referenced this pull request Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conformance Conformance test suite issues. loader Loader related feature/bug opencl OpenCL adapter specific issues ready to merge Added to PR's which are ready to merge specification Changes or additions to the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants