Re: [Mesa-dev] [PATCH] i965/vec4: Calculate register allocation q values manually.

2015-02-18 Thread Ian Romanick
On 02/18/2015 10:57 AM, Mark Janes wrote: > This patch fixes a performance regression that causes our piglit CI > system to slow by 50%. If anyone has time to take a look at this and > provide a RB, it will improve the rate at which you get your piglit > results. When you send a message like this

Re: [Mesa-dev] [PATCH] i965/vec4: Calculate register allocation q values manually.

2015-02-18 Thread Matt Turner
Reviewed-by: Matt Turner Thanks Curro! ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] i965/vec4: Calculate register allocation q values manually.

2015-02-18 Thread Mark Janes
This patch fixes a performance regression that causes our piglit CI system to slow by 50%. If anyone has time to take a look at this and provide a RB, it will improve the rate at which you get your piglit results. -Mark Francisco Jerez writes: > This fixes a regression in the running time of P

Re: [Mesa-dev] [PATCH] i965/vec4: Calculate register allocation q values manually.

2015-02-13 Thread Francisco Jerez
Connor Abbott writes: > I'll ask the same question I asked Jason when he did this for FS... > did you verify that the new q_values is the same as the old one? > Yeah, I did. > On Fri, Feb 13, 2015 at 8:02 AM, Francisco Jerez > wrote: >> This fixes a regression in the running time of Piglit int

Re: [Mesa-dev] [PATCH] i965/vec4: Calculate register allocation q values manually.

2015-02-13 Thread Connor Abbott
I'll ask the same question I asked Jason when he did this for FS... did you verify that the new q_values is the same as the old one? On Fri, Feb 13, 2015 at 8:02 AM, Francisco Jerez wrote: > This fixes a regression in the running time of Piglit introduced by > commit 78e9043475d4bed8b50f7e413963c

[Mesa-dev] [PATCH] i965/vec4: Calculate register allocation q values manually.

2015-02-13 Thread Francisco Jerez
This fixes a regression in the running time of Piglit introduced by commit 78e9043475d4bed8b50f7e413963c960fa0935bb, which increased the number of register allocation classes set up by the VEC4 back-end from 2 to 16. The algorithm used by ra_set_finalize() to calculate them is unnecessarily expens