<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 3, 2016 at 9:46 PM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, May 3, 2016 at 3:32 PM, Thomas Hindoe Paaboel Andersen<br>
<<a href="mailto:phomes@gmail.com">phomes@gmail.com</a>> wrote:<br>
> It seems that the intention was to check both for null as they are<br>
> dereferenced immediately after.<br>
><br>
> Added in d3636da9<br>
> ---<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..20b78d8 100644<br>
> --- a/src/compiler/nir/nir_lower_var_copies.c<br>
> +++ b/src/compiler/nir/nir_lower_var_copies.c<br>
> @@ -83,7 +83,7 @@ emit_copy_load_store(nir_intrinsic_instr *copy_instr,<br>
>     nir_deref *src_arr_parent = deref_next_wildcard_parent(src_tail);<br>
>     nir_deref *dest_arr_parent = deref_next_wildcard_parent(dest_tail);<br>
><br>
> -   if (src_arr_parent || dest_arr_parent) {<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>
<br>
</span>Actually I'm guessing that the assert was supposed to be<br>
src_arr_parent && dest_arr_parent. And the case where only one of them<br>
is null and the other is not should never happen.</blockquote><div><br></div><div>Yep, my mistake. That makes a lot more sense. What now? Do I send an updated patch or will you push your fix? No sure what the process is here.<br></div></div></div></div>