From c8a3bac8205108e47cbff05d0c19c53965c0e6bd Mon Sep 17 00:00:00 2001 From: Timothy Arceri Date: Tue, 23 May 2017 22:01:53 +1000 Subject: [PATCH] mesa: remove unrequired double calc MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit type_size() will already handle this correctly for us. Reviewed-by: Marek Olšák --- src/mesa/program/ir_to_mesa.cpp | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp index 3299ab722a2..4c30bea2f23 100644 --- a/src/mesa/program/ir_to_mesa.cpp +++ b/src/mesa/program/ir_to_mesa.cpp @@ -2445,20 +2445,10 @@ add_uniform_to_shader::visit_field(const glsl_type *type, const char *name, const enum glsl_interface_packing, bool /* last_field */) { - unsigned int size; - /* atomics don't get real storage */ if (type->contains_atomic()) return; - if (type->is_vector() || type->is_scalar()) { - size = type->vector_elements; - if (type->is_64bit()) - size *= 2; - } else { - size = type_size(type) * 4; - } - gl_register_file file; if (type->without_array()->is_sampler()) { file = PROGRAM_SAMPLER; @@ -2468,6 +2458,8 @@ add_uniform_to_shader::visit_field(const glsl_type *type, const char *name, int index = _mesa_lookup_parameter_index(params, name); if (index < 0) { + unsigned size = type_size(type) * 4; + index = _mesa_add_parameter(params, file, name, size, type->gl_type, NULL, NULL); -- 2.30.2