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

Author: Caio Marcelo de Oliveira Filho <[email protected]>
Date:   Mon Oct  5 14:25:10 2020 -0700

nir: Use a switch in nir_lower_explicit_io_instr

Reviewed-by: Jason Ekstrand <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7255>

---

 src/compiler/nir/nir_lower_io.c | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c
index 318e2a020fc..cddfaada155 100644
--- a/src/compiler/nir/nir_lower_io.c
+++ b/src/compiler/nir/nir_lower_io.c
@@ -1468,7 +1468,8 @@ nir_lower_explicit_io_instr(nir_builder *b,
       align_offset = 0;
    }
 
-   if (intrin->intrinsic == nir_intrinsic_load_deref) {
+   switch (intrin->intrinsic) {
+   case nir_intrinsic_load_deref: {
       nir_ssa_def *value;
       if (vec_stride > scalar_size) {
          nir_ssa_def *comps[NIR_MAX_VEC_COMPONENTS] = { NULL, };
@@ -1489,7 +1490,10 @@ nir_lower_explicit_io_instr(nir_builder *b,
                                         intrin->num_components);
       }
       nir_ssa_def_rewrite_uses(&intrin->dest.ssa, nir_src_for_ssa(value));
-   } else if (intrin->intrinsic == nir_intrinsic_store_deref) {
+      break;
+   }
+
+   case nir_intrinsic_store_deref: {
       assert(intrin->src[1].is_ssa);
       nir_ssa_def *value = intrin->src[1].ssa;
       nir_component_mask_t write_mask = nir_intrinsic_write_mask(intrin);
@@ -1511,10 +1515,15 @@ nir_lower_explicit_io_instr(nir_builder *b,
                                  align_mul, align_offset,
                                  value, write_mask);
       }
-   } else {
+      break;
+   }
+
+   default: {
       nir_ssa_def *value =
          build_explicit_io_atomic(b, intrin, addr, addr_format);
       nir_ssa_def_rewrite_uses(&intrin->dest.ssa, nir_src_for_ssa(value));
+      break;
+   }
    }
 
    nir_instr_remove(&intrin->instr);

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

Reply via email to