<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 9, 2015 at 11:34 AM, Connor Abbott <span dir="ltr"><<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">BTW, now that we're adding an extra possible source, do we need to<br>
expand the size of the source array?<br></blockquote><div><br></div><div>I don't know that we've added anything that wasn't there before.  Then again, it's not documented how you came to the number 4.  Also, I came to the conclusion that I don't like this patch anyway, so I'll probably drop it in favor of just memsetting the source to 0.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Anyways,<br>
Reviewed-by: Connor Abbott <<a href="mailto:cwabbott0@gmail.com">cwabbott0@gmail.com</a>><br>
<div><div class="h5"><br>
On Wed, Jan 7, 2015 at 9:03 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> This helps a lot with things like lowering passes that may need to add<br>
> sources.<br>
> ---<br>
>  src/glsl/nir/nir.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c<br>
> index 9b99c9e..c8d354d 100644<br>
> --- a/src/glsl/nir/nir.c<br>
> +++ b/src/glsl/nir/nir.c<br>
> @@ -458,7 +458,7 @@ nir_tex_instr_create(void *mem_ctx, unsigned num_srcs)<br>
>     dest_init(&instr->dest);<br>
><br>
>     instr->num_srcs = num_srcs;<br>
> -   for (unsigned i = 0; i < num_srcs; i++)<br>
> +   for (unsigned i = 0; i < 4; i++)<br>
>        src_init(&instr->src[i]);<br>
><br>
>     instr->has_predicate = false;<br>
> --<br>
> 2.2.0<br>
><br>
</div></div>> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>