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

Author: Vinson Lee <[email protected]>
Date:   Mon Oct 26 19:55:07 2020 -0700

gallium: Remove duplicate resource variable.

Fix defect reported by Coverity Scan.

Evaluation order violation (EVALUATION_ORDER)
write_write_typo: In resource = resource =
ntt_ureg_src_indirect(c, ureg_src_register(TGSI_FILE_IMAGE, 0U),
instr->src[0]), resource is written twice with the same value.

Fixes: 34cc6a804ec9 ("gallium: Add a nir-to-TGSI pass.")
Signed-off-by: Vinson Lee <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7326>

---

 src/gallium/auxiliary/nir/nir_to_tgsi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/gallium/auxiliary/nir/nir_to_tgsi.c 
b/src/gallium/auxiliary/nir/nir_to_tgsi.c
index 231192792c1..d74eafd06a3 100644
--- a/src/gallium/auxiliary/nir/nir_to_tgsi.c
+++ b/src/gallium/auxiliary/nir/nir_to_tgsi.c
@@ -1222,8 +1222,8 @@ ntt_emit_image_load_store(struct ntt_compile *c, 
nir_intrinsic_instr *instr)
                                    nir_intrinsic_image_array(instr));
 
    struct ureg_src resource =
-      resource = ntt_ureg_src_indirect(c, ureg_src_register(TGSI_FILE_IMAGE, 
0),
-                                       instr->src[0]);
+      ntt_ureg_src_indirect(c, ureg_src_register(TGSI_FILE_IMAGE, 0),
+                            instr->src[0]);
 
    struct ureg_dst dst;
    if (instr->intrinsic == nir_intrinsic_image_store) {

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to