nir: fix assert for wildcard pairs
authorThomas Hindoe Paaboel Andersen <phomes@gmail.com>
Wed, 4 May 2016 03:48:39 +0000 (05:48 +0200)
committerEduardo Lima Mitev <elima@igalia.com>
Thu, 5 May 2016 07:33:02 +0000 (09:33 +0200)
The assert was null checking dest_arr_parent twice. The intention
seems to be to check both dest_ and src_.

Added in d3636da9

Reviewed-by: Eduardo Lima Mitev <elima@igalia.com>
src/compiler/nir/nir_lower_var_copies.c

index 707d5af..1a7e2ee 100644 (file)
@@ -85,7 +85,7 @@ emit_copy_load_store(nir_intrinsic_instr *copy_instr,
 
    if (src_arr_parent || dest_arr_parent) {
       /* Wildcards had better come in matched pairs */
-      assert(dest_arr_parent && dest_arr_parent);
+      assert(src_arr_parent && dest_arr_parent);
 
       nir_deref_array *src_arr = nir_deref_as_array(src_arr_parent->child);
       nir_deref_array *dest_arr = nir_deref_as_array(dest_arr_parent->child);