Mesa (master): anv/pipeline: Only call remove_dead_variables once

Jason Ekstrand jekstrand at kemper.freedesktop.org
Sat Jan 7 23:45:39 UTC 2017


Module: Mesa
Branch: master
Commit: 43e0b0d4b255d910616c10e3e01bfec5db469e0e
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=43e0b0d4b255d910616c10e3e01bfec5db469e0e

Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Fri Jan  6 17:25:35 2017 -0800

anv/pipeline: Only call remove_dead_variables once

It can handle multiple modes at a time now so there's no reason to call
it repeatedly.

Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>

---

 src/intel/vulkan/anv_pipeline.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c
index db35d70..fadc76a 100644
--- a/src/intel/vulkan/anv_pipeline.c
+++ b/src/intel/vulkan/anv_pipeline.c
@@ -157,9 +157,9 @@ anv_shader_compile_to_nir(struct anv_device *device,
    assert(exec_list_length(&nir->functions) == 1);
    entry_point->name = ralloc_strdup(entry_point, "main");
 
-   nir_remove_dead_variables(nir, nir_var_shader_in);
-   nir_remove_dead_variables(nir, nir_var_shader_out);
-   nir_remove_dead_variables(nir, nir_var_system_value);
+   nir_remove_dead_variables(nir, nir_var_shader_in |
+                                  nir_var_shader_out |
+                                  nir_var_system_value);
    nir_validate_shader(nir);
 
    /* Now that we've deleted all but the main function, we can go ahead and




More information about the mesa-commit mailing list