From e5daef95877047d6c820953666dc570237c988db Mon Sep 17 00:00:00 2001 From: Karol Herbst Date: Tue, 15 Jan 2019 23:56:29 +0100 Subject: [PATCH] nir: rename nir_var_private to nir_var_shader_temp Signed-off-by: Karol Herbst Acked-by: Jason Ekstrand Reviewed-by: Eric Anholt Reviewed-by: Kenneth Graunke Reviewed-by: Bas Nieuwenhuizen --- src/compiler/glsl/glsl_to_nir.cpp | 4 ++-- src/compiler/nir/nir.c | 2 +- src/compiler/nir/nir.h | 2 +- src/compiler/nir/nir_linking_helpers.c | 2 +- src/compiler/nir/nir_lower_clip.c | 2 +- src/compiler/nir/nir_lower_constant_initializers.c | 2 +- src/compiler/nir/nir_lower_global_vars_to_local.c | 4 ++-- src/compiler/nir/nir_lower_io_to_temporaries.c | 2 +- src/compiler/nir/nir_opt_copy_prop_vars.c | 4 ++-- src/compiler/nir/nir_opt_dead_write_vars.c | 2 +- src/compiler/nir/nir_print.c | 4 ++-- src/compiler/nir/nir_remove_dead_variables.c | 4 ++-- src/compiler/nir/nir_split_vars.c | 16 ++++++++-------- src/compiler/nir/tests/vars_tests.cpp | 2 +- src/compiler/spirv/vtn_variables.c | 2 +- src/gallium/auxiliary/nir/tgsi_to_nir.c | 2 +- src/mesa/state_tracker/st_glsl_to_nir.cpp | 2 +- 17 files changed, 29 insertions(+), 29 deletions(-) diff --git a/src/compiler/glsl/glsl_to_nir.cpp b/src/compiler/glsl/glsl_to_nir.cpp index 8f78657..f04e0d5 100644 --- a/src/compiler/glsl/glsl_to_nir.cpp +++ b/src/compiler/glsl/glsl_to_nir.cpp @@ -324,7 +324,7 @@ nir_visitor::visit(ir_variable *ir) case ir_var_auto: case ir_var_temporary: if (is_global) - var->data.mode = nir_var_private; + var->data.mode = nir_var_shader_temp; else var->data.mode = nir_var_function; break; @@ -1564,7 +1564,7 @@ nir_visitor::visit(ir_expression *ir) * sense, we'll just turn it into a load which will probably * eventually end up as an SSA definition. */ - assert(this->deref->mode == nir_var_private); + assert(this->deref->mode == nir_var_shader_temp); op = nir_intrinsic_load_deref; } diff --git a/src/compiler/nir/nir.c b/src/compiler/nir/nir.c index 1fc09d9..8a92be9 100644 --- a/src/compiler/nir/nir.c +++ b/src/compiler/nir/nir.c @@ -129,7 +129,7 @@ nir_shader_add_variable(nir_shader *shader, nir_variable *var) assert(!"nir_shader_add_variable cannot be used for local variables"); break; - case nir_var_private: + case nir_var_shader_temp: exec_list_push_tail(&shader->globals, &var->node); break; diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index d12c499..534bdcd 100644 --- a/src/compiler/nir/nir.h +++ b/src/compiler/nir/nir.h @@ -97,7 +97,7 @@ typedef struct { typedef enum { nir_var_shader_in = (1 << 0), nir_var_shader_out = (1 << 1), - nir_var_private = (1 << 2), + nir_var_shader_temp = (1 << 2), nir_var_function = (1 << 3), nir_var_uniform = (1 << 4), nir_var_ubo = (1 << 5), diff --git a/src/compiler/nir/nir_linking_helpers.c b/src/compiler/nir/nir_linking_helpers.c index b85e129..8e67e84 100644 --- a/src/compiler/nir/nir_linking_helpers.c +++ b/src/compiler/nir/nir_linking_helpers.c @@ -133,7 +133,7 @@ nir_remove_unused_io_vars(nir_shader *shader, struct exec_list *var_list, if (!(other_stage & get_variable_io_mask(var, shader->info.stage))) { /* This one is invalid, make it a global variable instead */ var->data.location = 0; - var->data.mode = nir_var_private; + var->data.mode = nir_var_shader_temp; exec_node_remove(&var->node); exec_list_push_tail(&shader->globals, &var->node); diff --git a/src/compiler/nir/nir_lower_clip.c b/src/compiler/nir/nir_lower_clip.c index 6fdf1a9..474854d 100644 --- a/src/compiler/nir/nir_lower_clip.c +++ b/src/compiler/nir/nir_lower_clip.c @@ -212,7 +212,7 @@ nir_lower_clip_vs(nir_shader *shader, unsigned ucp_enables, bool use_vars) if (clipvertex) { exec_node_remove(&clipvertex->node); - clipvertex->data.mode = nir_var_private; + clipvertex->data.mode = nir_var_shader_temp; exec_list_push_tail(&shader->globals, &clipvertex->node); } } else { diff --git a/src/compiler/nir/nir_lower_constant_initializers.c b/src/compiler/nir/nir_lower_constant_initializers.c index 959d1ea..cc100dd 100644 --- a/src/compiler/nir/nir_lower_constant_initializers.c +++ b/src/compiler/nir/nir_lower_constant_initializers.c @@ -103,7 +103,7 @@ nir_lower_constant_initializers(nir_shader *shader, nir_variable_mode modes) if ((modes & nir_var_shader_out) && function->is_entrypoint) impl_progress |= lower_const_initializer(&builder, &shader->outputs); - if ((modes & nir_var_private) && function->is_entrypoint) + if ((modes & nir_var_shader_temp) && function->is_entrypoint) impl_progress |= lower_const_initializer(&builder, &shader->globals); if ((modes & nir_var_system_value) && function->is_entrypoint) diff --git a/src/compiler/nir/nir_lower_global_vars_to_local.c b/src/compiler/nir/nir_lower_global_vars_to_local.c index 9eb76c7..64eb6a6 100644 --- a/src/compiler/nir/nir_lower_global_vars_to_local.c +++ b/src/compiler/nir/nir_lower_global_vars_to_local.c @@ -36,7 +36,7 @@ static void register_var_use(nir_variable *var, nir_function_impl *impl, struct hash_table *var_func_table) { - if (var->data.mode != nir_var_private) + if (var->data.mode != nir_var_shader_temp) return; struct hash_entry *entry = @@ -87,7 +87,7 @@ nir_lower_global_vars_to_local(nir_shader *shader) nir_variable *var = (void *)entry->key; nir_function_impl *impl = entry->data; - assert(var->data.mode == nir_var_private); + assert(var->data.mode == nir_var_shader_temp); if (impl != NULL) { exec_node_remove(&var->node); diff --git a/src/compiler/nir/nir_lower_io_to_temporaries.c b/src/compiler/nir/nir_lower_io_to_temporaries.c index 2487add..7602637 100644 --- a/src/compiler/nir/nir_lower_io_to_temporaries.c +++ b/src/compiler/nir/nir_lower_io_to_temporaries.c @@ -134,7 +134,7 @@ create_shadow_temp(struct lower_io_state *state, nir_variable *var) /* Give the original a new name with @-temp appended */ const char *mode = (temp->data.mode == nir_var_shader_in) ? "in" : "out"; temp->name = ralloc_asprintf(var, "%s@%s-temp", mode, nvar->name); - temp->data.mode = nir_var_private; + temp->data.mode = nir_var_shader_temp; temp->data.read_only = false; temp->data.fb_fetch_output = false; temp->data.compact = false; diff --git a/src/compiler/nir/nir_opt_copy_prop_vars.c b/src/compiler/nir/nir_opt_copy_prop_vars.c index 8a5e115..21e58b5 100644 --- a/src/compiler/nir/nir_opt_copy_prop_vars.c +++ b/src/compiler/nir/nir_opt_copy_prop_vars.c @@ -133,7 +133,7 @@ gather_vars_written(struct copy_prop_var_state *state, nir_foreach_instr(instr, block) { if (instr->type == nir_instr_type_call) { written->modes |= nir_var_shader_out | - nir_var_private | + nir_var_shader_temp | nir_var_function | nir_var_ssbo | nir_var_shared; @@ -624,7 +624,7 @@ copy_prop_vars_block(struct copy_prop_var_state *state, nir_foreach_instr_safe(instr, block) { if (instr->type == nir_instr_type_call) { apply_barrier_for_modes(copies, nir_var_shader_out | - nir_var_private | + nir_var_shader_temp | nir_var_function | nir_var_ssbo | nir_var_shared); diff --git a/src/compiler/nir/nir_opt_dead_write_vars.c b/src/compiler/nir/nir_opt_dead_write_vars.c index 49c71cc..d8411e3 100644 --- a/src/compiler/nir/nir_opt_dead_write_vars.c +++ b/src/compiler/nir/nir_opt_dead_write_vars.c @@ -119,7 +119,7 @@ remove_dead_write_vars_local(void *mem_ctx, nir_block *block) nir_foreach_instr_safe(instr, block) { if (instr->type == nir_instr_type_call) { clear_unused_for_modes(&unused_writes, nir_var_shader_out | - nir_var_private | + nir_var_shader_temp | nir_var_function | nir_var_ssbo | nir_var_shared); diff --git a/src/compiler/nir/nir_print.c b/src/compiler/nir/nir_print.c index 88a82b1..f0142ab 100644 --- a/src/compiler/nir/nir_print.c +++ b/src/compiler/nir/nir_print.c @@ -421,8 +421,8 @@ get_variable_mode_str(nir_variable_mode mode, bool want_local_global_mode) return "ssbo"; case nir_var_shared: return "shared"; - case nir_var_private: - return want_local_global_mode ? "private" : ""; + case nir_var_shader_temp: + return want_local_global_mode ? "shader_temp" : ""; case nir_var_function: return want_local_global_mode ? "function" : ""; default: diff --git a/src/compiler/nir/nir_remove_dead_variables.c b/src/compiler/nir/nir_remove_dead_variables.c index 5c345bd..8804dfe 100644 --- a/src/compiler/nir/nir_remove_dead_variables.c +++ b/src/compiler/nir/nir_remove_dead_variables.c @@ -71,7 +71,7 @@ add_var_use_deref(nir_deref_instr *deref, struct set *live) * all means we need to keep it alive. */ assert(deref->mode == deref->var->data.mode); - if (!(deref->mode & (nir_var_function | nir_var_private | nir_var_shared)) || + if (!(deref->mode & (nir_var_function | nir_var_shader_temp | nir_var_shared)) || deref_used_for_not_store(deref)) _mesa_set_add(live, deref->var); } @@ -177,7 +177,7 @@ nir_remove_dead_variables(nir_shader *shader, nir_variable_mode modes) if (modes & nir_var_shader_out) progress = remove_dead_vars(&shader->outputs, live) || progress; - if (modes & nir_var_private) + if (modes & nir_var_shader_temp) progress = remove_dead_vars(&shader->globals, live) || progress; if (modes & nir_var_system_value) diff --git a/src/compiler/nir/nir_split_vars.c b/src/compiler/nir/nir_split_vars.c index 0cf3e51..6d4df61 100644 --- a/src/compiler/nir/nir_split_vars.c +++ b/src/compiler/nir/nir_split_vars.c @@ -259,10 +259,10 @@ nir_split_struct_vars(nir_shader *shader, nir_variable_mode modes) struct hash_table *var_field_map = _mesa_pointer_hash_table_create(mem_ctx); - assert((modes & (nir_var_private | nir_var_function)) == modes); + assert((modes & (nir_var_shader_temp | nir_var_function)) == modes); bool has_global_splits = false; - if (modes & nir_var_private) { + if (modes & nir_var_shader_temp) { has_global_splits = split_var_list_structs(shader, NULL, &shader->globals, var_field_map, mem_ctx); @@ -794,10 +794,10 @@ nir_split_array_vars(nir_shader *shader, nir_variable_mode modes) void *mem_ctx = ralloc_context(NULL); struct hash_table *var_info_map = _mesa_pointer_hash_table_create(mem_ctx); - assert((modes & (nir_var_private | nir_var_function)) == modes); + assert((modes & (nir_var_shader_temp | nir_var_function)) == modes); bool has_global_array = false; - if (modes & nir_var_private) { + if (modes & nir_var_shader_temp) { has_global_array = init_var_list_array_infos(&shader->globals, var_info_map, mem_ctx); } @@ -826,7 +826,7 @@ nir_split_array_vars(nir_shader *shader, nir_variable_mode modes) } bool has_global_splits = false; - if (modes & nir_var_private) { + if (modes & nir_var_shader_temp) { has_global_splits = split_var_list_arrays(shader, NULL, &shader->globals, var_info_map, mem_ctx); @@ -1491,7 +1491,7 @@ function_impl_has_vars_with_modes(nir_function_impl *impl, { nir_shader *shader = impl->function->shader; - if ((modes & nir_var_private) && !exec_list_is_empty(&shader->globals)) + if ((modes & nir_var_shader_temp) && !exec_list_is_empty(&shader->globals)) return true; if ((modes & nir_var_function) && !exec_list_is_empty(&impl->locals)) @@ -1512,7 +1512,7 @@ function_impl_has_vars_with_modes(nir_function_impl *impl, bool nir_shrink_vec_array_vars(nir_shader *shader, nir_variable_mode modes) { - assert((modes & (nir_var_private | nir_var_function)) == modes); + assert((modes & (nir_var_shader_temp | nir_var_function)) == modes); void *mem_ctx = ralloc_context(NULL); @@ -1540,7 +1540,7 @@ nir_shrink_vec_array_vars(nir_shader *shader, nir_variable_mode modes) } bool globals_shrunk = false; - if (modes & nir_var_private) + if (modes & nir_var_shader_temp) globals_shrunk = shrink_vec_var_list(&shader->globals, var_usage_map); bool progress = false; diff --git a/src/compiler/nir/tests/vars_tests.cpp b/src/compiler/nir/tests/vars_tests.cpp index 8e187b4..43d87f3 100644 --- a/src/compiler/nir/tests/vars_tests.cpp +++ b/src/compiler/nir/tests/vars_tests.cpp @@ -191,7 +191,7 @@ TEST_F(nir_redundant_load_vars_test, invalidate_inside_if_block) * if statement. They should be invalidated accordingly. */ - nir_variable **g = create_many_int(nir_var_private, "g", 3); + nir_variable **g = create_many_int(nir_var_shader_temp, "g", 3); nir_variable **out = create_many_int(nir_var_shader_out, "out", 3); nir_load_var(b, g[0]); diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index 6106ff3..9f42039 100644 --- a/src/compiler/spirv/vtn_variables.c +++ b/src/compiler/spirv/vtn_variables.c @@ -1694,7 +1694,7 @@ vtn_storage_class_to_mode(struct vtn_builder *b, break; case SpvStorageClassPrivate: mode = vtn_variable_mode_private; - nir_mode = nir_var_private; + nir_mode = nir_var_shader_temp; break; case SpvStorageClassFunction: mode = vtn_variable_mode_function; diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c b/src/gallium/auxiliary/nir/tgsi_to_nir.c index d4b9d29..e51da58 100644 --- a/src/gallium/auxiliary/nir/tgsi_to_nir.c +++ b/src/gallium/auxiliary/nir/tgsi_to_nir.c @@ -182,7 +182,7 @@ ttn_emit_declaration(struct ttn_compile *c) nir_variable *var = rzalloc(b->shader, nir_variable); var->type = glsl_array_type(glsl_vec4_type(), array_size, 0); - var->data.mode = nir_var_private; + var->data.mode = nir_var_shader_temp; var->name = ralloc_asprintf(var, "arr_%d", decl->Array.ArrayID); exec_list_push_tail(&b->shader->globals, &var->node); diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp b/src/mesa/state_tracker/st_glsl_to_nir.cpp index 3ab7af7..31cc9fa 100644 --- a/src/mesa/state_tracker/st_glsl_to_nir.cpp +++ b/src/mesa/state_tracker/st_glsl_to_nir.cpp @@ -103,7 +103,7 @@ st_nir_assign_vs_in_locations(nir_shader *nir) * set. */ exec_node_remove(&var->node); - var->data.mode = nir_var_private; + var->data.mode = nir_var_shader_temp; exec_list_push_tail(&nir->globals, &var->node); } } -- 2.7.4