[Mesa-dev] [PATCH v3 084/104] nir: Delete lower_io_types

Jason Ekstrand jason at jlekstrand.net
Tue Apr 3 18:39:28 UTC 2018


It's only used by the ir3 stand-alone compiler and Rob said we could
delete it.
---
 src/compiler/Makefile.sources                   |   1 -
 src/compiler/nir/meson.build                    |   1 -
 src/compiler/nir/nir.h                          |   1 -
 src/compiler/nir/nir_lower_io_types.c           | 176 ------------------------
 src/gallium/drivers/freedreno/ir3/ir3_cmdline.c |   1 -
 5 files changed, 180 deletions(-)
 delete mode 100644 src/compiler/nir/nir_lower_io_types.c

diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources
index 5a1b948..21b1f85 100644
--- a/src/compiler/Makefile.sources
+++ b/src/compiler/Makefile.sources
@@ -233,7 +233,6 @@ NIR_FILES = \
 	nir/nir_lower_io_arrays_to_elements.c \
 	nir/nir_lower_io_to_temporaries.c \
 	nir/nir_lower_io_to_scalar.c \
-	nir/nir_lower_io_types.c \
 	nir/nir_lower_passthrough_edgeflags.c \
 	nir/nir_lower_patch_vertices.c \
 	nir/nir_lower_phis_to_scalar.c \
