Re: [Mesa-dev] [PATCH 1/3] glsl: Use hash tables in opt_constant_variable().

2015-09-08 Thread Tapani Pälli
On 09/08/2015 12:34 PM, Kenneth Graunke wrote: On Tuesday, September 08, 2015 08:15:11 AM Tapani Pälli wrote: I did not get as impressive results with these 3 patches (how did you measure?) but on my machine (HSW GT2) complete shader_runner time goes from ~83 secs to ~70secs so it is definitel

Re: [Mesa-dev] [PATCH 1/3] glsl: Use hash tables in opt_constant_variable().

2015-09-08 Thread Kenneth Graunke
On Tuesday, September 08, 2015 08:15:11 AM Tapani Pälli wrote: > I did not get as impressive results with these 3 patches (how did you > measure?) but on my machine (HSW GT2) complete shader_runner time goes > from ~83 secs to ~70secs so it is definitely improving. > > I simply do 'time bin/shad

Re: [Mesa-dev] [PATCH 1/3] glsl: Use hash tables in opt_constant_variable().

2015-09-07 Thread Tapani Pälli
I did not get as impressive results with these 3 patches (how did you measure?) but on my machine (HSW GT2) complete shader_runner time goes from ~83 secs to ~70secs so it is definitely improving. I simply do 'time bin/shader_runner' for the measurement. Tested-by: Tapani Pälli On 09/05/2015

Re: [Mesa-dev] [PATCH 1/3] glsl: Use hash tables in opt_constant_variable().

2015-09-05 Thread Timothy Arceri
On Sat, 2015-09-05 at 02:21 -0700, Kenneth Graunke wrote: > Cuts compile/link time of the fragment shader in bug #91857 by 31% > (31.79 -> 21.64). It has over 8,000 variables so linked lists are > terrible. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91857 > Signed-off-by: Kenneth

[Mesa-dev] [PATCH 1/3] glsl: Use hash tables in opt_constant_variable().

2015-09-05 Thread Kenneth Graunke
Cuts compile/link time of the fragment shader in bug #91857 by 31% (31.79 -> 21.64). It has over 8,000 variables so linked lists are terrible. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91857 Signed-off-by: Kenneth Graunke --- src/glsl/opt_constant_variable.cpp | 39