From a094c1527080e413528b2b17839d4399ae0b69da Mon Sep 17 00:00:00 2001 From: Faith Ekstrand Date: Wed, 3 Dec 2025 14:00:19 -0500 Subject: [PATCH] pan/bi: Stop pretending to support f16 gl_Position It's always a full vec4 in the varying buffer. Reviewed-by: Lorenzo Rossi Acked-by: Eric R. Smith Part-of: --- src/panfrost/compiler/bifrost/bifrost_compile.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/panfrost/compiler/bifrost/bifrost_compile.c b/src/panfrost/compiler/bifrost/bifrost_compile.c index 3812e558486..34a7b96bc65 100644 --- a/src/panfrost/compiler/bifrost/bifrost_compile.c +++ b/src/panfrost/compiler/bifrost/bifrost_compile.c @@ -1115,8 +1115,8 @@ bi_emit_store_vary(bi_builder *b, nir_intrinsic_instr *instr) if (b->shader->arch <= 8 && b->shader->idvs == BI_IDVS_POSITION) { /* Bifrost position shaders have a fast path */ - assert(T == nir_type_float16 || T == nir_type_float32); - unsigned regfmt = (T == nir_type_float16) ? 0 : 1; + assert(T == nir_type_float32); + unsigned regfmt = 1; unsigned identity = (b->shader->arch == 6) ? 0x688 : 0; unsigned snap4 = 0x5E; uint32_t format = identity | (snap4 << 12) | (regfmt << 24);