Mesa (main): microsoft/compiler: dxil_nir_opt_alu_deref_srcs: return progress

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Dec 23 02:05:48 UTC 2021


Module: Mesa
Branch: main
Commit: e0ec818cfd86815ee9a3de62e1596c83bb791173
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e0ec818cfd86815ee9a3de62e1596c83bb791173

Author: Thomas H.P. Andersen <phomes at gmail.com>
Date:   Wed Dec 22 14:16:31 2021 +0100

microsoft/compiler: dxil_nir_opt_alu_deref_srcs: return progress

dxil_nir_opt_alu_deref_srcs will always return false because
the progress variable is declared both for the function and also
inside the loop.

Spotted by a unused-but-set-variable warning from clang

Reviewed-by: Jesse Natalie <jenatali at microsoft.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14290>

---

 src/microsoft/compiler/dxil_nir.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/microsoft/compiler/dxil_nir.c b/src/microsoft/compiler/dxil_nir.c
index 9c5a1a2d66d..1db5dd0e44d 100644
--- a/src/microsoft/compiler/dxil_nir.c
+++ b/src/microsoft/compiler/dxil_nir.c
@@ -934,7 +934,6 @@ dxil_nir_opt_alu_deref_srcs(nir_shader *nir)
          continue;
       assert(func->impl);
 
-      bool progress = false;
       nir_builder b;
       nir_builder_init(&b, func->impl);
 



More information about the mesa-commit mailing list