Skip to content

[Vulkan] Add support for depth textures#756

Merged
s-perron merged 2 commits intollvm:mainfrom
s-perron:depth
Mar 12, 2026
Merged

[Vulkan] Add support for depth textures#756
s-perron merged 2 commits intollvm:mainfrom
s-perron:depth

Conversation

@s-perron
Copy link
Contributor

Introduce the Depth32 data format to define a depth texture.
Tests are added for GatherCmp and SampleCmp in Texture2D, which use the
depth textures.

The depth texture are add in the Vulkan backend only.

@s-perron s-perron changed the title feat: Add support for depth textures [Vulkan] Add support for depth textures Feb 20, 2026
Introduce the Depth32 data format to define a depth texture.
Tests are added for GatherCmp and SampleCmp in Texture2D, which use the
depth textures.

The depth texture are add in the Vulkan backend only.
@Keenuts Keenuts added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label Feb 26, 2026
@Keenuts
Copy link
Contributor

Keenuts commented Feb 26, 2026

Added the test-all so we see results on other devices (didn't knew about this tag until recently)

@s-perron s-perron requested review from bogner and tex3d March 2, 2026 14:56
@s-perron
Copy link
Contributor Author

s-perron commented Mar 2, 2026

@tex3d @bogner As discussed in the HLSL meeting last tuesday. This adds depth textures, which are required on the Vulkan API side when doing a SampleCmp and GatherCmp. I do not know how DirectX works, so I didn't implement it. I want to make sure that DX can work with this before moving forward.

@s-perron
Copy link
Contributor Author

@bogner FYI.

@s-perron s-perron merged commit bac46d5 into llvm:main Mar 12, 2026
13 checks passed
@bogner
Copy link
Contributor

bogner commented Mar 12, 2026

@bogner FYI.

Looks reasonable. I'll attempt to implement the DX side this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-all When applied to a PR this will opt-in to additional pre-merge test configurations..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants