From d09442461f66955cd1ddc7367a0060db03d0fd0c Mon Sep 17 00:00:00 2001 From: Erik Faye-Lund Date: Thu, 15 Apr 2021 10:17:55 +0200 Subject: [PATCH] zink: fix stencil-export cap emission This cap should only ever be emitted for fragment-shaders, but we accidentally emit it for all shaders. Let's tighten the check to avoid a validator warning when emitting non-fragment shaders without support for VK_EXT_shader_stencil_export. Fixes: 8724d4fb361 ("zink: check shader stencil output") Reviewed-By: Mike Blumenkrantz Part-of: --- src/gallium/drivers/zink/nir_to_spirv/nir_to_spirv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/zink/nir_to_spirv/nir_to_spirv.c b/src/gallium/drivers/zink/nir_to_spirv/nir_to_spirv.c index 3a78b519127..969f718309c 100644 --- a/src/gallium/drivers/zink/nir_to_spirv/nir_to_spirv.c +++ b/src/gallium/drivers/zink/nir_to_spirv/nir_to_spirv.c @@ -3532,7 +3532,8 @@ nir_to_spirv(struct nir_shader *s, const struct zink_so_info *so_info) spirv_builder_emit_mem_model(&ctx.builder, SpvAddressingModelLogical, SpvMemoryModelGLSL450); - if (s->info.outputs_written & BITFIELD64_BIT(FRAG_RESULT_STENCIL)) { + if (s->info.stage == MESA_SHADER_FRAGMENT && + s->info.outputs_written & BITFIELD64_BIT(FRAG_RESULT_STENCIL)) { spirv_builder_emit_extension(&ctx.builder, "SPV_EXT_shader_stencil_export"); spirv_builder_emit_cap(&ctx.builder, SpvCapabilityStencilExportEXT); }