spirv: drop some dead code
authorRob Clark <robdclark@chromium.org>
Tue, 16 Jun 2020 18:23:26 +0000 (11:23 -0700)
committerMarge Bot <eric+marge@anholt.net>
Thu, 18 Jun 2020 03:40:54 +0000 (03:40 +0000)
This case is never hit, we don't have a nir intrinsic for this spirv
opcode.  And when we do, I'm not sure if it would be vectorized or not.
So best just to drop this case.

Signed-off-by: Rob Clark <robdclark@chromium.org>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5505>

src/compiler/spirv/spirv_to_nir.c

index 6fac715136c432a675967dbc90817c5dcb013d32..7565590fbd83088b174ebc5dc86cb0a024c534b4 100644 (file)
@@ -3116,7 +3116,6 @@ vtn_handle_atomics(struct vtn_builder *b, SpvOp opcode,
    /* uniform as "atomic counter uniform" */
    if (ptr->mode == vtn_variable_mode_uniform) {
       nir_deref_instr *deref = vtn_pointer_to_deref(b, ptr);
-      const struct glsl_type *deref_type = deref->type;
       nir_intrinsic_op op = get_uniform_nir_atomic_op(b, opcode);
       atomic = nir_intrinsic_instr_create(b->nb.shader, op);
       atomic->src[0] = nir_src_for_ssa(&deref->dest.ssa);
@@ -3127,11 +3126,6 @@ vtn_handle_atomics(struct vtn_builder *b, SpvOp opcode,
        */
 
       switch (opcode) {
-      case SpvOpAtomicStore:
-         atomic->num_components = glsl_get_vector_elements(deref_type);
-         nir_intrinsic_set_write_mask(atomic, (1 << atomic->num_components) - 1);
-         break;
-
       case SpvOpAtomicLoad:
       case SpvOpAtomicExchange:
       case SpvOpAtomicCompareExchange: