Skip to content

Stencil support for VK backend#9888

Merged
jorgeag-google merged 1 commit intogoogle:mainfrom
jorgeag-google:vk_stencil
Apr 22, 2026
Merged

Stencil support for VK backend#9888
jorgeag-google merged 1 commit intogoogle:mainfrom
jorgeag-google:vk_stencil

Conversation

@jorgeag-google
Copy link
Copy Markdown
Contributor

@jorgeag-google jorgeag-google commented Apr 8, 2026

Stencil support for VK backend

In order to support stencil op in VK we need to:

  1. Configure corresponding pipeline/render pass
  2. Configure attachment/FB.

Previous PR (#9716) covers 1)
This PR covers 2)

This PR also makes the the following tests pass on VK

  • BasicStencilBufferTest.StencilBuffer
  • BasicStencilBufferTest.DepthAndStencilBuffer

@poweifeng poweifeng added the internal Issue/PR does not affect clients label Apr 8, 2026
@poweifeng
Copy link
Copy Markdown
Contributor

@jorgeag-google , would you mind being more descriptive on why this change is necessary? (in the commit message, but then you'll have to copy the commit message over to the PR description).

@jorgeag-google jorgeag-google force-pushed the vk_stencil branch 2 times, most recently from df18337 to abcac7a Compare April 13, 2026 18:36
Comment thread filament/backend/src/vulkan/VulkanHandles.cpp Outdated
@jorgeag-google jorgeag-google force-pushed the vk_stencil branch 3 times, most recently from 96bc2b7 to 628fb1b Compare April 15, 2026 20:10
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanFboCache.h Outdated
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanHandles.h Outdated
@jorgeag-google jorgeag-google force-pushed the vk_stencil branch 2 times, most recently from 6b73d16 to 1d0fa49 Compare April 20, 2026 19:42
Comment thread filament/backend/src/vulkan/VulkanDriver.cpp
Comment thread filament/backend/src/vulkan/VulkanFboCache.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanFboCache.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanReadPixels.cpp Outdated
Comment thread filament/backend/src/vulkan/VulkanHandles.cpp Outdated
@jorgeag-google jorgeag-google force-pushed the vk_stencil branch 2 times, most recently from b535f51 to 5ce1f8f Compare April 22, 2026 17:23
In order to support stencil op in VK we need to:

1)Configure corresponding pipeline/render pass
2)Configure attachment/FB

Previous PR (google#9716) covers 1)
This PR covers 2)

This PR also makes the the following tests pass on VK

BasicStencilBufferTest.StencilBuffer
BasicStencilBufferTest.DepthAndStencilBuffer
@jorgeag-google jorgeag-google merged commit 6d5f1cf into google:main Apr 22, 2026
14 checks passed
@jorgeag-google jorgeag-google deleted the vk_stencil branch April 22, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Issue/PR does not affect clients

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants