Re: [Mesa-dev] [PATCH] nv50/ir: start LocalCSE with getFirst to merge PHI instructions

2016-10-25 Thread Karol Herbst
2016-10-25 11:18 GMT+02:00 Samuel Pitoiset : > > > On 10/08/2016 06:58 PM, Samuel Pitoiset wrote: >> >> This breaks a bunch of things, like: >> >> spec/glsl-4.30/execution/built-in-functions/cs-all-bvec2-using-if: fail >> spec/glsl-4.30/execution/built-in-functions/cs-all-bvec3-using-if: fail >> sp

Re: [Mesa-dev] [PATCH] nv50/ir: start LocalCSE with getFirst to merge PHI instructions

2016-10-25 Thread Samuel Pitoiset
On 10/08/2016 06:58 PM, Samuel Pitoiset wrote: This breaks a bunch of things, like: spec/glsl-4.30/execution/built-in-functions/cs-all-bvec2-using-if: fail spec/glsl-4.30/execution/built-in-functions/cs-all-bvec3-using-if: fail spec/glsl-4.30/execution/built-in-functions/cs-all-bvec4-using-if:

Re: [Mesa-dev] [PATCH] nv50/ir: start LocalCSE with getFirst to merge PHI instructions

2016-10-08 Thread Samuel Pitoiset
This breaks a bunch of things, like: spec/glsl-4.30/execution/built-in-functions/cs-all-bvec2-using-if: fail spec/glsl-4.30/execution/built-in-functions/cs-all-bvec3-using-if: fail spec/glsl-4.30/execution/built-in-functions/cs-all-bvec4-using-if: fail spec/glsl-4.30/execution/built-in-functions/

Re: [Mesa-dev] [PATCH] nv50/ir: start LocalCSE with getFirst to merge PHI instructions

2016-10-07 Thread Samuel Pitoiset
I will run piglit with that patch before pushing. Reviewed-by: Samuel Pitoiset On 10/06/2016 11:33 PM, Karol Herbst wrote: total instructions in shared programs : 2818606 -> 2818227 (-0.01%) total gprs used in shared programs: 379273 -> 379238 (-0.01%) total local used in shared programs

Re: [Mesa-dev] [PATCH] nv50/ir: start LocalCSE with getFirst to merge PHI instructions

2016-10-06 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Thu, Oct 6, 2016 at 5:33 PM, Karol Herbst wrote: > total instructions in shared programs : 2818606 -> 2818227 (-0.01%) > total gprs used in shared programs: 379273 -> 379238 (-0.01%) > total local used in shared programs : 9505 -> 9505 (0.00%) > total bytes used

[Mesa-dev] [PATCH] nv50/ir: start LocalCSE with getFirst to merge PHI instructions

2016-10-06 Thread Karol Herbst
total instructions in shared programs : 2818606 -> 2818227 (-0.01%) total gprs used in shared programs: 379273 -> 379238 (-0.01%) total local used in shared programs : 9505 -> 9505 (0.00%) total bytes used in shared programs : 25837192 -> 25833736 (-0.01%) localgpr