diff --git a/src/compiler/nir/meson.build b/src/compiler/nir/meson.build
index 01d75f5..1c13864 100644
--- a/src/compiler/nir/meson.build
+++ b/src/compiler/nir/meson.build
@@ -128,7 +128,6 @@ files_libnir = files(
   'nir_lower_io_arrays_to_elements.c',
   'nir_lower_io_to_temporaries.c',
   'nir_lower_io_to_scalar.c',
-  'nir_lower_io_types.c',
   'nir_lower_passthrough_edgeflags.c',
   'nir_lower_patch_vertices.c',
   'nir_lower_phis_to_scalar.c',
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 153e2b7..060d673 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -2647,7 +2647,6 @@ nir_src *nir_get_io_vertex_index_src(nir_intrinsic_instr *instr);
 
 bool nir_is_per_vertex_io(const nir_variable *var, gl_shader_stage stage);
 
-void nir_lower_io_types(nir_shader *shader);
 bool nir_lower_regs_to_ssa_impl(nir_function_impl *impl);
 bool nir_lower_regs_to_ssa(nir_shader *shader);
 bool nir_lower_vars_to_ssa(nir_shader *shader);
diff --git a/src/compiler/nir/nir_lower_io_types.c b/src/compiler/nir/nir_lower_io_types.c
deleted file mode 100644
index 795bbd8..0000000
--- a/src/compiler/nir/nir_lower_io_types.c
+++ /dev/null
@@ -1,176 +0,0 @@
-/*
- * Copyright © 2016 Red Hat
- *
- * 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"
-
-/* Lower complex (struct/array/mat) input and output vars to primitive types
- * (vec4) for linking.  All indirect input/output access should already be
- * lowered (ie. nir_lower_io_to_temporaries).
- */
-
-struct lower_io_types_state {
-   nir_shader *shader;
-   struct exec_list new_ins;
-   struct exec_list new_outs;
-};
-
-static nir_variable *
-get_new_var(struct lower_io_types_state *state, nir_variable *var,
-            const struct glsl_type *deref_type, unsigned off)
-{
-   struct exec_list *list;
-
-   if (var->data.mode == nir_var_shader_in) {
-      list = &state->new_ins;
-   } else {
-      assert(var->data.mode == nir_var_shader_out);
-      list = &state->new_outs;
-   }
-
-   nir_foreach_variable(nvar, list) {
-      if (nvar->data.location == (var->data.location + off))
-         return nvar;
-   }
-
-   /* doesn't already exist, so we need to create a new one: */
-   /* TODO figure out if we need to fixup interpolation mode for int vs float
-    * components of a struct, etc..
-    */
-   nir_variable *nvar = nir_variable_create(state->shader, var->data.mode,
-                                            deref_type, NULL);
-
-   nvar->name = ralloc_asprintf(nvar, "%s@%u", var->name, off);
-   nvar->data = var->data;
-   nvar->data.location += off;
-
-   /* nir_variable_create is too clever for its own good: */
-   exec_node_remove(&nvar->node);
-   exec_node_self_link(&nvar->node);      /* no delinit() :-( */
-
-   exec_list_push_tail(list, &nvar->node);
-
-   /* remove existing var from input/output list: */
-   exec_node_remove(&var->node);
-   exec_node_self_link(&var->node);
-
-   return nvar;
-}
-
-static unsigned
-get_deref_offset(struct lower_io_types_state *state, nir_deref *tail, bool vs_in)
-{
-   unsigned offset = 0;
-
-   while (tail->child != NULL) {
-      const struct glsl_type *parent_type = tail->type;
-      tail = tail->child;
-
-      if (tail->deref_type == nir_deref_type_array) {
-         nir_deref_array *deref_array = nir_deref_as_array(tail);
-
-         /* indirect inputs/outputs should already be lowered! */
-         assert(deref_array->deref_array_type == nir_deref_array_type_direct);
-
-         unsigned size = glsl_count_attribute_slots(tail->type, vs_in);
-
-         offset += size * deref_array->base_offset;
-      } else if (tail->deref_type == nir_deref_type_struct) {
-         nir_deref_struct *deref_struct = nir_deref_as_struct(tail);
-
-         for (unsigned i = 0; i < deref_struct->index; i++) {
-            const struct glsl_type *ft = glsl_get_struct_field(parent_type, i);
-            offset += glsl_count_attribute_slots(ft, vs_in);
-         }
-      }
-   }
-
-   return offset;
-}
-
-static bool
-lower_io_types_block(struct lower_io_types_state *state, nir_block *block)
-{
-   nir_foreach_instr(instr, block) {
-      if (instr->type != nir_instr_type_intrinsic)
-         continue;
-
-      nir_intrinsic_instr *intr = nir_instr_as_intrinsic(instr);
-
-      if ((intr->intrinsic != nir_intrinsic_load_var) &&
-          (intr->intrinsic != nir_intrinsic_store_var))
-         continue;
-
-      nir_variable *var = intr->variables[0]->var;
-
-      if ((var->data.mode != nir_var_shader_in) &&
-          (var->data.mode != nir_var_shader_out))
-         continue;
-
-      bool vs_in = (state->shader->info.stage == MESA_SHADER_VERTEX) &&
-                   (var->data.mode == nir_var_shader_in);
-      if (glsl_count_attribute_slots(var->type, vs_in) == 1)
-         continue;
-
-      unsigned off = get_deref_offset(state, &intr->variables[0]->deref, vs_in);
-      const struct glsl_type *deref_type =
-         nir_deref_tail(&intr->variables[0]->deref)->type;
-      nir_variable *nvar = get_new_var(state, var, deref_type, off);
-
-      /* and then re-write the load/store_var deref: */
-      intr->variables[0] = nir_deref_var_create(intr, nvar);
-   }
-
-   return true;
-}
-
-static void
-lower_io_types_impl(nir_function_impl *impl, struct lower_io_types_state *state)
-{
-   nir_foreach_block(block, impl) {
-      lower_io_types_block(state, block);
-   }
-
-   nir_metadata_preserve(impl, nir_metadata_block_index |
-                               nir_metadata_dominance);
-}
-
-
-void
-nir_lower_io_types(nir_shader *shader)
-{
-   struct lower_io_types_state state;
-
-   state.shader = shader;
-   exec_list_make_empty(&state.new_ins);
-   exec_list_make_empty(&state.new_outs);
-
-   nir_foreach_function(function, shader) {
-      if (function->impl)
-         lower_io_types_impl(function->impl, &state);
-   }
-
-   /* move new in/out vars to shader's lists: */
-   exec_list_append(&shader->inputs, &state.new_ins);
-   exec_list_append(&shader->outputs, &state.new_outs);
-}
diff --git a/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c b/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c
index 07e97c3..a79588e 100644
--- a/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c
+++ b/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c
@@ -126,7 +126,6 @@ load_glsl(unsigned num_files, char* const* files, gl_shader_stage stage)
 
 	NIR_PASS_V(nir, nir_split_var_copies);
 	NIR_PASS_V(nir, nir_lower_var_copies);
-	NIR_PASS_V(nir, nir_lower_io_types);
 
 	switch (stage) {
 	case MESA_SHADER_VERTEX:
-- 
2.5.0.400.gff86faf



More information about the mesa-dev mailing list