nir: remove old assert
authorTimothy Arceri <tarceri@itsqueeze.com>
Wed, 21 Feb 2018 03:36:09 +0000 (14:36 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Wed, 21 Feb 2018 22:31:00 +0000 (09:31 +1100)
This was originally intended to make sure the remap location
was not -1. However the code has changed alot since then,
the location is now never set to -1 and we also handle
components meaning this old assert has been doing comparisions
with the pointer to the array of component data.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105183

src/compiler/nir/nir_linking_helpers.c

index 6459c6a24d2548724c0121b8b0548e4ad68441ca..2b0a2668a33390f1f321f3739bcd11448a93fafa 100644 (file)
@@ -283,7 +283,6 @@ remap_slots_and_components(struct exec_list *var_list, gl_shader_stage stage,
       if (var->data.location >= VARYING_SLOT_VAR0 &&
           var->data.location - VARYING_SLOT_VAR0 < 32) {
          assert(var->data.location - VARYING_SLOT_VAR0 < 32);
-         assert(remap[var->data.location - VARYING_SLOT_VAR0] >= 0);
 
          const struct glsl_type *type = var->type;
          if (nir_is_per_vertex_io(var, stage)) {