Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Matt Turner
On Sat, May 16, 2015 at 1:07 PM, Jason Ekstrand wrote: > I found a variety of places where the user was doing, for instance, 2 > muls and 4 adds where the result of each mul is used twice. The > result is 6 instructions instead of just the 4 mad's. It's entirely > possible that, thanks to latanc

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Jason Ekstrand
On Sat, May 16, 2015 at 1:01 PM, Jason Ekstrand wrote: > On Sat, May 16, 2015 at 12:59 PM, Matt Turner wrote: >> On Sat, May 16, 2015 at 12:45 PM, Jason Ekstrand >> wrote: >>> On Sat, May 16, 2015 at 12:12 PM, Matt Turner wrote: On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke wrote

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Jason Ekstrand
On Sat, May 16, 2015 at 12:59 PM, Matt Turner wrote: > On Sat, May 16, 2015 at 12:45 PM, Jason Ekstrand wrote: >> On Sat, May 16, 2015 at 12:12 PM, Matt Turner wrote: >>> On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke >>> wrote: Looking at a couple of the shaders that are still worse off

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Matt Turner
On Sat, May 16, 2015 at 12:45 PM, Jason Ekstrand wrote: > On Sat, May 16, 2015 at 12:12 PM, Matt Turner wrote: >> On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke >> wrote: >>> Looking at a couple of the shaders that are still worse off...it looks >>> like a ton of Source shaders used to do MUL/

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Jason Ekstrand
On Sat, May 16, 2015 at 12:12 PM, Matt Turner wrote: > On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke wrote: >> Looking at a couple of the shaders that are still worse off...it looks >> like a ton of Source shaders used to do MUL/ADD with an attribute and >> two immediates, and now are doing MOV

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Ilia Mirkin
On Sat, May 16, 2015 at 3:12 PM, Matt Turner wrote: > On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke wrote: >> Looking at a couple of the shaders that are still worse off...it looks >> like a ton of Source shaders used to do MUL/ADD with an attribute and >> two immediates, and now are doing MOV/

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-16 Thread Matt Turner
On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke wrote: > Looking at a couple of the shaders that are still worse off...it looks > like a ton of Source shaders used to do MUL/ADD with an attribute and > two immediates, and now are doing MOV/MOV/MAD. I just looked, and thought that too for a minute

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-08 Thread Matt Turner
On Fri, May 8, 2015 at 10:08 AM, Jason Ekstrand wrote: > On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke wrote: >> On Thursday, May 07, 2015 06:17:46 PM Matt Turner wrote: >>> On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: >>> > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders o

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-08 Thread Jason Ekstrand
On Fri, May 8, 2015 at 3:27 AM, Kenneth Graunke wrote: > On Thursday, May 07, 2015 06:17:46 PM Matt Turner wrote: >> On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: >> > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: >> > >> >total instructions in shared progra

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-08 Thread Kenneth Graunke
On Thursday, May 07, 2015 06:17:46 PM Matt Turner wrote: > On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: > > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: > > > >total instructions in shared programs: 2724483 -> 2711790 (-0.47%) > >instructions in affecte

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Jason Ekstrand
On Thu, May 7, 2015 at 8:49 PM, Jason Ekstrand wrote: > On Thu, May 7, 2015 at 6:17 PM, Matt Turner wrote: >> On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: >>> GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: >>> >>>total instructions in shared programs: 27244

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Jason Ekstrand
On Thu, May 7, 2015 at 6:17 PM, Matt Turner wrote: > On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: >> GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: >> >>total instructions in shared programs: 2724483 -> 2711790 (-0.47%) >>instructions in affected program

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Jason Ekstrand
On Thu, May 7, 2015 at 6:17 PM, Matt Turner wrote: > On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: >> GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: >> >>total instructions in shared programs: 2724483 -> 2711790 (-0.47%) >>instructions in affected program

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Ian Romanick
On 05/07/2015 06:17 PM, Matt Turner wrote: > On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: >> GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: >> >>total instructions in shared programs: 2724483 -> 2711790 (-0.47%) >>instructions in affected programs: 18

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Matt Turner
On Thu, May 7, 2015 at 4:50 PM, Jason Ekstrand wrote: > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: > >total instructions in shared programs: 2724483 -> 2711790 (-0.47%) >instructions in affected programs: 1860859 -> 1848166 (-0.68%) >helped:

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Ian Romanick
On 05/07/2015 05:44 PM, Jason Ekstrand wrote: > > On May 7, 2015 5:38 PM, "Ian Romanick" > wrote: >> >> On 05/07/2015 04:50 PM, Jason Ekstrand wrote: >> > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: >> > >> >total instructions in share

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Jason Ekstrand
On May 7, 2015 5:38 PM, "Ian Romanick" wrote: > > On 05/07/2015 04:50 PM, Jason Ekstrand wrote: > > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: > > > >total instructions in shared programs: 2724483 -> 2711790 (-0.47%) > >instructions in affected programs: 1

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Ian Romanick
On 05/07/2015 04:50 PM, Jason Ekstrand wrote: > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: > >total instructions in shared programs: 2724483 -> 2711790 (-0.47%) >instructions in affected programs: 1860859 -> 1848166 (-0.68%) >helped:

Re: [Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Kenneth Graunke
On Thursday, May 07, 2015 04:50:39 PM Jason Ekstrand wrote: > GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: > >total instructions in shared programs: 2724483 -> 2711790 (-0.47%) >instructions in affected programs: 1860859 -> 1848166 (-0.68%) >helped:

[Mesa-dev] [PATCH] i965: Use NIR by default for vertex shaders on GEN8+

2015-05-07 Thread Jason Ekstrand
GLSL IR vs. NIR shader-db results for SIMD8 vertex shaders on Broadwell: total instructions in shared programs: 2724483 -> 2711790 (-0.47%) instructions in affected programs: 1860859 -> 1848166 (-0.68%) helped:4387 HURT: