Re: [Mesa-dev] [PATCH] nir/sweep: Reparent the shader name

2015-10-08 Thread Neil Roberts
Oops, I just made a similar patch without noticing this one. Feel free to take the commit message from my patch if you want. Either way this one is: Reviewed-by: Neil Roberts http://patchwork.freedesktop.org/patch/61369/ Sorry for the noise. Regards, - Neil Jason

[Mesa-dev] [PATCH] nir/sweep: Reparent the shader name

2015-10-07 Thread Jason Ekstrand
--- src/glsl/nir/nir_sweep.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/glsl/nir/nir_sweep.c b/src/glsl/nir/nir_sweep.c index d354975..b6ce43b 100644 --- a/src/glsl/nir/nir_sweep.c +++ b/src/glsl/nir/nir_sweep.c @@ -154,6 +154,8 @@ nir_sweep(nir_shader *nir) /* First, move

Re: [Mesa-dev] [PATCH] nir/sweep: Reparent the shader name

2015-10-07 Thread Matt Turner
On Wed, Oct 7, 2015 at 8:30 PM, Jason Ekstrand wrote: > --- Because otherwise it was being freed? > src/glsl/nir/nir_sweep.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/glsl/nir/nir_sweep.c b/src/glsl/nir/nir_sweep.c > index d354975..b6ce43b 100644 >

Re: [Mesa-dev] [PATCH] nir/sweep: Reparent the shader name

2015-10-07 Thread Matt Turner
On Wed, Oct 7, 2015 at 8:45 PM, Jason Ekstrand wrote: > On Oct 7, 2015 8:40 PM, "Matt Turner" wrote: >> On Wed, Oct 7, 2015 at 8:30 PM, Jason Ekstrand >> wrote: >> > --- >> >> Because otherwise it was being freed? > > Yes Cool.

Re: [Mesa-dev] [PATCH] nir/sweep: Reparent the shader name

2015-10-07 Thread Jason Ekstrand
On Oct 7, 2015 8:40 PM, "Matt Turner" wrote: > > On Wed, Oct 7, 2015 at 8:30 PM, Jason Ekstrand wrote: > > --- > > Because otherwise it was being freed? Yes > > src/glsl/nir/nir_sweep.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git