On Saturday, August 26, 2017 9:42:17 PM PDT Ilia Mirkin wrote: > The is_double_vertex_input needs to be set for arrays of doubles as > well. > > Fixes KHR-GL45.enhanced_layouts.varying_array_locations > > Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu> > Cc: mesa-sta...@lists.freedesktop.org > --- > src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp > b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp > index 9f021962e40..49e97004703 100644 > --- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp > +++ b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp > @@ -2718,7 +2718,7 @@ glsl_to_tgsi_visitor::visit(ir_dereference_variable *ir) > > this->result = st_src_reg(entry->file, entry->index, var->type, > entry->component, entry->array_id); > - if (this->shader->Stage == MESA_SHADER_VERTEX && var->data.mode == > ir_var_shader_in && var->type->is_double()) > + if (this->shader->Stage == MESA_SHADER_VERTEX && var->data.mode == > ir_var_shader_in && var->type->without_array()->is_double()) > this->result.is_double_vertex_input = true; > if (!native_integers) > this->result.type = GLSL_TYPE_FLOAT; >
Makes sense to me - structures don't exist as VS inputs, so unwrapping arrays should be sufficient. Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev