[PATCH v3 0/1] MR430: vkd3d-shader/tpf: Fix used temp registers accounting for dcl_temps.

[Nikolay Sivov (＠nsivov)]

Otherwise we always output "dcl_temps 1" even when no temp registers were used.

--
  v3: vkd3d-shader/tpf: Fix used temp registers accounting for dcl_temps.
 
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/430

[PATCH v3 1/1] vkd3d-shader/tpf: Fix used temp registers accounting for dcl_temps.

[Nikolay Sivov]

From: Nikolay Sivov <nsivov(a)codeweavers.com>

Otherwise we always output "dcl_temps 1" even when no temp registers were used.
---
 libs/vkd3d-shader/hlsl_codegen.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c
index 51c3d68f..6eac5d49 100644
--- a/libs/vkd3d-shader/hlsl_codegen.c
+++ b/libs/vkd3d-shader/hlsl_codegen.c
@@ -3461,22 +3461,20 @@ static void compute_liveness(struct hlsl_ctx *ctx, struct hlsl_ir_function_decl
 
 struct register_allocator
 {
-    size_t count, capacity;
-
-    /* Highest register index that has been allocated.
-     * Used to declare sm4 temp count. */
-    uint32_t max_reg;
-
     struct allocation
     {
         uint32_t reg;
         unsigned int writemask;
         unsigned int first_write, last_read;
     } *allocations;
+    size_t count, capacity;
 
     /* Indexable temps are allocated separately and always keep their index regardless of their
      * lifetime. */
     size_t indexable_count;
+
+    /* Total number of registers allocated so far. Used to declare sm4 temp count. */
+    uint32_t reg_count;
 };
 
 static unsigned int get_available_writemask(const struct register_allocator *allocator,
@@ -3519,7 +3517,7 @@ static void record_allocation(struct hlsl_ctx *ctx, struct register_allocator *a
     allocation->first_write = first_write;
     allocation->last_read = last_read;
 
-    allocator->max_reg = max(allocator->max_reg, reg_idx);
+    allocator->reg_count = max(allocator->reg_count, reg_idx + 1);
 }
 
 /* reg_size is the number of register components to be reserved, while component_count is the number
@@ -3986,7 +3984,7 @@ static void allocate_temp_registers(struct hlsl_ctx *ctx, struct hlsl_ir_functio
     }
 
     allocate_temp_registers_recurse(ctx, &entry_func->body, &allocator);
-    ctx->temp_count = allocator.max_reg + 1;
+    ctx->temp_count = allocator.reg_count;
     vkd3d_free(allocator.allocations);
 }
 
-- 
GitLab

https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/430