[Mesa-dev] [PATCH 3/6] nir: Add a large constants optimization pass
Jason Ekstrand
jason at jlekstrand.net
Fri Jun 29 20:06:52 UTC 2018
This pass searches for reasonably large local variables which can be
statically proven to be constant and moves them into shader constant
data. This is especially useful when large tables are baked into the
shader source code because they can be moved into a UBO by the driver to
reduce register pressure and make indirect access cheaper.
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/meson.build | 1 +
src/compiler/nir/nir.h | 4 +
src/compiler/nir/nir_opt_large_constants.c | 338 +++++++++++++++++++++
4 files changed, 344 insertions(+)
create mode 100644 src/compiler/nir/nir_opt_large_constants.c
diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources
index 0fcbc5c5c5b..9e3fbdc2612 100644
--- a/src/compiler/Makefile.sources
+++ b/src/compiler/Makefile.sources
@@ -276,6 +276,7 @@ NIR_FILES = \
nir/nir_opt_if.c \
nir/nir_opt_intrinsics.c \
nir/nir_opt_loop_unroll.c \
+ nir/nir_opt_large_constants.c \
nir/nir_opt_move_comparisons.c \
nir/nir_opt_move_load_ubo.c \
nir/nir_opt_peephole_select.c \
diff --git a/src/compiler/nir/meson.build b/src/compiler/nir/meson.build
index eb7fb7b121e..28aa8de7014 100644
--- a/src/compiler/nir/meson.build
+++ b/src/compiler/nir/meson.build
@@ -160,6 +160,7 @@ files_libnir = files(
'nir_opt_global_to_local.c',
'nir_opt_if.c',
'nir_opt_intrinsics.c',
+ 'nir_opt_large_constants.c',
'nir_opt_loop_unroll.c',
'nir_opt_move_comparisons.c',
'nir_opt_move_load_ubo.c',
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 6c49bce9aaa..106bc538427 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -2904,6 +2904,10 @@ bool nir_opt_if(nir_shader *shader);
bool nir_opt_intrinsics(nir_shader *shader);
+bool nir_opt_large_constants(nir_shader *shader,
+ int (*type_size)(const struct glsl_type *),
+ unsigned threshold);
+
bool nir_opt_loop_unroll(nir_shader *shader, nir_variable_mode indirect_mask);
bool nir_opt_move_comparisons(nir_shader *shader);
diff --git a/src/compiler/nir/nir_opt_large_constants.c b/src/compiler/nir/nir_opt_large_constants.c
new file mode 100644
index 00000000000..9dfbf9a83f8
--- /dev/null
+++ b/src/compiler/nir/nir_opt_large_constants.c
@@ -0,0 +1,338 @@
+/*
+ * Copyright © 2018 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ */
+
+#include "nir.h"
+#include "nir_builder.h"
+
+struct var_info {
+ bool is_constant;
+ bool found_read;
+};
+
+static unsigned
+struct_type_get_field_offset(const struct glsl_type *struct_type,
+ int (*type_size)(const struct glsl_type *),
+ unsigned field_idx)
+{
+ assert(glsl_type_is_struct(struct_type));
+ unsigned offset = 0;
+ for (unsigned i = 0; i < field_idx; i++)
+ offset += type_size(glsl_get_struct_field(struct_type, i));
+ return offset;
+}
+
+static nir_ssa_def *
+build_constant_load(nir_builder *b,
+ int (*type_size)(const struct glsl_type *),
+ nir_deref_instr *deref)
+{
+ const unsigned bit_size = glsl_get_bit_size(deref->type);
+ const unsigned num_components = glsl_get_vector_elements(deref->type);
+
+ nir_ssa_def *offset = nir_imm_int(b, 0);
+ while (deref->deref_type != nir_deref_type_var) {
+ nir_deref_instr *parent = nir_deref_instr_parent(deref);
+
+ if (deref->deref_type == nir_deref_type_array) {
+ nir_ssa_def *index = nir_ssa_for_src(b, deref->arr.index, 1);
+ nir_ssa_def *stride = nir_imm_int(b, type_size(deref->type));
+ offset = nir_iadd(b, offset, nir_imul(b, index, stride));
+ } else {
+ assert(deref->deref_type == nir_deref_type_struct);
+ unsigned field_offset =
+ struct_type_get_field_offset(parent->type, type_size,
+ deref->strct.index);
+ offset = nir_iadd(b, offset, nir_imm_int(b, field_offset));
+ }
+
+ deref = parent;
+ }
+
+ assert(deref->deref_type == nir_deref_type_var);
+ nir_variable *var = deref->var;
+
+ nir_intrinsic_instr *load =
+ nir_intrinsic_instr_create(b->shader, nir_intrinsic_load_constant);
+ load->num_components = num_components;
+ nir_intrinsic_set_base(load, var->data.location);
+ nir_intrinsic_set_range(load, type_size(var->type));
+ load->src[0] = nir_src_for_ssa(offset);
+ nir_ssa_dest_init(&load->instr, &load->dest,
+ num_components, bit_size, NULL);
+ nir_builder_instr_insert(b, &load->instr);
+
+ return &load->dest.ssa;
+}
+
+static void
+handle_constant_store(nir_builder *b,
+ int (*type_size)(const struct glsl_type *),
+ nir_intrinsic_instr *store)
+{
+ nir_deref_instr *deref = nir_src_as_deref(store->src[0]);
+
+ const unsigned bit_size = glsl_get_bit_size(deref->type);
+ const unsigned num_components = glsl_get_vector_elements(deref->type);
+
+ unsigned offset = 0;
+ while (deref->deref_type != nir_deref_type_var) {
+ nir_deref_instr *parent = nir_deref_instr_parent(deref);
+
+ if (deref->deref_type == nir_deref_type_array) {
+ offset += nir_src_as_const_value(deref->arr.index)->u32[0] *
+ type_size(deref->type);
+ } else {
+ assert(deref->deref_type == nir_deref_type_struct);
+ offset += struct_type_get_field_offset(parent->type, type_size,
+ deref->strct.index);
+ }
+
+ deref = parent;
+ }
+
+ assert(deref->deref_type == nir_deref_type_var);
+ offset += deref->var->data.location;
+
+ char *dst = (char *)b->shader->constant_data + offset;
+ nir_const_value *val = nir_src_as_const_value(store->src[1]);
+ switch (bit_size) {
+ case 8:
+ for (unsigned i = 0; i < num_components; i++)
+ ((uint8_t *)dst)[i] = val->u8[i];
+ break;
+
+ case 16:
+ for (unsigned i = 0; i < num_components; i++)
+ ((uint16_t *)dst)[i] = val->u16[i];
+ break;
+
+ case 32:
+ for (unsigned i = 0; i < num_components; i++)
+ ((uint32_t *)dst)[i] = val->u32[i];
+ break;
+
+ case 64:
+ for (unsigned i = 0; i < num_components; i++)
+ ((uint64_t *)dst)[i] = val->u64[i];
+ break;
+
+ default:
+ unreachable("Invalid bit size");
+ }
+}
+
+/** Lower large constant variables to shader constant data
+ *
+ * This pass looks for large (type_size(var->type) > threshold) variables
+ * which are statically constant and moves them into shader constant data.
+ * This is especially useful when large tables are baked into the shader
+ * source code because they can be moved into a UBO by the driver to reduce
+ * register pressure and make indirect access cheaper.
+ */
+bool
+nir_opt_large_constants(nir_shader *shader,
+ int (*type_size)(const struct glsl_type *),
+ unsigned threshold)
+{
+ /* This only works with a single entrypoint */
+ nir_function_impl *impl = nir_shader_get_entrypoint(shader);
+
+ /* This pass can only be run once */
+ assert(shader->constant_data == NULL && shader->num_constants == 0);
+
+ /* The index parameter is unused for local variables so we'll use it for
+ * indexing into our array of variable metadata.
+ */
+ unsigned num_locals = 0;
+ nir_foreach_variable(var, &impl->locals)
+ var->data.index = num_locals++;
+
+ struct var_info *var_infos = malloc(num_locals * sizeof(struct var_info));
+ for (unsigned i = 0; i < num_locals; i++) {
+ var_infos[i] = (struct var_info) {
+ .is_constant = true,
+ .found_read = false,
+ };
+ }
+
+ /* First, walk through the shader and figure out what variables we can
+ * lower to the constant blob.
+ */
+ bool first_block = true;
+ nir_foreach_block(block, impl) {
+ nir_foreach_instr(instr, block) {
+ if (instr->type != nir_instr_type_intrinsic)
+ continue;
+
+ nir_intrinsic_instr *intrin = nir_instr_as_intrinsic(instr);
+
+ bool src_is_const = false;
+ nir_deref_instr *src_deref = NULL, *dst_deref = NULL;
+ switch (intrin->intrinsic) {
+ case nir_intrinsic_store_deref:
+ dst_deref = nir_src_as_deref(intrin->src[0]);
+ src_is_const = nir_src_as_const_value(intrin->src[1]);
+ break;
+
+ case nir_intrinsic_load_deref:
+ src_deref = nir_src_as_deref(intrin->src[0]);
+ break;
+
+ case nir_intrinsic_copy_deref:
+ dst_deref = nir_src_as_deref(intrin->src[0]);
+ src_deref = nir_src_as_deref(intrin->src[1]);
+ src_is_const = false;
+ break;
+
+ default:
+ continue;
+ }
+
+ if (dst_deref && dst_deref->mode == nir_var_local) {
+ nir_variable *var = nir_deref_instr_get_variable(dst_deref);
+ assert(var->data.mode == nir_var_local);
+
+ /* We only consider variables constant if they only have constant
+ * stores, all the stores come before any reads, and all stores
+ * come in the first block. We also can't handle indirect stores.
+ */
+ struct var_info *info = &var_infos[var->data.index];
+ if (!src_is_const || info->found_read || !first_block ||
+ nir_deref_instr_has_indirect(dst_deref))
+ info->is_constant = false;
+ }
+
+ if (src_deref && src_deref->mode == nir_var_local) {
+ nir_variable *var = nir_deref_instr_get_variable(src_deref);
+ assert(var->data.mode == nir_var_local);
+
+ var_infos[var->data.index].found_read = true;
+ }
+ }
+ first_block = false;
+ }
+
+ shader->num_constants = 0;
+ nir_foreach_variable(var, &impl->locals) {
+ struct var_info *info = &var_infos[var->data.index];
+ if (!info->is_constant)
+ continue;
+
+ unsigned var_size = type_size(var->type);
+ if (var_size <= threshold || !info->found_read) {
+ /* Don't bother lowering small stuff or data that's never read */
+ info->is_constant = false;
+ continue;
+ }
+
+ var->data.location = shader->num_constants;
+ shader->num_constants += var_size;
+ }
+
+ if (shader->num_constants == 0) {
+ free(var_infos);
+ return false;
+ }
+
+ shader->constant_data = rzalloc_size(shader, shader->num_constants);
+
+ nir_builder b;
+ nir_builder_init(&b, impl);
+
+ nir_foreach_block(block, impl) {
+ nir_foreach_instr_safe(instr, block) {
+ if (instr->type != nir_instr_type_intrinsic)
+ continue;
+
+ nir_intrinsic_instr *intrin = nir_instr_as_intrinsic(instr);
+
+ switch (intrin->intrinsic) {
+ case nir_intrinsic_load_deref: {
+ nir_deref_instr *deref = nir_src_as_deref(intrin->src[0]);
+ if (deref->mode != nir_var_local)
+ continue;
+
+ nir_variable *var = nir_deref_instr_get_variable(deref);
+ struct var_info *info = &var_infos[var->data.index];
+ if (info->is_constant) {
+ b.cursor = nir_after_instr(&intrin->instr);
+ nir_ssa_def *val = build_constant_load(&b, type_size, deref);
+ nir_ssa_def_rewrite_uses(&intrin->dest.ssa,
+ nir_src_for_ssa(val));
+ nir_instr_remove(&intrin->instr);
+ nir_deref_instr_remove_if_unused(deref);
+ }
+ break;
+ }
+
+ case nir_intrinsic_store_deref: {
+ nir_deref_instr *deref = nir_src_as_deref(intrin->src[0]);
+ if (deref->mode != nir_var_local)
+ continue;
+
+ nir_variable *var = nir_deref_instr_get_variable(deref);
+ struct var_info *info = &var_infos[var->data.index];
+ if (info->is_constant) {
+ b.cursor = nir_after_instr(&intrin->instr);
+ handle_constant_store(&b, type_size, intrin);
+ nir_instr_remove(&intrin->instr);
+ nir_deref_instr_remove_if_unused(deref);
+ }
+ break;
+ }
+
+ case nir_intrinsic_copy_deref: {
+ nir_deref_instr *deref = nir_src_as_deref(intrin->src[1]);
+ if (deref->mode != nir_var_local)
+ continue;
+
+ nir_variable *var = nir_deref_instr_get_variable(deref);
+ struct var_info *info = &var_infos[var->data.index];
+ if (info->is_constant) {
+ b.cursor = nir_after_instr(&intrin->instr);
+ nir_ssa_def *val = build_constant_load(&b, type_size, deref);
+ nir_store_deref(&b, nir_src_as_deref(intrin->src[0]), val, ~0);
+ nir_instr_remove(&intrin->instr);
+ nir_deref_instr_remove_if_unused(deref);
+ }
+ break;
+ }
+
+ default:
+ continue;
+ }
+ }
+ }
+
+ /* Clean up the now unused variables */
+ nir_foreach_variable_safe(var, &impl->locals) {
+ if (var_infos[var->data.index].is_constant)
+ exec_node_remove(&var->node);
+ }
+
+ free(var_infos);
+
+ nir_metadata_preserve(impl, nir_metadata_block_index |
+ nir_metadata_dominance);
+ return true;
+}
--
2.17.1
More information about the mesa-dev
mailing list