diff options
author | Jason Ekstrand <[email protected]> | 2014-11-19 16:06:32 -0800 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2015-01-15 07:19:02 -0800 |
commit | 6a52d2af2f7594fcd76fcf6158eca531e48af1e3 (patch) | |
tree | 89d015f2fa2288f98551f858c655688ae4e8e9bb /src/glsl/nir/nir.c | |
parent | 829aa98320fcd529407d16991b476b71af017479 (diff) |
nir: Don't require a function in ssa_def_init
Instead, we give SSA definitions a temporary index of 0xFFFFFFFF if the
instruction does not have a block and a proper index when it actually gets
added to the list.
Reviewed-by: Connor Abbott <[email protected]>
Diffstat (limited to 'src/glsl/nir/nir.c')
-rw-r--r-- | src/glsl/nir/nir.c | 42 |
1 files changed, 32 insertions, 10 deletions
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c index 3f04de2fee6..281683a542d 100644 --- a/src/glsl/nir/nir.c +++ b/src/glsl/nir/nir.c @@ -1188,17 +1188,27 @@ add_use_cb(nir_src *src, void *state) return true; } +static void +add_ssa_def(nir_instr *instr, nir_ssa_def *def) +{ + if (instr->block && def->index == UINT_MAX) { + nir_function_impl *impl = + nir_cf_node_get_function(&instr->block->cf_node); + + def->index = impl->ssa_alloc++; + } +} + static bool add_def_cb(nir_dest *dest, void *state) { nir_instr *instr = (nir_instr *) state; - if (dest->is_ssa) - return true; - - nir_register *reg = dest->reg.reg; - - _mesa_set_add(reg->defs, _mesa_hash_pointer(instr), instr); + if (dest->is_ssa) { + add_ssa_def(instr, &dest->ssa); + } else { + _mesa_set_add(dest->reg.reg->defs, _mesa_hash_pointer(instr), instr); + } return true; } @@ -1206,8 +1216,12 @@ add_def_cb(nir_dest *dest, void *state) static void add_defs_uses(nir_instr *instr) { - nir_foreach_src(instr, add_use_cb, instr); - nir_foreach_dest(instr, add_def_cb, instr); + if (instr->type == nir_instr_type_ssa_undef) { + add_ssa_def(instr, &nir_instr_as_ssa_undef(instr)->def); + } else { + nir_foreach_src(instr, add_use_cb, instr); + nir_foreach_dest(instr, add_def_cb, instr); + } } void @@ -1748,17 +1762,25 @@ nir_instr_rewrite_src(nir_instr *instr, nir_src *src, nir_src new_src) } void -nir_ssa_def_init(nir_function_impl *impl, nir_instr *instr, nir_ssa_def *def, +nir_ssa_def_init(nir_instr *instr, nir_ssa_def *def, unsigned num_components, const char *name) { void *mem_ctx = ralloc_parent(instr); def->name = name; - def->index = impl->ssa_alloc++; def->parent_instr = instr; def->uses = _mesa_set_create(mem_ctx, _mesa_key_pointer_equal); def->if_uses = _mesa_set_create(mem_ctx, _mesa_key_pointer_equal); def->num_components = num_components; + + if (instr->block) { + nir_function_impl *impl = + nir_cf_node_get_function(&instr->block->cf_node); + + def->index = impl->ssa_alloc++; + } else { + def->index = UINT_MAX; + } } struct ssa_def_rewrite_state { |