*/
bool is_direct;
+ /* Set on a root node for a variable to indicate that variable is used by a
+ * cast or passed through some other sequence of instructions that are not
+ * derefs.
+ */
+ bool has_complex_use;
+
struct deref_node *wildcard;
struct deref_node *indirect;
struct deref_node *children[0];
};
+#define UNDEF_NODE ((struct deref_node *)(uintptr_t)1)
+
struct lower_variables_state {
nir_shader *shader;
void *dead_ctx;
if (deref->deref_type == nir_deref_type_var)
return get_deref_node_for_var(deref->var, state);
+ if (deref->deref_type == nir_deref_type_cast)
+ return NULL;
+
struct deref_node *parent =
get_deref_node_recur(nir_deref_instr_parent(deref), state);
+ if (parent == NULL)
+ return NULL;
switch (deref->deref_type) {
case nir_deref_type_struct:
- assert(glsl_type_is_struct(parent->type));
+ assert(glsl_type_is_struct_or_ifc(parent->type));
assert(deref->strct.index < glsl_get_length(parent->type));
if (parent->children[deref->strct.index] == NULL) {
return parent->children[deref->strct.index];
case nir_deref_type_array: {
- nir_const_value *const_index = nir_src_as_const_value(deref->arr.index);
- if (const_index) {
- uint32_t index = const_index->u32[0];
+ if (nir_src_is_const(deref->arr.index)) {
+ uint32_t index = nir_src_as_uint(deref->arr.index);
/* This is possible if a loop unrolls and generates an
* out-of-bounds offset. We need to handle this at least
* somewhat gracefully.
*/
if (index >= glsl_get_length(parent->type))
- return NULL;
+ return UNDEF_NODE;
if (parent->children[index] == NULL) {
parent->children[index] =
static struct deref_node *
get_deref_node(nir_deref_instr *deref, struct lower_variables_state *state)
{
+ /* This pass only works on local variables. Just ignore any derefs with
+ * a non-local mode.
+ */
+ if (deref->mode != nir_var_function_temp)
+ return NULL;
+
struct deref_node *node = get_deref_node_recur(deref, state);
if (!node)
return NULL;
return;
case nir_deref_type_array: {
- nir_const_value *const_index = nir_src_as_const_value((*path)->arr.index);
- assert(const_index);
- uint32_t index = const_index->u32[0];
+ uint32_t index = nir_src_as_uint((*path)->arr.index);
if (node->children[index]) {
foreach_deref_node_worker(node->children[index],
}
case nir_deref_type_array: {
- nir_const_value *const_index = nir_src_as_const_value((*path)->arr.index);
- if (!const_index)
+ if (!nir_src_is_const((*path)->arr.index))
return true;
- uint32_t index = const_index->u32[0];
+ uint32_t index = nir_src_as_uint((*path)->arr.index);
/* If there is an indirect at this level, we're aliased. */
if (node->indirect)
{
assert(path->path[0]->deref_type == nir_deref_type_var);
nir_variable *var = path->path[0]->var;
+ struct deref_node *var_node = get_deref_node_for_var(var, state);
- return path_may_be_aliased_node(get_deref_node_for_var(var, state),
- &path->path[1], state);
+ /* First see if this variable is ever used by anything other than a
+ * load/store. If there's even so much as a cast in the way, we have to
+ * assume aliasing and bail.
+ */
+ if (var_node->has_complex_use)
+ return true;
+
+ return path_may_be_aliased_node(var_node, &path->path[1], state);
+}
+
+static void
+register_complex_use(nir_deref_instr *deref,
+ struct lower_variables_state *state)
+{
+ assert(deref->deref_type == nir_deref_type_var);
+ struct deref_node *node = get_deref_node_for_var(deref->var, state);
+ if (node == NULL)
+ return;
+
+ node->has_complex_use = true;
}
static void
return;
if (node->loads == NULL)
- node->loads = _mesa_set_create(state->dead_ctx, _mesa_hash_pointer,
- _mesa_key_pointer_equal);
+ node->loads = _mesa_pointer_set_create(state->dead_ctx);
_mesa_set_add(node->loads, load_instr);
}
return;
if (node->stores == NULL)
- node->stores = _mesa_set_create(state->dead_ctx, _mesa_hash_pointer,
- _mesa_key_pointer_equal);
+ node->stores = _mesa_pointer_set_create(state->dead_ctx);
_mesa_set_add(node->stores, store_instr);
}
continue;
if (node->copies == NULL)
- node->copies = _mesa_set_create(state->dead_ctx, _mesa_hash_pointer,
- _mesa_key_pointer_equal);
+ node->copies = _mesa_pointer_set_create(state->dead_ctx);
_mesa_set_add(node->copies, copy_instr);
}
{
nir_foreach_block(block, impl) {
nir_foreach_instr_safe(instr, block) {
- if (instr->type != nir_instr_type_intrinsic)
- continue;
+ switch (instr->type) {
+ case nir_instr_type_deref: {
+ nir_deref_instr *deref = nir_instr_as_deref(instr);
- nir_intrinsic_instr *intrin = nir_instr_as_intrinsic(instr);
+ if (deref->deref_type == nir_deref_type_var &&
+ nir_deref_instr_has_complex_use(deref))
+ register_complex_use(deref, state);
- switch (intrin->intrinsic) {
- case nir_intrinsic_load_deref:
- register_load_instr(intrin, state);
break;
+ }
- case nir_intrinsic_store_deref:
- register_store_instr(intrin, state);
- break;
+ case nir_instr_type_intrinsic: {
+ nir_intrinsic_instr *intrin = nir_instr_as_intrinsic(instr);
+
+ switch (intrin->intrinsic) {
+ case nir_intrinsic_load_deref:
+ register_load_instr(intrin, state);
+ break;
- case nir_intrinsic_copy_deref:
- register_copy_instr(intrin, state);
+ case nir_intrinsic_store_deref:
+ register_store_instr(intrin, state);
+ break;
+
+ case nir_intrinsic_copy_deref:
+ register_copy_instr(intrin, state);
+ break;
+
+ default:
+ continue;
+ }
break;
+ }
default:
- continue;
+ break;
}
}
}
nir_builder b;
nir_builder_init(&b, state->impl);
- struct set_entry *copy_entry;
set_foreach(node->copies, copy_entry) {
nir_intrinsic_instr *copy = (void *)copy_entry->key;
switch (intrin->intrinsic) {
case nir_intrinsic_load_deref: {
nir_deref_instr *deref = nir_src_as_deref(intrin->src[0]);
+ if (deref->mode != nir_var_function_temp)
+ continue;
+
struct deref_node *node = get_deref_node(deref, state);
- if (node == NULL) {
+ if (node == NULL)
+ continue;
+
+ if (node == UNDEF_NODE) {
/* If we hit this path then we are referencing an invalid
* value. Most likely, we unrolled something and are
* reading past the end of some array. In any case, this
continue;
nir_alu_instr *mov = nir_alu_instr_create(state->shader,
- nir_op_imov);
+ nir_op_mov);
mov->src[0].src = nir_src_for_ssa(
nir_phi_builder_value_get_block_def(node->pb_value, block));
- for (unsigned i = intrin->num_components; i < 4; i++)
+ for (unsigned i = intrin->num_components; i < NIR_MAX_VEC_COMPONENTS; i++)
mov->src[0].swizzle[i] = 0;
assert(intrin->dest.is_ssa);
case nir_intrinsic_store_deref: {
nir_deref_instr *deref = nir_src_as_deref(intrin->src[0]);
+ if (deref->mode != nir_var_function_temp)
+ continue;
+
struct deref_node *node = get_deref_node(deref, state);
+ if (node == NULL)
+ continue;
assert(intrin->src[1].is_ssa);
nir_ssa_def *value = intrin->src[1].ssa;
- if (node == NULL) {
+ if (node == UNDEF_NODE) {
/* Probably an out-of-bounds array store. That should be a
* no-op. */
nir_instr_remove(&intrin->instr);
* intrin->num_components and value->num_components
* may differ.
*/
- unsigned swiz[4];
- for (unsigned i = 0; i < 4; i++)
+ unsigned swiz[NIR_MAX_VEC_COMPONENTS];
+ for (unsigned i = 0; i < NIR_MAX_VEC_COMPONENTS; i++)
swiz[i] = i < intrin->num_components ? i : 0;
new_def = nir_swizzle(&b, value, swiz,
- intrin->num_components, false);
+ intrin->num_components);
} else {
nir_ssa_def *old_def =
nir_phi_builder_value_get_block_def(node->pb_value, block);
* written values with the existing contents of unwritten
* channels, creating a new SSA value for the whole vector.
*/
- nir_ssa_def *srcs[4];
+ nir_ssa_def *srcs[NIR_MAX_VEC_COMPONENTS];
for (unsigned i = 0; i < intrin->num_components; i++) {
if (wrmask & (1 << i)) {
srcs[i] = nir_channel(&b, value, i);
state.dead_ctx = ralloc_context(state.shader);
state.impl = impl;
- state.deref_var_nodes = _mesa_hash_table_create(state.dead_ctx,
- _mesa_hash_pointer,
- _mesa_key_pointer_equal);
+ state.deref_var_nodes = _mesa_pointer_hash_table_create(state.dead_ctx);
exec_list_make_empty(&state.direct_deref_nodes);
/* Build the initial deref structures and direct_deref_nodes table */
nir_deref_path *path = &node->path;
assert(path->path[0]->deref_type == nir_deref_type_var);
- nir_variable *var = path->path[0]->var;
- if (var->data.mode != nir_var_local) {
- exec_node_remove(&node->direct_derefs_link);
- continue;
- }
+ /* We don't build deref nodes for non-local variables */
+ assert(path->path[0]->var->data.mode == nir_var_function_temp);
if (path_may_be_aliased(path, &state)) {
exec_node_remove(&node->direct_derefs_link);
foreach_deref_node_match(path, lower_copies_to_load_store, &state);
}
- if (!progress)
+ if (!progress) {
+#ifndef NDEBUG
+ impl->valid_metadata &= ~nir_metadata_not_properly_reset;
+#endif
return false;
+ }
nir_metadata_require(impl, nir_metadata_dominance);
assert(node->path.path[0]->var->constant_initializer == NULL);
if (node->stores) {
- struct set_entry *store_entry;
set_foreach(node->stores, store_entry) {
nir_intrinsic_instr *store =
(nir_intrinsic_instr *)store_entry->key;
{
bool progress = false;
- nir_assert_unlowered_derefs(shader, nir_lower_load_store_derefs);
-
nir_foreach_function(function, shader) {
if (function->impl)
progress |= nir_lower_vars_to_ssa_impl(function->impl);