From 5bfef2de66be739a54e64275feaa42c2d340b548 Mon Sep 17 00:00:00 2001 From: Tony Wasserka Date: Thu, 29 Apr 2021 18:06:53 +0200 Subject: [PATCH] aco/ra: Fix off-by-one-error in print_regs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reviewed-by: Daniel Schürmann Fixes: 3675aefa84e ("aco/ra: Fix build with print_regs enabled") Part-of: --- src/amd/compiler/aco_register_allocation.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/amd/compiler/aco_register_allocation.cpp b/src/amd/compiler/aco_register_allocation.cpp index ab643bfdc85..42ef1fd5e14 100644 --- a/src/amd/compiler/aco_register_allocation.cpp +++ b/src/amd/compiler/aco_register_allocation.cpp @@ -355,8 +355,7 @@ private: /* helper function for debugging */ UNUSED void print_regs(ra_ctx& ctx, bool vgprs, RegisterFile& reg_file) { - unsigned max = vgprs ? ctx.program->max_reg_demand.vgpr : ctx.program->max_reg_demand.sgpr; - PhysRegInterval regs { vgprs ? PhysReg{256} : PhysReg{0}, max }; + PhysRegInterval regs = get_reg_bounds(ctx.program, vgprs ? RegType::vgpr : RegType::sgpr); char reg_char = vgprs ? 'v' : 's'; /* print markers */ @@ -387,7 +386,7 @@ UNUSED void print_regs(ra_ctx& ctx, bool vgprs, RegisterFile& reg_file) } printf("\n"); - printf("%u/%u used, %u/%u free\n", max - free_regs, max, free_regs, max); + printf("%u/%u used, %u/%u free\n", regs.size - free_regs, regs.size, free_regs, regs.size); /* print assignments */ prev = 0;