Skip to content

[SYCL] Fix validation layers for Vulkan in bindless images interop tests#21271

Merged
sarnex merged 7 commits intointel:syclfrom
dyniols:fix_validation_layers_vulkan
Feb 17, 2026
Merged

[SYCL] Fix validation layers for Vulkan in bindless images interop tests#21271
sarnex merged 7 commits intointel:syclfrom
dyniols:fix_validation_layers_vulkan

Conversation

@dyniols
Copy link
Contributor

@dyniols dyniols commented Feb 11, 2026

This PR fixes issue which we always had false for return layer.layerName == "VK_LAYER_KHRONOS_validation" because it's comparison of C-style char array to a string literal. The content is ommited.

I removed that whole check. We need for CI always Vulkan validation layers to check for issues on Vulkan side in bindless interop tests.

@dyniols dyniols changed the title [SYCL] Fix validation layers for bindless images Vulkan interop tests [SYCL] Fix validation layers for Vulkan in bindless images interop tests Feb 11, 2026
@dyniols dyniols marked this pull request as ready for review February 11, 2026 20:31
@dyniols dyniols requested a review from a team as a code owner February 11, 2026 20:31
@dyniols dyniols marked this pull request as draft February 11, 2026 20:55
@dyniols dyniols marked this pull request as ready for review February 11, 2026 21:11
@dyniols dyniols marked this pull request as draft February 11, 2026 22:48
@dyniols dyniols marked this pull request as ready for review February 16, 2026 20:13
@dyniols dyniols requested a review from 0x12CC February 16, 2026 20:13
@dyniols
Copy link
Contributor Author

dyniols commented Feb 17, 2026

@intel/llvm-gatekeepers Can we merge it? Thanks :)

@sarnex sarnex merged commit 9c41152 into intel:sycl Feb 17, 2026
51 of 54 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.

4 participants

Comments