Re: [Mesa-dev] [PATCH 2/2] i965: Delete linked GLSL IR when using NIR.

2015-06-26 Thread Kenneth Graunke
On Thursday, June 25, 2015 02:29:13 PM Tapani Pälli wrote: > Huh I see this went in already, I've noticed a problem and thought to > share it. > > Currently program resource list (used by gl api shader queries) is > generated in linker, before backend LinkShader call. What this means is > that

Re: [Mesa-dev] [PATCH 2/2] i965: Delete linked GLSL IR when using NIR.

2015-06-25 Thread Tapani Pälli
Huh I see this went in already, I've noticed a problem and thought to share it. Currently program resource list (used by gl api shader queries) is generated in linker, before backend LinkShader call. What this means is that it relies on frontend optimization passes to get rid of dead inputs a

Re: [Mesa-dev] [PATCH 2/2] i965: Delete linked GLSL IR when using NIR.

2015-06-11 Thread Jason Ekstrand
On Thu, Jun 11, 2015 at 12:41 AM, Tapani Pälli wrote: > This is based on Kenneth's patch to delete 'most of the IR'. Due to > linker changes to clone variables, we can now free all of IR. > > Saves 58MB of memory when replaying a Dota 2 trace on Broadwell. I think we've saved ~50 MB 3 times now o

Re: [Mesa-dev] [PATCH 2/2] i965: Delete linked GLSL IR when using NIR.

2015-06-11 Thread Kenneth Graunke
On Thursday, June 11, 2015 10:41:53 AM Tapani Pälli wrote: > This is based on Kenneth's patch to delete 'most of the IR'. Due to > linker changes to clone variables, we can now free all of IR. > > Saves 58MB of memory when replaying a Dota 2 trace on Broadwell. > > Signed-off-by: Tapani Pälli >

[Mesa-dev] [PATCH 2/2] i965: Delete linked GLSL IR when using NIR.

2015-06-11 Thread Tapani Pälli
This is based on Kenneth's patch to delete 'most of the IR'. Due to linker changes to clone variables, we can now free all of IR. Saves 58MB of memory when replaying a Dota 2 trace on Broadwell. Signed-off-by: Tapani Pälli --- src/mesa/drivers/dri/i965/brw_shader.cpp | 5 - 1 file changed,