From ecc19e12dca95d2571d3761dea6dec24b061013c Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Thu, 20 Apr 2017 11:44:01 -0700 Subject: [PATCH] intel/fs: Use regs_written() in spilling cost heuristic for improved accuracy. This is what we use later on to compute the number of registers that will actually get spilled to memory, so it's more likely to match reality than the current open-coded approximation. Cc: Reviewed-by: Plamena Manolova Reviewed-by: Jason Ekstrand --- src/intel/compiler/brw_fs_reg_allocate.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/intel/compiler/brw_fs_reg_allocate.cpp b/src/intel/compiler/brw_fs_reg_allocate.cpp index c981d72e4f2..2d4d46ef334 100644 --- a/src/intel/compiler/brw_fs_reg_allocate.cpp +++ b/src/intel/compiler/brw_fs_reg_allocate.cpp @@ -826,8 +826,7 @@ fs_visitor::choose_spill_reg(struct ra_graph *g) } if (inst->dst.file == VGRF) - spill_costs[inst->dst.nr] += DIV_ROUND_UP(inst->size_written, REG_SIZE) - * block_scale; + spill_costs[inst->dst.nr] += regs_written(inst) * block_scale; switch (inst->opcode) { -- 2.30.2