summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorConnor Abbott <cwabbott0@gmail.com>2019-08-22 13:21:17 +0200
committerConnor Abbott <cwabbott0@gmail.com>2019-08-23 11:05:31 +0200
commit97d592c855ce10e465900ede4db0dcea3c699ec7 (patch)
treea695ca8f4f3c2f3b280ba43c068ca586e343d8f4
parent3eb4aeed6042693b861e710d47624af010d53bfb (diff)
radeonsi/nir: Don't recompute num_inputs and num_outputs
Don't repeat what mesa/st already does. Reviewed-by: Marek Olšák <marek.olsak@amd.com>
-rw-r--r--src/gallium/drivers/radeonsi/si_shader_nir.c27
1 files changed, 3 insertions, 24 deletions
diff --git a/src/gallium/drivers/radeonsi/si_shader_nir.c b/src/gallium/drivers/radeonsi/si_shader_nir.c
index c6ca057ea9fa..0589b6278bf9 100644
--- a/src/gallium/drivers/radeonsi/si_shader_nir.c
+++ b/src/gallium/drivers/radeonsi/si_shader_nir.c
@@ -518,13 +518,8 @@ void si_nir_scan_shader(const struct nir_shader *nir,
* tracker has already mapped them to attributes via
* variable->data.driver_location.
*/
- if (nir->info.stage == MESA_SHADER_VERTEX) {
- processed_inputs |= 1ull << i;
-
- if (glsl_type_is_dual_slot(glsl_without_array(variable->type)))
- processed_inputs |= 2ull << i;
+ if (nir->info.stage == MESA_SHADER_VERTEX)
continue;
- }
for (unsigned j = 0; j < attrib_count; j++, i++) {
@@ -587,8 +582,6 @@ void si_nir_scan_shader(const struct nir_shader *nir,
}
}
- i = 0;
- uint64_t processed_outputs = 0;
nir_foreach_variable(variable, &nir->outputs) {
unsigned semantic_name, semantic_index;
@@ -683,14 +676,6 @@ void si_nir_scan_shader(const struct nir_shader *nir,
info->num_stream_output_components[streamw]++;
}
- /* make sure we only count this location once against
- * the num_outputs counter.
- */
- if (processed_outputs & ((uint64_t)1 << i))
- continue;
-
- processed_outputs |= ((uint64_t)1 << i);
-
info->output_semantic_name[i] = semantic_name;
info->output_semantic_index[i] = semantic_index;
@@ -740,14 +725,8 @@ void si_nir_scan_shader(const struct nir_shader *nir,
}
}
- info->num_inputs = util_last_bit64(processed_inputs);
- info->num_outputs = util_last_bit64(processed_outputs);
-
- /* Inputs and outputs can't have holes. If this fails, use
- * nir_assign_io_var_locations to re-assign driver_location.
- */
- assert(processed_inputs == u_bit_consecutive64(0, info->num_inputs));
- assert(processed_outputs == u_bit_consecutive64(0, info->num_outputs));
+ info->num_inputs = nir->num_inputs;
+ info->num_outputs = nir->num_outputs;
struct set *ubo_set = _mesa_set_create(NULL, _mesa_hash_pointer,
_mesa_key_pointer_equal);