On Fri, Jan 9, 2015 at 11:34 AM, Connor Abbott <cwabbo...@gmail.com> wrote:

> BTW, now that we're adding an extra possible source, do we need to
> expand the size of the source array?
>

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.


>
> Anyways,
> Reviewed-by: Connor Abbott <cwabbo...@gmail.com>
>
> On Wed, Jan 7, 2015 at 9:03 PM, Jason Ekstrand <ja...@jlekstrand.net>
> wrote:
> > This helps a lot with things like lowering passes that may need to add
> > sources.
> > ---
> >  src/glsl/nir/nir.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c
> > index 9b99c9e..c8d354d 100644
> > --- a/src/glsl/nir/nir.c
> > +++ b/src/glsl/nir/nir.c
> > @@ -458,7 +458,7 @@ nir_tex_instr_create(void *mem_ctx, unsigned
> num_srcs)
> >     dest_init(&instr->dest);
> >
> >     instr->num_srcs = num_srcs;
> > -   for (unsigned i = 0; i < num_srcs; i++)
> > +   for (unsigned i = 0; i < 4; i++)
> >        src_init(&instr->src[i]);
> >
> >     instr->has_predicate = false;
> > --
> > 2.2.0
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to