Mesa (master): nir/load_const_to_scalar: Get rid of a bit size switch statement

[GitLab Mirror]

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

Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Wed Mar 27 18:31:01 2019 -0500

nir/load_const_to_scalar: Get rid of a bit size switch statement

Now that nir_const_value is a scalar, we don't need the switch on bit
size in order to pluck off components properly.

Reviewed-by: Karol Herbst <kherbst at redhat.com>

---

 src/compiler/nir/nir_lower_load_const_to_scalar.c | 20 +-------------------
 1 file changed, 1 insertion(+), 19 deletions(-)

diff --git a/src/compiler/nir/nir_lower_load_const_to_scalar.c b/src/compiler/nir/nir_lower_load_const_to_scalar.c
index 47dc83613ef..a65dcb06d0c 100644
--- a/src/compiler/nir/nir_lower_load_const_to_scalar.c
+++ b/src/compiler/nir/nir_lower_load_const_to_scalar.c
@@ -50,25 +50,7 @@ lower_load_const_instr_scalar(nir_load_const_instr *lower)
    for (unsigned i = 0; i < lower->def.num_components; i++) {
       nir_load_const_instr *load_comp =
          nir_load_const_instr_create(b.shader, 1, lower->def.bit_size);
-      switch (lower->def.bit_size) {
-      case 64:
-         load_comp->value[0].u64 = lower->value[i].u64;
-         break;
-      case 32:
-         load_comp->value[0].u32 = lower->value[i].u32;
-         break;
-      case 16:
-         load_comp->value[0].u16 = lower->value[i].u16;
-         break;
-      case 8:
-         load_comp->value[0].u8 = lower->value[i].u8;
-         break;
-      case 1:
-         load_comp->value[0].b = lower->value[i].b;
-         break;
-      default:
-         assert(!"invalid bit size");
-      }
+      load_comp->value[0] = lower->value[i];
       nir_builder_instr_insert(&b, &load_comp->instr);
       loads[i] = &load_comp->def;
    }