From 669d4d7f77648948800abce59bc99a29a338a3ad Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Fri, 9 Aug 2013 18:07:45 -0700 Subject: [PATCH] i965/fs: Rename setup_kills() to setup_initial_values(). Although this function currently only initializes the KILL set, it will soon initialize other data flow sets as well. Signed-off-by: Kenneth Graunke Reviewed-by: Paul Berry --- src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp index 144a43f85ac..634eb3d0a79 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp @@ -73,7 +73,7 @@ public: fs_copy_prop_dataflow(void *mem_ctx, cfg_t *cfg, exec_list *out_acp[ACP_HASH_SIZE]); - void setup_kills(); + void setup_initial_values(); void run(); void *mem_ctx; @@ -125,16 +125,16 @@ fs_copy_prop_dataflow::fs_copy_prop_dataflow(void *mem_ctx, cfg_t *cfg, assert(next_acp == num_acp); - setup_kills(); + setup_initial_values(); run(); } /** - * Walk the set of instructions in the block, marking which entries in the acp - * are killed by the block. + * Set up initial values for each of the data flow sets, prior to running + * the fixed-point algorithm. */ void -fs_copy_prop_dataflow::setup_kills() +fs_copy_prop_dataflow::setup_initial_values() { for (int b = 0; b < cfg->num_blocks; b++) { bblock_t *block = cfg->blocks[b]; @@ -145,6 +145,7 @@ fs_copy_prop_dataflow::setup_kills() if (inst->dst.file != GRF) continue; + /* Mark ACP entries which are killed by this instruction. */ for (int i = 0; i < num_acp; i++) { if (inst->overwrites_reg(acp[i]->dst) || inst->overwrites_reg(acp[i]->src)) { -- 2.30.2