<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 4, 2016 at 7:46 AM, Eduardo Lima Mitev <span dir="ltr"><<a href="mailto:elima@igalia.com" target="_blank">elima@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Good catch!<br>
<br>
Reviewed-by: Eduardo Lima Mitev <<a href="mailto:elima@igalia.com">elima@igalia.com</a>><br></blockquote><div><br></div><div>Thanks! I do not have commit access. Can I ask you to push? <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On 05/04/2016 05:48 AM, Thomas Hindoe Paaboel Andersen wrote:<br>
> The assert was null checking dest_arr_parent twice. The intention<br>
> seems to be to check both dest_ and src_.<br>
><br>
> Added in d3636da9<br>
> ---<br>
> v2:<br>
>   Fix the assert rather than checking both in the if(). Hat tip to Ilia.<br>
>  src/compiler/nir/nir_lower_var_copies.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/compiler/nir/nir_lower_var_copies.c b/src/compiler/nir/nir_lower_var_copies.c<br>
> index 707d5af..1a7e2ee 100644<br>
> --- a/src/compiler/nir/nir_lower_var_copies.c<br>
> +++ b/src/compiler/nir/nir_lower_var_copies.c<br>
> @@ -85,7 +85,7 @@ emit_copy_load_store(nir_intrinsic_instr *copy_instr,<br>
><br>
>     if (src_arr_parent || dest_arr_parent) {<br>
>        /* Wildcards had better come in matched pairs */<br>
> -      assert(dest_arr_parent && dest_arr_parent);<br>
> +      assert(src_arr_parent && dest_arr_parent);<br>
><br>
>        nir_deref_array *src_arr = nir_deref_as_array(src_arr_parent->child);<br>
>        nir_deref_array *dest_arr = nir_deref_as_array(dest_arr_parent->child);<br>
><br>
<br>
</div></div></blockquote></div><br></div></div>