Skip to content

Gate multiview builtin shaders on multiview feature check#3831

Merged
baldurk merged 1 commit intobaldurk:v1.xfrom
xen2:fix-multiview-shader-feature-check
Apr 20, 2026
Merged

Gate multiview builtin shaders on multiview feature check#3831
baldurk merged 1 commit intobaldurk:v1.xfrom
xen2:fix-multiview-shader-feature-check

Conversation

@xen2
Copy link
Copy Markdown
Contributor

@xen2 xen2 commented Apr 20, 2026

Description

CheckerboardMultiviewFS and TrisizeMultiviewFS compile with #define USE_MULTIVIEW 1, emitting OpCapability MultiView.

Without a FeatureCheck::MultiView gate, they get compiled and passed to vkCreateShaderModule on devices that didn't enable multiview, producing spurious validation errors at capture init (when using RenderDoc API).

Other shaders QuadResolve/WriteMultiviewFS already gate with FeatureCheck::MultiView.
This applies the same gate to the remaining two.
It seems safe because both variants are only selected when the captured content uses multiview.

@baldurk baldurk merged commit ceeeaa5 into baldurk:v1.x Apr 20, 2026
16 checks passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants