st/glsl_to_nir: remove dead local variables
authorTimothy Arceri <tarceri@itsqueeze.com>
Thu, 31 Jan 2019 04:59:18 +0000 (15:59 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Fri, 1 Feb 2019 04:56:02 +0000 (15:56 +1100)
Without this we do not end up with a deterministic NIR because
temporary register variables are added in random order. NIR must
be deterministic because we use it to produce a sha for the
radeonsi backends disk cache.

This fixes the shader cache for a bunch of shaders.

Another positive is that this results in a large reduction in the
size of the NIR that the state tracker stores to the disk cache.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/state_tracker/st_glsl_to_nir.cpp

index d7f2e3e6eaa98a201a0134f946f2d6526ace3b7a..271a30eab708ebc9cdf203294967cdc810b8589d 100644 (file)
@@ -463,6 +463,9 @@ st_glsl_to_nir_post_opts(struct st_context *st, struct gl_program *prog,
    NIR_PASS_V(nir, st_nir_lower_builtin);
    NIR_PASS_V(nir, gl_nir_lower_atomics, shader_program, true);
 
+   nir_variable_mode mask = nir_var_function_temp;
+   nir_remove_dead_variables(nir, mask);
+
    if (st->ctx->_Shader->Flags & GLSL_DUMP) {
       _mesa_log("\n");
       _mesa_log("NIR IR for linked %s program %d:\n",