On Thursday, January 19, 2017 1:16:18 PM PST Rob Clark wrote:
> glsl_to_nir expects matrix ops to be lowered away to vector by the time
> it runs, but standalone wasn't bothering to call this pass.
> 
> Signed-off-by: Rob Clark <robdcl...@gmail.com>
> ---
>  src/compiler/glsl/standalone.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/compiler/glsl/standalone.cpp 
> b/src/compiler/glsl/standalone.cpp
> index 9a8d75d..20fe231 100644
> --- a/src/compiler/glsl/standalone.cpp
> +++ b/src/compiler/glsl/standalone.cpp
> @@ -381,6 +381,7 @@ compile_shader(struct gl_context *ctx, struct gl_shader 
> *shader)
>        new(shader) _mesa_glsl_parse_state(ctx, shader->Stage, shader);
>  
>     _mesa_glsl_compile_shader(ctx, shader, options->dump_ast, 
> options->dump_hir);
> +   do_mat_op_to_vec(shader->ir);
>  
>     /* Print out the resulting IR */
>     if (!state->error && options->dump_lir) {
> 

The standalone compiler can optionally link shaders.  You don't want to
lower matrices prior to linking, as that could break varying matching...

Attachment: 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

Reply via email to