intel / intel/llvm

Drop `CommonUtils/vulkan_common.hpp` from bindless images e2e tests and migrate to `vulkan_setup.hpp`

Open
#22,885 0 comments 0 reactions 1 assignee Claimed by @mianowski View on GitHub
enhancement good first issue sycl-bindless-images
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

### Describe the bug

The bindless images Vulkan interop e2e tests still rely on `CommonUtils/vulkan_common.hpp`, which is inconsistent with the new setup. This legacy dependency should be removed, and all tests that still use it should be migrated to the newer common Vulkan setup header `vulkan_setup.hpp`.

Tests still impacted:
`test-e2e/bindless_images/vulkan_interop/depth_format.cpp`
`test-e2e/bindless_images/vulkan_interop/mipmaps.cpp`
`test-e2e/MemoryExport/export_memory_to_vulkan.cpp`

### To reproduce

### Environment

- OS: [e.g Windows/Linux]
- Target device and vendor: [e.g. Intel GPU]
- DPC++ version: [e.g. commit hash or output of `clang++ --version`]
- Dependencies version: [e.g. the output of `sycl-ls --verbose`]

### Additional context

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.