diff options
author | Matt Turner <mattst88@gmail.com> | 2017-03-09 11:49:57 -0800 |
---|---|---|
committer | Matt Turner <mattst88@gmail.com> | 2017-03-23 14:34:44 -0700 |
commit | ef71af7356ae5d19a90b3bb5e83f14341173b8f3 (patch) | |
tree | bfeb59f3b4409b57a16565ceb0fff3ab29ae96b2 /src/compiler | |
parent | abc8a702d0f01852f85705a87c9d624300c1efec (diff) |
nir: Return progress from nir_convert_from_ssa().
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Diffstat (limited to 'src/compiler')
-rw-r--r-- | src/compiler/nir/nir.h | 2 | ||||
-rw-r--r-- | src/compiler/nir/nir_from_ssa.c | 21 |
2 files changed, 15 insertions, 8 deletions
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index 388b5c85aef..d032787a2a0 100644 --- a/src/compiler/nir/nir.h +++ b/src/compiler/nir/nir.h @@ -2588,7 +2588,7 @@ void nir_convert_loop_to_lcssa(nir_loop *loop); * registers. If false, convert all values (even those not involved in a phi * node) to registers. */ -void nir_convert_from_ssa(nir_shader *shader, bool phi_webs_only); +bool nir_convert_from_ssa(nir_shader *shader, bool phi_webs_only); bool nir_lower_phis_to_regs_block(nir_block *block); bool nir_lower_ssa_defs_to_regs_block(nir_block *block); diff --git a/src/compiler/nir/nir_from_ssa.c b/src/compiler/nir/nir_from_ssa.c index d2646c63c47..1aa35509b11 100644 --- a/src/compiler/nir/nir_from_ssa.c +++ b/src/compiler/nir/nir_from_ssa.c @@ -41,6 +41,7 @@ struct from_ssa_state { bool phi_webs_only; struct hash_table *merge_node_table; nir_instr *instr; + bool progress; }; /* Returns true if a dominates b */ @@ -503,6 +504,7 @@ rewrite_ssa_def(nir_ssa_def *def, void *void_state) nir_instr *parent_instr = def->parent_instr; nir_instr_remove(parent_instr); ralloc_steal(state->dead_ctx, parent_instr); + state->progress = true; return true; } @@ -514,14 +516,14 @@ rewrite_ssa_def(nir_ssa_def *def, void *void_state) nir_dest *dest = exec_node_data(nir_dest, def, ssa); nir_instr_rewrite_dest(state->instr, dest, nir_dest_for_reg(reg)); - + state->progress = true; return true; } /* Resolves ssa definitions to registers. While we're at it, we also * remove phi nodes. */ -static bool +static void resolve_registers_block(nir_block *block, struct from_ssa_state *state) { nir_foreach_instr_safe(instr, block) { @@ -531,11 +533,10 @@ resolve_registers_block(nir_block *block, struct from_ssa_state *state) if (instr->type == nir_instr_type_phi) { nir_instr_remove(instr); ralloc_steal(state->dead_ctx, instr); + state->progress = true; } } state->instr = NULL; - - return true; } static void @@ -756,7 +757,7 @@ resolve_parallel_copies_block(nir_block *block, struct from_ssa_state *state) return true; } -static void +static bool nir_convert_from_ssa_impl(nir_function_impl *impl, bool phi_webs_only) { struct from_ssa_state state; @@ -766,6 +767,7 @@ nir_convert_from_ssa_impl(nir_function_impl *impl, bool phi_webs_only) state.phi_webs_only = phi_webs_only; state.merge_node_table = _mesa_hash_table_create(NULL, _mesa_hash_pointer, _mesa_key_pointer_equal); + state.progress = false; nir_foreach_block(block, impl) { add_parallel_copy_to_end_of_block(block, state.dead_ctx); @@ -804,15 +806,20 @@ nir_convert_from_ssa_impl(nir_function_impl *impl, bool phi_webs_only) /* Clean up dead instructions and the hash tables */ _mesa_hash_table_destroy(state.merge_node_table, NULL); ralloc_free(state.dead_ctx); + return state.progress; } -void +bool nir_convert_from_ssa(nir_shader *shader, bool phi_webs_only) { + bool progress = false; + nir_foreach_function(function, shader) { if (function->impl) - nir_convert_from_ssa_impl(function->impl, phi_webs_only); + progress |= nir_convert_from_ssa_impl(function->impl, phi_webs_only); } + + return progress; } |